diff options
author | wittman@chromium.org <wittman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-05 22:15:35 +0000 |
---|---|---|
committer | wittman@chromium.org <wittman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-02-05 22:15:35 +0000 |
commit | 84685520948dcbf750736faa3e2bcc9dc430e861 (patch) | |
tree | 4bfc4086238f2f94355a689c142c5543b9664a09 /ash/desktop_background/desktop_background_view.cc | |
parent | f84973412773063d475d1b9f91540e6f6fa6e4da (diff) | |
download | chromium_src-84685520948dcbf750736faa3e2bcc9dc430e861.zip chromium_src-84685520948dcbf750736faa3e2bcc9dc430e861.tar.gz chromium_src-84685520948dcbf750736faa3e2bcc9dc430e861.tar.bz2 |
Add synchronous Widget closing event
Adds a WidgetObserver event that is guaranteed to be invoked synchronously
when a Widget is requested to close. Rename the existing event, which
is invoked at Widget destruction time, typically via a task on the event
loop, to OnWidgetDestroying.
The new event will be used in Views-specific WebContentsModalDialogManager
functionality to get close events for Widgets, so that we don't risk
adverse interleaving of Widget close events with other relevant events
(e.g. IPC). This will allow us to avoids test flakiness observed
in https://codereview.chromium.org/12045037 due to event races in
PrintPreviewTest.PrintCommands
BUG=157161
Review URL: https://chromiumcodereview.appspot.com/12089061
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@180796 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ash/desktop_background/desktop_background_view.cc')
-rw-r--r-- | ash/desktop_background/desktop_background_view.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ash/desktop_background/desktop_background_view.cc b/ash/desktop_background/desktop_background_view.cc index 3590853..0675bc9 100644 --- a/ash/desktop_background/desktop_background_view.cc +++ b/ash/desktop_background/desktop_background_view.cc @@ -82,7 +82,7 @@ class ShowWallpaperAnimationObserver : public ui::ImplicitAnimationObserver, } // Overridden from views::WidgetObserver. - virtual void OnWidgetClosing(views::Widget* widget) OVERRIDE { + virtual void OnWidgetDestroying(views::Widget* widget) OVERRIDE { desktop_widget_->RemoveObserver(this); desktop_widget_ = NULL; } |