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

Unified Diff: chrome/browser/chromeos/frame/panel_browser_view.cc

Issue 7748036: Restoring a session should restore window minimization state on Windows (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: GetSavedWindowPlacement consolidation, Test, and All case. Created 9 years, 4 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/chromeos/frame/panel_browser_view.cc
diff --git a/chrome/browser/chromeos/frame/panel_browser_view.cc b/chrome/browser/chromeos/frame/panel_browser_view.cc
index fba8fa8f3629063091ec76ed1d9a5513473055ae..4d20dcbd028726e8fad8a5abe633dad2f1063f8f 100644
--- a/chrome/browser/chromeos/frame/panel_browser_view.cc
+++ b/chrome/browser/chromeos/frame/panel_browser_view.cc
@@ -110,16 +110,16 @@ WindowOpenDisposition PanelBrowserView::GetDispositionForPopupBounds(
return chromeos::BrowserView::DispositionForPopupBounds(bounds);
}
-bool PanelBrowserView::GetSavedWindowBounds(gfx::Rect* bounds) const {
- bool res = ::BrowserView::GetSavedWindowBounds(bounds);
- if (res)
+bool PanelBrowserView::GetSavedWindowPlacement(
+ gfx::Rect* bounds,
+ ui::WindowShowState* show_state) const {
+ bool result = ::BrowserView::GetSavedWindowPlacement(bounds, show_state);
+ if (result) {
LimitBounds(bounds);
- return res;
-}
-
-bool PanelBrowserView::GetSavedMaximizedState(bool* maximized) const {
- // Panels have no maximized state.
- return false;
+ // Panels have no maximized state.
+ show_state = ui::SHOW_STATE_NORMAL;
sky 2011/08/26 22:29:56 *show_state = ...
dhollowa 2011/08/26 23:18:23 Done.
+ }
+ return result;
}
////////////////////////////////////////////////////////////////////////////////

Powered by Google App Engine
This is Rietveld 408576698