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

Issue 650952 link

Starred by 2 users
Status: Duplicate
Merged: issue 631513
Owner:
█ nasko@chromium.org
Out until 24 Jan
Closed: Oct 2016
Cc:
rob@robwu.nl
nyerramilli@chromium.org
█ ranjitkan@chromium.org
wfh@chromium.org
creis@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Print preview does not get open for embedded pdf after adding 'PDF viewer extension'.

Reported by rk...@etouch.net, Sep 28 2016 
Chrome Version:55.0.2873.3 Revision 714c807de9a9710084add5fddbaa72d02a91096e-refs/branch-heads/2873@{#4}
OS: Windows(7,8,10),Mac(10.10.5,10.11.4)

URL: https://chrome.google.com/webstore/detail/pdf-viewer/oemmndcbldboiebfnladdacbdfmadadm/related?hl=en

What steps will reproduce the problem?
(1) Launch chrome,navigate to above url and click on 'ADD TO CHROME' button 
(2) Then navigate to https://msu.edu/~urban/sme865/resources/embedded_pdf.html and click on 'Print' option.
(3) Observe.

Actual: Print preview does not gets opened for pdf after adding PDF viewer extension.

Expected: Print preview should seen.

This is a regression issue broken in 'M-55', will soon update the other info:

Comment 1 by rk...@etouch.net, Sep 28 2016

Labels: OS-Linux
Good Build: 55.0.2841.0
Bad Build: 55.0.2842.0
Actual_Print.mp4
1.0 MB View Download
Expected_Print.mp4
711 KB View Download

Comment 2 by brajkumar@chromium.org, Sep 28 2016

Components: UI>Browser>PrintPreview
Labels: hasbisect-per-revision
Owner: jdufault@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build:55.0.2841.0(Revision: 414854).
Bad build: 55.0.2842.0(Revision: 414942).

You are probably looking for a change made after 414882 (known good), but no later than 414883 (first known bad).

CHANGELOG URL:
-----------------
https://chromium.googlesource.com/chromium/src/+log/f05e88ae004f438fcd8fb2023b51ba19a7903e48..390ccf8ff50b6ff69c2339076a410c7c7f70a357

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

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

Thanks!

Comment 3 by jdufault@chromium.org, Sep 28 2016

Owner: brajkumar@chromium.org
There are quite a few CLs within the good/bad tags. I'd recommend continuing the bisect on a per-commit basis (instead of a per-tag basis) as the CL you linked to is entirely unrelated to this. Glancing through the commit list I didn't see anything obvious that would be responsible for this bug.

Comment 4 by thestig@chromium.org, Oct 6 2016

Cc: rob@robwu.nl
Owner: ----
Status: Untriaged (was: Assigned)
I'll bisect again...

Comment 5 by thestig@chromium.org, Oct 6 2016

Summary: Regression: Print preview does not get open for embedded pdf after adding 'PDF viewer extension'. (was: Regression: Print preview does not get open for emmedded pdf after adding 'PDF viewer extension'.)

Comment 7 by rob@robwu.nl, Oct 6 2016

Components: Internals>Sandbox>SiteIsolation
Minimal repro:

Start server at localhost port 8080, start Chrome with and without --site-per-process (with the flag print preview is broken):

frame.html
Calling print();
<script>print();</script>

index.html                                                                       
<iframe src="http://127.0.0.1:8080/frame.html"</iframe>

Comment 8 by alex...@chromium.org, Oct 6 2016

Mergedinto: 631513
Status: Duplicate (was: Untriaged)

Comment 9 by rob@robwu.nl, Oct 13 2016 

FYI this issue is also tracked at https://github.com/mozilla/pdf.js/issues/7706

With a work-around that is far from beautiful.
►

Sign in to add a comment