New issue
Advanced search Search tips

Issue 622137 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Disappearing TABs at some zoom-levels, with white-space: pre-wrap

Reported by eric.rannaud@gmail.com, Jun 22 2016

Issue description

UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/51.0.2704.84 Safari/537.36

Example URL:
badtab4.html in attachment

Steps to reproduce the problem:
1. Open badtab4.html in attachment
2. Change zoom level

What is the expected behavior?
At all zoom levels, there should be a red rectangle on each row (div.line) (5 rows in total).

What went wrong?
At zoom levels <=50%, 75%, 110%, 175%, 200%, 300%, the only lines that show a red box are:
- with white-space: pre, not pre-wrap
- with less than two TABs (&#9;) before the SPAN

At all the other zoom levels, each line has a red rectangle.

And lines with only one TAB before the SPAN (which itself contains a TAB) are fine.

Does it occur on multiple sites: N/A

Is it a problem with a plugin? No 

Did this work before? Yes I think it worked before Chromium 51, but I'm not 100% positive.

Does this work in other browsers? Yes 

Chrome version: 51.0.2704.84  Channel: n/a
OS Version: Linux nc052 4.6.2-1-ARCH #1 SMP PREEMPT Wed Jun 8 08:40:59 CEST 2016 x86_64 GNU/Linux
Flash Version:
 
badtab4.html
664 bytes View Download
Cc: svil...@igalia.com f...@opera.com
Components: Blink
Labels: M-51 OS-Mac OS-Windows
Owner: yoshiki@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce the issue on Ubuntu 14.04,Win 7 and Mac 10.11.5 using latest stable 51.0.2704.103.
Its working fine on latest Beta # 52.0.2743.41 and canary 53.0.2774.3.

Reverse Bisect info:
===================
Good:52.0.2716.0
Bad :52.0.2715.0
Change Log:
============
https://chromium.googlesource.com/chromium/src/+log/67d8ccb45177b214dbdb51375f1ed1ba31ffa07b..c7c0d3ab719b356d8ccef636506507b2cead7ff1

Possible CLs to merge to M51:
============================
1) https://chromium.googlesource.com/chromium/src/+/7b1b0f17c7a1eac0283fc9daa519328a42e0139f
2) https://chromium.googlesource.com/chromium/src/+/60c83f745a5192816cffd56d4b7a41c992c5328d
3) https://chromium.googlesource.com/chromium/src/+/fe8421b9573ca696e596248811edf0a5d23fc191

yoshiki@ : Unable to find a proper Cl to be resolving this,still assigned to you(based on Suspect #2). Could you please help on this further if a merge is required here to M51.
svillar@igalia.com, fs@opera.com : cced you both if your changes are related to this.

Comment 2 by f...@opera.com, Jun 22 2016

Cc: e...@chromium.org
In that range of commits, https://chromium.googlesource.com/chromium/src/+/957ee1cf9558f59e65c943a85fab2ec7ce94ebe2 looks more likely to have made a difference here. +eae

Comment 3 by pdr@chromium.org, Jun 23 2016

Cc: -e...@chromium.org kojii@chromium.org
Components: -Blink Blink>Layout
Labels: -Type-Compat Type-Bug-Regression
Owner: e...@chromium.org
Because "&#9;" is a tab character I think this is https://chromium.googlesource.com/chromium/src/+/957ee1cf9558f59e65c943a85fab2ec7ce94ebe2 after all.

Comment 4 by e...@chromium.org, Jul 15 2016

Status: Fixed (was: Assigned)
Fixed in 52.

Sign in to add a comment