Regression: Buttons on screenshot appears misplaced in ' Network' section of devtools.
Reported by
rp...@etouch.net,
Sep 7 2017
|
|||||||||
Issue descriptionVersion: 63.0.3208.0 5bfe3ad1decdfd840726e31694347e6d973a3f09-refs/heads/master@{#500160} OS: Windows (7,8,8.1,10),Linux (14.04 LTS,Mac OS X(10.11.6,10.12.3) What steps will reproduce the problem? 1. Launch chrome, navigate to NTP and open devtools. 2. Now go to 'Network' section and click on 'Capture screenshot' icon and then press 'F5' to reload and capture filmstrip. 3. Now double click to zoom image and drag devtools window to RHS and press tab key to bring focus on 'Next frame' button on screenshot,observe Actual: Buttons(Previous frame and Next frame) on screenshot appears misplaced after pressing 'Tab' key Expected: Buttons(Previous frame and Next frame) on screenshot shouldn't appear misplaced after pressing 'Tab' key This is regression issue, broken in ‘M 58’ and will soon update other info : Good build:58.0.3017.0 Bad build: 58.0.3018.0
,
Sep 7 2017
Using the per-revision bisect providing the bisect results, Good build: 58.0.3017.0 (Revision: 451477). Bad build: 58.0.3018.0 (Revision: 451537). You are probably looking for a change made after 451522 (known good), but no lat er than 451523 (first known bad). CHANGELOG URL: The script might not always return single CL as suspect as some perf builds might get missing due to failure. https://chromium.googlesource.com/chromium/src/+log/0fea64cb0a598ecd2051bb7c58e6fbf9422b1aef..3e3774f369e1e642d06e895276b08814b279c20c?pretty=fuller&n=100 From the CL above, assigning the issue to the concern owner @dgozman- 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. Suspect : https://chromium.googlesource.com/chromium/src/+/3e3774f369e1e642d06e895276b08814b279c20c Thanks!
,
Sep 8 2017
Erik, mind taking a look?
,
Nov 30 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/e105595288ee836d3d793e7aec419cb40d3de10d commit e105595288ee836d3d793e7aec419cb40d3de10d Author: Erik Luo <luoe@chromium.org> Date: Thu Nov 30 03:01:24 2017 DevTools: add scroller to filmstrip dialog image By placing the image inside a scrolling container, none of the dialog's children will overflow after this CL. Screenshot: https://imgur.com/a/CIkCU Bug: 762893 Change-Id: I8ba3dc7cc09e629f09917c1fa24309c698777d3f Reviewed-on: https://chromium-review.googlesource.com/772787 Commit-Queue: Erik Luo <luoe@chromium.org> Reviewed-by: Andrey Lushnikov <lushnikov@chromium.org> Cr-Commit-Position: refs/heads/master@{#520402} [modify] https://crrev.com/e105595288ee836d3d793e7aec419cb40d3de10d/third_party/WebKit/Source/devtools/front_end/perf_ui/FilmStripView.js [modify] https://crrev.com/e105595288ee836d3d793e7aec419cb40d3de10d/third_party/WebKit/Source/devtools/front_end/perf_ui/filmStripDialog.css
,
Nov 30 2017
,
Dec 1 2017
As per Comment# 6 adding appropriate TE-Verified labels. Thank You.
,
Dec 1 2017
,
Dec 1 2017
,
Dec 1 2017
Apologies for the wrong comments# 7,8,9. Removing TE-Verified labels.
,
Dec 1 2017
Thanks for the verification, it's not fixed... I'll take another look.
,
Dec 8 2017
Extreme UI conditions. |
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 Deleted