New issue
Advanced search Search tips

Issue 680699 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 682060
Owner:
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 3
Type: Bug



Sign in to add a comment

pdf viewer locks up across tabs and windows when "alert" in pdf goes away

Reported by mig...@thedeanda.com, Jan 12 2017

Issue description

Chrome Version       : 57.0.2980.0 (Developer Build) (64-bit) 16325c0ba632bc13701728830fe14a873ef8ab53-refs/heads/master@{#443307}
URLs (if applicable) : http://home.thedeanda.com/pdfissue/break_pdf.html
Other browsers tested:

Windows 10 Chrome (native) - 55.0.2883.87 (Official Build) m (64-bit) - OK
Windows 10 Chrome (via browserstack) - 56.0.2924.14 (Official Build) dev (32-bit) - FAIL

What steps will reproduce the problem?
Open PDF in tab or window or embedded to same page
Open PDF with alert/prompt in it, then have javascript yank it off the dom
Existing PDFs will be in some form of deadlock (since alert wasn't manually closed)

What is the expected result?
PDFs in separate tabs or windows shouldn't be affected


What happens instead?
Existing PDFs are unresponsive and often reloading prevents them from opening until _all_ pdf's and page with "yanking" get reloaded.


Please provide any additional information below. Attach a screenshot if
possible.

The link provided includes a PDF that causes the problems and provides an easy way to reproduce the issue.
 
files used to reproduce the issue. in case link goes away
break_pdf.html
1.3 KB View Download
alert.pdf
50.8 KB Download
Cc: kkaluri@chromium.org
Components: Internals>Plugins>PDF
Labels: Needs-Feedback
Unable to reproduce the issue on windows 10 with chrome version #55.0.0.2883.87

These are the steps followed :

1. Download the attached files from comment #1
2. Loaded the "break_pdf.html" and open few pdf's in new tab, window
3. Loaded the alert.pdf and close the alert pop-up using "Esc" button

Didn't observe any non-responsiveness from the other pdf tabs.

Attaching the screencast for your reference, please look into it and let us know  any steps i have missed.

Issue 680699.mp4
6.3 MB View Download
Sorry if I caused confusion. The issue does not happen on chrome 55 but it started happening on chrome 56 beta and I see the issue as well on chromium pulled by the https://github.com/scheib/chromium-latest-linux.git script: 

57.0.2980.0 (Developer Build) (64-bit)
57.0.2981.0 (Developer Build) (64-bit)


Also note that you don't have to open all of embedded, popup and tab... I just wanted to show that all lock up. Basically any existing pdf will lock up when the "break pdf" button is used. It also appears that new pdf's continue to not work until all "stuck" pdf's are reloaded or closed. 
pdfissue.mp4
10.3 MB View Download
chromium.mp4
5.1 MB View Download

Comment 4 by ajha@chromium.org, Jan 16 2017

Labels: Needs-Bisect Needs-Triage-M57
Labels: -Needs-Feedback -Needs-Bisect -Needs-Triage-M57 hasbisect-per-revision M-56
Owner: a...@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce this issue on windows 10, Ubuntu 14.04 and Mac 10.12.2 on latest chrome Beta #56.0.2924.67, dev  #57.0.2986.0 and Canary #58.0.2988.0	
Issue is broken in M56. 


Bisect Info:
===========
Good build : 56.0.2888.0,  Revision Range- 424625
Bad build  : 56.0.2889.0,  Revision Range- 424926

After executing the per-revision-bisect script, i got the following CL's between good and bad build versions
============================================
https://chromium.googlesource.com/chromium/src/+log/158941f08d24fb3cbaa9466486e1bb0a994668b4..5d3b869bc09c586f076e3d78d8b8966b75dcf6c9

The suspecting Change Log is :
-----------
https://chromium.googlesource.com/chromium/src/+/5d3b869bc09c586f076e3d78d8b8966b75dcf6c9

From the above CL suspecting the below change
--------------------------------------
https://codereview.chromium.org/2412843002

avi@- Could you please look into this issue, if it's related to your change?  if not could you please help us to reassign this issue to the right owner.



Thank You...

Labels: OS-Linux OS-Mac OS-Windows
retested today on the following build and it seems to be fully corrected now:

Chromium	58.0.3002.0 (Developer Build) (64-bit)
Revision	3ee4869818e9ef4da49d21018707c2aae820a0db-refs/heads/master@{#447949}

Comment 8 by a...@chromium.org, Feb 3 2017

Mergedinto: 682060
Status: Duplicate (was: Assigned)
I made a fix for dialogs that addressed a different issue, but probably fixed this accidentally as well. If you can no longer repro this, we're good.

Sign in to add a comment