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

Issue 616167 link

Starred by 2 users
Status: Fixed
Owner:
█ robhogan@chromium.org
Use other robhogan account instead.
Closed: Jul 2016
Cc:
█ kavvaru@chromium.org
mustaq@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Firefox, IE, and Edge all collapse a hidden div on the left of the linked page. On Chrome it does not collapse.

Reported by rama...@gmail.com, May 31 2016 
Chrome Version       : 51.0.2704.63 m
URLs (if applicable) : https://jsfiddle.net/307vnezk/1/
Other browsers tested:
  Add OK or FAIL, along with the version, after other browsers where you
have tested this issue:
     Safari: 
    Firefox: OK - 46.0.1
         IE: OK - 11.306.10586.0
    MS Edge: OK - 25.10586.0.0

What steps will reproduce the problem?
(1) Visit https://jsfiddle.net/307vnezk/1/
(2) Click "header 2"
(3) Click "header 2" again

What is the expected result?

     On Firefox, IE, and Edge, "header 3"  moves back up when the elements under "header 2" are hidden.

What happens instead?

     On Chrome, "header 3" stays far beneath "header 2", even though the elements between "header 2" and "header 3" are hidden.


Please provide any additional information below. Attach a screenshot if
possible.

Attached is a screenshot, as well as a standalone HTML file which can be used to reproduce this difference.
chrome_rendered_differently.png
53.6 KB View Download
chrome_renders_differently.html
2.7 KB View Download

Comment 1 by kavvaru@chromium.org, Jun 1 2016

Cc: kavvaru@chromium.org
Components: Blink
Labels: -Type-Bug -Pri-3 M-53 hasbisect OS-Linux OS-Mac OS-Windows Pri-2 Type-Bug-Regression
Owner: mustaq@chromium.org
Status: Assigned (was: Unconfirmed)
able to reproduce the issue on windows 7, Linux Ubuntu 14.04 and Mac 10.11.5 using chrome version 51.0.2704.63 and canary 53.0.2754.0.
This is regression issue broken in M46.Please find the bisect information as below.

Narrow bisect::
Good ::46.0.2489.0  --   (official build 344633)
Bad:: 46.0.2490.0  --   (official build 344925)

Tool bisect is giving all good builds.

CHANGELOG URL:  https://chromium.googlesource.com/chromium/src/+log/f0f354c6874641783563bceaa59a15e261f25b95..67dbfc77c12dcb8a14dc86aca6646e36d26d4cb4

Blink CL::
https://chromium.googlesource.com/chromium/blink/+log/68814c3..57a9141

Possible suspect from the above blink CL
https://chromium.googlesource.com/chromium/blink/+/57a91411faaa4a3f1b9717212b258d225290ac56

mustaq@ Could you please look into this issue if it is related to your change,else please route this to an appropriate dev person.

Thanks,

Comment 2 by mustaq@chromium.org, Jun 3 2016

Labels: Hotlist-Input-Dev

Comment 3 by cbiesin...@chromium.org, Jun 6 2016

Cc: mustaq@chromium.org
Components: -Blink Blink>Layout
Owner: robhogan@chromium.org
I'd rather suspect robhogan's CL from the blink roll: https://chromium.googlesource.com/chromium/blink/+/d2666b048c44dd8b9c728df619a66da97a6c21ac

Comment 4 by mustaq@chromium.org, Jun 7 2016 

Yes, this is a layout bug. Just confirmed that reverting crrev.com/1308433003 fixes the problem locally. Thanks cbiesinger@.
Project Member

Comment 5 by bugdroid1@chromium.org, Jun 17 2016 

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

commit f1a02d60a6f16a56e35f69ab6e4efaeaf64f6820
Author: Xianzhu Wang <wangxianzhu@chromium.org>
Date: Fri Jun 17 05:04:29 2016

NeedsRebaseline for  crbug.com/616167 

BUG= 616167 

Review URL: https://codereview.chromium.org/2077793002 .

Cr-Commit-Position: refs/heads/master@{#400360}

[modify] https://crrev.com/f1a02d60a6f16a56e35f69ab6e4efaeaf64f6820/third_party/WebKit/LayoutTests/TestExpectations
Project Member

Comment 6 by bugdroid1@chromium.org, Jun 17 2016 

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

commit 6a77a19486664462e90710b5f2ac835fd0724350
Author: Rebaseline Bot <blink-rebaseline-bot@chromium.org>
Date: Fri Jun 17 06:30:47 2016

Auto-rebaseline for r400360

https://chromium.googlesource.com/chromium/src/+/f1a02d60a

BUG= 616167 
TBR=wangxianzhu@chromium.org

Review URL: https://codereview.chromium.org/2078843002 .

Cr-Commit-Position: refs/heads/master@{#400380}

[modify] https://crrev.com/6a77a19486664462e90710b5f2ac835fd0724350/third_party/WebKit/LayoutTests/TestExpectations
[modify] https://crrev.com/6a77a19486664462e90710b5f2ac835fd0724350/third_party/WebKit/LayoutTests/platform/mac/fast/block/do-not-strip-anonymous-blocks-when-block-child-becomes-float-and-other-block-on-line-expected.txt
[modify] https://crrev.com/6a77a19486664462e90710b5f2ac835fd0724350/third_party/WebKit/LayoutTests/platform/mac/fast/repaint/line-flow-with-floats-10-expected.txt
[modify] https://crrev.com/6a77a19486664462e90710b5f2ac835fd0724350/third_party/WebKit/LayoutTests/platform/mac/fast/repaint/line-flow-with-floats-3-expected.txt
[modify] https://crrev.com/6a77a19486664462e90710b5f2ac835fd0724350/third_party/WebKit/LayoutTests/platform/mac/fast/repaint/line-flow-with-floats-4-expected.txt
[modify] https://crrev.com/6a77a19486664462e90710b5f2ac835fd0724350/third_party/WebKit/LayoutTests/platform/mac/fast/repaint/line-flow-with-floats-5-expected.txt
[modify] https://crrev.com/6a77a19486664462e90710b5f2ac835fd0724350/third_party/WebKit/LayoutTests/platform/mac/fast/repaint/line-flow-with-floats-6-expected.txt
[modify] https://crrev.com/6a77a19486664462e90710b5f2ac835fd0724350/third_party/WebKit/LayoutTests/platform/mac/fast/repaint/line-flow-with-floats-9-expected.txt
[modify] https://crrev.com/6a77a19486664462e90710b5f2ac835fd0724350/third_party/WebKit/LayoutTests/platform/win/fast/block/do-not-strip-anonymous-blocks-when-block-child-becomes-float-and-other-block-on-line-expected.txt
[modify] https://crrev.com/6a77a19486664462e90710b5f2ac835fd0724350/third_party/WebKit/LayoutTests/platform/win/fast/repaint/line-flow-with-floats-10-expected.txt
[modify] https://crrev.com/6a77a19486664462e90710b5f2ac835fd0724350/third_party/WebKit/LayoutTests/platform/win/fast/repaint/line-flow-with-floats-3-expected.txt
[modify] https://crrev.com/6a77a19486664462e90710b5f2ac835fd0724350/third_party/WebKit/LayoutTests/platform/win/fast/repaint/line-flow-with-floats-4-expected.txt
[modify] https://crrev.com/6a77a19486664462e90710b5f2ac835fd0724350/third_party/WebKit/LayoutTests/platform/win/fast/repaint/line-flow-with-floats-5-expected.txt
[modify] https://crrev.com/6a77a19486664462e90710b5f2ac835fd0724350/third_party/WebKit/LayoutTests/platform/win/fast/repaint/line-flow-with-floats-6-expected.txt
[modify] https://crrev.com/6a77a19486664462e90710b5f2ac835fd0724350/third_party/WebKit/LayoutTests/platform/win/fast/repaint/line-flow-with-floats-9-expected.txt
Project Member

Comment 7 by sheriffbot@chromium.org, Jul 7 2016

Labels: -M-53 M-54 MovedFrom-53
Moving this nonessential bug to the next milestone.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 8 by rama...@gmail.com, Jul 20 2016 

Are there any easy workarounds?

Perhaps some javascript I could call to force it to re-layout the divs in this example?

Comment 9 by robhogan@chromium.org, Jul 20 2016

Status: Fixed (was: Assigned)
This is fixed now. Not sure why the CL that fixed it didn't post here.

https://codereview.chromium.org/2046863006
►

Sign in to add a comment