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

Implement Document.readyState. Prevent iframes from notifying the compos... #3759

Closed
wants to merge 1 commit into from

Conversation

jdm
Copy link
Member

@jdm jdm commented Oct 21, 2014

...itor after the initial parse. Fixes #1720. Fixes #3738.

r? @Ms2ger or @Manishearth

@hoppipolla-critic-bot
Copy link

Critic review: https://critic.hoppipolla.co.uk/r/2938

This is an external review system which you may optionally use for the code review of your pull request.

In order to help critic track your changes, please do not make in-place history rewrites (e.g. via git rebase -i or git commit --amend) when updating this pull request.

@larsbergstrom
Copy link
Contributor

Can either @Ms2ger or @Manishearth please take a look at this? I believe it's blocking a PR from @kmcallister which is in turn blocking a PR from @mbrubeck :-)

@Manishearth
Copy link
Member

In a way #3776 blocks this one, lengthening the chain ;P

@Manishearth
Copy link
Member

@mbrubeck You probably know this, but be a bit careful about the interactive ready state (scroll up) for #3721 once this merges.

bors-servo pushed a commit that referenced this pull request Oct 23, 2014
...itor after the initial parse. Fixes #1720. Fixes #3738.

r? @Ms2ger or @Manishearth
@bors-servo bors-servo closed this Oct 23, 2014
@hsivonen
Copy link
Contributor

As noted in another PR, in the case of Gecko, I found that it would be beneficial to have state tracking across the navigation. If such a state tracking object existed, it would be logical for the true readyState to be kept on that object such that once Document is created, it would get a reference to the associated navigation tracking object and the Document's readyState could then call through to that object to ask for the state.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Intermittent compositor failure in wpt Implement Document.readyState
7 participants