New issue
Advanced search Search tips

Issue 668762 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 4
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Chrome PDF Viewer : text filled with gradient doesn't use the correct opacity

Reported by antlafra...@gmail.com, Nov 26 2016

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.99 Safari/537.36

Example URL:

Steps to reproduce the problem:
Open the attached file in Chrome PDF Viewer

What is the expected behavior?
The text should be filled with a gradient and have 20% opacity

What went wrong?
The text has the gradient, but is opaque.

Does it occur on multiple sites: N/A

Is it a problem with a plugin? Yes Chrome PDF Viewer

Did this work before? N/A 

Does this work in other browsers? Yes

Chrome version: 54.0.2840.99  Channel: stable
OS Version: 10.0
Flash Version: Shockwave Flash 23.0 r0

The problem happens only with text, when it uses a gradient fill and there is a "non-stroking alpha constant" defined in the PDF document.
There is NO similar problem with rectangles and other shape.
 
fill text opacity.pdf
1.4 KB Download
Adobe reader.png
48.0 KB View Download
Chrome viewer.png
31.2 KB View Download

Comment 1 by ajha@chromium.org, Nov 27 2016

Components: Internals>Plugins>PDF
Labels: M-54
Labels: -Type-Compat -M-54 M-56 Type-Bug
Status: Untriaged (was: Unconfirmed)
Able to reproduce the issue on MAC 10.11.6, Windows 10 and Ubuntu 14.04 using chrome reported version #54.0.2840.99 and latest canary #57.0.2934.0.

This is a non regression issue as it is observed from M30 old builds.

Hence, marking it as untriaged to get more inputs from dev team.

Thanks,

Labels: OS-Linux OS-Mac

Comment 4 by weili@chromium.org, Nov 28 2016

Status: Available (was: Untriaged)
Project Member

Comment 5 by sheriffbot@chromium.org, Nov 29 2017

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
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. If you change it back, also remove the "Hotlist-Recharge-Cold" label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -M-56 -Hotlist-Recharge-Cold OS-Chrome
Status: Available (was: Untriaged)
Did you create the test PDF? If so, can we add it to our test repository?
Yes I created it with PDFKit. You can do whatever you want with it.
Thank you. We'll try to look at this sooner rather than later.
Owner: thestig@chromium.org
Status: Started (was: Available)
1 line fix: https://pdfium-review.googlesource.com/c/pdfium/+/35991
Project Member

Comment 10 by bugdroid1@chromium.org, Aug 1

The following revision refers to this bug:
  https://pdfium.googlesource.com/pdfium_tests/+/96223aa3d32dcb606f4f33c5647abfd4e413f220

commit 96223aa3d32dcb606f4f33c5647abfd4e413f220
Author: Lei Zhang <thestig@chromium.org>
Date: Wed Aug 01 00:17:10 2018

Add a test case for gradients with opacity.

BUG= chromium:668762 

Change-Id: I99de143c6c981336994c17d8470dd9c1862bdfa3
Reviewed-on: https://pdfium-review.googlesource.com/39290
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>

[add] https://crrev.com/96223aa3d32dcb606f4f33c5647abfd4e413f220/pdfium/bug_668762_expected.pdf.0.png
[add] https://crrev.com/96223aa3d32dcb606f4f33c5647abfd4e413f220/pdfium/bug_668762.pdf

Project Member

Comment 11 by bugdroid1@chromium.org, Aug 4

The following revision refers to this bug:
  https://pdfium.googlesource.com/pdfium/+/a7a957f7cec2380b1f04f7166780dba50c0d5d85

commit a7a957f7cec2380b1f04f7166780dba50c0d5d85
Author: Lei Zhang <thestig@chromium.org>
Date: Sat Aug 04 04:29:30 2018

Use more text object state in DrawTextPathWithPattern().

BUG= chromium:668762 

Change-Id: Iaf3fc9bcf68b8da2c947bbd78c5d72f78195b621
Reviewed-on: https://pdfium-review.googlesource.com/35991
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>

[modify] https://crrev.com/a7a957f7cec2380b1f04f7166780dba50c0d5d85/core/fpdfapi/render/cpdf_renderstatus.cpp
[modify] https://crrev.com/a7a957f7cec2380b1f04f7166780dba50c0d5d85/DEPS

Status: Fixed (was: Started)
Project Member

Comment 13 by bugdroid1@chromium.org, Aug 4

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/9103c32638d2537173750f01d777a94a12fbe00e

commit 9103c32638d2537173750f01d777a94a12fbe00e
Author: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Date: Sat Aug 04 07:35:10 2018

Roll src/third_party/pdfium 03395da5d582..a7a957f7cec2 (2 commits)

https://pdfium.googlesource.com/pdfium.git/+log/03395da5d582..a7a957f7cec2


git log 03395da5d582..a7a957f7cec2 --date=short --no-merges --format='%ad %ae %s'
2018-08-04 thestig@chromium.org Use more text object state in DrawTextPathWithPattern().
2018-08-04 thestig@chromium.org Clarify integer types in CPDF_Parser::LoadCrossRefV5().


Created with:
  gclient setdep -r src/third_party/pdfium@a7a957f7cec2

The AutoRoll server is located here: https://pdfium-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.



BUG= chromium:668762 
TBR=dsinclair@chromium.org

Change-Id: I8152b379a6bd55f71dd12efff57aedf1ee7cd898
Reviewed-on: https://chromium-review.googlesource.com/1162814
Reviewed-by: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#580752}
[modify] https://crrev.com/9103c32638d2537173750f01d777a94a12fbe00e/DEPS

Sign in to add a comment