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

Unified Diff: chrome/browser/ui/views/frame/browser_view.cc

Issue 11348311: Fix logic for BrowserView::GetSavedWindowPlacement (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ui/views/frame/browser_view.cc
diff --git a/chrome/browser/ui/views/frame/browser_view.cc b/chrome/browser/ui/views/frame/browser_view.cc
index c2b4258b4c7886eed2f9cf9022bd1409e2a9e513..f3374ebd490d15240e9deacb0ea6d0bb19513947 100644
--- a/chrome/browser/ui/views/frame/browser_view.cc
+++ b/chrome/browser/ui/views/frame/browser_view.cc
@@ -1846,11 +1846,12 @@ bool BrowserView::GetSavedWindowPlacement(
}
#endif
- if ((browser_->is_type_popup() &&
- !browser_->is_devtools() && !browser_->is_app()) ||
- (browser_->is_type_panel())) {
- // We are a popup window. The value passed in |bounds| represents two
- // pieces of information:
+ if ((browser_->is_type_popup() || browser_->is_type_panel()) &&
+ bounds->width() == browser_->override_bounds().width() &&
+ bounds->height() == browser_->override_bounds().height() &&
+ !browser_->is_devtools()) {
+ // This is a popup window that has not been resized. The value passed in
+ // |bounds| represents two pieces of information:
// - the position of the window, in screen coordinates (outer position).
// - the size of the content area (inner size).
// We need to use these values to determine the appropriate size and
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698