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

Calling current_styles before or during styling #13962

Open
k-bx opened this issue Oct 28, 2016 · 4 comments
Open

Calling current_styles before or during styling #13962

k-bx opened this issue Oct 28, 2016 · 4 comments
Labels

Comments

@k-bx
Copy link

@k-bx k-bx commented Oct 28, 2016

Calling current_styles before or during styling

URL:

https://blog.devteam.space/new-macbook-pro-is-not-a-laptop-for-developers-anymore-d0d4b1b8b7de?gi=7649b9baeae5#.tpt1vssco

Servo Version:

Servo 0.0.1-b4a882f

Backtrace:


WARNING: <ScriptThread PipelineId { namespace_id: PipelineNamespaceId(10), index: PipelineIndex(10) }>: stop_parsing not implemented, full speed ahead!
WARNING: <ScriptThread PipelineId { namespace_id: PipelineNamespaceId(10), index: PipelineIndex(11) }>: stop_parsing not implemented, full speed ahead!
WARNING: <ScriptThread PipelineId { namespace_id: PipelineNamespaceId(1), index: PipelineIndex(11) }>: stop_parsing not implemented, full speed ahead!
WARNING: <main>: Compositor layer has an unknown pipeline (PipelineId { namespace_id: PipelineNamespaceId(10), index: PipelineIndex(3) }).
WARNING: <main>: Compositor layer has an unknown pipeline (PipelineId { namespace_id: PipelineNamespaceId(10), index: PipelineIndex(5) }).
WARNING: <main>: Compositor layer has an unknown pipeline (PipelineId { namespace_id: PipelineNamespaceId(10), index: PipelineIndex(1) }).
WARNING: <main>: Compositor layer has an unknown pipeline (PipelineId { namespace_id: PipelineNamespaceId(10), index: PipelineIndex(7) }).
WARNING: <ScriptThread PipelineId { namespace_id: PipelineNamespaceId(1), index: PipelineIndex(11) }>: FIXME: </script> not fully implemented
WARNING: <ScriptThread PipelineId { namespace_id: PipelineNamespaceId(1), index: PipelineIndex(11) }>: stop_parsing not implemented, full speed ahead!
WARNING: <main>: Compositor layer has an unknown pipeline (PipelineId { namespace_id: PipelineNamespaceId(10), index: PipelineIndex(3) }).
WARNIN
...

This report was generated by the browser.html issue reporter.

@emilio
Copy link
Member

@emilio emilio commented Oct 29, 2016

@emilio
Copy link
Member

@emilio emilio commented Oct 30, 2016

So I investigated this a bit, and I couldn't dig a lot more because I can't make it crash in debug builds, but it seems deterministic (that's good), and related with the reflow we issue when the hash is changed.

I suspect the reflow may be suppressed and the next restyle we find something unexpected. I may investigate a bit more later.

@emilio
Copy link
Member

@emilio emilio commented Oct 30, 2016

Nope, the reflow is actually due to a script query. No idea what can trigger it exactly though :(

@bholley
Copy link
Contributor

@bholley bholley commented Oct 31, 2016

I'm still refactoring this code a bit, so give me a few days and we'll see if the problem persists.

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

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.