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

[WIP] Script to embedder communication #20706

Conversation

@gterzian
Copy link
Member

gterzian commented Apr 28, 2018

#19992 (comment)

Following the pattern of ScriptToConstellationChan.


  • ./mach build -d does not report any errors
  • ./mach build-geckolib does not report any errors
  • ./mach test-tidy does not report any errors
  • These changes fix #19992 (github issue number if applicable).
  • There are tests for these changes OR
  • These changes do not require tests because _____

This change is Reviewable

@highfive
Copy link

highfive commented Apr 28, 2018

Heads up! This PR modifies the following files:

  • @asajeffrey: components/constellation/pipeline.rs, components/script/dom/globalscope.rs, components/constellation/constellation.rs, components/script/script_thread.rs, components/script/dom/window.rs
  • @cbrewster: components/constellation/pipeline.rs, components/constellation/constellation.rs
  • @paulrouget: components/constellation/pipeline.rs, components/constellation/constellation.rs
  • @fitzgen: components/script/dom/globalscope.rs, components/script/script_thread.rs, components/script/dom/window.rs, components/script_traits/lib.rs
  • @KiChjang: components/script/dom/globalscope.rs, components/script/script_thread.rs, components/script/dom/window.rs, components/script_traits/lib.rs
@highfive
Copy link

highfive commented Apr 28, 2018

warning Warning warning

  • These commits modify script code, but no tests are modified. Please consider adding a test!
@gterzian gterzian changed the title [WIP] Script to embedder comminication [WIP] Script to embedder communication Apr 28, 2018
@gterzian gterzian force-pushed the gterzian:embedder_handling_of_prompts_and_alerts branch from 4f1e1dd to 3ea8ecf Apr 28, 2018
@gterzian
Copy link
Member Author

gterzian commented Apr 28, 2018

In favor of #20707

@gterzian gterzian closed this Apr 28, 2018

/// A Script to Embedder channel.
#[derive(Clone, Deserialize, Serialize)]
pub struct ScriptToEmbedderChan {

This comment has been minimized.

@gterzian

gterzian Apr 28, 2018

Author Member

This is not necessary, since top_level_browsing_context_id would available directly at the point where could be used to send an EmbedderMsg...

@gterzian gterzian deleted the gterzian:embedder_handling_of_prompts_and_alerts branch Jan 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants
You can’t perform that action at this time.