Skip to content

/ servo

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

Remove the script listener thread. #11410

Merged
merged 2 commits into from May 25, 2016
Merged

Remove the script listener thread. #11410

merged 2 commits into from May 25, 2016
+108 −172

Commits on May 25, 2016

  1. Remove the script listener thread (fixes #11345). 

    We needed a separate thread in the chrome process because communication to the
compositor is done through a trait object, and cross-process virtual calls are
forbidden.

Also, the fact that these messages are ultimately handled by the compositor is
an implementation detail; conceptually, the relevant constellation is supposed
to handle these messages.

So instead, the script thread will now send the messages to the constellation,
which will ask the compositor to handle them.
    @Ms2ger
    Ms2ger committed May 25, 2016
    4113eb6

  2. Merge PrivilegedPipelineContent::{start_all, start_paint_thread}. 

    They now do the same thing.
    @Ms2ger
    Ms2ger committed May 25, 2016
    Loading status checks…
    e7f75ca
You can’t perform that action at this time.