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

Unified Diff: chrome/browser/extensions/api/app_window/app_window_api.cc

Issue 25449002: Add chrome.app.window.[get|set][Min|Max][Width|Height] (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase. Use ShellWindow::Set[Min|Max]imumSize. Update tests. Created 7 years, 2 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/extensions/api/app_window/app_window_api.cc
diff --git a/chrome/browser/extensions/api/app_window/app_window_api.cc b/chrome/browser/extensions/api/app_window/app_window_api.cc
index f21546a62d1bcee875a0b64c75d07d1390e0027c..3a50fea8343754a028a5c95fb43b7930eb45bd62 100644
--- a/chrome/browser/extensions/api/app_window/app_window_api.cc
+++ b/chrome/browser/extensions/api/app_window/app_window_api.cc
@@ -100,6 +100,15 @@ void SetCreateResultFromShellWindow(ShellWindow* window,
boundsValue->SetInteger("width", bounds.width());
boundsValue->SetInteger("height", bounds.height());
result->Set("bounds", boundsValue);
+
+ const ShellWindow::SizeConstraints& size_constraints =
tapted 2013/10/16 23:31:52 Hm. this is identical to the code in AppWindowCont
+ window->size_constraints();
+ gfx::Size min_size = size_constraints.GetMinimumSize();
+ gfx::Size max_size = size_constraints.GetMaximumSize();
+ result->SetInteger("minWidth", min_size.width());
+ result->SetInteger("minHeight", min_size.height());
+ result->SetInteger("maxWidth", max_size.width());
+ result->SetInteger("maxHeight", max_size.height());
}
} // namespace

Powered by Google App Engine
This is Rietveld 408576698