New issue
Advanced search Search tips

Issue 870199 link

Starred by 3 users

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Wrong hyperlinks in PDF when multiline

Reported by pavel.m...@gmail.com, Aug 2

Issue description

UserAgent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/68.0.3440.84 Safari/537.36

Steps to reproduce the problem:
1. Try to print the attached HTML to PDF in Chrome.
2. See that Link1, Link2, Link3 all navigate to test.cz/link3, but they should be test.cz/link1, test.cz/link2 and test.cz/link3
3. The problem does not appear if Link3 is not split into 2 lines (try it by uncommenting style in the HTML)

The wrong PDF is attached too.

What is the expected behavior?
The expected output is correct links even with multiline hyperlinks. 

Some PDF viewers show all hyperlinks correctly, but Acrobat Reader and Chrome embedded viewer don't.

What went wrong?
The error has been reported to Skia:
https://bugs.chromium.org/p/skia/issues/detail?id=8198

It won't be fixed in Skia due to:
"Blink is calling SkAnnotateRectWithURL() with a single rect that is the bounding box of all of the `Link3` text.  It should be called twice with two smaller rects so it doesn't overlap the other links."

Did this work before? N/A 

Chrome version: 68.0.3440.84  Channel: stable
OS Version: Ubuntu 18.04
Flash Version:
 
bad_links.pdf
45.9 KB Download
multiline_hyperlink.html
373 bytes View Download
Labels: Needs-Triage-M68
Cc: halcanary@google.com
Components: Internals>Printing Internals>Compositing
Status: Untriaged (was: Unconfirmed)
https://bugs.chromium.org/p/skia/issues/detail?id=8198 says this is due to the SkAnnotateRectWithURL() calls, which are all in cc/paint, so over to Internals>Compositing for triaging.
Cc: enne@chromium.org
Components: -Internals>Compositing Blink>Paint
This forwards directly from GraphicsContext::SetUrlForRect with no coalescing, so passing off to paint.
Labels: OS-Android OS-Chrome OS-Mac OS-Windows
Status: Available (was: Untriaged)
This might be tricky to fix. We'll see.
Cc: -halcanary@google.com

Sign in to add a comment