[pull] master from web-platform-tests:master

css/css-fonts/WEB_FEATURES.yml

@@ -73,6 +73,12 @@ features:
   - crash-font-face-invalid-descriptor.html
   - web-font-no-longer-accessible-when-stylesheet-removed.html
   - test_datafont_same_origin.html
+- name: font-feature-settings
+  files:
+  - font-feature-settings-*
+  - font-default-03.html
+  - font-default-04.html
+  - variable-in-feature-crash.html
 - name: font-weight
   files:
   - font-weight-*

css/css-fonts/parsing/WEB_FEATURES.yml

@@ -52,3 +52,6 @@ features:
 - name: font-variant-position
   files:
   - font-variant-position-*
+- name: font-feature-settings
+  files:
+  - font-feature-settings-*

css/css-overflow/line-clamp/WEB_FEATURES.yml

@@ -0,0 +1,3 @@
+features:
+- name: line-clamp
+  files: "**"

css/css-overflow/parsing/WEB_FEATURES.yml

@@ -7,4 +7,8 @@ features:
   - scroll-button*
 - name: scrollbar-gutter
   files:
-  - scrollbar-gutter-*
+  - scrollbar-gutter-*
+- name: line-clamp
+  files:
+  - line-clamp-*
+  - webkit-line-clamp-*

css/css-typed-om/the-stylepropertymap/properties/WEB_FEATURES.yml

@@ -2,6 +2,9 @@ features:
 - name: speak
   files:
   - speak.html
+- name: font-feature-settings
+  files:
+  - font-feature-settings.html
 - name: font-variant
   files:
   - font-variant.html

document-picture-in-picture/base-uri.https.html

@@ -9,14 +9,6 @@
 promise_test(async (t) => {
   await test_driver.bless('request PiP window from top window');
   const pipWindow = await documentPictureInPicture.requestWindow();
-
-  if (pipWindow.document.readyState != "complete") {
-    // about:blank should load synchronous, but Gecko is still working on that...
-    // The initial placeholder is in many ways not initialized, e.g. doesn't have the right base
-    assert_true(true, "Waiting for pip window to load");
-    await new Promise(res => pipWindow.addEventListener("load", res, { once: true }));
-  }
-
   assert_equals(pipWindow.document.baseURI, document.baseURI, "Base URIs match");
 });
 </script>

document-picture-in-picture/display-mode.https.html

@@ -10,13 +10,6 @@
   await test_driver.bless('request PiP window');
   const pipWindow = await documentPictureInPicture.requestWindow();
 
-  if (pipWindow.document.readyState != "complete") {
-    // about:blank should load synchronous, but Gecko is still working on that...
-    // Ensure the async load doesn't blow away the iframe later on.
-    assert_true(true, "Waiting for pip window to load");
-    await new Promise(res => pipWindow.addEventListener("load", res, { once: true }));
-  }
-
   await new Promise(requestAnimationFrame);
   assert_true(
     !!pipWindow.matchMedia('(display-mode: picture-in-picture)'.matches),

document-picture-in-picture/focus-opener.https.html

@@ -9,16 +9,19 @@
   <iframe></iframe>
 <script>
 promise_test(async (t) => {
-  await test_driver.bless('request PiP window from top window');
-  const pipWindow = await documentPictureInPicture.requestWindow();
-
-  // Blur this window by focusing the iframe. This will allow us to detect that
-  // the document picture-in-picture window has focused us.
+  // Ensure this window is blurred so that we can detect that the
+  // document PiP window has focused us.
   const blurPromise = new Promise(async (resolve) => {
     window.addEventListener('blur', resolve, { once: true });
-    await test_driver.bless('focus inner iframe to blur window');
-    document.getElementsByTagName('iframe')[0].focus();
   });
+
+  await test_driver.bless('request PiP window from top window');
+  const pipWindow = await documentPictureInPicture.requestWindow();
+
+  // The PiP might be focused automatically.
+  // But to be safe that this window is blured, focus the iframe.
+  await test_driver.bless('focus inner iframe to blur window');
+  document.getElementsByTagName('iframe')[0].focus();
   await blurPromise;
 
   // Now focus this window from the document picture-in-picture window's

document-picture-in-picture/pip-fullscreen.tentative.https.html

@@ -28,13 +28,6 @@
   await test_driver.bless('request PiP window from top window');
   const pipWindow = await documentPictureInPicture.requestWindow();
 
-  if (pipWindow.document.readyState != "complete") {
-    // about:blank should load synchronous, but Gecko is still working on that...
-    // The initial placeholder is in many ways not initialized, e.g. doesn't have the right base
-    assert_true(true, "Waiting for pip window to load");
-    await new Promise(res => pipWindow.addEventListener("load", res, { once: true }));
-  }
-
   const fsResult = new Promise((res, rej) => {
     document.body.addEventListener('fullscreenchange', res);
     document.body.addEventListener('fullscreenerror', rej);

document-picture-in-picture/pip-resize.https.html

@@ -17,7 +17,7 @@
 
   await assert_throws_dom('NotAllowedError', pipWindow.DOMException, () =>
     pipWindow.resizeTo(pipWindow.outerWidth, pipWindow.outerHeight + 100)
-  , 'resizeTo requires user acivation');
+  , 'resizeTo requires user activation');
 
   await test_driver.bless('resize window');
   let resized = new Promise(res => pipWindow.addEventListener("resize", res, { once: true }));
@@ -38,7 +38,7 @@
 
   await assert_throws_dom('NotAllowedError', pipWindow.DOMException, () =>
     pipWindow.resizeBy(100, 0)
-  , 'resizeBy requires user acivation');
+  , 'resizeBy requires user activation');
 
   await test_driver.bless('resize window');
   let resized = new Promise(res => pipWindow.addEventListener("resize", res, { once: true }));

document-picture-in-picture/propagate-user-activation-from-opener.https.html

@@ -49,12 +49,6 @@
   await test_driver.bless('request PiP window');
   const pipWindow = await documentPictureInPicture.requestWindow();
 
-  if (pipWindow.document.readyState != "complete") {
-    // about:blank should load synchronous, but Gecko is still working on that...
-    assert_true(true, "Waiting for pip window to load");
-    await new Promise(res => pipWindow.addEventListener("load", res, { once: true }));
-  }
-
   const ifr = pipWindow.document.createElement("iframe");
   pipWindow.document.body.append(ifr);
 

document-picture-in-picture/propagate-user-activation-to-opener.https.html

@@ -4,20 +4,14 @@
 <script src="/resources/testharnessreport.js"></script>
 <script src="/resources/testdriver.js"></script>
 <script src="/resources/testdriver-vendor.js"></script>
+<script src="/common/get-host-info.sub.js"></script>
 <iframe id="same-origin-iframe" src="/common/blank.html"></iframe>
 <body>
 <script>
 promise_test(async (t) => {
   await test_driver.bless('request PiP window');
   const pipWindow = await documentPictureInPicture.requestWindow();
 
-  if (pipWindow.document.readyState != "complete") {
-    // about:blank should load synchronous, but Gecko is still working on that...
-    // The async load could blow away our document while blessing causing an error
-    assert_true(true, "Waiting for pip window to load");
-    await new Promise(res => pipWindow.addEventListener("load", res, { once: true }));
-  }
-
   assert_false(navigator.userActivation.isActive, 'the opener should initially not have user activation');
   assert_false(pipWindow.navigator.userActivation.isActive, 'the PiP window should initially not have user activation');
 
@@ -36,42 +30,26 @@
   await test_driver.bless('request PiP window');
   const pipWindow = await documentPictureInPicture.requestWindow();
 
-  if (pipWindow.document.readyState != "complete") {
-    // about:blank should load synchronous, but Gecko is still working on that...
-    // The async load could blow away our document while blessing causing an error
-    assert_true(true, "Waiting for pip window to load");
-    await new Promise(res => pipWindow.addEventListener("load", res, { once: true }));
-  }
-
   const ifr = pipWindow.document.createElement("iframe");
-  ifr.src = 'https://{{hosts[alt][www]}}:{{ports[https][0]}}/common/blank.html';
   pipWindow.document.body.append(ifr);
-  await new Promise(res => ifr.addEventListener('load', res, { once: true }));
 
   assert_false(navigator.userActivation.isActive, 'opener initially not active');
   assert_false(pipWindow.navigator.userActivation.isActive, 'PiP initially not active');
 
-  await test_driver.bless('activate cross-origin iframe', null, ifr.contentWindow);
+  await test_driver.bless('activate iframe', null, ifr.contentWindow);
 
   assert_true(navigator.userActivation.isActive, 'activation propagated to opener');
 
   pipWindow.open().close();
 
   assert_false(navigator.userActivation.isActive, 'activation was consumed in opener');
   assert_false(pipWindow.navigator.userActivation.isActive, 'activation was consumed in PiP');
-}, 'user activation propagates from cross-origin iframe in PiP to opener');
+}, 'user activation propagates from iframe in PiP to opener');
 
 promise_test(async (t) => {
 await test_driver.bless('request PiP window');
   const pipWindow = await documentPictureInPicture.requestWindow();
 
-  if (pipWindow.document.readyState != "complete") {
-    // about:blank should load synchronous, but Gecko is still working on that...
-    // The async load could blow away our document while blessing causing an error
-    assert_true(true, "Waiting for pip window to load");
-    await new Promise(res => pipWindow.addEventListener("load", res, { once: true }));
-  }
-
   const ifr = document.getElementById("same-origin-iframe");
 
   assert_false(navigator.userActivation.isActive, 'opener initially not active');
@@ -89,13 +67,6 @@
 await test_driver.bless('request PiP window');
   const pipWindow = await documentPictureInPicture.requestWindow();
 
-  if (pipWindow.document.readyState != "complete") {
-    // about:blank should load synchronous, but Gecko is still working on that...
-    // The async load could blow away our document while blessing causing an error
-    assert_true(true, "Waiting for pip window to load");
-    await new Promise(res => pipWindow.addEventListener("load", res, { once: true }));
-  }
-
   const ifr = document.getElementById("same-origin-iframe");
 
   assert_false(navigator.userActivation.isActive, 'opener initially not active');

document-picture-in-picture/returns-window-with-document.https.html

@@ -9,17 +9,29 @@
   <div id="div"></div>
 <script>
 
+function waitForEnter() {
+  return new Promise(resolve =>
+    documentPictureInPicture.addEventListener("enter", resolve, { once: true })
+  );
+}
+
 promise_test(async (t) => {
+  const enter = waitForEnter();
+
   await test_driver.bless('request PiP window');
   const pipWindow = await documentPictureInPicture.requestWindow();
 
   assert_true(!!pipWindow.document,
           'Window should contain a document');
   assert_true(documentPictureInPicture.window === pipWindow,
       'DocumentPictureInPicture.window should match the current window');
+
+  await enter;  // avoid issues in the next subtest
 }, 'requestWindow resolves with the PiP window');
 
 promise_test(async (t) => {
+  const enter = waitForEnter();
+
   await test_driver.bless('request PiP window');
   const pipWindow = await documentPictureInPicture.requestWindow();
   const div = document.getElementById('div');
@@ -34,12 +46,49 @@
       'The div should have moved away from the original document');
   assert_true(pipWindow.document.body.contains(div),
       'The div should have moved to the PiP document');
+
+  await enter;  // avoid issues in the next subtest
 }, 'Elements can be moved from opener to PiP document');
 
 promise_test(async (t) => {
+  const enter = waitForEnter();
+
   await test_driver.bless('request PiP window');
   const pipWindow = await documentPictureInPicture.requestWindow();
   assert_equals(pipWindow.document.readyState, "complete", "PiP document ready state is complete");
+
+  await enter;  // avoid issues in the next subtest
 }, 'PiP document is fully loaded');
+
+promise_test(async (t) => {
+  let events = [];
+
+  const enter = new Promise(resolve => {
+    documentPictureInPicture.onenter = () => {
+      events.push("enter");
+      resolve();
+    }
+  });
+
+  await test_driver.bless('request PiP window');
+  const request = documentPictureInPicture.requestWindow()
+    .then(() => {
+      events.push("requestWindow");
+    });
+
+    // Microtask to resolve request should already be queued
+  const microtask = Promise.resolve().then(() => events.push("microtask"));
+
+  assert_true(!!documentPictureInPicture.window, "window property should be set synchronous");
+
+  await Promise.all([enter, request, microtask]);
+
+  assert_equals(
+    events.join(),
+    "requestWindow,microtask,enter",
+    "Got the expected order of actions"
+  );
+
+}, "requestWindow timing");
 </script>
 </body>

document-picture-in-picture/support/popup-opens-pip.html

@@ -33,11 +33,6 @@
   if (data.type == 'request-pip') {
     pipWindow = await documentPictureInPicture.requestWindow();
 
-    if (pipWindow.document.readyState != "complete") {
-      // about:blank should load synchronous, but Gecko is still working on that...
-      await new Promise(res => pipWindow.addEventListener("load", res, { once: true }));
-    }
-
     const script = pipWindow.document.createElement('script');
     script.innerHTML = get_pip_code(data.channelName);
     pipWindow.document.body.append(script);

dom/collections/WEB_FEATURES.yml

@@ -0,0 +1,4 @@
+features:
+- name: dataset
+  files:
+  - domstringmap-supported-property-names.html

editing/crashtests/justifyfull-before-textarea-and-non-editable-invisible-br.html

@@ -0,0 +1,20 @@
+<!doctype html>
+<html>
+<head>
+<meta charset="utf-8">
+<script>
+"use strict";
+
+document.addEventListener("DOMContentLoaded", () => {
+  const dialog = document.querySelector("dialog");
+  dialog.close();
+  dialog.show();
+  getSelection().collapse(dialog, 1); // After the <hr>
+  document.execCommand("justifyFull");
+}, {once: true});
+</script>
+</head>
+<body>
+<dialog open contenteditable><hr><textarea>
+</textarea>
+<br contenteditable="false">

html/browsers/offline/browser-state/WEB_FEATURES.yml

@@ -0,0 +1,3 @@
+features:
+- name: online
+  files: "**"

html/dom/elements/global-attributes/WEB_FEATURES.yml

@@ -10,3 +10,8 @@ features:
 - name: title-attr
   files:
   - title-manual.html
+- name: dataset
+  files:
+  - dataset-*
+  - custom-attrs.html
+  - data_unicode_attr.html