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 Broadcastchannel #25796

Merged
merged 2 commits into from Feb 25, 2020
Merged

Conversation

@gterzian
Copy link
Member

gterzian commented Feb 18, 2020

FIX #21025


  • ./mach build -d does not report any errors
  • ./mach test-tidy does not report any errors
  • These changes fix #___ (GitHub issue number if applicable)
  • There are tests for these changes OR
  • These changes do not require tests because ___
@highfive
Copy link

highfive commented Feb 18, 2020

Heads up! This PR modifies the following files:

  • @asajeffrey: components/script/dom/mod.rs, components/script/dom/webidls/BroadcastChannel.webidl, components/script/dom/broadcastchannel.rs
  • @KiChjang: components/script/dom/mod.rs, components/script/dom/webidls/BroadcastChannel.webidl, components/script/dom/broadcastchannel.rs
@highfive
Copy link

highfive commented Feb 18, 2020

warning Warning warning

  • These commits modify script code, but no tests are modified. Please consider adding a test!
@gterzian gterzian force-pushed the gterzian:implement_broadcast_channel branch from a7ecb75 to bf11296 Feb 18, 2020
@Manishearth
Copy link
Member

Manishearth commented Feb 18, 2020

Please @ me when you're ready for review!

@gterzian gterzian force-pushed the gterzian:implement_broadcast_channel branch 5 times, most recently from 8e0b1e3 to 575ba2a Feb 19, 2020
@gterzian gterzian force-pushed the gterzian:implement_broadcast_channel branch from 24c9d8a to 55690c4 Feb 19, 2020
@gterzian
Copy link
Member Author

gterzian commented Feb 19, 2020

@bors-servo try=wpt

@bors-servo
Copy link
Contributor

bors-servo commented Feb 19, 2020

Trying commit 55690c4 with merge f2cd8cf...

bors-servo added a commit that referenced this pull request Feb 19, 2020
[WIP] Implement Broadcastchannel

<!-- Please describe your changes on the following line: -->

FIX #21025

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: -->
- [ ] `./mach build -d` does not report any errors
- [ ] `./mach test-tidy` does not report any errors
- [ ] These changes fix #___ (GitHub issue number if applicable)

<!-- Either: -->
- [ ] There are tests for these changes OR
- [ ] These changes do not require tests because ___

<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
@gterzian gterzian force-pushed the gterzian:implement_broadcast_channel branch from 55690c4 to 9b75536 Feb 19, 2020
@gterzian
Copy link
Member Author

gterzian commented Feb 20, 2020

@bors-servo try=wpt

@bors-servo
Copy link
Contributor

bors-servo commented Feb 20, 2020

Trying commit 9b75536 with merge 7f774e1...

bors-servo added a commit that referenced this pull request Feb 20, 2020
[WIP] Implement Broadcastchannel

<!-- Please describe your changes on the following line: -->

FIX #21025

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: -->
- [ ] `./mach build -d` does not report any errors
- [ ] `./mach test-tidy` does not report any errors
- [ ] These changes fix #___ (GitHub issue number if applicable)

<!-- Either: -->
- [ ] There are tests for these changes OR
- [ ] These changes do not require tests because ___

<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
@bors-servo
Copy link
Contributor

bors-servo commented Feb 20, 2020

💔 Test failed - status-taskcluster

@gterzian
Copy link
Member Author

gterzian commented Feb 20, 2020

9 unexpected results that are NOT known-intermittents:
  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/sandbox.html:
  â”” PASS [expected FAIL] Creating BroadcastChannel in an opaque origin


  â–¶ Unexpected subtest result in /webmessaging/MessageEvent-trusted.html:
  â”” PASS [expected FAIL] With a BroadcastChannel


  â–¶ CRASH [expected OK] /webmessaging/broadcastchannel/blobs.html
  │ 
  │ 
  │ 
  │ 
  │ The SC holder does not have any blob impls (thread ScriptThread PipelineId { namespace_id: PipelineNamespaceId(1), index: PipelineIndex(1) }, at components/script/dom/blob.rs:158)
  │ stack backtrace:
  │    0: servo::backtrace::print
  │    1: servo::main::{{closure}}
  │    2: std::panicking::rust_panic_with_hook
  │              at src/libstd/panicking.rs:474
  │    3: rust_begin_unwind
  │              at src/libstd/panicking.rs:378
  │    4: core::panicking::panic_fmt
  │              at src/libcore/panicking.rs:85
  │    5: core::option::expect_failed
  │              at src/libcore/option.rs:1203
  │    6: <script::dom::blob::Blob as script::dom::bindings::serializable::Serializable>::deserialize
  │    7: script::dom::bindings::structuredclone::read_callback
  │    8: _ZN23JSStructuredCloneReader9startReadEN2JS13MutableHandleINS0_5ValueEEE
  │              at /root/.cargo/git/checkouts/mozjs-fa11ffc7d4f1cc2d/5906588/mozjs/js/src/vm/StructuredClone.cpp:2591
  │    9: _ZN23JSStructuredCloneReader4readEN2JS13MutableHandleINS0_5ValueEEE
  │              at /root/.cargo/git/checkouts/mozjs-fa11ffc7d4f1cc2d/5906588/mozjs/js/src/vm/StructuredClone.cpp:2955
  │   10: _Z19ReadStructuredCloneP9JSContextR21JSStructuredCloneDataN2JS20StructuredCloneScopeENS3_13MutableHandleINS3_5ValueEEEPK26JSStructuredCloneCallbacksPv
  │              at /root/.cargo/git/checkouts/mozjs-fa11ffc7d4f1cc2d/5906588/mozjs/js/src/vm/StructuredClone.cpp:639
  │   11: script::dom::bindings::structuredclone::read
  │   12: <script::dom::globalscope::GlobalScope::broadcast_message_event::process_pending_port_messages<F> as script::task::TaskOnce>::run_once
  │   13: <T as script::task::TaskBox>::run_box
  │   14: script::script_thread::ScriptThread::handle_msg_from_script
  │   15: _ZN6script13script_thread12ScriptThread11handle_msgs17h24bdb8b2675b7816E.llvm.515341110360646871
  │   16: script::script_thread::ScriptThread::start
  │   17: profile_traits::mem::ProfilerChan::run_with_memory_reporting
  │   18: std::sys_common::backtrace::__rust_begin_short_backtrace
  │   19: _ZN3std9panicking3try7do_call17h5cf35d4fef2ac511E.llvm.16767328407926134309
  │   20: __rust_maybe_catch_panic
  │              at src/libpanic_unwind/lib.rs:86
  │   21: core::ops::function::FnOnce::call_once{{vtable.shim}}
  │   22: <alloc::boxed::Box<F> as core::ops::function::FnOnce<A>>::call_once
  │              at /rustc/5d04ce67fd14538d03fa47a2598f80d49fd564c6/src/liballoc/boxed.rs:1017
  │   23: <alloc::boxed::Box<F> as core::ops::function::FnOnce<A>>::call_once
  │              at /rustc/5d04ce67fd14538d03fa47a2598f80d49fd564c6/src/liballoc/boxed.rs:1017
  │       std::sys_common::thread::start_thread
  │              at src/libstd/sys_common/thread.rs:13
  │       std::sys::unix::thread::Thread::new::thread_start
  │              at src/libstd/sys/unix/thread.rs:80
  │   24: start_thread
  │   25: __clone
  │ [2020-02-20T05:12:38Z ERROR servo] The SC holder does not have any blob impls
  │ Pipeline failed in hard-fail mode.  Crashing!
  │ thread panicked while processing panic. aborting.
  â”” thread panicked while processing panic. aborting.


  â–¶ Unexpected subtest result in /html/dom/idlharness.https.html?exclude=(Document|Window|HTML.*):
  â”” PASS [expected FAIL] BroadcastChannel interface: existence and properties of interface object

  â–¶ Unexpected subtest result in /html/dom/idlharness.https.html?exclude=(Document|Window|HTML.*):
  â”” PASS [expected FAIL] BroadcastChannel interface object length

  â–¶ Unexpected subtest result in /html/dom/idlharness.https.html?exclude=(Document|Window|HTML.*):
  â”” PASS [expected FAIL] BroadcastChannel interface object name

  â–¶ Unexpected subtest result in /html/dom/idlharness.https.html?exclude=(Document|Window|HTML.*):
  â”” PASS [expected FAIL] BroadcastChannel interface: existence and properties of interface prototype object

  â–¶ Unexpected subtest result in /html/dom/idlharness.https.html?exclude=(Document|Window|HTML.*):
  â”” PASS [expected FAIL] BroadcastChannel interface: existence and properties of interface prototype object's "constructor" property

  â–¶ Unexpected subtest result in /html/dom/idlharness.https.html?exclude=(Document|Window|HTML.*):
  â”” PASS [expected FAIL] BroadcastChannel interface: existence and properties of interface prototype object's @@unscopables property

  â–¶ Unexpected subtest result in /html/dom/idlharness.https.html?exclude=(Document|Window|HTML.*):
  â”” PASS [expected FAIL] BroadcastChannel interface: attribute name

  â–¶ Unexpected subtest result in /html/dom/idlharness.https.html?exclude=(Document|Window|HTML.*):
  â”” PASS [expected FAIL] BroadcastChannel interface: operation postMessage(any)

  â–¶ Unexpected subtest result in /html/dom/idlharness.https.html?exclude=(Document|Window|HTML.*):
  â”” PASS [expected FAIL] BroadcastChannel interface: operation close()

  â–¶ Unexpected subtest result in /html/dom/idlharness.https.html?exclude=(Document|Window|HTML.*):
  â”” PASS [expected FAIL] BroadcastChannel interface: attribute onmessage

  â–¶ Unexpected subtest result in /html/dom/idlharness.https.html?exclude=(Document|Window|HTML.*):
  â”” PASS [expected FAIL] BroadcastChannel interface: attribute onmessageerror


  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/interface.html:
  â”” PASS [expected FAIL] Should throw if no name is provided

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/interface.html:
  â”” PASS [expected FAIL] Null name should not throw

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/interface.html:
  â”” PASS [expected FAIL] Undefined name should not throw

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/interface.html:
  â”” PASS [expected FAIL] Non-empty name should not throw

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/interface.html:
  â”” PASS [expected FAIL] Non-string name should not throw

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/interface.html:
  â”” PASS [expected FAIL] postMessage without parameters should throw

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/interface.html:
  â”” PASS [expected FAIL] postMessage with null should not throw

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/interface.html:
  â”” PASS [expected FAIL] close should not throw

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/interface.html:
  â”” PASS [expected FAIL] close should not throw when called multiple times

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/interface.html:
  â”” PASS [expected FAIL] postMessage after close should throw

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/interface.html:
  â”” PASS [expected FAIL] BroadcastChannel should have an onmessage event

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/interface.html:
  â”” PASS [expected FAIL] postMessage should throw with uncloneable data

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/interface.html:
  â”” PASS [expected FAIL] postMessage should throw InvalidStateError after close, even with uncloneable data


  â–¶ OK [expected TIMEOUT] /webmessaging/broadcastchannel/workers.html

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/workers.html:
  â”” PASS [expected FAIL] BroadcastChannel works in workers

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/workers.html:
  â”” PASS [expected FAIL] Closing and re-opening a channel works.

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/workers.html:
  â”” PASS [expected TIMEOUT] BroadcastChannel created after a worker self.close()

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/workers.html:
  â”” PASS [expected FAIL] BroadcastChannel used after a worker self.close()


  â–¶ Unexpected subtest result in /workers/semantics/interface-objects/001.worker.html:
  â”” PASS [expected FAIL] The BroadcastChannel interface object should be exposed.


  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/basics.html:
  â”” PASS [expected FAIL] postMessage results in correct event

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/basics.html:
  â”” PASS [expected FAIL] messages are delivered in port creation order

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/basics.html:
  â”” PASS [expected FAIL] messages aren't delivered to a closed port

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/basics.html:
  â”” PASS [expected FAIL] closing and creating channels during message delivery works correctly


  â–¶ OK [expected TIMEOUT] /webmessaging/broadcastchannel/origin.window.html

  â–¶ Unexpected subtest result in /webmessaging/broadcastchannel/origin.window.html:
  â”” PASS [expected TIMEOUT] Serialization of BroadcastChannel origin
@gterzian
Copy link
Member Author

gterzian commented Feb 20, 2020

9 unexpected results that are NOT known-intermittents:
  â–¶ Unexpected subtest result in /html/browsers/the-window-object/noopener-noreferrer-sizing.window.html:
  â”” PASS [expected FAIL] window.open() with noopener should have equal viewport width and height

  â–¶ Unexpected subtest result in /html/browsers/the-window-object/noopener-noreferrer-sizing.window.html:
  â”” PASS [expected FAIL] window.open() with noreferrer should have equal viewport width and height


  â–¶ Unexpected subtest result in /html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/no-coop-coep.https.any.html:
  â”” PASS [expected FAIL] SharedArrayBuffer over BroadcastChannel without COOP+COEP


  â–¶ Unexpected subtest result in /html/browsers/the-window-object/window-open-noreferrer.html:
  â”” PASS [expected FAIL] window.open() with "noreferrer" tests


  â–¶ OK [expected ERROR] /html/browsers/the-window-object/window-open-noopener.html?indexed

  â–¶ Unexpected subtest result in /html/browsers/the-window-object/window-open-noopener.html?indexed:
  â”” PASS [expected FAIL] window.open() with 'noopener' should reuse existing target

  â–¶ Unexpected subtest result in /html/browsers/the-window-object/window-open-noopener.html?indexed:
  â”” PASS [expected NOTRUN] noopener=1 means the same as noopener

  â–¶ Unexpected subtest result in /html/browsers/the-window-object/window-open-noopener.html?indexed:
  â”” PASS [expected NOTRUN] noopener=0 means lack of noopener

  â–¶ Unexpected subtest result in /html/browsers/the-window-object/window-open-noopener.html?indexed:
  â”” PASS [expected NOTRUN] noopener separated only by spaces should work

  â–¶ Unexpected subtest result in /html/browsers/the-window-object/window-open-noopener.html?indexed:
  â”” PASS [expected NOTRUN] Trailing noopener should work

  â–¶ Unexpected subtest result in /html/browsers/the-window-object/window-open-noopener.html?indexed:
  â”” PASS [expected NOTRUN] Leading noopener should work

  â–¶ Unexpected subtest result in /html/browsers/the-window-object/window-open-noopener.html?indexed:
  â”” PASS [expected NOTRUN] Interior noopener should work

  â–¶ Unexpected subtest result in /html/browsers/the-window-object/window-open-noopener.html?indexed:
  â”” PASS [expected NOTRUN] noreferrer should also suppress opener when reusing existing target


  â–¶ Unexpected subtest result in /html/infrastructure/safe-passing-of-structured-data/shared-array-buffers/no-coop-coep.https.any.worker.html:
  â”” PASS [expected FAIL] SharedArrayBuffer over BroadcastChannel without COOP+COEP


  â–¶ TIMEOUT [expected OK] /html/browsers/the-window-object/noopener-noreferrer-BarProp.window.html
  │ 
  │ 
  │ 
  │ 
  │ [2020-02-20T05:09:18Z ERROR servo::browser] Multiple top level browsing contexts not supported yet.
  │ [2020-02-20T05:09:18Z ERROR servo::browser] Multiple top level browsing contexts not supported yet.
  │ [2020-02-20T05:09:18Z ERROR script::dom::bindings::error] Error at http://web-platform.test:8000/html/browsers/the-window-object/support/BarProp-target.html?5454noopener34324:8:5 window[prop] is undefined
  â”” [2020-02-20T05:09:19Z ERROR script::dom::bindings::error] Error at http://web-platform.test:8000/html/browsers/the-window-object/support/BarProp-target.html?5454noreferrer34324:8:5 window[prop] is undefined

  â–¶ Unexpected subtest result in /html/browsers/the-window-object/noopener-noreferrer-BarProp.window.html:
  │ TIMEOUT [expected FAIL] window.open() with noopener should have all bars visible
  └   → Test timed out

  â–¶ Unexpected subtest result in /html/browsers/the-window-object/noopener-noreferrer-BarProp.window.html:
  │ TIMEOUT [expected FAIL] window.open() with noreferrer should have all bars visible
  └   → Test timed out


  â–¶ TIMEOUT [expected OK] /html/semantics/links/links-created-by-a-and-area-elements/target_blank_implicit_noopener.html
  │ 
  │ 
  │ 
  │ 
  │ [2020-02-20T05:09:09Z ERROR servo::browser] Multiple top level browsing contexts not supported yet.
  │ [2020-02-20T05:09:09Z ERROR servo::browser] Multiple top level browsing contexts not supported yet.
  │ [2020-02-20T05:09:09Z ERROR servo::browser] Multiple top level browsing contexts not supported yet.
  │ [2020-02-20T05:09:09Z ERROR servo::browser] Multiple top level browsing contexts not supported yet.
  │ [2020-02-20T05:09:09Z ERROR servo::browser] Multiple top level browsing contexts not supported yet.
  │ [2020-02-20T05:09:10Z ERROR servo::browser] Multiple top level browsing contexts not supported yet.
  │ [2020-02-20T05:09:10Z ERROR servo::browser] Multiple top level browsing contexts not supported yet.
  â”” [2020-02-20T05:09:10Z ERROR servo::browser] Multiple top level browsing contexts not supported yet.

  â–¶ Unexpected subtest result in /html/semantics/links/links-created-by-a-and-area-elements/target_blank_implicit_noopener.html:
  â”” PASS [expected FAIL] Anchor element with target=_blank with rel=noopener

  â–¶ Unexpected subtest result in /html/semantics/links/links-created-by-a-and-area-elements/target_blank_implicit_noopener.html:
  â”” PASS [expected FAIL] Anchor element with target=_blank with rel=opener

  â–¶ Unexpected subtest result in /html/semantics/links/links-created-by-a-and-area-elements/target_blank_implicit_noopener.html:
  â”” PASS [expected FAIL] Anchor element with target=_blank with rel=opener+noopener

  â–¶ Unexpected subtest result in /html/semantics/links/links-created-by-a-and-area-elements/target_blank_implicit_noopener.html:
  â”” PASS [expected FAIL] Anchor element with target=_blank with rel=noopener+opener

  â–¶ Unexpected subtest result in /html/semantics/links/links-created-by-a-and-area-elements/target_blank_implicit_noopener.html:
  â”” PASS [expected FAIL] Anchor element with target=_blank with rel=noreferrer

  â–¶ Unexpected subtest result in /html/semantics/links/links-created-by-a-and-area-elements/target_blank_implicit_noopener.html:
  â”” PASS [expected FAIL] Anchor element with target=_blank with rel=opener+noreferrer

  â–¶ Unexpected subtest result in /html/semantics/links/links-created-by-a-and-area-elements/target_blank_implicit_noopener.html:
  â”” PASS [expected FAIL] Anchor element with target=_blank with rel=noopener+opener+noreferrer

  â–¶ Unexpected subtest result in /html/semantics/links/links-created-by-a-and-area-elements/target_blank_implicit_noopener.html:
  │ TIMEOUT [expected FAIL] Area element with target=_blank with rel=noopener
  └   → Test timed out

  â–¶ Unexpected subtest result in /html/semantics/links/links-created-by-a-and-area-elements/target_blank_implicit_noopener.html:
  │ TIMEOUT [expected FAIL] Area element with target=_blank with rel=opener
  └   → Test timed out

  â–¶ Unexpected subtest result in /html/semantics/links/links-created-by-a-and-area-elements/target_blank_implicit_noopener.html:
  │ TIMEOUT [expected FAIL] Area element with target=_blank with implicit rel=noopener
  └   → Test timed out

  â–¶ Unexpected subtest result in /html/semantics/links/links-created-by-a-and-area-elements/target_blank_implicit_noopener.html:
  │ TIMEOUT [expected FAIL] Area element with target=_blank with rel=opener+noopener
  └   → Test timed out

  â–¶ Unexpected subtest result in /html/semantics/links/links-created-by-a-and-area-elements/target_blank_implicit_noopener.html:
  │ TIMEOUT [expected FAIL] Area element with target=_blank with rel=noopener+opener
  └   → Test timed out


  â–¶ TIMEOUT [expected OK] /html/semantics/links/links-created-by-a-and-area-elements/target_blank_implicit_noopener_base.html
  │ 
  │ 
  │ 
  â”” 


  â–¶ Unexpected subtest result in /workers/opaque-origin.html:
  â”” PASS [expected FAIL] Worker can access BroadcastChannel
@gterzian
Copy link
Member Author

gterzian commented Feb 20, 2020

1 unexpected results that are NOT known-intermittents:
  â–¶ TIMEOUT [expected OK] /html/semantics/forms/form-submission-target/rel-base-target.html
  │ 
  │ 
  │ 
  â”” 

@gterzian
Copy link
Member Author

gterzian commented Feb 20, 2020

4 unexpected results that are NOT known-intermittents:
  â–¶ FAIL [expected PASS] /css/css-flexbox/flex-items-flexibility.html
  │   → /css/css-flexbox/flex-items-flexibility.html 54a9df64f1476dd12020019d7cf22ac34d727bc0
  │   → /css/css-flexbox/reference/flex-items-flexibility.html 19723eb00eaef8bf076dbc242bd87043ddc61933
  └   → Screenshot is solid color 0xFFFFFF for /css/css-flexbox/flex-items-flexibility.html


  â–¶ FAIL [expected PASS] /css/css-flexbox/flexbox_flex-1-1-Npercent.html
  │   → /css/css-flexbox/flexbox_flex-1-1-Npercent.html 54a9df64f1476dd12020019d7cf22ac34d727bc0
  │   → /css/css-flexbox/flexbox_flex-1-1-Npercent-ref.html 8c19f8bbbb9d9fb7af16cb0b08d47b791c9bdf17
  └   → Screenshot is solid color 0xFFFFFF for /css/css-flexbox/flexbox_flex-1-1-Npercent.html


  â–¶ Unexpected subtest result in /_mozilla/mozilla/interfaces.worker.html:
  │ FAIL [expected PASS] interfaces
  │   → assert_true: If this is failing: DANGER, are you sure you want to expose the new interface BroadcastChannel to all webpages as a property on the global? Do not make a change to this file without review from jdm or Ms2ger for that specific change! expected true got false
  │ 
  │ test_interfaces/<@http://web-platform.test:8000/_mozilla/mozilla/interfaces.js:72:7
  │ Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1867:25
  │ test@http://web-platform.test:8000/resources/testharness.js:548:30
  │ test_interfaces@http://web-platform.test:8000/_mozilla/mozilla/interfaces.js:2:3
  â”” @http://web-platform.test:8000/_mozilla/mozilla/interfaces.worker.js:9:1


  â–¶ Unexpected subtest result in /_mozilla/mozilla/interfaces.html:
  │ FAIL [expected PASS] Interfaces exposed on the window
  │   → assert_true: If this is failing: DANGER, are you sure you want to expose the new interface BroadcastChannel to all webpages as a property on the global? Do not make a change to this file without review from jdm or Ms2ger for that specific change! expected true got false
  │ 
  │ test_interfaces/<@http://web-platform.test:8000/_mozilla/mozilla/interfaces.js:72:7
  │ Test.prototype.step@http://web-platform.test:8000/resources/testharness.js:1867:25
  │ test@http://web-platform.test:8000/resources/testharness.js:548:30
  │ test_interfaces@http://web-platform.test:8000/_mozilla/mozilla/interfaces.js:2:3
  â”” @http://web-platform.test:8000/_mozilla/mozilla/interfaces.html:13:1
@gterzian gterzian force-pushed the gterzian:implement_broadcast_channel branch 3 times, most recently from 86d028a to cf71484 Feb 20, 2020
bors-servo added a commit that referenced this pull request Feb 25, 2020
…earth

Implement Broadcastchannel

<!-- Please describe your changes on the following line: -->

FIX #21025

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `___` with appropriate data: -->
- [ ] `./mach build -d` does not report any errors
- [ ] `./mach test-tidy` does not report any errors
- [ ] These changes fix #___ (GitHub issue number if applicable)

<!-- Either: -->
- [ ] There are tests for these changes OR
- [ ] These changes do not require tests because ___

<!-- Also, please make sure that "Allow edits from maintainers" checkbox is checked, so that we can help you if you get stuck somewhere along the way.-->

<!-- Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. -->
@bors-servo
Copy link
Contributor

bors-servo commented Feb 25, 2020

💔 Test failed - status-taskcluster

@gterzian gterzian force-pushed the gterzian:implement_broadcast_channel branch from b0adb83 to 1cb1f16 Feb 25, 2020
@servo-wpt-sync
Copy link
Collaborator

servo-wpt-sync commented Feb 25, 2020

Transplanted upstreamable changes to existing PR.

Completed upstream sync of web-platform-test changes at web-platform-tests/wpt#21895.

@gterzian
Copy link
Member Author

gterzian commented Feb 25, 2020

Just made a small correction to a doc comment, and it looks like I've missed a few test updates...

@gterzian gterzian force-pushed the gterzian:implement_broadcast_channel branch from 1cb1f16 to fe8d77f Feb 25, 2020
@servo-wpt-sync
Copy link
Collaborator

servo-wpt-sync commented Feb 25, 2020

Transplanted upstreamable changes to existing PR.

Completed upstream sync of web-platform-test changes at web-platform-tests/wpt#21895.

@gterzian gterzian force-pushed the gterzian:implement_broadcast_channel branch from fe8d77f to 99e4d74 Feb 25, 2020
@gterzian
Copy link
Member Author

gterzian commented Feb 25, 2020

@bors-servo r=Manishearth

Let's see if the below is a new intermittent, I marked it as TIMEOUT for now.

TIMEOUT [expected OK] /html/semantics/forms/form-submission-target/rel-base-target.html

@bors-servo
Copy link
Contributor

bors-servo commented Feb 25, 2020

📌 Commit 99e4d74 has been approved by Manishearth

@bors-servo
Copy link
Contributor

bors-servo commented Feb 25, 2020

Testing commit 99e4d74 with merge c9c5f8b...

@servo-wpt-sync
Copy link
Collaborator

servo-wpt-sync commented Feb 25, 2020

Transplanted upstreamable changes to existing PR.

Completed upstream sync of web-platform-test changes at web-platform-tests/wpt#21895.

@bors-servo
Copy link
Contributor

bors-servo commented Feb 25, 2020

☀️ Test successful - status-taskcluster
Approved by: Manishearth
Pushing c9c5f8b to master...

@bors-servo bors-servo merged commit c9c5f8b into servo:master Feb 25, 2020
2 checks passed
2 checks passed
Community-TC (pull_request) TaskGroup: success
Details
homu Test successful
Details
@servo-wpt-sync
Copy link
Collaborator

servo-wpt-sync commented Feb 25, 2020

Error syncing changes upstream. Logs saved in error-snapshot-1582615371429.

@gterzian gterzian deleted the gterzian:implement_broadcast_channel branch Feb 25, 2020
@bors-servo bors-servo mentioned this pull request Feb 25, 2020
@CYBAI
Copy link
Collaborator

CYBAI commented Feb 25, 2020

web-platform-tests/wpt#21895 is not merged yet due to the syncing error 👀 Might need a manual merge.

@jdm
Copy link
Member

jdm commented Feb 25, 2020

Manually merged. Thanks!

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.

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