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

Unified Diff: views/widget/native_widget_views.cc

Issue 8417025: aura: Update how the tooltip manager works. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: compile fixes Created 9 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: views/widget/native_widget_views.cc
diff --git a/views/widget/native_widget_views.cc b/views/widget/native_widget_views.cc
index 4ed77ca53a64d6e2eee477ff724fe467a7de5f46..5778e64e859337493c33b65a7170388b0b953d07 100644
--- a/views/widget/native_widget_views.cc
+++ b/views/widget/native_widget_views.cc
@@ -10,6 +10,7 @@
#include "views/views_delegate.h"
#include "views/widget/native_widget_view.h"
#include "views/widget/root_view.h"
+#include "views/widget/tooltip_manager_views.h"
#include "views/widget/window_manager.h"
#if defined(HAVE_IBUS)
@@ -125,6 +126,12 @@ void NativeWidgetViews::OnBoundsChanged(const gfx::Rect& new_bounds,
}
bool NativeWidgetViews::OnMouseEvent(const MouseEvent& event) {
+#if defined(TOUCH_UI) || defined(USE_AURA)
varunjain 2011/10/28 16:09:23 I am wondering if we really need a views tooltip m
sadrul 2011/10/28 16:26:16 Indeed. Right now, apart from aura, only TOUCH_UI
+ TooltipManagerViews* tooltip_manager =
+ static_cast<TooltipManagerViews*>(GetTooltipManager());
sky 2011/10/28 17:47:27 Same comment about changing type of field to avoid
sadrul 2011/10/28 21:42:52 This is complicated here since NWViews doesn't act
+ if (tooltip_manager)
+ tooltip_manager->UpdateForMouseEvent(event);
+#endif
return HandleWindowOperation(event) ? true : delegate_->OnMouseEvent(event);
}

Powered by Google App Engine
This is Rietveld 408576698