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

Unified Diff: cc/trees/layer_tree_host_impl.cc

Issue 2384053002: cancel overlay-scrollbar hover state when moving pointer out of window for ChromeOS (Closed)
Patch Set: fix a bug Created 4 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
« no previous file with comments | « cc/trees/layer_tree_host_impl.h ('k') | ui/events/blink/input_handler_proxy.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/layer_tree_host_impl.cc
diff --git a/cc/trees/layer_tree_host_impl.cc b/cc/trees/layer_tree_host_impl.cc
index 4aa0886dfb80d80425acf157f5d0e7d8daf1e872..d6c95117dd2bb45c54703c68b291ddc95097a28e 100644
--- a/cc/trees/layer_tree_host_impl.cc
+++ b/cc/trees/layer_tree_host_impl.cc
@@ -3266,6 +3266,16 @@ void LayerTreeHostImpl::MouseMoveAt(const gfx::Point& viewport_point) {
active_tree_->device_scale_factor());
}
+void LayerTreeHostImpl::MouseLeave() {
+ ScrollbarAnimationController* animation_controller =
bokan 2016/10/04 20:57:41 What if the mouse isn't over the scrollbar though,
chaopeng 2016/10/05 02:29:29 Yes, It doesnot work for near-moveout. I add a new
+ ScrollbarAnimationControllerForId(
+ scroll_layer_id_when_mouse_over_scrollbar_);
+
+ if (animation_controller) {
dtapuska 2016/10/04 20:50:22 don't think you need the scope brackets :-)
+ animation_controller->DidMouseLeave();
+ }
+}
+
void LayerTreeHostImpl::HandleMouseOverScrollbar(LayerImpl* layer_impl) {
int new_id = Layer::INVALID_ID;
if (layer_impl && layer_impl->ToScrollbarLayer())
« no previous file with comments | « cc/trees/layer_tree_host_impl.h ('k') | ui/events/blink/input_handler_proxy.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698