diff options
| author | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-14 18:03:47 +0000 |
|---|---|---|
| committer | sky@chromium.org <sky@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-10-14 18:03:47 +0000 |
| commit | 08a3b713ae113c58893e7daddbbc4afd178d446f (patch) | |
| tree | d2f1dee12be5cb34f4acdc1cf269021e5948a7ef /views/view.h | |
| parent | 56008bf3dc79ab9d76f09769317ac4307c385b08 (diff) | |
| download | chromium_src-08a3b713ae113c58893e7daddbbc4afd178d446f.zip chromium_src-08a3b713ae113c58893e7daddbbc4afd178d446f.tar.gz chromium_src-08a3b713ae113c58893e7daddbbc4afd178d446f.tar.bz2 | |
Fixes regression in menu button. I converted a call from GetWidget to
GetWindow, the problem is in certain scenarios GetWindow can return
null. I've changed the code back to GetWidget and converted Screen
from taking a NativeWindow to a NativeView.
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/269076
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@28986 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'views/view.h')
| -rw-r--r-- | views/view.h | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/views/view.h b/views/view.h index cb44e2a..dd7662b 100644 --- a/views/view.h +++ b/views/view.h @@ -428,6 +428,9 @@ class View : public AcceleratorTarget { virtual Widget* GetWidget() const; // Gets the Widget that most closely contains this View, if any. + // NOTE: almost all views displayed on screen have a Widget, but not + // necessarily a Window. This is due to widgets being able to create top + // level windows (as is done for popups, bubbles and menus). virtual Window* GetWindow() const; // Get the containing RootView |