Issue metadata
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 descriptionChrome 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).
,
Sep 13 2017
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.
,
Oct 3 2017
@alph: mind taking a look whether the platform is doing something wrong or we use it incorrectly?
,
Oct 6 2017
over to reviewer...
,
Oct 9 2017
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.
,
Oct 10 2017
This is a bug with custom scrollbars and mouse-wheel scrolling. Here's a minimized testcase: http://output.jsbin.com/viheyek/quiet
,
Oct 10 2017
,
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
,
Oct 12 2017
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by rbasuvula@chromium.org
, Sep 12 2017Status: Untriaged (was: Unconfirmed)