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 260413 46b968653f4d
parent 260412 503f9aa61c25
child 260414 3e9805c11aa3
push id775
push userryanvm@gmail.com
push date2015-05-07 00:07 +0000
treeherdermozilla-release@46b968653f4d [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) {