summaryrefslogtreecommitdiffstats
path: root/extensions/components
diff options
context:
space:
mode:
authordmazzoni <dmazzoni@chromium.org>2015-06-22 11:37:48 -0700
committerCommit bot <commit-bot@chromium.org>2015-06-22 18:39:25 +0000
commit1fbc9540c7a729001f34e2c0d8418f16186ef679 (patch)
treef0b84cbf6d67b9dac0afbb25c040e8cbd338caea /extensions/components
parent706ec4ea51514b39b6deba706ec8eee7c496240d (diff)
downloadchromium_src-1fbc9540c7a729001f34e2c0d8418f16186ef679.zip
chromium_src-1fbc9540c7a729001f34e2c0d8418f16186ef679.tar.gz
chromium_src-1fbc9540c7a729001f34e2c0d8418f16186ef679.tar.bz2
Revert of Revert of Revert of [MacViews] Implement AppWindowNativeWidgetMac::Maximize. (patchset #1 id:1 of https://codereview.chromium.org/1198403003/)
Reason for revert: Un-reverting this seems to have caused these test failures: https://build.chromium.org/p/chromium.mac/builders/Mac10.6%20Tests/builds/3755 Original issue's description: > Revert of Revert of [MacViews] Implement AppWindowNativeWidgetMac::Maximize. (patchset #1 id:1 of https://codereview.chromium.org/1192373006/) > > Reason for revert: > The failing test cleared up without this revert, so relanding. > > Original issue's description: > > Revert of [MacViews] Implement AppWindowNativeWidgetMac::Maximize. (patchset #5 id:80001 of https://codereview.chromium.org/1184643003/) > > > > Reason for revert: > > This seems the most likely cause of a reliable Mac browser test failure. > > > > e.g https://build.chromium.org/p/chromium.webkit/builders/Mac10.6%20Tests/builds/29726 > > > > Original issue's description: > > > [MacViews] Implement AppWindowNativeWidgetMac::Maximize. > > > > > > This makes MacViews behavior consistent with the existing Cocoa > > > implementation and adds a heap of tests for interactions between > > > Minimize, Restore, Maximize, and Fullscreen. > > > > > > This also changes NativeAppWindowCocoa behavior to be more > > > consistent with other platforms: > > > - isMaximized, isMinimized, isFullscreen are mutually exclusive, > > > - maximize() shows the window if it is minimized, > > > - restore() always restores the window size (in addition to unminimizing > > > and exiting fullscreen). > > > > > > This also fixes a bug in the Cocoa implementation where Maximize > > > changes the window frame while in fullscreen. > > > > > > BUG=459877 > > > > > > Committed: https://crrev.com/5c69ddcaa25a8d5a6963a7cab4bb6b6e5e636b78 > > > Cr-Commit-Position: refs/heads/master@{#335465} > > > > TBR=tapted@chromium.org,jackhou@chromium.org > > NOPRESUBMIT=true > > NOTREECHECKS=true > > NOTRY=true > > BUG=459877 > > > > Committed: https://crrev.com/21d068fd8a2094ea45cb067a22c8f8bd87c4b1be > > Cr-Commit-Position: refs/heads/master@{#335493} > > TBR=tapted@chromium.org,jackhou@chromium.org > NOPRESUBMIT=true > NOTREECHECKS=true > NOTRY=true > BUG=459877 > > Committed: https://crrev.com/2cf765dddad3b6f67e13c2ac1a4db4cfdd847f2c > Cr-Commit-Position: refs/heads/master@{#335513} TBR=tapted@chromium.org,jackhou@chromium.org,schenney@chromium.org NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=459877 Review URL: https://codereview.chromium.org/1196533003 Cr-Commit-Position: refs/heads/master@{#335531}
Diffstat (limited to 'extensions/components')
-rw-r--r--extensions/components/native_app_window/native_app_window_views.cc2
1 files changed, 1 insertions, 1 deletions
diff --git a/extensions/components/native_app_window/native_app_window_views.cc b/extensions/components/native_app_window/native_app_window_views.cc
index 0233623..e80e852 100644
--- a/extensions/components/native_app_window/native_app_window_views.cc
+++ b/extensions/components/native_app_window/native_app_window_views.cc
@@ -44,10 +44,10 @@ void NativeAppWindowViews::Init(AppWindow* app_window,
Observe(app_window_->web_contents());
widget_ = new views::Widget;
- widget_->AddObserver(this);
InitializeWindow(app_window, create_params);
OnViewWasResized();
+ widget_->AddObserver(this);
}
NativeAppWindowViews::~NativeAppWindowViews() {