Print selection is missing some contents / pages
Reported by
alex131...@gmail.com,
Oct 10 2016
|
|||||||||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/53.0.2785.143 Safari/537.36 Example URL: https://reflets.info/le-monde-selon-telegram/ Steps to reproduce the problem: 1. Open the URL 2. Select the content of the article, from "Par Jef Mathiot 6 octobre 2016" to "Excellent conseil." 3. Right-click > Print // Sandwich menu > Print 4. Destination is PDF 5. Ensure "Print selection" is checked What is the expected behavior? The preview display the ~9 pages with all selected content, so is the resulting PDF. What went wrong? The preview displays only 4 pages, stopping to "et impose l’utilisation du DC auquel l’utilisateur est associé.", so is the resulting PDF. Does it occur on multiple sites: N/A Is it a problem with a plugin? No Did this work before? N/A Does this work in other browsers? N/A Chrome version: 53.0.2785.143 x64 Channel: stable OS Version: 10.0 x64 Flash Version:
,
Mar 9 2017
Took a quick look. The code PrepareFrameAndViewForPrint::CopySelection() is copying all the selected text. It then escape the text and append it to: data:text/html;charset=utf-8, to make a long data: URL. Then it creates a new blink::WebView, sets that as the WebFrame, and tell it to load the data: URL. I have no idea what happens after that inside of Blink though.
,
Mar 10 2017
,
Mar 11 2017
Uh, this is definitely a printing + Blink integration issue and I need help from the Blink side to help understand what's wrong. So here's another wild stab at setting the Blink component.
,
Mar 11 2017
,
Mar 11 2017
nainar@, any idea what's happening here? If this isn't related with your work, I'll try other labels, please let me know.
,
Mar 13 2017
I have zero expertise in loading code, sorry. thestig@ could you provide function calls on the Chromium side of things? I can possibly take it from there to explore. Definitely not CSS though.
,
Mar 13 2017
Sorry, I thought PrintBrowser is in your team. It looks like issue 699841 was for paint, is this something paint can look into?
,
Mar 13 2017
Blink paint is responsible for generating the paint command stream. Skia generates the pdf.
,
Mar 13 2017
I mentioned the code in comment 2. Please take a look at PrepareFrameAndViewForPrint::CopySelection().
,
Mar 13 2017
I have some familiarity here, will look soon.
,
Mar 13 2017
wkorman: Thank you for taking a look. Some of that code dates back to 2009, so it's "just the way we've always done it" which may not make perfect sense in 2017.
,
May 11 2017
Unfortunately have not been able to get to this. Unassigning self as not clear will be able to look soon.
,
May 14 2018
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
May 14 2018
The page seems to be behind a paywall now. Closing the bug due to the absence of a way to test/verify. |
|||||||||||||
►
Sign in to add a comment |
|||||||||||||
Comment 1 by thestig@chromium.org
, Oct 10 2016Labels: -Type-Compat -Arch-x86_64 OS-Chrome OS-Linux OS-Mac Type-Bug
Status: Available (was: Unconfirmed)