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

Issue 662698 link

Starred by 5 users
Status: Fixed
Owner:
tsepez@chromium.org
Closed: Nov 2016
Cc:
█ hdodda@chromium.org
nyerramilli@chromium.org
thestig@chromium.org
█ msrchandra@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug



Sign in to add a comment

Regression: Webview process of pdf viewer crashed after giving 'Print' command

Reported by chromium...@gmail.com, Nov 6 2016 
UserAgent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2910.0 Safari/537.36

Steps to reproduce the problem:
(1) Lunch chrome, navigate to testcase.pdf
(2) Click on Print icon(print preview opened), close preview page and observe

What is the expected behavior?
No Crash

What went wrong?
Crash

Crashed report ID: 13ea20d1-424f-4250-b1bc-3d2344ae91f5

How much crashed? Just one tab

Is it a problem with a plugin? N/A 

Did this work before? N/A 

Chrome version: 56.0.2910.0  Channel: canary
OS Version: 6.1 (Windows 7, Windows Server 2008 R2)
Flash Version: Shockwave Flash 24.0 r0

Good Build: 56.0.2909.0 
Bad Build: 56.0.2910.0
testcase.pdf
139 KB Download

Comment 1 by hdodda@chromium.org, Nov 7 2016

Cc: hdodda@chromium.org
Components: Internals>Plugins>PDF
Labels: -Pri-2 hasbisect-per-revision ReleaseBlock-Beta M-56 OS-Linux OS-Mac Pri-1
Owner: thestig@chromium.org
Status: Assigned (was: Unconfirmed)
Using the per-revision bisect providing the bisect results,
Good build: 56.0.2909.0(Revision: 429737).
Bad build: 56.0.2910.0(Revision: 430103).

You are probably looking for a change made after 430088 (known good), but no later than 430089 (first known bad).
CHANGELOG URL:

The script might not always return single CL as suspect as some perf builds might get missing due to failure.
  
https://chromium.googlesource.com/chromium/src/+log/84ed75069855bf68ed20423c4a3803c628dd48a8..4093035dccdddf3a51f4035dfe9d3c90809ffb94

From the CL's above, assigning the issue to the concern owner 

@thestig - Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner.

Review-Url: https://codereview.chromium.org/2475923003

Note : Adding RB-Beta for now, please feel free to edit it.

Thanks!

Comment 2 by thestig@chromium.org, Nov 7 2016

Owner: tsepez@chromium.org

Comment 3 by thestig@chromium.org, Nov 7 2016 

Revert or rework https://codereview.chromium.org/2471263005 ?

Please note at ToT there might be another printing bug.

Comment 4 by thestig@chromium.org, Nov 8 2016

Cc: msrchandra@chromium.org nyerramilli@chromium.org
 Issue 663257  has been merged into this issue.

Comment 5 by tsepez@chromium.org, Nov 8 2016

Cc: thestig@chromium.org
Added emebedder tests at https://codereview.chromium.org/2489653003/
Removing CHECK() from suspect CL doesn't fix issue, however, as we just segv later.
Project Member

Comment 6 by bugdroid1@chromium.org, Nov 9 2016 

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

commit 5fad936b6ec8dd5a6885d3eb321851d5f42e8143
Author: pdfium-deps-roller <pdfium-deps-roller@chromium.org>
Date: Wed Nov 09 03:47:47 2016

Roll src/third_party/pdfium/ 28c888b8e..d0ecd899d (2 commits).

https://pdfium.googlesource.com/pdfium.git/+log/28c888b8ee6d..d0ecd899d632

$ git log 28c888b8e..d0ecd899d --date=short --no-merges --format='%ad %ae %s'
2016-11-08 tsepez Fix abort above FPDFPage_Flatten
2016-11-08 npm Add FPDF_FreeDefaultSystemFontInfo API

BUG= 662698 

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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls

TBR=dsinclair@chromium.org

Review-Url: https://codereview.chromium.org/2486023003
Cr-Commit-Position: refs/heads/master@{#430849}

[modify] https://crrev.com/5fad936b6ec8dd5a6885d3eb321851d5f42e8143/DEPS

Comment 7 by thestig@chromium.org, Nov 9 2016

Status: Fixed (was: Assigned)

Comment 8 by tsepez@chromium.org, Nov 11 2016 

Issue 664492 has been merged into this issue.

Comment 9 by thestig@chromium.org, Nov 19 2016 

Issue 666826 has been merged into this issue.
►

Sign in to add a comment