Bug 1341514 - Using the valid update url provider in test. r=gcp

MozReview-Commit-ID: 50NcRd2e9hB

--HG--
extra : rebase_source : 9ae7f98578837aa42b20bfd802d71934c9f3bd28

toolkit/components/url-classifier/tests/mochitest/chrome.ini

@@ -26,4 +26,3 @@ tags = trackingprotection
 [test_donottrack.html]
 [test_classifier_changetablepref.html]
 [test_reporturl.html]
-skip-if = true # Bug 1341514

toolkit/components/url-classifier/tests/mochitest/test_reporturl.html

@@ -94,6 +94,7 @@ function testOnWindow(aTestData, aCallback, aTestCreater) {
         is(button, undefined, "Report button should not be showed");
         win.close();
         resolve();
+        return;
       }
 
       button.click();
@@ -158,11 +159,13 @@ var testDatas = [
     expectedReportUri: "http://itisaphishingsite.org/phishing.html"
   },
 
-  // Non-google provider, no report button is showed
+  // Non-google provider, no report button is showed.
+  // Test provider needs a valid update URL (mozilla for example) otherwise
+  // the updates inserting the test data will fail.
   { topUrl: "http://fakeitisaphishingsite.org/phishing.html",
     testUrl: "fakeitisaphishingsite.org/phishing.html",
     list: "fake-phish-simple",
-    provider: "fake",
+    provider: "mozilla",
     blockCreater : createBlockedPage
   },