Bug 1172009 - don't show reader mode button for file:///, r=margaret
authorGijs Kruitbosch <gijskruitbosch@gmail.com>
Tue, 09 Jun 2015 13:48:11 +0100
changeset 248123 09369dc533d36ca9204d165536acd32b87906dd1
parent 248122 0d0fbf21e23e4168a40e85b157e0f4404365fe86
child 248124 043466f5a7f1b797d53724d7c4e1e20f3ee4a743
push id60888
push userkwierso@gmail.com
push dateThu, 11 Jun 2015 01:38:38 +0000
treeherdermozilla-inbound@39e638ed06bf [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmargaret
bugs1172009
milestone41.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1172009 - don't show reader mode button for file:///, r=margaret
toolkit/components/reader/ReaderMode.jsm
--- a/toolkit/components/reader/ReaderMode.jsm
+++ b/toolkit/components/reader/ReaderMode.jsm
@@ -285,17 +285,17 @@ this.ReaderMode = {
   _blockedHosts: [
     "twitter.com",
     "mail.google.com",
     "github.com",
     "reddit.com",
   ],
 
   _shouldCheckUri: function (uri) {
-    if (!(uri.schemeIs("http") || uri.schemeIs("https") || uri.schemeIs("file"))) {
+    if (!(uri.schemeIs("http") || uri.schemeIs("https"))) {
       this.log("Not parsing URI scheme: " + uri.scheme);
       return false;
     }
 
     try {
       uri.QueryInterface(Ci.nsIURL);
     } catch (ex) {
       // If this doesn't work, presumably the URL is not well-formed or something