Rip Out the Sidebar API

chrome/browser/resources/print_preview/margin_settings.js

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 cr.define('print_preview', function() {
   'use strict';
 
   /**
    * Creates a Margins object that holds four margin values. The units in which
    * the values are expressed can be any numeric value.
@@ skipping 309 matching lines @@
         return;
 
       if (this.customMargins_.isEqual(this.previousCustomMargins_))
         return;
 
       this.previousCustomMargins_ = this.customMargins_.clone();
       setDefaultValuesAndRegeneratePreview(false);
     },
 
     /**
-     * Listener executed when the mouse is over the sidebar. If the custom
-     * margin lines are displayed, then, it fades them out.
-     * @private
-     */
-    onSidebarMouseOver_: function(e) {

[Aaron Boodman, 2011/12/21 08:38:20]

The occurrences of 'sidebar' in print_preview should also stay.

-      $('mainview').onmouseover = this.onMainviewMouseOver_.bind(this);
-      $('navbar-container').onmouseover = null;
-      if (!this.forceDisplayingMarginLines_)
-        this.marginsUI.hide(false);
-    },
-
-    /**
      * Listener executed when the mouse is over the main view. If the custom
      * margin lines are hidden, then, it fades them in.
      * @private
      */
     onMainviewMouseOver_: function() {
       $('mainview').onmouseover = null;
-      $('navbar-container').onmouseover = this.onSidebarMouseOver_.bind(this);
       this.forceDisplayingMarginLines_ = false;
       this.marginsUI.show();
     },
 
     /**
      * Adds listeners to all margin related controls.
      * @private
      */
     addEventListeners_: function() {
       this.marginList_.onchange = this.onMarginsChanged_.bind(this);
@@ skipping 128 matching lines @@
       }
       return this.marginsUI_;
     },
 
     /**
      * Adds listeners when the custom margins option is selected.
      * @private
      */
     addCustomMarginEventListeners_: function() {
       $('mainview').onmouseover = this.onMainviewMouseOver_.bind(this);
-      $('navbar-container').onmouseover = this.onSidebarMouseOver_.bind(this);
       this.eventTracker_.add(this.marginsUI,
                              customEvents.MARGIN_LINE_DRAG,
                              this.onMarginsLineDrag_.bind(this),
                              false);
       this.eventTracker_.add(document, customEvents.MARGIN_TEXTBOX_FOCUSED,
                              this.onMarginTextboxFocused_.bind(this), false);
     },
 
     /**
      * Removes the event listeners associated with the custom margins option.
@@ skipping 196 matching lines @@
       this.pageWidth_ = this.currentDefaultPageLayout.pageWidth;
       this.pageHeight_ = this.currentDefaultPageLayout.pageHeight;
     }
   };
 
   return {
     MarginSettings: MarginSettings,
     PageLayout: PageLayout,
   };
 });