Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add page.emulateCPUThrottling #7343

Merged
merged 5 commits into from Jun 22, 2021
Merged

feat: add page.emulateCPUThrottling #7343

merged 5 commits into from Jun 22, 2021

Conversation

jschfflr
Copy link
Contributor

No description provided.

@google-cla google-cla bot added the cla: yes label Jun 21, 2021
@jschfflr jschfflr requested a review from OrKoN June 21, 2021 20:39
@jschfflr
Copy link
Contributor Author

@OrKoN Do you have a suggestion for how to test this feature in a better way?

docs/api.md Outdated Show resolved Hide resolved
Copy link
Member

@mathiasbynens mathiasbynens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mathiasbynens mathiasbynens changed the title feat: add Page.emulateCPUThrottling feat: add page.emulateCPUThrottling Jun 22, 2021
@jschfflr jschfflr enabled auto-merge (squash) June 22, 2021 10:15
docs/api.md Outdated Show resolved Hide resolved
@jschfflr jschfflr merged commit 4ce4110 into main Jun 22, 2021
@jschfflr jschfflr deleted the emulate-cpu-throttling branch June 22, 2021 11:43
tasneemkoushar added a commit that referenced this pull request Jun 22, 2021
* fix: modified comment for method product, platform and newPage

* fix: added comment for browsercontext, StartCSSCoverage, StartJSCoverage

* fix: corrected comments for JSONValue, asElement, evaluateHandle

* fix: corrected comments for JSONValue, asElement, evaluateHandle

* fix: added comments for some of the method

* fix: added proper comments

* fix: added comment for some methods in page.ts

* fix: rectified the comments

* fix: changed some of the comments

* fix: added comments for 3 more methods of class page

* fix: added comments for 3 more methods of class page

* fix: added comments for 5 more methods in class page

* fix: removed the extra whitespace

* fix: added comments for 5 more method in page class

* fix: added comments for 5 more methods in class page

* fix: commented 12 more methods in class Page

* fix: commented 12 methods in class Page

* fix: commented 12 more methods in class Page

* fix: commented 12 more methods in class Page

* chore: fix typo in connection.spec.js (#7326)

recieved -> received

* chore: synchronize Bug 1710839: update comment about firefox preference remote.enabled (#7324)

* fix: added comments for methods in class page

* docs: add download location to the FAQ (#7339)

Co-authored-by: Mathias Bynens <mathias@qiwi.be>

* fix: added comments

* fix: comments added

* feat: add page.emulateCPUThrottling (#7343)

* fix: added comments

Co-authored-by: Jack Franklin <jacktfranklin@chromium.org>
Co-authored-by: Ikko Ashimine <eltociear@gmail.com>
Co-authored-by: Julian Descottes <julian.descottes@gmail.com>
Co-authored-by: Diego Fernandez <dfernandez79@users.noreply.github.com>
Co-authored-by: Mathias Bynens <mathias@qiwi.be>
Co-authored-by: Jan Scheffler <janscheffler@chromium.org>
This was referenced May 30, 2022
This was referenced May 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants