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

Issue 7790010: Revert 98679 - Restoring a session should restore window minimization state on Windows (Closed)

Created:
9 years, 3 months ago by dhollowa
Modified:
9 years, 3 months ago
Reviewers:
dhollowa
CC:
chromium-reviews, zork+watch_chromium.org, jennb, hashimoto+watch_chromium.org, aboxhall+watch_chromium.org, davidbarr+watch_chromium.org, yoshiki+watch_chromium.org, Dmitry Titov, jianli, prasadt, yuzo+watch_chromium.org, davemoore+watch_chromium.org, stevenjb, dcheng, dtseng+watch_chromium.org, Paweł Hajdan Jr., dhollowa, ctguil+watch_chromium.org, dmazzoni+watch_chromium.org
Visibility:
Public.

Description

Revert 98679 - Restoring a session should restore window minimization state on Windows These changes enable save and restore of minimized window state on Windows. (Mac and Linux came prior). The window state is now saved whenever size-changed notifications fire (in addition to the original activation and close cases). BUG=43274 TEST=Manual. Launch Chrome on Windows, create two browser windows, minimize "a" window, quit. Relaunch, expect "a" to be minimized. Review URL: http://codereview.chromium.org/7748036 TBR=dhollowa@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=98699

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+223 lines, -265 lines) Patch
M chrome/browser/chromeos/frame/browser_view.h View 1 chunk +1 line, -3 lines 0 comments Download
M chrome/browser/chromeos/frame/browser_view.cc View 2 chunks +2 lines, -5 lines 0 comments Download
M chrome/browser/chromeos/frame/panel_browser_view.h View 1 chunk +2 lines, -3 lines 0 comments Download
M chrome/browser/chromeos/frame/panel_browser_view.cc View 1 chunk +9 lines, -9 lines 0 comments Download
M chrome/browser/sessions/session_restore.cc View 2 chunks +3 lines, -17 lines 0 comments Download
M chrome/browser/sessions/session_service.cc View 1 chunk +1 line, -3 lines 0 comments Download
MM chrome/browser/ui/browser_browsertest.cc View 1 chunk +0 lines, -14 lines 0 comments Download
M chrome/browser/ui/panels/panel_browser_view.h View 1 chunk +1 line, -3 lines 0 comments Download
M chrome/browser/ui/panels/panel_browser_view.cc View 1 chunk +1 line, -4 lines 0 comments Download
M chrome/browser/ui/views/accessibility_event_router_views_unittest.cc View 1 chunk +7 lines, -5 lines 0 comments Download
M chrome/browser/ui/views/bookmarks/bookmark_bar_view_test.cc View 1 chunk +7 lines, -5 lines 0 comments Download
M chrome/browser/ui/views/chrome_views_delegate.h View 1 chunk +5 lines, -5 lines 0 comments Download
M chrome/browser/ui/views/chrome_views_delegate.cc View 4 chunks +18 lines, -12 lines 0 comments Download
M chrome/browser/ui/views/frame/browser_frame.cc View 1 chunk +2 lines, -1 line 0 comments Download
M chrome/browser/ui/views/frame/browser_view.h View 1 chunk +3 lines, -4 lines 0 comments Download
M chrome/browser/ui/views/frame/browser_view.cc View 3 chunks +19 lines, -9 lines 0 comments Download
M chrome/browser/ui/views/menu_model_adapter_test.cc View 1 chunk +8 lines, -5 lines 0 comments Download
M ui/base/ui_base_types.h View 1 chunk +1 line, -2 lines 0 comments Download
M views/desktop/desktop_views_delegate.h View 1 chunk +5 lines, -5 lines 0 comments Download
M views/desktop/desktop_views_delegate.cc View 1 chunk +8 lines, -4 lines 0 comments Download
M views/desktop/desktop_window_view.cc View 1 chunk +1 line, -1 line 0 comments Download
M views/test/test_views_delegate.h View 1 chunk +6 lines, -5 lines 0 comments Download
M views/test/test_views_delegate.cc View 1 chunk +5 lines, -8 lines 0 comments Download
M views/views_delegate.h View 2 chunks +11 lines, -9 lines 0 comments Download
M views/widget/native_widget_gtk.h View 2 chunks +3 lines, -4 lines 0 comments Download
M views/widget/native_widget_gtk.cc View 3 chunks +6 lines, -11 lines 0 comments Download
M views/widget/native_widget_private.h View 3 chunks +10 lines, -5 lines 0 comments Download
M views/widget/native_widget_views.h View 2 chunks +3 lines, -4 lines 0 comments Download
M views/widget/native_widget_views.cc View 2 chunks +4 lines, -4 lines 0 comments Download
M views/widget/native_widget_win.h View 2 chunks +3 lines, -4 lines 0 comments Download
M views/widget/native_widget_win.cc View 5 chunks +11 lines, -23 lines 0 comments Download
MM views/widget/widget.h View 5 chunks +9 lines, -14 lines 0 comments Download
MM views/widget/widget.cc View 14 chunks +29 lines, -41 lines 0 comments Download
MM views/widget/widget_delegate.h View 2 chunks +5 lines, -7 lines 0 comments Download
MM views/widget/widget_delegate.cc View 1 chunk +14 lines, -7 lines 0 comments Download

Messages

Total messages: 1 (0 generated)
dhollowa
9 years, 3 months ago (2011-08-29 21:06:41 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698