New issue
Advanced search Search tips

Issue 679804 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

scroll_layer_impl is null in LayerTreeHostImpl::MouseMoveAt

Project Member Reported by chaopeng@chromium.org, Jan 10 2017

Issue description

Chrome Version: 55.0.2883.87 (64-bit)
OS: Linux
 
temp.html
594 bytes View Download
Project Member

Comment 1 by bugdroid1@chromium.org, Jan 16 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/38ec90922bb8f6e9c2b6aaacdc27c13ae6dde8f2

commit 38ec90922bb8f6e9c2b6aaacdc27c13ae6dde8f2
Author: chaopeng <chaopeng@chromium.org>
Date: Mon Jan 16 21:14:23 2017

Return main thread scroll layer FindScrollLayerForDeviceViewportPoint

In FindScrollLayerForDeviceViewportPoint, we return null for main thread scroll
layer. But some calls only use the layer to look up animation controller and
that null return will make it wrong.

In the patch we return a layer without checking it is scroll in main thread.

BUG= 679804 
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2621133002
Cr-Commit-Position: refs/heads/master@{#443943}

[modify] https://crrev.com/38ec90922bb8f6e9c2b6aaacdc27c13ae6dde8f2/cc/trees/layer_tree_host_impl.cc
[modify] https://crrev.com/38ec90922bb8f6e9c2b6aaacdc27c13ae6dde8f2/cc/trees/layer_tree_host_impl_unittest.cc

Status: Fixed (was: Started)

Sign in to add a comment