Regression : PDF page turns black for a moment on resizing it.
Reported by
rp...@etouch.net,
Oct 30 2017
|
|||||||||||||
Issue descriptionVersion: 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
,
Oct 30 2017
Tagging with blocker label, please undo if not the case.
,
Oct 30 2017
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.
,
Oct 30 2017
With response to comment #3 : Re-bisected this issue on different windows machine and getting the same range as mentioned in comment #1.
,
Oct 30 2017
This is probably related to my change. I will investigate today.
,
Oct 31 2017
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.
,
Oct 31 2017
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.
,
Nov 1 2017
,
Nov 1 2017
,
Nov 2 2017
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.
,
Nov 7 2017
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?
,
Dec 12 2017
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.
,
Sep 4
Setting PDF bugs assigned to me back to untriaged so they can get re-assigned as needed.
,
Sep 5
Since this is just in devtools, I don't think it's worth fixing. |
|||||||||||||
►
Sign in to add a comment |
|||||||||||||
Comment 1 by rp...@etouch.net
, Oct 30 2017Owner: kojii@chromium.org
Status: Assigned (was: Unconfirmed)