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(chromium): roll Chromium to r782078 #6250
Conversation
@@ -49,7 +49,7 @@ describe('ElementHandle specs', function () { | |||
const elementHandle = await nestedFrame.$('div'); | |||
const box = await elementHandle.boundingBox(); | |||
if (isChrome) | |||
expect(box).toEqual({ x: 28, y: 260, width: 264, height: 18 }); | |||
expect(box).toEqual({ x: 28, y: 182, width: 264, height: 18 }); | |||
else expect(box).toEqual({ x: 28, y: 182, width: 254, height: 18 }); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now these two expect
calls are the same, I think we can lose the conditional.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The width
is different though?
@@ -163,7 +164,7 @@ async function checkRollCandidate() { | |||
10 | |||
); | |||
const currentRevision = parseInt( | |||
require('../lib/cjs/revisions').PUPPETEER_REVISIONS.chromium, | |||
require('../lib/cjs/puppeteer/revisions').PUPPETEER_REVISIONS.chromium, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah sorry I missed these when I did the restructure; thank you for fixing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but let Mathias check as well as I've not done a roll before :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This corresponds to Chromium 85.0.4182.0
This roll includes:
Closes #6151