Bug 1158228 - Disable visibility check helper function to avoid test bustage. a=lizzard
authorMargaret Leibovic <margaret.leibovic@gmail.com>
Sat, 02 May 2015 07:29:12 -0700
changeset 265879 5de0834fe8a9a2d81867a26c497da9c4fa29ba6c
parent 265878 1f1699b0cec116270a238af733008655c70f2589
child 265880 57fa7552c3c3691ea47fd8a65d99f12139d25069
push id4718
push userraliiev@mozilla.com
push dateMon, 11 May 2015 18:39:53 +0000
treeherdermozilla-beta@c20c4ef55f08 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerslizzard
bugs1158228
milestone39.0a2
Bug 1158228 - Disable visibility check helper function to avoid test bustage. a=lizzard
toolkit/components/reader/ReaderMode.jsm
--- a/toolkit/components/reader/ReaderMode.jsm
+++ b/toolkit/components/reader/ReaderMode.jsm
@@ -112,17 +112,18 @@ this.ReaderMode = {
     if (!this._shouldCheckUri(uri)) {
       return false;
     }
 
     let utils = this.getUtilsForWin(doc.defaultView);
     // We pass in a helper function to determine if a node is visible, because
     // it uses gecko APIs that the engine-agnostic readability code can't rely
     // upon.
-    return new Readability(uri, doc).isProbablyReaderable(this.isNodeVisible.bind(this, utils));
+    // NOTE: This is currently disabled, see bug 1158228.
+    return new Readability(uri, doc).isProbablyReaderable(/*this.isNodeVisible.bind(this, utils)*/);
   },
 
   isNodeVisible: function(utils, node) {
     let bounds = utils.getBoundsWithoutFlushing(node);
     return bounds.height > 0 && bounds.width > 0;
   },
 
   getUtilsForWin: function(win) {