HTML: the condition for opening a popup by window.open, and BarProp v…

…alues for each case For whatwg/html#5872 and whatwg/html#4431
arai-a · Oct 27, 2021 · 85ce509 · 85ce509
1 parent 0119dc6
commit 85ce509
Show file tree

Hide file tree

Showing 2 changed files with 155 additions and 0 deletions.
diff --git a/...r-creating-and-navigating-browsing-contexts-by-name/open-features-is-popup-condition.html b/...r-creating-and-navigating-browsing-contexts-by-name/open-features-is-popup-condition.html
@@ -0,0 +1,140 @@
+<!DOCTYPE html>
+<meta charset="utf-8">
+<title>HTML: window.open `features`: condition for is popup</title>
+<meta name="variant" content="?single-1">
+<meta name="variant" content="?single-2">
+<meta name="variant" content="?position">
+<meta name="variant" content="?combination">
+<meta name=timeout content=long>
+<link rel="help" href="https://html.spec.whatwg.org/multipage/window-object.html#window-open-steps">
+
+<!-- user agents are not required to support open features other than `noopener`
+     and on some platforms position and size features don't make sense -->
+<meta name="flags" content="may" />
+
+<script src="/resources/testharness.js"></script>
+<script src="/resources/testharnessreport.js"></script>
+<script src="/common/PrefixedPostMessage.js"></script>
+<script>
+var windowURL = 'resources/is-popup-barprop.html';
+
+var target = document.location.search.substring(1);
+
+// features, visible
+// NOTE: visible == !isPopup
+var tests = {
+  "single-1": [
+    // Empty feature results in non-popup.
+    [undefined, true],
+
+    // The explicit popup feature.
+    ["popup", false],
+    ["popup=1", false],
+    ["popup=0", true],
+
+    // Other feature alone results in popup.
+    ["location", false],
+    ["location=yes", false],
+    ["location=no", false],
+
+    ["toolbar", false],
+    ["toolbar=yes", false],
+    ["toolbar=no", false],
+
+    ["menubar", false],
+    ["menubar=yes", false],
+    ["menubar=no", false],
+
+    ["resizable", false],
+    ["resizable=yes", false],
+    ["resizable=no", false],
+  ],
+  "single-2": [
+    ["scrollbars", false],
+    ["scrollbars=yes", false],
+    ["scrollbars=no", false],
+
+    ["status", false],
+    ["status=yes", false],
+    ["status=no", false],
+
+    ["titlebar", false],
+    ["titlebar=yes", false],
+    ["titlebar=no", false],
+
+    ["close", false],
+    ["close=yes", false],
+    ["close=no", false],
+
+    ["minimizable", false],
+    ["minimizable=yes", false],
+    ["minimizable=no", false],
+
+    ["personalbar", false],
+    ["personalbar=yes", false],
+    ["personalbar=no", false],
+  ],
+  "position": [
+    ["left=500", false],
+    ["screenX=500", false],
+
+    ["top=500", false],
+    ["screenY=500", false],
+
+    ["width=500", false],
+    ["innerWidth=500", false],
+
+    ["outerWidth=500", false],
+
+    ["height=500", false],
+    ["innerHeight=500", false],
+
+    ["outerHeight=500", false],
+  ],
+  "combination": [
+    // The following combination results in non-popup.
+    ["location,toolbar,menubar,resizable,scrollbars,status", true],
+
+    // Either location or toolbar is required for non-popup.
+    ["location,menubar,resizable,scrollbars,status", true],
+    ["toolbar,menubar,resizable,scrollbars,status", true],
+
+    ["resizable,scrollbars,status", false],
+    ["location=no,menubar=no,resizable,scrollbars,status", false],
+
+    // menubar is required for non-popup.
+    ["location,toolbar,resizable,scrollbars,status", false],
+
+    // resizable is required for non-popup, but defaults to true
+    ["location,toolbar,menubar,scrollbars,status", true],
+    ["location,toolbar,menubar,resizable=no,scrollbars,status", false],
+
+    // scrollbars is required for non-popup.
+    ["location,toolbar,menubar,resizable,status", false],
+
+    // status is required for non-popup.
+    ["location,toolbar,menubar,resizable,scrollbars", false],
+
+    // The explicit popup feature has priority than others.
+    ["popup=1,location,toolbar,menubar,resizable,scrollbars,status", false],
+    ["popup=0,location,toolbar,menubar,resizable,scrollbars", true],
+  ],
+};
+
+tests[target].forEach(([features, visible]) => {
+  async_test(t => {
+    var prefixedMessage = new PrefixedMessageTest();
+    prefixedMessage.onMessage(t.step_func_done((data, e) => {
+      e.source.close();
+      assert_equals(data.locationbar, visible, `window.locationbar.visible`);
+      assert_equals(data.menubar, visible, `window.menubar.visible`);
+      assert_equals(data.personalbar, visible, `window.personalbar.visible`);
+      assert_equals(data.scrollbars, visible, `window.scrollbars.visible`);
+      assert_equals(data.statusbar, visible, `window.statusbar.visible`);
+      assert_equals(data.toolbar, visible, `window.toolbar.visible`);
+    }));
+    var win = window.open(prefixedMessage.url(windowURL), '', features);
+  }, `${format_value(features)} should set BarProp visibility to ${visible}`);
+});
+
+</script>
diff --git a/...pis-for-creating-and-navigating-browsing-contexts-by-name/resources/is-popup-barprop.html b/...pis-for-creating-and-navigating-browsing-contexts-by-name/resources/is-popup-barprop.html
@@ -0,0 +1,15 @@
+<script src="/common/PrefixedPostMessage.js"></script>
+<script>
+var prefixedMessage = new PrefixedMessageResource();
+function sendBarProps() {
+  prefixedMessage.postToOpener({
+    locationbar: window.locationbar.visible,
+    menubar: window.menubar.visible,
+    personalbar: window.personalbar.visible,
+    scrollbars: window.scrollbars.visible,
+    statusbar: window.statusbar.visible,
+    toolbar: window.toolbar.visible,
+  });
+}
+window.addEventListener('load', sendBarProps);
+</script>