Bug 1618539 - Remove content-process use of nsIFocusManager::GetActiveWindow. r=NeilDeakin

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

--HG--
extra : moz-landing-system : lando

browser/components/urlbar/tests/browser/browser_locationBarExternalLoad.js

@@ -76,14 +76,6 @@ async function testURL(url, loadFunc, endFunc) {
         null,
         "focusedElement not null"
       );
-
-      if (arg.isRemote) {
-        Assert.equal(
-          Services.focus.activeWindow,
-          content,
-          "activeWindow not correct"
-        );
-      }
     }
   );
 

dom/base/nsFocusManager.cpp

@@ -359,7 +359,8 @@ InputContextAction::Cause nsFocusManager::GetFocusMoveActionCause(
 
 NS_IMETHODIMP
 nsFocusManager::GetActiveWindow(mozIDOMWindowProxy** aWindow) {
-  // TODO mActiveWindow in content process
+  MOZ_ASSERT(XRE_IsParentProcess(),
+             "Must not be called outside the parent process.");
   NS_IF_ADDREF(*aWindow = mActiveWindow);
   return NS_OK;
 }

dom/base/test/chrome/bug418986-1.js

@@ -7,8 +7,10 @@ var test = function(isContent) {
     set: [["security.allow_eval_with_system_principal", true]],
   });
 
-  let { ww } = SpecialPowers.Services;
-  window.chromeWindow = ww.activeWindow;
+  if (!isContent) {
+    let { ww } = SpecialPowers.Services;
+    window.chromeWindow = ww.activeWindow;
+  }
 
   // The pairs of values expected to be the same when
   // fingerprinting resistance is enabled.
@@ -62,11 +64,11 @@ var test = function(isContent) {
         pairs.map(function([item, onVal]) {
           if (resisting) {
             checkPair("window." + item, onVal);
-          } else if (!item.startsWith("moz")) {
+          } else if (!isContent && !item.startsWith("moz")) {
             checkPair("window." + item, "chromeWindow." + item);
           }
         });
-        if (!resisting) {
+        if (!isContent && !resisting) {
           // Hard to predict these values, but we can enforce constraints:
           ok(
             window.mozInnerScreenX >= chromeWindow.mozInnerScreenX,

dom/interfaces/base/nsIFocusManager.idl

@@ -45,7 +45,7 @@ interface nsIFocusManager : nsISupports
 {
   /**
    * The most active (frontmost) window, or null if no window that is part of
-   * the application is active.
+   * the application is active. Do not use outside the parent process.
    */
   readonly attribute mozIDOMWindowProxy activeWindow;