Skip to content

Commit

Permalink
Roll chromium to 508268 (#1019)
Browse files Browse the repository at this point in the history
This gets us:

- Plural metrics names https://crrev.com/507885
- TargetInfoChanged on URL changes https://crrev.com/508049
- Sending proper resource types for requests https://crrev.com/667504

Fixes #734
  • Loading branch information
JoelEinbinder authored and aslushnikov committed Oct 12, 2017
1 parent c893bf3 commit 0426e3c
Show file tree
Hide file tree
Showing 4 changed files with 15 additions and 15 deletions.
12 changes: 6 additions & 6 deletions docs/api.md
Expand Up @@ -671,10 +671,10 @@ If there's no element matching `selector`, the method throws an error.
#### page.getMetrics()
- returns: <[Object]> Object containing metrics as key/value pairs.
- `Timestamp` <[number]> The timestamp when the metrics sample was taken.
- `DocumentCount` <[number]> Number of documents in the page.
- `FrameCount` <[number]> Number of frames in the page.
- `JSEventListenerCount` <[number]> Number of events in the page.
- `NodeCount` <[number]> Number of DOM nodes in the page.
- `Documents` <[number]> Number of documents in the page.
- `Frames` <[number]> Number of frames in the page.
- `JSEventListeners` <[number]> Number of events in the page.
- `Nodes` <[number]> Number of DOM nodes in the page.
- `LayoutCount` <[number]> Total number of full or partial page layout.
- `RecalcStyleCount` <[number]> Total number of page style recalculations.
- `LayoutDuration` <[number]> Combined durations of all page layouts.
Expand Down Expand Up @@ -820,7 +820,7 @@ The `format` options are:
The method iterates javascript heap and finds all the objects with the given prototype.

```js
// Create a Map object
// Create a Map object
await page.evaluate(() => window.map = new Map());
// Get a handle to the Map object prototype
const mapPrototype = await page.evaluateHandle(() => Map.prototype);
Expand Down Expand Up @@ -1543,7 +1543,7 @@ await resultHandle.dispose();
The method iterates javascript heap and finds all the objects with the given prototype.

```js
// Create a Map object
// Create a Map object
await page.evaluate(() => window.map = new Map());
// Get a handle to the Map object prototype
const mapPrototype = await page.evaluateHandle(() => Map.prototype);
Expand Down
8 changes: 4 additions & 4 deletions lib/Page.js
Expand Up @@ -832,10 +832,10 @@ class Page extends EventEmitter {
/** @type {!Set<string>} */
const supportedMetrics = new Set([
'Timestamp',
'DocumentCount',
'FrameCount',
'JSEventListenerCount',
'NodeCount',
'Documents',
'Frames',
'JSEventListeners',
'Nodes',
'LayoutCount',
'RecalcStyleCount',
'LayoutDuration',
Expand Down
2 changes: 1 addition & 1 deletion package.json
Expand Up @@ -34,7 +34,7 @@
"ws": "^3.0.0"
},
"puppeteer": {
"chromium_revision": "507565"
"chromium_revision": "508268"
},
"devDependencies": {
"@types/debug": "0.0.30",
Expand Down
8 changes: 4 additions & 4 deletions test/test.js
Expand Up @@ -796,10 +796,10 @@ describe('Page', function() {
function checkMetrics(metrics) {
const metricsToCheck = new Set([
'Timestamp',
'DocumentCount',
'FrameCount',
'JSEventListenerCount',
'NodeCount',
'Documents',
'Frames',
'JSEventListeners',
'Nodes',
'LayoutCount',
'RecalcStyleCount',
'LayoutDuration',
Expand Down

0 comments on commit 0426e3c

Please sign in to comment.