Heads up! This PR modifies the following files:

• @asajeffrey: components/constellation/constellation.rs
• @KiChjang: components/script/dom/bindings/global.rs, components/script/layout_interface.rs, components/script/dom/window.rs, components/script_traits/lib.rs, components/script_traits/lib.rs, components/script/dom/document.rs, components/script/dom/htmllinkelement.rs, components/script/clipboard_provider.rs, components/script/dom/workerglobalscope.rs, components/script/dom/htmltextareaelement.rs, components/script/script_thread.rs, components/script/dom/htmlinputelement.rs, components/script/dom/htmlbodyelement.rs, components/script/dom/htmliframeelement.rs, components/script/dom/webglrenderingcontext.rs, components/script/dom/bindings/trace.rs, components/script/dom/canvasrenderingcontext2d.rs
• @emilio: components/script/dom/webglrenderingcontext.rs

Warning

• These commits modify gfx, layout, and script code, but no tests are modified. Please consider adding a test!

Yes yes yes!

Reviewed 24 of 24 files at r1.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

components/constellation/constellation.rs, line 327 [r1] (raw file):

    pub fn start(state: InitialConstellationState) -> Sender<FromCompositorMsg> {
        let (ipc_script_sender, ipc_script_receiver) =
            ipc::channel::<FromScriptMsg>().expect("ipc channel failure");

Do we need to provide the type arguments here? Can we not just use ipc::channel().expect("ipc channel failure")?

components/layout/layout_thread.rs, line 1084 [r1] (raw file):

                rw_data.constellation_chan
                       .send(ConstellationMsg::ViewportConstrained(self.id, constraints))
                       .unwrap();

This is so much cleaner, hooray!

Comments from Reviewable

New code was committed to pull request.

@bors-servo r+

📌 Commit cc2b2b5 has been approved by asajeffrey

@bors-servo r-

Homu confused, will r+ again.

@bors-servo r=asajeffrey

📌 Commit cc2b2b5 has been approved by asajeffrey

⌛ Testing commit cc2b2b5 with merge 27c25e8...

☀️ Test successful - android, arm32, arm64, linux-dev, linux-rel, mac-dev-unit, mac-rel-css, mac-rel-wpt, windows