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

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

Issue 11410024: ui: Remove gfx::Size::ClampToNonNegative, prevent negative sizes always. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: no negative sizes Created 8 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: 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 2bed9f500d9b5babe969c3658015476cf1ceae03..f6f85c058d40cbfbcf694b705990877a760cf612 100644
--- a/chrome/browser/extensions/api/app_window/app_window_api.cc
+++ b/chrome/browser/extensions/api/app_window/app_window_api.cc
@@ -171,14 +171,14 @@ bool AppWindowCreateFunction::RunImpl() {
if (maximum_size.width() &&
create_params.bounds.width() > maximum_size.width())
create_params.bounds.set_width(maximum_size.width());
- if (create_params.bounds.width() != INT_MIN &&
+ if (create_params.bounds.width() != 0 &&
create_params.bounds.width() < minimum_size.width())
create_params.bounds.set_width(minimum_size.width());
if (maximum_size.height() &&
create_params.bounds.height() > maximum_size.height())
create_params.bounds.set_height(maximum_size.height());
- if (create_params.bounds.height() != INT_MIN &&
+ if (create_params.bounds.height() != 0 &&
create_params.bounds.height() < minimum_size.height())
create_params.bounds.set_height(minimum_size.height());

Powered by Google App Engine
This is Rietveld 408576698