Bug 513957: disable annoying dumps, r=joe, r=mfinkle
authorGavin Sharp <gavin@mozilla.com>
Tue, 01 Sep 2009 14:14:37 -0400
changeset 65495 5f7f0e55316a654e655379976acfd1462e288838
parent 65494 a69079e758a40854e6160286867ede018fc67f4e
child 65496 bc3a55d8f9e95223de4af005e0b5181408766fa1
push id1
push userroot
push dateTue, 26 Apr 2011 22:38:44 +0000
treeherdermozilla-beta@bfdb6e623a36 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjoe, mfinkle
bugs513957
Bug 513957: disable annoying dumps, r=joe, r=mfinkle
mobile/chrome/content/browser.js
--- a/mobile/chrome/content/browser.js
+++ b/mobile/chrome/content/browser.js
@@ -333,17 +333,17 @@ var Browser = {
   contentScrollbox: null,
   contentScrollboxScroller: null,
   controlsScrollbox: null,
   controlsScrollboxScroller: null,
 
   startup: function() {
     var self = this;
 
-    dump("begin startup\n");
+    //dump("begin startup\n");
 
     let container = document.getElementById("tile-container");
     let bv = this._browserView = new BrowserView(container, Browser.getVisibleRect);
 
     /* handles dispatching clicks on tiles into clicks in content or zooms */
     container.customClicker = this._createContentCustomClicker(bv);
 
     /* vertically scrolling box that contains tiles and the urlbar */
@@ -364,17 +364,17 @@ var Browser = {
     // during startup a lot of viewportHandler calls happen due to content and window resizes
     bv.beginBatchOperation();
 
     function resizeHandler(e) {
 
       if (e.target != window)
         return;
 
-      dump(window.innerWidth + "," + window.innerHeight + "\n");
+      //dump(window.innerWidth + "," + window.innerHeight + "\n");
       // XXX is this code right here actually needed?
       let w = window.innerWidth;
       let h = window.innerHeight;
       let maximize = (document.documentElement.getAttribute("sizemode") == "maximized");
       if (maximize && w > screen.width)
         return;
 
       bv.beginBatchOperation();
@@ -533,17 +533,17 @@ var Browser = {
     // this code eventually...
     if (gPrefService.prefHasUserValue("temporary.disablePlugins")) {
       gPrefService.clearUserPref("temporary.disablePlugins");
       this.setPluginState(true);
     }
 
     bv.commitBatchOperation();
 
-    dump("end startup\n");
+    //dump("end startup\n");
   },
 
   shutdown: function() {
     this._browserView.setBrowser(null, null, false);
 
     BrowserUI.uninit();
 
     var os = Cc["@mozilla.org/observer-service;1"].getService(Ci.nsIObserverService);
@@ -682,18 +682,18 @@ var Browser = {
     bv.forceContainerResize();
 
     // XXX these should probably be computed less hackily so they don't
     //   potentially break if we change something in browser.xul
     let offY = Math.round(document.getElementById("toolbar-container").getBoundingClientRect().height);
     let restoreX = Math.max(0, tab.browserViewportState.visibleX);
     let restoreY = Math.max(0, tab.browserViewportState.visibleY) + offY;
 
-    dump('Switch tab scrolls to: ' + restoreX
-                            + ', ' + restoreY + '\n');
+    //dump('Switch tab scrolls to: ' + restoreX
+    //                        + ', ' + restoreY + '\n');
 
     Browser.contentScrollboxScroller.scrollTo(restoreX, restoreY);
 
     document.getElementById("tabs").selectedItem = tab.chromeTab;
 
     if (!firstTab) {
       let webProgress = this.selectedBrowser.webProgress;
       let securityUI = this.selectedBrowser.securityUI;
@@ -1158,17 +1158,17 @@ var Browser = {
     return this.clientToBrowserView(cX, cY).map(this._browserView.viewportToBrowser);
   },
 
   /**
    * return element at client coordinates of browser, returns null if
    * there's no active browser or if no element can be found
    */
   elementFromPoint: function elementFromPoint(x, y) {
-    Util.dumpLn("*** elementFromPoint: page ", x, ",", y);
+    //Util.dumpLn("*** elementFromPoint: page ", x, ",", y);
 
     let browser = this._browserView.getBrowser();
     if (!browser) return null;
 
     let cwu = BrowserView.Util.getBrowserDOMWindowUtils(browser);
 
     let scrollX = { value: 0 }, scrollY = { value: 0 };
     cwu.getScrollXY(false, scrollX, scrollY);
@@ -1244,18 +1244,18 @@ Browser.MainDragger.prototype = {
     this.draggedFrame = null;
 
     if (this._targetIsContent(target)) {
       // since we're dealing with content, look to see if user has started
       // a drag while over a IFRAME/FRAME element
       let [x, y] = Browser.transformClientToBrowser(clientX, clientY);
       let element = Browser.elementFromPoint(x, y);
       if (element && element.ownerDocument != Browser.selectedBrowser.contentDocument) {
-        Util.dumpLn("*** dragStart got element ", element, " ownerDoc ", element.ownerDocument,
-                    " selectedBrowser.contentDoc ", Browser.selectedBrowser.contentDocument);
+        //Util.dumpLn("*** dragStart got element ", element, " ownerDoc ", element.ownerDocument,
+        //            " selectedBrowser.contentDoc ", Browser.selectedBrowser.contentDocument);
         this.draggedFrame = element.ownerDocument.defaultView;
       }
     }
 
     this.bv.pauseRendering();
   },
 
   dragStop: function dragStop(dx, dy, scroller) {