New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 764271 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Vertical scroll bar does not work using mouse wheel in devtools.

Reported by pranjali...@etouch.net, Sep 12 2017

Issue description

Chrome Version:62.0.3202.18 7ba42bf8da38fd2a88b71bea05df3ebfd29ee96c-refs/branch-heads/3202@{#152}

OS: MAC (Retina 10.12)

Steps to reproduce:
1)Launch chrome ,open devtools in NTP and go to performance section.
2)Record some activities for 3-4 sec and click on stop button.
3)Now scroll devtools window using mouse wheel and observe.

Actual Result:  Vertical scroll bar does not work using mouse wheel in performance section on devtools.
Expected Result: Vertical scroll bar should be work using mouse wheel in performance section on devtools.

This is regression issue broken in 'M-61' and will soon update the bisect info.

Manual Bisect:
Good build: 61.0.3136.0
Bad build: 61.0.3137.0

Note:Issue is not seen on Windows(7,8,10), Linux(14.04 LTS) and Mac(10.11.6, 10.12.3, 10.12.5).


 
Actual_result.mov
3.2 MB Download
Expected_result.mov
3.2 MB Download
Labels: Needs-Bisect
Status: Untriaged (was: Unconfirmed)
Labels: -Needs-Bisect hasbisect
Owner: hayleyferr@chromium.org
Status: Assigned (was: Untriaged)
Narrow Bisect:
https://chromium.googlesource.com/chromium/src/+log/b1efd1e7b6d9eb46c2a367a84f8350623093ec14..8e641dfb7ee85c52e2e7957ab26c102cc9dd67f9?pretty=fuller&n=10000

Suspect: r480803?

@hayleyferr:  Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner.
Cc: alph@chromium.org
@alph: mind taking a look whether the platform is doing something wrong or we use it incorrectly?
Owner: vmp...@chromium.org
over to reviewer...
Cc: vmp...@chromium.org
Owner: pdr@chromium.org
pdr@, could you take a look since you've been looking at scrolling recently. I can verify that the window doesn't scroll when the mouse is over the scrollbar. Note that if the mouse is over the content of the window then it has a custom action of zooming in instead of scrolling.

Comment 6 by pdr@chromium.org, Oct 10 2017

Components: -Platform>DevTools Blink>Compositing
This is a bug with custom scrollbars and mouse-wheel scrolling. Here's a minimized testcase: http://output.jsbin.com/viheyek/quiet
Components: -Blink>Compositing Blink>Scroll
Project Member

Comment 8 by bugdroid1@chromium.org, Oct 12 2017

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

commit 176756e93d0cf910e461547f6c7c0e7bfbce36a3
Author: Philip Rogers <pdr@chromium.org>
Date: Thu Oct 12 22:59:39 2017

Ensure LayerImpls that draw content are hit testable

[1] introduced a bit on LayerImpl for hit testing layers that do not
draw content. There was a bug in how this bit was pushed (Layer to
LayerImpl) where a Layer that draws content would have a LayerImpl
that was not marked as being hit testable because the (false)
should_hit_test bit was pushed after the (true) draws_content bit.

This patch renames should_hit_test to hit_testable_without_draws_content
and makes should_hit_test depend on both drawing content and
hit_testable_without_draws_content. A test has been added that covers
the Layer->LayerImpl property push behavior.

[1] https://chromium-review.googlesource.com/c/chromium/src/+/529426

Bug:  764271 
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Id5d9f2cf9cd6a5a6e33808b3729bf3c16f481037
Reviewed-on: https://chromium-review.googlesource.com/711136
Reviewed-by: vmpstr <vmpstr@chromium.org>
Reviewed-by: Xida Chen <xidachen@chromium.org>
Commit-Queue: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#508519}
[modify] https://crrev.com/176756e93d0cf910e461547f6c7c0e7bfbce36a3/cc/blink/web_layer_impl.cc
[modify] https://crrev.com/176756e93d0cf910e461547f6c7c0e7bfbce36a3/cc/blink/web_layer_impl.h
[modify] https://crrev.com/176756e93d0cf910e461547f6c7c0e7bfbce36a3/cc/layers/layer.cc
[modify] https://crrev.com/176756e93d0cf910e461547f6c7c0e7bfbce36a3/cc/layers/layer.h
[modify] https://crrev.com/176756e93d0cf910e461547f6c7c0e7bfbce36a3/cc/layers/layer_impl.cc
[modify] https://crrev.com/176756e93d0cf910e461547f6c7c0e7bfbce36a3/cc/layers/layer_impl.h
[modify] https://crrev.com/176756e93d0cf910e461547f6c7c0e7bfbce36a3/cc/layers/layer_unittest.cc
[modify] https://crrev.com/176756e93d0cf910e461547f6c7c0e7bfbce36a3/cc/trees/layer_tree_host_impl_unittest.cc
[modify] https://crrev.com/176756e93d0cf910e461547f6c7c0e7bfbce36a3/third_party/WebKit/Source/core/paint/compositing/CompositedLayerMapping.cpp
[modify] https://crrev.com/176756e93d0cf910e461547f6c7c0e7bfbce36a3/third_party/WebKit/Source/core/paint/compositing/CompositedLayerMapping.h
[modify] https://crrev.com/176756e93d0cf910e461547f6c7c0e7bfbce36a3/third_party/WebKit/Source/core/paint/compositing/CompositedLayerMappingTest.cpp
[modify] https://crrev.com/176756e93d0cf910e461547f6c7c0e7bfbce36a3/third_party/WebKit/Source/platform/graphics/GraphicsLayer.cpp
[modify] https://crrev.com/176756e93d0cf910e461547f6c7c0e7bfbce36a3/third_party/WebKit/Source/platform/graphics/GraphicsLayer.h
[modify] https://crrev.com/176756e93d0cf910e461547f6c7c0e7bfbce36a3/third_party/WebKit/public/platform/WebLayer.h

Comment 9 by pdr@chromium.org, Oct 12 2017

Status: Fixed (was: Assigned)

Sign in to add a comment