Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3342)

Unified Diff: chrome/browser/resources/print_preview/margin_settings.js

Issue 9006027: Rip Out the Sidebar API (Closed) Base URL: http://src.chromium.org/svn/trunk/src/
Patch Set: '' Created 9 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/print_preview/margin_settings.js
===================================================================
--- chrome/browser/resources/print_preview/margin_settings.js (revision 115251)
+++ chrome/browser/resources/print_preview/margin_settings.js (working copy)
@@ -327,25 +327,12 @@
},
/**
- * 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 shou
- $('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();
},
@@ -494,7 +481,6 @@
*/
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),

Powered by Google App Engine
This is Rietveld 408576698