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

Clarify Document and Window's browsing context slot #4368

Merged
merged 3 commits into from
Feb 19, 2019

Conversation

Copy link
Member

@domenic domenic left a comment

Choose a reason for hiding this comment

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

LGTM, very nice. Suggested commit message (in particular I don't think Window has a BC "slot" per se):

Clarify Document and Window's browsing contexts

Previously, a Document could sometimes "have no browsing context". Now,
"a Document's browsing context" is either a browsing context or null.

Previously, "a Window's browsing context" was not defined, despite being
referenced extensively. Now, it is defined to be the Window's associated
Document's browsing context.

Related to #3846, #4363, and #4366.

source Show resolved Hide resolved
source Show resolved Hide resolved
annevk added a commit to web-platform-tests/wpt that referenced this pull request Feb 17, 2019
@annevk
Copy link
Member Author

annevk commented Feb 18, 2019

I tweaked that sentence. This should be in order now. Tests for rewritten length getter have landed.

@annevk annevk merged commit 4fe0ac1 into master Feb 19, 2019
@annevk annevk deleted the annevk/browsing-context-assocation branch February 19, 2019 10:40
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Mar 15, 2019
…testonly

Automatic update from web-platform-tests
HTML: window.length and named access

For whatwg/html#4368.

--

wpt-commits: 3a43f99a56a4c016e3f1cda41330f1a2c0e780ff
wpt-pr: 15414
mykmelez pushed a commit to mykmelez/gecko that referenced this pull request Mar 16, 2019
…testonly

Automatic update from web-platform-tests
HTML: window.length and named access

For whatwg/html#4368.

--

wpt-commits: 3a43f99a56a4c016e3f1cda41330f1a2c0e780ff
wpt-pr: 15414
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Mar 16, 2019
…testonly

Automatic update from web-platform-tests
HTML: window.length and named access

For whatwg/html#4368.

--

wpt-commits: 3a43f99a56a4c016e3f1cda41330f1a2c0e780ff
wpt-pr: 15414
mykmelez pushed a commit to mykmelez/gecko that referenced this pull request Mar 17, 2019
…testonly

Automatic update from web-platform-tests
HTML: window.length and named access

For whatwg/html#4368.

--

wpt-commits: 3a43f99a56a4c016e3f1cda41330f1a2c0e780ff
wpt-pr: 15414
marcoscaceres pushed a commit to web-platform-tests/wpt that referenced this pull request Jul 23, 2019
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request Oct 4, 2019
…testonly

Automatic update from web-platform-tests
HTML: window.length and named access

For whatwg/html#4368.

--

wpt-commits: 3a43f99a56a4c016e3f1cda41330f1a2c0e780ff
wpt-pr: 15414

UltraBlame original commit: 3d06a920925d1d2beb8dbe5d3fb7a686155a7000
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request Oct 4, 2019
…testonly

Automatic update from web-platform-tests
HTML: window.length and named access

For whatwg/html#4368.

--

wpt-commits: 3a43f99a56a4c016e3f1cda41330f1a2c0e780ff
wpt-pr: 15414

UltraBlame original commit: 8566ae4d152605453ebe6715ec4f2f8af60a95d3
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request Oct 4, 2019
…testonly

Automatic update from web-platform-tests
HTML: window.length and named access

For whatwg/html#4368.

--

wpt-commits: 3a43f99a56a4c016e3f1cda41330f1a2c0e780ff
wpt-pr: 15414

UltraBlame original commit: 3d06a920925d1d2beb8dbe5d3fb7a686155a7000
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request Oct 4, 2019
…testonly

Automatic update from web-platform-tests
HTML: window.length and named access

For whatwg/html#4368.

--

wpt-commits: 3a43f99a56a4c016e3f1cda41330f1a2c0e780ff
wpt-pr: 15414

UltraBlame original commit: 8566ae4d152605453ebe6715ec4f2f8af60a95d3
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request Oct 4, 2019
…testonly

Automatic update from web-platform-tests
HTML: window.length and named access

For whatwg/html#4368.

--

wpt-commits: 3a43f99a56a4c016e3f1cda41330f1a2c0e780ff
wpt-pr: 15414

UltraBlame original commit: 3d06a920925d1d2beb8dbe5d3fb7a686155a7000
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request Oct 4, 2019
…testonly

Automatic update from web-platform-tests
HTML: window.length and named access

For whatwg/html#4368.

--

wpt-commits: 3a43f99a56a4c016e3f1cda41330f1a2c0e780ff
wpt-pr: 15414

UltraBlame original commit: 8566ae4d152605453ebe6715ec4f2f8af60a95d3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants