From 89263f03b422c87c5e2842707a38658253022d5a Mon Sep 17 00:00:00 2001 From: wpt-sync Date: Tue, 5 Dec 2017 19:47:10 +0000 Subject: [PATCH 01/26] Fix race in wpt /wasm/wasm_local_iframe_test.html () bugzilla-url: https://bugzilla-dev.allizom.org/show_bug.cgi?id=Bug 1383683 gecko-commit: e219b2b152d795da623814d7aeae3a89b1d2726e gecko-integration-branch: central gecko-reviewers: bz --- wasm/wasm_local_iframe_test.html | 24 +++++++++++++----------- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/wasm/wasm_local_iframe_test.html b/wasm/wasm_local_iframe_test.html index 9ccebdc75394a1..0f4fbd0c7191ec 100644 --- a/wasm/wasm_local_iframe_test.html +++ b/wasm/wasm_local_iframe_test.html @@ -2,16 +2,18 @@ - + From f8f63e252b375635e242bc4a2ccba9dd387424ee Mon Sep 17 00:00:00 2001 From: wpt-sync Date: Tue, 5 Dec 2017 20:11:25 +0000 Subject: [PATCH 02/26] part 1 - Add internal overflow-clip-box-block/-inline properties and make overflow-clip-box a shorthand. bugzilla-url: https://bugzilla-dev.allizom.org/show_bug.cgi?id=Bug 1422839 gecko-commit: cf93ccd03ca7d452aa1e1e9bff575f0992df7874 gecko-integration-branch: autoland gecko-reviewers: dholbert --- .../animation-model/animation-types/property-list.js | 6 ------ 1 file changed, 6 deletions(-) diff --git a/web-animations/animation-model/animation-types/property-list.js b/web-animations/animation-model/animation-types/property-list.js index 46829c31495400..32bf63d3a82175 100644 --- a/web-animations/animation-model/animation-types/property-list.js +++ b/web-animations/animation-model/animation-types/property-list.js @@ -1054,12 +1054,6 @@ const gCSSProperties = { types: [ ] }, - 'overflow-clip-box': { - // https://developer.mozilla.org/en/docs/Web/CSS/overflow-clip-box - types: [ - { type: 'discrete', options: [ [ 'padding-box', 'content-box' ] ] } - ] - }, 'overflow-wrap': { // https://drafts.csswg.org/css-text-3/#propdef-overflow-wrap types: [ From 79e857b55d76794edde1ff15420c2fcf9cff3725 Mon Sep 17 00:00:00 2001 From: wpt-sync Date: Thu, 7 Dec 2017 16:24:20 +0000 Subject: [PATCH 03/26] P5 Fix race in about-blank-replacement.https.html popup test case. bugzilla-url: https://bugzilla-dev.allizom.org/show_bug.cgi?id=Bug 1423412 gecko-commit: 7750a46652fa83acc6b8cd3e3bee8f428d24bfd5 gecko-integration-branch: central gecko-reviewers: baku --- .../resources/about-blank-replacement-popup-frame.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/service-workers/service-worker/resources/about-blank-replacement-popup-frame.py b/service-workers/service-worker/resources/about-blank-replacement-popup-frame.py index be5294539833c7..f0b8cd578e3e05 100644 --- a/service-workers/service-worker/resources/about-blank-replacement-popup-frame.py +++ b/service-workers/service-worker/resources/about-blank-replacement-popup-frame.py @@ -12,12 +12,15 @@ def main(request, response): ") + session.url = inline("" % server_config["domains"][""]) result = session.transport.send("GET", "session/%s/cookie" % session.session_id) assert result.status == 200 assert "value" in result.body From a59f44649f1beef175896409cb657e5a88f26961 Mon Sep 17 00:00:00 2001 From: wpt-sync Date: Thu, 7 Dec 2017 18:20:21 +0000 Subject: [PATCH 06/26] Move element retrieval WPT tests to single directory. bugzilla-url: https://bugzilla-dev.allizom.org/show_bug.cgi?id=Bug 1421675 gecko-commit: 4bdcd7f6e1978b4769629208fb479d93a859018a gecko-integration-branch: central gecko-reviewers: maja_zf --- webdriver/tests/{retrieval => element_retrieval}/__init__.py | 0 webdriver/tests/{retrieval => element_retrieval}/find_element.py | 0 .../{retrieval => element_retrieval}/find_element_from_element.py | 0 .../find_element_from_elements.py | 0 webdriver/tests/{retrieval => element_retrieval}/find_elements.py | 0 5 files changed, 0 insertions(+), 0 deletions(-) rename webdriver/tests/{retrieval => element_retrieval}/__init__.py (100%) rename webdriver/tests/{retrieval => element_retrieval}/find_element.py (100%) rename webdriver/tests/{retrieval => element_retrieval}/find_element_from_element.py (100%) rename webdriver/tests/{retrieval => element_retrieval}/find_element_from_elements.py (100%) rename webdriver/tests/{retrieval => element_retrieval}/find_elements.py (100%) diff --git a/webdriver/tests/retrieval/__init__.py b/webdriver/tests/element_retrieval/__init__.py similarity index 100% rename from webdriver/tests/retrieval/__init__.py rename to webdriver/tests/element_retrieval/__init__.py diff --git a/webdriver/tests/retrieval/find_element.py b/webdriver/tests/element_retrieval/find_element.py similarity index 100% rename from webdriver/tests/retrieval/find_element.py rename to webdriver/tests/element_retrieval/find_element.py diff --git a/webdriver/tests/retrieval/find_element_from_element.py b/webdriver/tests/element_retrieval/find_element_from_element.py similarity index 100% rename from webdriver/tests/retrieval/find_element_from_element.py rename to webdriver/tests/element_retrieval/find_element_from_element.py diff --git a/webdriver/tests/retrieval/find_element_from_elements.py b/webdriver/tests/element_retrieval/find_element_from_elements.py similarity index 100% rename from webdriver/tests/retrieval/find_element_from_elements.py rename to webdriver/tests/element_retrieval/find_element_from_elements.py diff --git a/webdriver/tests/retrieval/find_elements.py b/webdriver/tests/element_retrieval/find_elements.py similarity index 100% rename from webdriver/tests/retrieval/find_elements.py rename to webdriver/tests/element_retrieval/find_elements.py From 8ba9bef8ef6ebedca600042557c3edebd4230045 Mon Sep 17 00:00:00 2001 From: wpt-sync Date: Thu, 7 Dec 2017 18:17:44 +0000 Subject: [PATCH 07/26] Fix test failures. bugzilla-url: https://bugzilla-dev.allizom.org/show_bug.cgi?id=Bug 1037335 gecko-commit: 7678ac4b15adf33769ce558814fe44f84447ebe6 gecko-integration-branch: central gecko-reviewers: ckerschb --- content-security-policy/securitypolicyviolation/idl.html | 3 +++ 1 file changed, 3 insertions(+) diff --git a/content-security-policy/securitypolicyviolation/idl.html b/content-security-policy/securitypolicyviolation/idl.html index 17f492e7344de6..a4f9e68f8314db 100644 --- a/content-security-policy/securitypolicyviolation/idl.html +++ b/content-security-policy/securitypolicyviolation/idl.html @@ -21,6 +21,9 @@ long lineNumber; long columnNumber; }; + + interface Event { + }; -
From 6be87daac58455f0e7ffe7f6c24459c9728668a6 Mon Sep 17 00:00:00 2001 From: wpt-sync Date: Thu, 7 Dec 2017 18:21:09 +0000 Subject: [PATCH 09/26] Remove duplicated WebDriver WebIDL test. This test is duplicated in testing/web-platform/tests/webdriver/tests/interface.html. bugzilla-url: https://bugzilla-dev.allizom.org/show_bug.cgi?id=Bug 1421676 gecko-commit: ad8f6896b87dfabdbbf0d6db28bdfaaf16158f6a gecko-integration-branch: central gecko-reviewers: maja_zf --- webdriver/interface/interface.html | 40 ------------------------------ 1 file changed, 40 deletions(-) delete mode 100644 webdriver/interface/interface.html diff --git a/webdriver/interface/interface.html b/webdriver/interface/interface.html deleted file mode 100644 index f3c149b1cee06b..00000000000000 --- a/webdriver/interface/interface.html +++ /dev/null @@ -1,40 +0,0 @@ - - - - - - - - - From 229d087a1f14d2df3e5df3336c3fc5c70737ac69 Mon Sep 17 00:00:00 2001 From: moz-wptsync-bot Date: Thu, 11 Jan 2018 12:12:35 +0000 Subject: [PATCH 10/26] Remove animationiteration from legacy event listener test, make the transition tests more robust, and re-enable that test. According to :birtles, it is not guaranteed that animationiteration event will be fired. This event is sample-based rather than event-based, and such behavior has been clarified in CSS Animations Level 2: https://drafts.csswg.org/css-animations-2/#event-dispatch Also, Chromium has the same issue with this test: https://bugs.chromium.org/p/chromium/issues/detail?id=701445 bugzilla-url: https://bugzilla-dev.allizom.org/show_bug.cgi?id=1351409 gecko-commit: fa855df2e888bb6edbed237d7a6b78ab6476c96b gecko-integration-branch: central gecko-reviewers: jmaher --- dom/events/EventListener-invoke-legacy.html | 14 ++++---------- 1 file changed, 4 insertions(+), 10 deletions(-) diff --git a/dom/events/EventListener-invoke-legacy.html b/dom/events/EventListener-invoke-legacy.html index 85a4b0a5fe64ef..a01afcd8d1985c 100644 --- a/dom/events/EventListener-invoke-legacy.html +++ b/dom/events/EventListener-invoke-legacy.html @@ -51,22 +51,16 @@ } function setupTransition(elem) { - elem.style.transition = ''; - requestAnimationFrame(function() { - elem.style.color = 'red'; - elem.style.transition = 'color 30ms'; - requestAnimationFrame(function() { - elem.style.color = 'green'; - }); - }); + getComputedStyle(elem).color; + elem.style.color = 'green'; + elem.style.transition = 'color 30ms'; } function setupAnimation(elem) { - elem.style.animation = 'test 30ms 2'; + elem.style.animation = 'test 30ms'; } runLegacyEventTest('transitionend', 'webkitTransitionEnd', "TransitionEvent", setupTransition); runLegacyEventTest('animationend', 'webkitAnimationEnd', "AnimationEvent", setupAnimation); -runLegacyEventTest('animationiteration', 'webkitAnimationIteration', "AnimationEvent", setupAnimation); runLegacyEventTest('animationstart', 'webkitAnimationStart', "AnimationEvent", setupAnimation); From 653d2fb82095855042cc334076417c374c461683 Mon Sep 17 00:00:00 2001 From: moz-wptsync-bot Date: Thu, 11 Jan 2018 12:12:59 +0000 Subject: [PATCH 11/26] Handle errors for inline module scripts and ensure we update the module map after fetch errors bugzilla-url: https://bugzilla-dev.allizom.org/show_bug.cgi?id=1426140 gecko-commit: a8f97458d0cc43a62b048a5ebb4a8b1663d18b20 gecko-integration-branch: central gecko-reviewers: baku --- .../the-script-element/module/load-error-events-inline.html | 1 - 1 file changed, 1 deletion(-) diff --git a/html/semantics/scripting-1/the-script-element/module/load-error-events-inline.html b/html/semantics/scripting-1/the-script-element/module/load-error-events-inline.html index c4dd0807f62a62..58397dd07d48b4 100644 --- a/html/semantics/scripting-1/the-script-element/module/load-error-events-inline.html +++ b/html/semantics/scripting-1/the-script-element/module/load-error-events-inline.html @@ -59,4 +59,3 @@ - From 90466ecb204beff2d11578a71e4181c3253022e5 Mon Sep 17 00:00:00 2001 From: moz-wptsync-bot Date: Thu, 11 Jan 2018 12:12:48 +0000 Subject: [PATCH 12/26] P1: Update the wpt tests to expect to reject the cors synthesized response to a same origin request. bugzilla-url: https://bugzilla-dev.allizom.org/show_bug.cgi?id=1427978 gecko-commit: e3edde0c2242804300ebd96b0a0f648ace682688 gecko-integration-branch: central gecko-reviewers: bkelly --- .../service-worker/fetch-response-taint.https.html | 6 ++++++ .../service-worker/worker-interception.https.html | 12 +++++++++--- 2 files changed, 15 insertions(+), 3 deletions(-) diff --git a/service-workers/service-worker/fetch-response-taint.https.html b/service-workers/service-worker/fetch-response-taint.https.html index a6e7f984ea2449..8ebee0c07374c3 100644 --- a/service-workers/service-worker/fetch-response-taint.https.html +++ b/service-workers/service-worker/fetch-response-taint.https.html @@ -193,6 +193,9 @@ // Fetch to the other origin with same-origin mode should fail. if (origin == OTHER_ORIGIN && mode == 'same-origin') { ng_test(url, mode, credentials); + } else if (origin == BASE_ORIGIN && mode == 'same-origin') { + // Cors type response to a same-origin mode request should fail + ng_test(url, mode, credentials); } else { // The response from the SW should be cors. ok_test(url, mode, credentials, 'cors', 'undefined'); @@ -208,6 +211,9 @@ // Fetch to the other origin with same-origin mode should fail. if (origin == OTHER_ORIGIN && mode == 'same-origin') { ng_test(url, mode, credentials); + } else if (origin == BASE_ORIGIN && mode == 'same-origin') { + // Cors type response to a same-origin mode request should fail + ng_test(url, mode, credentials); } else { // The response from the SW should be cors. ok_test(url, mode, credentials, 'cors', 'username1s'); diff --git a/service-workers/service-worker/worker-interception.https.html b/service-workers/service-worker/worker-interception.https.html index 3ec66a54b6e1e5..4f14746d917f9e 100644 --- a/service-workers/service-worker/worker-interception.https.html +++ b/service-workers/service-worker/worker-interception.https.html @@ -81,10 +81,16 @@ }); }) .then(function(data) { - assert_equals(data, 'dummy-worker-script loaded'); + assert_unreached('intercepted cors response to a same-origin mode ' + + 'worker load should fail'); service_worker_unregister_and_done(t, scope); - }); - }, 'Verify worker script intercepted by cors response succeeds'); + }) + .catch(function(e) { + assert_true(true, 'intercepted cors response to a same-origin mode ' + + 'worker load should fail'); + service_worker_unregister_and_done(t, scope); + }); + }, 'Verify worker script intercepted by cors response fails'); promise_test(function(t) { var worker_url = 'resources/dummy-no-cors-worker.js'; From 980addd1a04de0da2953a598f8692fa1e3910dd3 Mon Sep 17 00:00:00 2001 From: moz-wptsync-bot Date: Thu, 11 Jan 2018 12:13:49 +0000 Subject: [PATCH 13/26] Fix test code that makes incorrect assumptions about when scripts are run bugzilla-url: https://bugzilla-dev.allizom.org/show_bug.cgi?id=1426675 gecko-commit: afef4f6c98755643a56ad15e39b3ac885877eb7d gecko-integration-branch: central gecko-reviewers: jgraham --- .../the-script-element/module/instantiation-error-3.html | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/html/semantics/scripting-1/the-script-element/module/instantiation-error-3.html b/html/semantics/scripting-1/the-script-element/module/instantiation-error-3.html index ab510c675ad170..27e50f7fd69537 100644 --- a/html/semantics/scripting-1/the-script-element/module/instantiation-error-3.html +++ b/html/semantics/scripting-1/the-script-element/module/instantiation-error-3.html @@ -26,4 +26,4 @@ + onerror="unreachable()" onload="log.push(3)"> From bec3819ff8abe11c9fe93c9c1782d11a143653c0 Mon Sep 17 00:00:00 2001 From: moz-wptsync-bot Date: Thu, 11 Jan 2018 12:14:01 +0000 Subject: [PATCH 14/26] Fix test that didn't take account of module scripts' defer scheduling bugzilla-url: https://bugzilla-dev.allizom.org/show_bug.cgi?id=1426642 gecko-commit: f6b74e7c3c0943a212e0d2efd73340b8d2157dbb gecko-integration-branch: central gecko-reviewers: jgraham --- .../the-script-element/module/module-in-xhtml.xhtml | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/html/semantics/scripting-1/the-script-element/module/module-in-xhtml.xhtml b/html/semantics/scripting-1/the-script-element/module/module-in-xhtml.xhtml index 724c9e750743f8..1655e61a7c54e3 100644 --- a/html/semantics/scripting-1/the-script-element/module/module-in-xhtml.xhtml +++ b/html/semantics/scripting-1/the-script-element/module/module-in-xhtml.xhtml @@ -10,7 +10,11 @@ window.evaluated_module_script = true; From da026901fc528f4dc40168de3d457e2604f44191 Mon Sep 17 00:00:00 2001 From: moz-wptsync-bot Date: Thu, 11 Jan 2018 12:14:15 +0000 Subject: [PATCH 15/26] Fix incorrect custom element type in CustomElementData. f=echen, bugzilla-url: https://bugzilla-dev.allizom.org/show_bug.cgi?id=1419662 gecko-commit: 48bb9add84a85b085b2739d37798086fc6fd91d9 gecko-integration-branch: central gecko-reviewers: smaug --- custom-elements/Document-createElement.html | 15 ++++++ ...er-constructs-custom-elements-with-is.html | 51 +++++++++++++++++++ custom-elements/upgrading/Node-cloneNode.html | 19 +++++++ 3 files changed, 85 insertions(+) create mode 100644 custom-elements/parser/parser-constructs-custom-elements-with-is.html diff --git a/custom-elements/Document-createElement.html b/custom-elements/Document-createElement.html index e446c507ca865a..52a68e8e178976 100644 --- a/custom-elements/Document-createElement.html +++ b/custom-elements/Document-createElement.html @@ -29,6 +29,21 @@ }, 'document.createElement must create an instance of custom elements'); +test(function () { + class AutonomousCustomElement extends HTMLElement {}; + class IsCustomElement extends HTMLElement {}; + + customElements.define('autonomous-custom-element', AutonomousCustomElement); + customElements.define('is-custom-element', IsCustomElement); + + var instance = document.createElement('autonomous-custom-element', { is: "is-custom-element"}); + + assert_true(instance instanceof AutonomousCustomElement); + assert_equals(instance.localName, 'autonomous-custom-element'); + assert_equals(instance.namespaceURI, 'http://www.w3.org/1999/xhtml', 'A custom element HTML must use HTML namespace'); + +}, 'document.createElement must create an instance of autonomous custom elements when it has is attribute'); + function assert_reports(expected, testFunction, message) { var uncaughtError = null; window.onerror = function (message, url, lineNumber, columnNumber, error) { uncaughtError = error; return true; } diff --git a/custom-elements/parser/parser-constructs-custom-elements-with-is.html b/custom-elements/parser/parser-constructs-custom-elements-with-is.html new file mode 100644 index 00000000000000..96c00278a3d7a0 --- /dev/null +++ b/custom-elements/parser/parser-constructs-custom-elements-with-is.html @@ -0,0 +1,51 @@ + + + +Custom Elements: Changes to the HTML parser + + + + + + + + +
+ + + + + + diff --git a/custom-elements/upgrading/Node-cloneNode.html b/custom-elements/upgrading/Node-cloneNode.html index 0d158fd59e4863..0492e1f39a131d 100644 --- a/custom-elements/upgrading/Node-cloneNode.html +++ b/custom-elements/upgrading/Node-cloneNode.html @@ -30,6 +30,25 @@ 'A cloned custom element must be an instance of the custom element'); }, 'Node.prototype.cloneNode(false) must be able to clone a custom element'); +test(function () { + class AutonomousCustomElement extends HTMLElement {}; + class IsCustomElement extends HTMLElement {}; + + customElements.define('autonomous-custom-element', AutonomousCustomElement); + customElements.define('is-custom-element', IsCustomElement); + + var instance = document.createElement('autonomous-custom-element', { is: "is-custom-element"}); + assert_true(instance instanceof HTMLElement); + assert_true(instance instanceof AutonomousCustomElement); + + var clone = instance.cloneNode(false); + assert_not_equals(instance, clone); + assert_true(clone instanceof HTMLElement, + 'A cloned custom element must be an instance of HTMLElement'); + assert_true(clone instanceof AutonomousCustomElement, + 'A cloned custom element must be an instance of the custom element'); +}, 'Node.prototype.cloneNode(false) must be able to clone as a autonomous custom element when it contains is attribute'); + test_with_window(function (contentWindow) { var contentDocument = contentWindow.document; class MyCustomElement extends contentWindow.HTMLElement {} From 728ba2842e9092a2ddb6cf7e7e8a93680842ec59 Mon Sep 17 00:00:00 2001 From: moz-wptsync-bot Date: Thu, 11 Jan 2018 12:14:27 +0000 Subject: [PATCH 16/26] Fix inheritance of animation and transition properties of mismatched length. At least when the animation-name length is bigger than the animation properties, we mess up inheritance and only set properly the specified counts, then don't cycle it. The nicer fix for this is making these vectors properly, and move the cycling logic at used-value time (bug 1420928). Same for transitions. bugzilla-url: https://bugzilla-dev.allizom.org/show_bug.cgi?id=1426246 gecko-commit: 064e0be1cc29617de74098aba4b3bb706775315a gecko-integration-branch: central gecko-reviewers: hiro --- css/css-animations/animation-delay-011.html | 24 +++++++++++++++++++++ 1 file changed, 24 insertions(+) create mode 100644 css/css-animations/animation-delay-011.html diff --git a/css/css-animations/animation-delay-011.html b/css/css-animations/animation-delay-011.html new file mode 100644 index 00000000000000..415a5747553797 --- /dev/null +++ b/css/css-animations/animation-delay-011.html @@ -0,0 +1,24 @@ + + +CSS Animations Test: inherited animation-delay with mismatched animation-name length + + + + +
From 7add142c339b1c438bfe720b70c58edcd4b70e9b Mon Sep 17 00:00:00 2001 From: moz-wptsync-bot Date: Thu, 11 Jan 2018 12:14:39 +0000 Subject: [PATCH 17/26] Stop toggling selectedness state of