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(chromium): roll Chromium to r782078 #6250

Merged
merged 5 commits into from Jul 20, 2020
Merged

feat(chromium): roll Chromium to r782078 #6250

merged 5 commits into from Jul 20, 2020

Conversation

johanbay
Copy link
Collaborator

@johanbay johanbay commented Jul 20, 2020

This corresponds to Chromium 85.0.4182.0
This roll includes:

Closes #6151

@@ -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 });
Copy link
Collaborator

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.

Copy link
Collaborator Author

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,
Copy link
Collaborator

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 👍

Copy link
Collaborator

@jackfranklin jackfranklin left a 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 :)

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 merged commit e89e264 into main Jul 20, 2020
@mathiasbynens mathiasbynens deleted the roll-m85 branch July 20, 2020 15:17
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.

Roll Chromium 85
4 participants