Bug 1388494 - Fix 'undefined' text issue after you clicked "Ignore this warning". draft
authorHenry Chang <hchang@mozilla.com>
Fri, 11 Aug 2017 17:18:58 +0800
changeset 647160 bf5b6175b6af315a7d1a6312aff5845ed283eb26
parent 647159 a92f5d2e955f1c0662f2ddacb780a190e68e3a8d
child 726415 492fae560df1761a3a20b4cd02840a29d1d9234d
push id74309
push userhchang@mozilla.com
push dateWed, 16 Aug 2017 02:01:09 +0000
bugs1388494
milestone57.0a1
Bug 1388494 - Fix 'undefined' text issue after you clicked "Ignore this warning". MozReview-Commit-ID: CLb1lVsybEg
browser/base/content/browser.js
browser/locales/en-US/chrome/browser/browser.properties
--- a/browser/base/content/browser.js
+++ b/browser/base/content/browser.js
@@ -3188,16 +3188,20 @@ var BrowserOnClick = {
             openUILinkIn(reportUrl, "tab");
           }
         };
       }
     } else if (reason === "unwanted") {
       title = gNavigatorBundle.getString("safebrowsing.reportedUnwantedSite");
       // There is no button for reporting errors since Google doesn't currently
       // provide a URL endpoint for these reports.
+    } else if (reason === "harmful") {
+      title = gNavigatorBundle.getString("safebrowsing.reportedHarmfulSite");
+      // There is no button for reporting errors since Google doesn't currently
+      // provide a URL endpoint for these reports.
     }
 
     let notificationBox = gBrowser.getNotificationBox();
     let value = "blocked-badware-page";
 
     let previousNotification = notificationBox.getNotificationWithValue(value);
     if (previousNotification) {
       notificationBox.removeNotification(previousNotification);
--- a/browser/locales/en-US/chrome/browser/browser.properties
+++ b/browser/locales/en-US/chrome/browser/browser.properties
@@ -544,16 +544,17 @@ safebrowsing.getMeOutOfHereButton.label=
 safebrowsing.getMeOutOfHereButton.accessKey=G
 safebrowsing.deceptiveSite=Deceptive Site!
 safebrowsing.notADeceptiveSiteButton.label=This isn’t a deceptive site…
 safebrowsing.notADeceptiveSiteButton.accessKey=D
 safebrowsing.reportedAttackSite=Reported Attack Site!
 safebrowsing.notAnAttackButton.label=This isn’t an attack site…
 safebrowsing.notAnAttackButton.accessKey=A
 safebrowsing.reportedUnwantedSite=Reported Unwanted Software Site!
+safebrowsing.reportedHarmfulSite=Reported Harmful Site!
 
 # Ctrl-Tab
 # LOCALIZATION NOTE (ctrlTab.listAllTabs.label): #1 represents the number
 # of tabs in the current browser window. It will always be 2 at least.
 # See: http://developer.mozilla.org/en/docs/Localization_and_Plurals
 ctrlTab.listAllTabs.label=;List All #1 Tabs
 
 # LOCALIZATION NOTE (addKeywordTitleAutoFill): %S will be replaced by the page's title