Make chrome/ be documentElement/body agnostic with regards to scrollTop/Left

chrome/browser/resources/help/help_base_page.js

Index: chrome/browser/resources/help/help_base_page.js
diff --git a/chrome/browser/resources/help/help_base_page.js b/chrome/browser/resources/help/help_base_page.js
index f07a9f1e0cd58df9eaf3c7ef0ce582f47325e0cf..52d30591bd989fbcf34b1101dc8cb9d5c2bd23a9 100644
--- a/chrome/browser/resources/help/help_base_page.js
+++ b/chrome/browser/resources/help/help_base_page.js
@@ -115,13 +115,16 @@ cr.define('help', function() {
      * @private
      */
     freeze_: function(freeze) {
+      var scrollTop = scrollTopForDocument(document);
+      var scrollLeft = scrollLeftForDocument(document);
+
       if (freeze) {
-        this.lastScrollTop = document.documentElement.scrollTop;
+        this.lastScrollTop = scrollTop;

[Dan Beam, 2013/11/13 23:31:07]

nit: replace `var scrollTop = ...;` with:

  this.lastScrollTop = scrollTopForDocument(document);

so it's only requested when if (freeze) is true.

         document.body.style.overflow = 'hidden';
-        window.scroll(document.documentElement.scrollLeft, 0);
+        window.scroll(scrollLeft, 0);
       } else {
         document.body.style.overflow = 'auto';
-        window.scroll(document.documentElement.scrollLeft, this.lastScrollTop);
+        window.scroll(scrollLeft, this.lastScrollTop);
       }
     },