From 53b704d204e99b00e45dc2b0d3bab1afd4e0e245 Mon Sep 17 00:00:00 2001
From: Edgar Chen <echen@mozilla.com>
Date: Tue, 31 May 2022 20:03:46 +0000
Subject: [PATCH] Do not forward focus to child frame when calling
 window.focus() on parent frame

Differential Revision: https://phabricator.services.mozilla.com/D146446

bugzilla-url: https://bugzilla.mozilla.org/show_bug.cgi?id=1757531
gecko-commit: c424ce3bbdf1f959575ad753feeceb97b97dd31b
gecko-reviewers: hsivonen
---
 ...lement-after-calling-window-focus.sub.html | 88 +++++++++++++++++++
 ...ment-after-calling-window-focus-inner.html | 38 ++++++++
 ...after-calling-window-focus-middle.sub.html | 30 +++++++
 ...ling-window-focus-outer-different.sub.html | 31 +++++++
 ...r-calling-window-focus-outer-same.sub.html | 31 +++++++
 5 files changed, 218 insertions(+)
 create mode 100644 focus/activeelement-after-calling-window-focus.sub.html
 create mode 100644 focus/support/activeelement-after-calling-window-focus-inner.html
 create mode 100644 focus/support/activeelement-after-calling-window-focus-middle.sub.html
 create mode 100644 focus/support/activeelement-after-calling-window-focus-outer-different.sub.html
 create mode 100644 focus/support/activeelement-after-calling-window-focus-outer-same.sub.html
diff --git a/focus/activeelement-after-calling-window-focus.sub.html b/focus/activeelement-after-calling-window-focus.sub.html
new file mode 100644
index 00000000000000..f854ffb36ce462
--- /dev/null
+++ b/focus/activeelement-after-calling-window-focus.sub.html
@@ -0,0 +1,88 @@
+<!doctype html>
+<meta charset=utf-8>
+<title>activeElement after calling window.focus()</title>
+<meta name="timeout" content="long">
+<script src=/resources/testharness.js></script>
+<script src=/resources/testharnessreport.js></script>
+<script>
+function waitForEvent(target, event, checkFn) {
+  return new Promise(resolve => {
+    target.addEventListener(event, e => {
+      if (checkFn && !checkFn(e)) {
+        return;
+      }
+      resolve();
+    }, { once: true });
+  });
+}
+
+function focusInnerInput(w) {
+  w.postMessage("focusinnerinput", "*");
+}
+
+function focusInnerFrame(w) {
+  w.postMessage("focusinner", "*");
+}
+
+function focusMiddleFrame(w) {
+  w.postMessage("focusmiddle", "*");
+}
+
+function focusOuterFrame(w) {
+  w.postMessage("focusouter", "*");
+}
+
+// This will send message to outer frame and also inner frame to ask them
+// send the log they collect back, the logs of outer and inner will be
+// concatenated.
+async function getLog(w) {
+  let log = "";
+  step_timeout(function() {
+    w.postMessage("getlog", "*");
+  }, 0);
+  await waitForEvent(window, "message", (e) => {
+    log = e.data;
+    return true;
+  });
+  return log;
+}
+
+async function runTest(t, url) {
+  let w = window.open(url);
+  t.add_cleanup(() => { w.close(); });
+  await waitForEvent(window, "message", e => e.data === "ready");
+  // Calling input.focus() on inner iframe should move its document.activeElement to INPUT.
+  focusInnerInput(w);
+  assert_equals(await getLog(w), 'outerlog:windowblur,middlelog:innerlog:windowfocus,INPUT,');
+  // Calling window.focus() on inner iframe should NOT reset its document.activeElement to BODY
+  // because it's focused element is not an iframe.
+  focusInnerFrame(w);
+  assert_equals(await getLog(w), 'outerlog:windowblur,middlelog:innerlog:windowfocus,INPUT,INPUT,');
+  // Calling window.focus() on middle iframe should reset its document.activeElement to BODY.
+  focusMiddleFrame(w);
+  assert_equals(await getLog(w), 'outerlog:windowblur,middlelog:windowfocus,BODY,innerlog:windowfocus,INPUT,INPUT,windowblur,');
+  // Calling window.focus() on top-level frame should reset its document.activeElement to BODY.
+  focusOuterFrame(w);
+  assert_equals(await getLog(w), 'outerlog:windowblur,windowfocus,BODY,middlelog:windowfocus,BODY,windowblur,innerlog:windowfocus,INPUT,INPUT,windowblur,');
+}
+
+promise_test(async t => {
+  await runTest(t, "https://{{hosts[][www]}}:{{ports[https][0]}}/focus/support/activeelement-after-calling-window-focus-outer-same.sub.html");
+}, "Tests for all frames are in same origin");
+
+promise_test(async t => {
+  await runTest(t, "https://{{hosts[alt][www]}}:{{ports[https][0]}}/focus/support/activeelement-after-calling-window-focus-outer-same.sub.html");
+}, "Tests for middle frame and inner frame are in same origin and outer frame is in different origin");
+
+promise_test(async t => {
+  await runTest(t, "https://{{hosts[alt][www]}}:{{ports[https][0]}}/focus/support/activeelement-after-calling-window-focus-outer-different.sub.html");
+}, "Tests for outer frame and middle frame are in same origin and inner frame is in different origin");
+
+promise_test(async t => {
+  await runTest(t, "https://{{hosts[][www]}}:{{ports[https][0]}}/focus/support/activeelement-after-calling-window-focus-outer-different.sub.html");
+}, "Tests for outer frame and inner frame are in same origin and middle frame is in different origin");
+
+promise_test(async t => {
+  await runTest(t, "support/activeelement-after-calling-window-focus-outer-different.sub.html");
+}, "Tests for all frames are in different origin");
+</script>
diff --git a/focus/support/activeelement-after-calling-window-focus-inner.html b/focus/support/activeelement-after-calling-window-focus-inner.html
new file mode 100644
index 00000000000000..1fa14d4b8a24de
--- /dev/null
+++ b/focus/support/activeelement-after-calling-window-focus-inner.html
@@ -0,0 +1,38 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <meta charset="utf-8">
+  <title>Focus test inner document</title>
+</head>
+<body>
+<h1>Inner</h1>
+<input>
+<script>
+let innerlog = "innerlog:";
+
+window.onmessage = function(e) {
+  if (e.data == "focusinnerinput") {
+    document.querySelector("input").focus();
+    innerlog += document.activeElement.tagName + ",";
+  } else if (e.data == "focusinner") {
+    window.focus();
+    innerlog += document.activeElement.tagName + ",";
+  } else if (e.data == "getlog") {
+    parent.postMessage(innerlog, "*");
+  }
+};
+
+window.onfocus = function() {
+  innerlog += "windowfocus,";
+};
+
+window.onblur = function() {
+  innerlog += "windowblur,";
+};
+
+window.onload = function() {
+  parent.postMessage("ready", "*");
+}
+</script>
+</body>
+</html>
diff --git a/focus/support/activeelement-after-calling-window-focus-middle.sub.html b/focus/support/activeelement-after-calling-window-focus-middle.sub.html
new file mode 100644
index 00000000000000..c5a0d60a71de37
--- /dev/null
+++ b/focus/support/activeelement-after-calling-window-focus-middle.sub.html
@@ -0,0 +1,30 @@
+<!doctype html>
+<meta charset="utf-8">
+<title>Focus test middle document</title>
+<h1>Middle</h1><br>
+<iframe src="https://{{hosts[][www]}}:{{ports[https][0]}}/focus/support/activeelement-after-calling-window-focus-inner.html"></iframe>
+<script>
+let middlelog = "middlelog:";
+
+let iframe = document.querySelector("iframe");
+window.onmessage = function(e) {
+  if (e.data == "ready") {
+    parent.postMessage(e.data, "*");
+  } else if (e.data == "focusinnerinput" || e.data == "focusinner" || e.data == "getlog") {
+    iframe.contentWindow.postMessage(e.data, "*");
+  } else if (e.data == "focusmiddle") {
+    window.focus();
+    middlelog += document.activeElement.tagName + ",";
+  } else {
+    parent.postMessage(middlelog + e.data, "*");
+  }
+};
+
+window.onfocus = function() {
+  middlelog += "windowfocus,";
+};
+
+window.onblur = function() {
+  middlelog += "windowblur,";
+};
+</script>
diff --git a/focus/support/activeelement-after-calling-window-focus-outer-different.sub.html b/focus/support/activeelement-after-calling-window-focus-outer-different.sub.html
new file mode 100644
index 00000000000000..0c2f8f792da461
--- /dev/null
+++ b/focus/support/activeelement-after-calling-window-focus-outer-different.sub.html
@@ -0,0 +1,31 @@
+<!doctype html>
+<meta charset="utf-8">
+<title>Focus test outer document</title>
+<iframe src="https://{{hosts[alt][www]}}:{{ports[https][0]}}/focus/support/activeelement-after-calling-window-focus-middle.sub.html"></iframe>
+<script>
+let outerlog = "outerlog:";
+
+let iframe = document.querySelector("iframe");
+window.onmessage = function(e) {
+   if (e.data == "ready") {
+    opener.postMessage(e.data, "*");
+  } else if (e.data == "focusinnerinput" || e.data == "focusinner" || e.data == "focusmiddle" || e.data == "getlog") {
+    iframe.contentWindow.postMessage(e.data, "*");
+  } else if (e.data == "focusouter") {
+    window.focus();
+    outerlog += document.activeElement.tagName + ",";
+  } else {
+    opener.postMessage(outerlog + e.data, "*");
+  }
+};
+
+window.onload = function() {
+  window.onfocus = function() {
+    outerlog += "windowfocus,";
+  };
+};
+
+window.onblur = function() {
+  outerlog += "windowblur,";
+};
+</script>
diff --git a/focus/support/activeelement-after-calling-window-focus-outer-same.sub.html b/focus/support/activeelement-after-calling-window-focus-outer-same.sub.html
new file mode 100644
index 00000000000000..cabdb26b347e05
--- /dev/null
+++ b/focus/support/activeelement-after-calling-window-focus-outer-same.sub.html
@@ -0,0 +1,31 @@
+<!doctype html>
+<meta charset="utf-8">
+<title>Focus test outer document</title>
+<iframe src="https://{{hosts[][www]}}:{{ports[https][0]}}/focus/support/activeelement-after-calling-window-focus-middle.sub.html"></iframe>
+<script>
+let outerlog = "outerlog:";
+
+let iframe = document.querySelector("iframe");
+window.onmessage = function(e) {
+   if (e.data == "ready") {
+    opener.postMessage(e.data, "*");
+  } else if (e.data == "focusinnerinput" || e.data == "focusinner" || e.data == "focusmiddle" || e.data == "getlog") {
+    iframe.contentWindow.postMessage(e.data, "*");
+  } else if (e.data == "focusouter") {
+    window.focus();
+    outerlog += document.activeElement.tagName + ",";
+  } else {
+    opener.postMessage(outerlog + e.data, "*");
+  }
+};
+
+window.onload = function() {
+  window.onfocus = function() {
+    outerlog += "windowfocus,";
+  };
+};
+
+window.onblur = function() {
+  outerlog += "windowblur,";
+};
+</script>
