Bug 1496191 - When test_big_icon is listening for search engine modifications, ensure it QIs the engine to have the proper interface information. r=mikedeboer

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

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

toolkit/components/search/tests/xpcshell/test_big_icon.js

@@ -9,7 +9,10 @@ add_task(async function test_big_icon() {
   await asyncInit();
 
   let promiseChanged = TestUtils.topicObserved("browser-search-engine-modified",
-    (engine, verb) => verb == "engine-changed" && engine.name == "BigIcon" && engine.iconURI);
+    (engine, verb) => {
+      engine.QueryInterface(Ci.nsISearchEngine);
+      return verb == "engine-changed" && engine.name == "BigIcon" && engine.iconURI;
+    });
 
   let iconUrl = gDataUrl + "big_icon.ico";
   await addTestEngines([

toolkit/components/search/tests/xpcshell/xpcshell.ini

@@ -31,7 +31,6 @@ support-files =
 
 [test_nocache.js]
 [test_big_icon.js]
-skip-if = (os == "win" && processor == "x86_64" && !debug) # Bug 1496191
 [test_bug930456.js]
 [test_bug930456_child.js]
 [test_engine_set_alias.js]