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

Unified Diff: Source/devtools/front_end/SplitView.js

Issue 71633003: DevTools: added "overlayContents" mode, where DevTools content is placed around and underneath inse… (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 7 years, 1 month 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: Source/devtools/front_end/SplitView.js
diff --git a/Source/devtools/front_end/SplitView.js b/Source/devtools/front_end/SplitView.js
index 41b5489cc159946101bf94fdb2c1ea8425d3fdd3..cf9cd931cceb67d29fb0e17ce8334809803191f2 100644
--- a/Source/devtools/front_end/SplitView.js
+++ b/Source/devtools/front_end/SplitView.js
@@ -308,6 +308,8 @@ WebInspector.SplitView.prototype = {
this._sidebarSize = size;
+ this.onLayoutUpdated();
+
// No need to recalculate this._sidebarSize and this._totalSize again.
this._muteOnResize = true;
this.doResize();
@@ -390,6 +392,11 @@ WebInspector.SplitView.prototype = {
this._updateLayout();
},
+ onLayoutUpdated: function()
pfeldman 2013/11/18 15:07:42 There is an event in SidebarView for that, all you
dgozman 2013/11/21 16:38:51 Looks like that event is only fired on the whole e
+ {
+ // Override this method to do something when layout changes.
+ },
+
/**
* @param {Event} event
* @return {boolean}

Powered by Google App Engine
This is Rietveld 408576698