Bug 1158228 - Disable visibility check helper function to avoid test bustage. a=sledru
authorMargaret Leibovic <margaret.leibovic@gmail.com>
Sat, 02 May 2015 07:29:12 -0700
changeset 259077 46b968653f4d0635401787fc260a430fb2ce0cf8
parent 259076 503f9aa61c25d7110f18c5509fc50d2a6fe8ab96
child 259078 3e9805c11aa3df7ddbec2b8b1f53c3f8d85e1676
push id4717
push userryanvm@gmail.com
push dateMon, 11 May 2015 14:48:36 +0000
treeherdermozilla-beta@85229fbaf017 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerssledru
bugs1158228
milestone38.0
Bug 1158228 - Disable visibility check helper function to avoid test bustage. a=sledru
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) {