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

Issue 736001 link

Starred by 3 users
Status: Assigned
Owner:
weili@chromium.org
Cc:
jam@chromium.org
clamy@chromium.org
weili@chromium.org
thestig@chromium.org
creis@chromium.org
dcheng@chromium.org
arthurso...@chromium.org
carlosk@chromium.org
█ nasko@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 3
Type: Bug



Sign in to add a comment

iframes not printing when selecting part of a page

Project Member Reported by jam@chromium.org, Jun 22 2017 
Branching from  bug 732780  (where we discovered this caused a crash with plznavigate, but is broken with both).


repro:
(1) Visit www.phrases.org.uk/meanings/proverbs.html
(2) Select the same area as in the video in order to select at least the ads (iframe).
(3) Right click > Print...

Lei found this regressed in r414879.

This is probably low priority, since we haven't noticed it was broken for 10 months. Probably most times this happens the iframe is an ad.

Comment 1 by thestig@chromium.org, Jun 22 2017

Components: Internals>Printing
Labels: OS-Chrome OS-Linux OS-Mac OS-Windows
I haven't looked into why r414879 caused this to fail. Off the top of my head, the PrintWebViewHelper (really should be renamed to PrintWebFrameHelper someday) that is generating the PDF for printing does not use any extension functionality. Will need to look into it.

BTW, one can make much nicer webpages for testing, rather than using the one listed above with ads.

Comment 2 by nasko@chromium.org, Jun 29 2017 

We usually use http://csreis.github.io/tests/, which has a bunch of different test pages with same site and cross-site iframes. Feel free to use those.

Comment 3 by shrike@chromium.org, Jul 6 2017

Owner: thestig@chromium.org
Status: Untriaged (was: Unconfirmed)
[MacTriage]

Comment 4 by weili@chromium.org, Jul 11 2017

Owner: weili@chromium.org
Status: Assigned (was: Untriaged)
Put into my queue for checking
►

Sign in to add a comment