Bug 778582 - Part 4. r=lucasr

mobile/android/chrome/content/aboutReader.html

@@ -27,8 +27,7 @@
     <li><a id="toggle-button" class="button toggle-button" href="#"></a></li>
   </ul>
 
-  <script type="application/javascript;version=1.8" src="chrome://browser/content/aboutReader.js">
-  </script>
+  <script type="application/javascript;version=1.8" src="chrome://browser/content/aboutReader.js"></script>
 </body>
 
 </html>

mobile/android/chrome/content/aboutReader.js

@@ -48,8 +48,6 @@ let AboutReader = {
 
     this._toolbarEnabled = false;
 
-    this._scrollOffset = window.pageYOffset;
-
     this._frame.contentDocument.addEventListener("DOMSubtreeModified", this, false);
     this._frame.contentDocument.addEventListener("MozScrolledAreaChanged", this, false);
     this._frame.contentDocument.addEventListener("touchstart", this, false);
@@ -146,7 +144,7 @@ let AboutReader = {
   uninit: function Reader_uninit() {
     dump("Uninit()");
 
-    Services.obs.removeObserver(this, "Reader:FaviconReturn", false);
+    Services.obs.removeObserver(this, "Reader:FaviconReturn");
 
     this._frame.contentDocument.removeEventListener("touchstart", this, false);
     this._frame.contentDocument.removeEventListener("click", this, false);

mobile/android/components/AboutRedirector.js

@@ -61,7 +61,8 @@ let modules = {
   },
   reader: {
     uri: "chrome://browser/content/aboutReader.html",
-    privileged: true
+    privileged: true,
+    hide: true
   },
   readercontent: {
     uri: "chrome://browser/content/aboutReaderContent.html",

mobile/android/themes/core/aboutReader.css

@@ -2,9 +2,6 @@
  * License, v. 2.0. If a copy of the MPL was not distributed with this file,
  * You can obtain one at http://mozilla.org/MPL/2.0/. */
 
-%filter substitution
-%include defines.inc
-
 body {
   font-family: "OpenSansRegular","Droid Sans",sans-serif;
 }
@@ -24,7 +21,7 @@ body {
 
 .toolbar {
   font-family: "Droid Sans",helvetica,arial,clean,sans-serif;
-  -moz-transition-property: visibility opacity;
+  -moz-transition-property: visibility, opacity;
   -moz-transition-duration: 0.7s;
   visibility: visible;
   opacity: 1.0;
@@ -39,7 +36,7 @@ body {
 }
 
 .toolbar-hidden {
-  -moz-transition-property: visibility opacity;
+  -moz-transition-property: visibility, opacity;
   -moz-transition-duration: 0.7s;
   visibility: hidden;
   opacity: 0.0;

mobile/android/themes/core/jar.mn

@@ -11,7 +11,7 @@ chrome.jar:
   skin/aboutAddons.css                      (aboutAddons.css)
   skin/aboutApps.css                        (aboutApps.css)
 * skin/aboutDownloads.css                   (aboutDownloads.css)
-* skin/aboutReader.css                      (aboutReader.css)
+  skin/aboutReader.css                      (aboutReader.css)
   skin/aboutReaderContent.css               (aboutReaderContent.css)
 * skin/browser.css                          (browser.css)
 * skin/content.css                          (content.css)