Regression : Unwanted movement of text message is seen in 'Application' section of devtools.
Reported by
rp...@etouch.net,
Mar 30 2017
|
||||||||||||
Issue descriptionVersion: 59.0.3056.0 abd1360936725f296381ceb3c194307a29137c53-refs/heads/master@{#460603} OS: Windows (7,8,8.1,10),Linux (14.04 LTS),Mac OS X(10.11.6,10.12.1) What steps will reproduce the problem? 1. Launch chrome, navigate to NTP and open devtools 2. Now go to 'Application' section and hover mouse over 'Top' option under 'Frames' section and at the same time observe 'No manifest detected' message in RHS Actual: Unwanted movement of text message is seen after mouse is hovered upon 'Top' option at first instance. Expected: Unwanted movement of text message should not be seen after mouse is hovered upon 'Top' option at first instance. This is regression issue, broken in ‘M 59’ and will soon update other info : Good build:59.0.3048.0 Bad build: 59.0.3049.0
,
Mar 30 2017
,
Mar 30 2017
Adding RB Label as this is a recent Regression. Please remove if not required. Thank You.
,
Apr 5 2017
I am not able to reproduce it. Could it be that it was fixed or is it Windows specific? Also, I don't think this blocks the release.
,
Apr 5 2017
Issue 703080 has been merged into this issue.
,
Apr 5 2017
With response to comment #4 : Rechecked the above issue on Windows OS with latest canary chrome version : 59.0.3062.0 and the issue is still reproducible.Kindly refer the attached screen cast for your reference
,
Apr 5 2017
Ok, I managed to reproduce this. Seems to require specific width of the devtools panel.
,
Apr 5 2017
Looks like a layout bug.
Here's what happens:
1. Line from a mouse event handler dies this.listItemElement.classList.add('hovered');
2. This triggers CSS rule that makes the nested span visible with the change display: (none -> block).
No relayout is caused when the class is removed.
I only see this when the class is added from a mouse handler - e.g. doing the same from the console does not trigger the layout (even though the span becomes visible)
,
Apr 6 2017
There is only a single change in the regression range and it specifically involves the pane in question: https://chromium.googlesource.com/chromium/src/+/8f7c3a1ade6b01cb90a8648d1e4dbfeab14bd8a5
,
Apr 6 2017
The change in question only fixed a bug in DevTools that was hiding the layout issue. Jumping is caused by a style change in the onmousemove handler. Positioning does not revert to the previous state after the style change is reverted (as the class gets removed).
,
Apr 6 2017
,
Apr 7 2017
This is a P1 regression where the CL that caused the regression is known. Whether the root cause is elsewhere is irrelevant. Our policy is to fix the manifestation of the regression (usually by reverting the relevant change) and then look into the root cause when the time pressure has been lifted. While this might be a layout bug it seems somewhat unlikely given the bug only manifests when triggered from a mouse handler, layout doesn't care about the cause of a layout. Switching between display: none and block is also a very common thing to do, again suggestion that there is more to it. Either way, if you believe this is layout issue then please file a separate bug for that and assign it to me or leave it Untriaged. *This* bug is about a P1 regression in devtools caused by https://chromium.googlesource.com/chromium/src/+/8f7c3a1ade6b01cb90a8648d1e4dbfeab14bd8a5
,
Apr 7 2017
As far as DevTools is concerned, this is a P3. It seems to be a more general issue with the layout. I defer to you to prioritize.
,
Apr 7 2017
Closing as WontFIX based on comment 13. |
||||||||||||
►
Sign in to add a comment |
||||||||||||
Comment 1 by rbasuvula@chromium.org
, Mar 30 2017Labels: hasbisect-per-revision
Owner: eostroukhov@chromium.org
Status: Assigned (was: Unconfirmed)