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

Add a nextLocalAncestor helper for Fullscreen #4321

Merged
merged 1 commit into from Dec 13, 2016
Merged

Conversation

jeffcarp
Copy link
Contributor

Refactoring only, no observable change intended.

A test is added that fails if the continue statement were dropped, as
apparently there was no coverage for this.

Note: This CL is also a test for the WPT export process.

BUG=402376

Review-Url: https://codereview.chromium.org/2565183002
Cr-Commit-Position: refs/heads/master@{#438076}

Refactoring only, no observable change intended.

A test is added that fails if the continue statement were dropped, as
apparently there was no coverage for this.

Note: This CL is also a test for the WPT export process.

BUG=402376

Review-Url: https://codereview.chromium.org/2565183002
Cr-Commit-Position: refs/heads/master@{#438076}
@wpt-pr-bot
Copy link
Collaborator

Notifying @aliams, @foolip, @jernoble, and @upsuper. (Learn how reviewing works.)

@wpt-stability-bot
Copy link

Firefox

Testing revision 02e9918
Starting 10 test iterations
No tests run.

@wpt-stability-bot
Copy link

Chrome

Testing revision 02e9918
Starting 10 test iterations
No tests run.

@jeffcarp jeffcarp merged commit 0234c35 into master Dec 13, 2016
@foolip foolip deleted the chromium-export-try branch December 13, 2016 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants