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

Issue 658932 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Compat



Sign in to add a comment

display of New York Times article with complicated HTML is messed up

Project Member Reported by dbloch@google.com, Oct 24 2016

Issue description

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

Example URL:
http://www.nytimes.com/interactive/2016/10/24/world/asia/living-in-chinas-expanding-deserts.html?_r=1

Steps to reproduce the problem:
1. Go to http://www.nytimes.com/interactive/2016/10/24/world/asia/living-in-chinas-expanding-deserts.html?_r=1

What is the expected behavior?
Fairly complicated.  It shows a video, which takes up the whole tab.  You can scroll through the article with any of the usual methods, and it has text, images, more videos, and a map with animation.

What went wrong?
Starts by displaying a map from near the bottom.  Scrolling doesn't have any effect until you get half-way through the vertical size of the article, then the contents are garbled.

Does it occur on multiple sites: N/A

Is it a problem with a plugin? No 

Did this work before? Yes 54.0.2840.71 (Official Build) (64-bit)

Does this work in other browsers? Yes

Chrome version: 55.0.2883.21  Channel: beta
OS Version: 
Flash Version: Shockwave Flash 23.0 r0

Fails the same way on Windows.
 

Comment 1 by hdodda@chromium.org, Oct 25 2016

Cc: hdodda@chromium.org
Components: UI
Labels: -Pri-2 -Type-Compat hasbisect-per-revision ReleaseBlock-Stable M-56 OS-Mac OS-Windows Pri-1 Type-Bug-Regression
Owner: trchen@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build:55.0.2844.0(Revision: 415023).
Bad build: 55.0.2845.0: (Revision:415522).

You are probably looking for a change made after 415439 (known good), but no later than 415440 (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/540399d5f34abfcae1828e2c36835ffae391e319..2ef7df4c03595db39c1b0c303ec8a822f0bf30c9

From the CL above, assigning the issue to the concern owner 

@trchen - 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.

Review-Url: https://codereview.chromium.org/2295483003

Note: Adding RB-Stable label since this is a recent regression, please feel free to edit it if this not the case.

Thanks !
Cc: ranjitkan@chromium.org
Just to update, still able to reproduce the issue on Windows 10, MAC 10.11.6 for chrome version 56.0.2900.0. Issue is marked with a blocker label and M55 is already in Beta.

@trchen: request you to please take a look into it and provide an update on it.

Thanks.!
Cc: shrike@chromium.org
Components: -UI Blink
Labels: -M-56 M-55
This is a M55 regression, it should block stable for M55, not M56.
Components: -Blink Blink>Compositing

Comment 5 by gov...@chromium.org, Oct 26 2016

**** Bulk edit -  please ignore if not applicable ****

A friendly reminder that M55 Stable is launch is coming soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch ASAP so it gets enough baking time in Beta (before Stable promotion). Thank you!

Comment 6 by trchen@chromium.org, Oct 26 2016

Labels: -ReleaseBlock-Stable
Status: ExternalDependency (was: Assigned)
Will not fix. Working as intended.

Their "picture book" effect abused a compositing bug previously in WebKit/Blink. It worked in Firefox because their framework applied different CSS to workaround the correct behavior in Firefox.

Full story: (you may TL;DR)

Overflow clip should not affect elements that are not positioned by it. For example:
<div style="position:relative; overflow:hidden">
    <div style="position:fixed"></div>
</div>
The fixed-pos element will ignore the clip. Prior to my CL, our compositing code path made a mistake not to escape clip correctly. (The software path wasn't affected by this issue.)

To achieve the intended behavior, CSS clip can be used instead. Unlike overflow clip, CSS clip affect all descendants. So something like this can be done:
<div id="g-map-sandstorms">
  <div style="position:absolute; left:0; top:0; width:100%; height:100%; clip:rect(auto,auto,auto,auto);">
    <div class="g-text-block">...</div>
    <div class="g-media g-ai2html-poster">...</div>
  </div>
</div>

Comment 7 by trchen@chromium.org, Oct 26 2016

Labels: -OS-Linux -OS-Windows -Pri-1 -Type-Bug-Regression -OS-Mac OS-All Pri-2 Type-Compat
I've reached out to techfeedback@nytimes.com . Let me know if we've worked with them before and know a contact person.
Just to to update the latest behavior of the bug, Issue is still observed on chrome latest Canary M57-57.0.2926.0

trchen @ Could you please let us know is there any recent update available on this issue?

Thanks!

Comment 9 by trchen@chromium.org, Nov 22 2016

Status: WontFix (was: ExternalDependency)
Didn't get a reply from them. Nothing we can do if they don't fix their own bug.

Sign in to add a comment