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

Issue 779459 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner: ----
Closed: Sep 5
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 3
Type: Bug-Regression



Sign in to add a comment

Regression : PDF page turns black for a moment on resizing it.

Reported by rp...@etouch.net, Oct 30 2017

Issue description

Version: 64.0.3253.0 (32/64-bit)3163a676b23fd74bf3d1c5c13a1141d26d8dd6aa-refs/heads/master@{#512377}
OS: Windows (7,8,8.1,10),Linux (14.04 LTS),Mac OS X(10.12.6,10.13.1)
URL : http://unec.edu.az/application/uploads/2014/12/pdf-sample.pdf

What steps will reproduce the problem?
1. Launch chrome, navigate to above url and open devtools.
2. Now open emulated view and try to resize PDF page from moving it to Left and right,observe

Actual: PDF page turns black for a moment on resizing it 
Expected: PDF page should not turn black for a moment on resizing it 

This is regression issue, broken in ‘M 64’ and will soon update other info :
Good build:64.0.3251.0
Bad build: 64.0.3252.0
 
Actual_video.mov
5.7 MB Download
Expected_video.mov
5.4 MB Download

Comment 1 by rp...@etouch.net, Oct 30 2017

Labels: hasbisect
Owner: kojii@chromium.org
Status: Assigned (was: Unconfirmed)
Narrow Bisect info : 
https://chromium.googlesource.com/chromium/src/+log/d7cf0e8a1e7dfdc71476a2ec4169c4d4d48fc943..85a1bf73c32a1cda0d34fceb7a20f96b26bba035?pretty=fuller&n=10000

Suspecting: r512173 from Narrow bisect

@kojii : Could you please help to reassign if your change is not the cause for this change.

Note : Unable to bisect using per-revision bisect hence providing bisect using old script.
Labels: ReleaseBlock-Stable
Tagging with blocker label, please undo if not the case.

Comment 3 by kojii@chromium.org, Oct 30 2017

Cc: kojii@chromium.org
Labels: -hasbisect Needs-Bisect
Owner: fsam...@chromium.org
Not that one, it affects only when LayoutNG runtime flag is enabled, and it's off by default unless the user turns it on by chrome:flags.

This looks like a paint issue, but I can't find anything in the range.

fsamuel@, any ideas? If not, it'd be great to try another bisect. I added the label.

Comment 4 by rp...@etouch.net, Oct 30 2017

Labels: -Needs-Bisect hasbisect
With response to comment #3 :
Re-bisected this issue on different windows machine and getting the same range as mentioned in comment #1.

Comment 5 by fsamuel@google.com, Oct 30 2017

This is probably related to my change. I will investigate today.
Owner: dsinclair@chromium.org
I'm not sure this is actually a bug. It seems like before we didn't update ScreenInfo when you resize using dev tools and so nothing happened now we do, and so layout causes flicker. Reassigning to dsinclair@ for triage.
Labels: -ReleaseBlock-Stable
Owner: ----
Status: Untriaged (was: Assigned)
If this is just flickering when resized in devtools, I don't think it needs to be RBS. Removing label and sending to the PDF triage queue.
Components: -Platform>DevTools

Comment 9 by kojii@chromium.org, Nov 1 2017

Cc: -kojii@chromium.org
Cc: fsam...@chromium.org
Labels: -Pri-1 -M-64 Pri-3
Is this something that's easy to fix?

Even in the "before" video, the non-content area is already painted black upon resizing. It's just now that sometimes the content area goes black too. As most users aren't doing this with DevTools open, I'm going to lower the priority.
Cc: -fsam...@chromium.org
Owner: fsam...@chromium.org
Status: Assigned (was: Untriaged)
Is the updating of ScreenInfo something that the pdf plugin controls? or is this done at a higher level?

i.e., is there anything we can do at the pdf plugin level to fix this?
Owner: dsinclair@chromium.org
The propagation of ScreenInfo happens in BrowserPlugin. I'm not sure why the PDF is flashing though. It may be doing a full repaint which maybe isn't necessary? I'm honestly not sure. Passing back to dsinclair@ to investigate. If you can verify that it's not a PDF bug then I'll investigate on the compositing side. At the moment, only PDF does this.
Owner: ----
Status: Untriaged (was: Assigned)
Setting PDF bugs assigned to me back to untriaged so they can get re-assigned as needed.
Status: WontFix (was: Untriaged)
Since this is just in devtools, I don't think it's worth fixing.

Sign in to add a comment