From 2d82a8160b726349984aa7756304a30b8b0f348d Mon Sep 17 00:00:00 2001 From: "sky@chromium.org" Date: Thu, 28 May 2009 16:38:07 +0000 Subject: Splits TooltipManager so that it can be ported and stubs out the GTK side. BUG=none TEST=none Review URL: http://codereview.chromium.org/114054 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17076 0039d316-1c4b-4281-b951-d872f2087c98 --- views/widget/widget_win.h | 14 +++++++++++--- 1 file changed, 11 insertions(+), 3 deletions(-) (limited to 'views/widget/widget_win.h') diff --git a/views/widget/widget_win.h b/views/widget/widget_win.h index ebf6bfa..fa6bab6 100644 --- a/views/widget/widget_win.h +++ b/views/widget/widget_win.h @@ -24,7 +24,7 @@ class Rect; namespace views { class RootView; -class TooltipManager; +class TooltipManagerWin; class DefaultThemeProvider; class Window; @@ -506,7 +506,17 @@ class WidgetWin : public Widget, // Returns true if this WidgetWin is opaque. bool opaque() const { return opaque_; } + // The TooltipManager. + // WARNING: RootView's destructor calls into the TooltipManager. As such, this + // must be destroyed AFTER root_view_. This really only matters during + // WM_SESSIONEND, as normally the hwnd is destroyed which tiggers unsetting + // the widget in the RootView so that RootView's destructor doesn't call into + // the TooltipManager. + scoped_ptr tooltip_manager_; + // The root of the View hierarchy attached to this window. + // WARNING: see warning in tooltip_manager_ for ordering dependencies with + // this and tooltip_manager_. scoped_ptr root_view_; // Whether or not we have capture the mouse. @@ -515,8 +525,6 @@ class WidgetWin : public Widget, // If true, the mouse is currently down. bool is_mouse_down_; - scoped_ptr tooltip_manager_; - // Are a subclass of WindowWin? bool is_window_; -- cgit v1.1