New issue
Advanced search Search tips

Issue 882867 link

Starred by 3 users

Issue metadata

Status: WontFix
Owner:
Closed: Sep 13
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Mac
Pri: 2
Type: Bug



Sign in to add a comment

[Flash] Unable to print through system dialog

Reported by xlro...@gmail.com, Sep 11

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_6) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/69.0.3497.81 Safari/537.36

Steps to reproduce the problem:
1. Launch Chrome and goto https://storyclassic.adobe.com
2. When asked, choose to allow flash player plugin to run on this site
3. Sign in with this dummy account (email: roverma+chrome@adobetest.com, password: Chrome@123)
4. After entering this credentials you will be on a page with loading animation. You will notice a prompt in the URL bar for flash plugin. Please do allow it to run on this page. Otherwise site won't load.
5. Once you are in, click on the 'PROJECTS' button and then double click on 'Paladin Script' to open that document
6. Once that document is successfully open, please choose 'Print' option from Application's 'File' menu (not the Chrome's one).
7. Click on 'OK' button in the subsequent dialog.
8. In the Chrome's print preview dialog, please choose 'Print using system dialog'
9. Click on 'PDF' dropdown in the next system dialog and choose 'Open in Preview'
10. In a new tab open this URL chrome://flags/#site-isolation-trial-opt-out and set it to Opt-out
11. Relaunch Chrome and repeat steps 1-9

What is the expected behavior?
In previous version of Chrome (v68.0.3440.106), print used to work when setting the site-isolation flag to Opt-out. Now that is broken too.

What went wrong?
Unable to print via system dialog

Did this work before? Yes v68.0.3440.106

Chrome version: 69.0.3497.81  Channel: stable
OS Version: OS X 10.13.6
Flash Version: 

NOTE: This issue is very much similar to this issue: https://bugs.chromium.org/p/chromium/issues/detail?id=859481
With only difference being that the workaround that was suggested in that bug (to opt out from site-isolation flag), is no longer working
 
Screen-recording of the above steps
ChromeIssue480p.mov
16.7 MB Download
We will really appreciate if you can tell us some workaround to this issue.
Labels: Needs-Bisect Needs-Triage-M69
Components: Internals>Printing
Cc: pbomm...@chromium.org thestig@chromium.org gov...@chromium.org
Cc: kenrb@chromium.org creis@chromium.org
Components: Internals>Sandbox>SiteIsolation
Owner: weili@chromium.org
Status: Assigned (was: Unconfirmed)
Tentatively assigning to weili@ / assumming that this might be related to UsePdfCompositorServiceForPrint.

xlrohit@ - does the issue repro if you launch Chrome with the following cmdline flag: --disable-features=UsePdfCompositorServiceForPrint?
This is effectively  bug 859481  for Chrome 69, right? Chrome 67 and Chrome 68 has an opt out option. Chrome 70 has this fixed. Chrome 69 is caught in the middle.

Assuming this is the case, one possibility is to merge the fix, r577712, to Chrome 69.
Owner: thestig@chromium.org
Based on #8, re-assign to thestig@
Cc: weili@chromium.org
weili: Do you know why chrome://flags/#site-isolation-trial-opt-out stopped working?
Cc: swarnasree.mukkala@chromium.org
Labels: -Needs-Bisect ReleaseBlock-Stable Triaged-ET M-69 Target-69 FoundIn-69
Tested the issue on reported chrome version #69.0.3497.81, latest dev #70.0.3538.9 and latest canary 71.0.3549.0 on Mac 10.13.6 and below are the observations.

Observations:
===========
1.Able to reproduce the issue on reported chrome version #69.0.3497.81, observed that after clicking 'Open in Preview', no action is being performed.
2.Unable to reproduce the issue on both latest M-70 (#70.0.3538.9) and latest canary #71.0.3549.0, observed that after clicking 'Open in Preview', it opens the file in preview.

As per comment#8, the root cause of issue seems to be identified, hence removing Needs-Bisect label. Requesting thestig@ to merge the issue to M-69, so adding ReleaseBlock-Stable label.
Note: Please feel free to remove the ReleaseBlock-Stable label if it is not applicable.

Thanks.!
@lukasza: Thanks for the workaround! By disabling pdf-compositor-service-for-print I am able to use print options in system dialog (print, open in preview and save as pdf)

Tried both of the following ways to disable the service and it was working afterwards.
 - launching Chrome with this cmdline flag: --disable-features=UsePdfCompositorServiceForPrint 
 - by disabling the same via chrome://flags/#use-pdf-compositor-service-for-print
RE: #c10: thestig@: Why chrome://flags/#site-isolation-trial-opt-out stopped working?

Because to turn off print compositing, *both* 1) Site Isolation and 2) kUsePdfCompositorServiceForPrint feature need to be disabled:

//components/printing/browser/print_manager_utils.cc:
    void CreateCompositeClientIfNeeded(content::WebContents* web_contents) {
    ...
        if (content::SiteIsolationPolicy::ShouldPdfCompositorBeEnabledForOopifs() ||
          base::FeatureList::IsEnabled(
              features::kUsePdfCompositorServiceForPrint)) {
        PrintCompositeClient::CreateForWebContents(web_contents);
        SetOopifEnabled();
      }
    }

r565018 (landed in 69.0.3452.0) made kUsePdfCompositorServiceForPrint turned on by default, which meant that turning off Site Isolation is not sufficient anymore for turning off print compositing.
bug reporter: Looks like we found a workaround. Are you ok with the workaround for Chrome 69? You won't need it in Chrome 70.

re: comment 13: Right, thanks for reminding me.
@thestig: Yes, I am okay with the workaround as this issue won't be there in v70.
Labels: -M-69 -Target-69 Target-70 Target-71 M-71 M-70
Per comments #14 and #15, not a blocker for M69. Pls target fix for M70.
Status: WontFix (was: Assigned)
bug reporter: Thanks for letting us know. We won't take any actions on this issue then.

Please also consider trying out Chrome Beta Channel. If you find issues on the Beta Channel and report them, we can fix them earlier, so the issues never affect most users who are on Stable Channel.
@thstig: Thanks! I will make sure to check Beta channel periodically.

Sign in to add a comment