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

Unified Diff: chrome/browser/devtools/devtools_window.cc

Issue 12871013: DevTools: respect minimum width/height when resizing. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 7 years, 9 months 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/devtools/devtools_window.cc
===================================================================
--- chrome/browser/devtools/devtools_window.cc (revision 187863)
+++ chrome/browser/devtools/devtools_window.cc (working copy)
@@ -90,7 +90,7 @@
// Minimal height of devtools pane or content pane when devtools are docked
// to the browser window.
-const int kMinDevToolsHeight = 50;
+const int kMinDevToolsHeight = 23;
pfeldman 2013/03/19 10:42:06 I'd leave this 50.
dgozman 2013/03/21 09:55:26 Done.
const int kMinDevToolsWidth = 150;
const int kMinContentsSize = 50;
@@ -396,6 +396,14 @@
return height_;
}
+int DevToolsWindow::GetMinimumWidth() {
+ return kMinDevToolsWidth;
+}
+
+int DevToolsWindow::GetMinimumHeight() {
+ return kMinDevToolsHeight;
+}
+
void DevToolsWindow::SetWidth(int width) {
width_ = width;
profile_->GetPrefs()->SetInteger(prefs::kDevToolsVSplitLocation, width);

Powered by Google App Engine
This is Rietveld 408576698