Bug 463491 - Do not reduce the popup blocker abuse level when using command event. r=jst

2011-08-11 18:31:33 +02:00 · 2011-08-11 18:31:33 +02:00 · a75399b568
--- a/content/events/src/nsDOMEvent.cpp
+++ b/content/events/src/nsDOMEvent.cpp
@ -984,9 +984,6 @@ nsDOMEvent::GetEventPopupControlState(nsEvent *aEvent)
        if (::PopupAllowedForEvent("change"))
          abuse = openControlled;
        break;
-      case NS_XUL_COMMAND:
-        abuse = openControlled;
-        break;
      }
    }
    break;
--- a/dom/tests/browser/Makefile.in
+++ b/dom/tests/browser/Makefile.in
@ -51,6 +51,7 @@ _BROWSER_FILES = \
 		browser_ConsoleAPITests.js \
 		test-console-api.html \
 		browser_autofocus_preference.js \
+		browser_popup_blocker_save_open_panel.js \
 		$(NULL)

 libs::	$(_BROWSER_FILES)
--- a/dom/tests/browser/browser_popup_blocker_save_open_panel.js
+++ b/dom/tests/browser/browser_popup_blocker_save_open_panel.js
@ -0,0 +1,101 @@
+/**
+ * In this test, we call the save panel with CTRL+S. When shown, we load a
+ * webpage that is going to open malicious popups. These popups should be
+ * disallowed.
+ */
+
+var gLoaded = false;
+
+Cc["@mozilla.org/moz/jssubscript-loader;1"]
+  .getService(Ci.mozIJSSubScriptLoader)
+  .loadSubScript("chrome://mochitests/content/browser/toolkit/content/tests/browser/common/mockObjects.js",
+                 this);
+
+function MockFilePicker() { }
+
+MockFilePicker.prototype = {
+  QueryInterface: XPCOMUtils.generateQI([Ci.nsIFilePicker]),
+
+  init: function() { },
+
+  appendFilters: function(val) { },
+  appendFilter: function(val) { },
+
+  // constants
+  modeOpen: 0,
+  modeSave: 1,
+  modeGetFolder: 2,
+  modeOpenMultiple: 3,
+  returnOK: 0,
+  returnCancel: 1,
+  returnReplace: 2,
+  filterAll: 1,
+  filterHTML: 2,
+  filterText: 4,
+  filterImages: 8,
+  filterXML: 16,
+  filterXUL: 32,
+  filterApps: 64,
+  filterAllowURLs: 128,
+  filterAudio: 256,
+  filterVideo: 512,
+
+  // properties
+  defaultExtension: "",
+  defaultString: "",
+  get displayDirectory() { return null; },
+  set displayDirectory(val) { },
+  file: null,
+  get files() { return null; },
+  get fileURL() { return null; },
+  filterIndex: 0,
+
+  show: function() {
+    gBrowser.selectedTab.linkedBrowser.addEventListener("load", function () {
+      gBrowser.selectedTab.linkedBrowser.removeEventListener("load", arguments.callee, true);
+      executeSoon(function() {
+        gLoaded = true;
+      });
+    }, true);
+
+    gBrowser.selectedTab.linkedBrowser.loadURI("data:text/html,<!DOCTYPE html><html><body onload='window.open(\"about:blank\", \"\", \"width=200,height=200\");'></body></html>");
+
+    let curThread = Components.classes["@mozilla.org/thread-manager;1"]
+                              .getService().currentThread;
+    while (!gLoaded) {
+      curThread.processNextEvent(true);
+    }
+
+    return this.returnCancel;
+  },
+};
+
+function test() {
+  waitForExplicitFinish();
+
+  var mockFilePickerRegisterer =
+    new MockObjectRegisterer("@mozilla.org/filepicker;1", MockFilePicker);
+  mockFilePickerRegisterer.register();
+
+  var prefs = Components.classes["@mozilla.org/preferences-service;1"]
+            .getService(Components.interfaces.nsIPrefBranch);
+  var gDisableLoadPref = prefs.getBoolPref("dom.disable_open_during_load");
+  prefs.setBoolPref("dom.disable_open_during_load", true);
+
+  gBrowser.addEventListener("DOMPopupBlocked", function() {
+    gBrowser.removeEventListener("DOMPopupBlocked", arguments.callee, true);
+    ok(true, "The popup has been blocked");
+    prefs.setBoolPref("dom.disable_open_during_load", gDisableLoadPref);
+
+    mockFilePickerRegisterer.unregister();
+
+    finish();
+  }, true)
+
+  if (navigator.platform.indexOf("Mac") == 0) {
+    // MacOS use metaKey instead of ctrlKey.
+    EventUtils.synthesizeKey("s", { metaKey: true, });
+  } else {
+    EventUtils.synthesizeKey("s", { ctrlKey: true, });
+  }
+}