Bug 164993 - REGRESSION (Safari 10): onbeforeunload does not show up after the first time Leave is clicked
Summary: REGRESSION (Safari 10): onbeforeunload does not show up after the first time ...
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: Page Loading (show other bugs)
Version: WebKit Nightly Build
Hardware: Mac macOS 10.12
: P2 Normal
Assignee: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2016-11-19 15:14 PST by Manuel Perez
Modified: 2016-12-09 10:34 PST (History)
6 users (show)

See Also:


Attachments
a test page that demonstrates and explains the problem (999 bytes, text/html)
2016-11-19 15:14 PST, Manuel Perez
no flags Details
page 1 out of 2 to better demonstrate the problem (1.11 KB, text/html)
2016-11-24 03:06 PST, Manuel Perez
no flags Details
page 2 out of 2 to better demonstrate the problem (200 bytes, text/html)
2016-11-24 03:07 PST, Manuel Perez
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Manuel Perez 2016-11-19 15:14:54 PST
Created attachment 295272 [details]
a test page that demonstrates and explains the problem

After the first time the user is presented with the dialog triggered by onbeforeunload and the user chooses to leave, the dialog does not show up any more regardless of the way used to navigate out of the page.

We found this issue on Safari 10.0 (12602.1.50.0.10) and then we were able to reproduce it on Version 10.0.1 and the latest nightly build.

See the attachment for complete description and steps to follow to reproduce the issue.
Comment 1 Alexey Proskuryakov 2016-11-23 00:20:20 PST
Do you know if this is a regression? In other words, did this issue occur with Safari 9?

In the attached test case, I assume that one needs to press the Back button between steps 2 and 3.
Comment 2 Manuel Perez 2016-11-23 05:22:52 PST
> Do you know if this is a regression? In other words, did this issue occur with Safari 9?

We just tested it and the issue cannot be reproduced on Safari 9.

> In the attached test case, I assume that one needs to press the Back button between steps 2 and 3.

If you test it directly in https://bug-164993-attachments.webkit.org/attachment.cgi?id=295272 without downloading the attachment then yes.
Comment 3 Brady Eidson 2016-11-23 23:13:15 PST
This is the back/forward cache, as I'm sure Alexey suspected.
Comment 4 Manuel Perez 2016-11-24 03:06:10 PST
Created attachment 295400 [details]
page 1 out of 2 to better demonstrate the problem
Comment 5 Manuel Perez 2016-11-24 03:07:11 PST
Created attachment 295402 [details]
page 2 out of 2 to better demonstrate the problem
Comment 6 Manuel Perez 2016-11-24 03:16:45 PST
Not sure it's related to the back/forward cache.

I've just uploaded a couple of attachments to demonstrate the issue without requiring to hit the back button. This time you'll have to download the pages in order to check the issue out.

Thanks in advance.
Comment 7 Radar WebKit Bug Importer 2016-12-01 12:37:53 PST
<rdar://problem/29464654>
Comment 8 Daniel Bates 2016-12-07 13:02:25 PST
This is a Safari bug. That is, it is not a WebKit bug. Marking this bug Resolved Invalid.
Comment 9 Manuel Perez 2016-12-07 15:13:51 PST
I think this is not a safari specific issue as it can be reproduced using WebKit-SVN-r208586.dmg
Comment 10 Daniel Bates 2016-12-07 16:17:34 PST
(In reply to comment #8)
> This is a Safari bug. That is, it is not a WebKit bug. Marking this bug
> Resolved Invalid.

For completeness the Safari bug affects Safari version 9.1 (601.5.17) and latter.
Comment 11 Daniel Bates 2016-12-07 16:17:54 PST
(In reply to comment #9)
> I think this is not a safari specific issue as it can be reproduced using
> WebKit-SVN-r208586.dmg

Notice that the nightlies use system Safari. What version of Safari are you using?
Comment 12 Daniel Bates 2016-12-07 16:19:42 PST
(In reply to comment #11)
> (In reply to comment #9)
> > I think this is not a safari specific issue as it can be reproduced using
> > WebKit-SVN-r208586.dmg
> 
Are you saying that you were able to reproduce this issue with Safari version < 9.1 (601.5.17) and WebKit-SVN-r208586.dmg?
Comment 13 Edward Rudd 2016-12-07 18:25:46 PST
(In reply to comment #12)
> (In reply to comment #11)
> > (In reply to comment #9)
> > > I think this is not a safari specific issue as it can be reproduced using
> > > WebKit-SVN-r208586.dmg
> > 
> Are you saying that you were able to reproduce this issue with Safari
> version < 9.1 (601.5.17) and WebKit-SVN-r208586.dmg?

We will check though our QA systems with older Safari and report back.. If it is indeed a safari only issue.  Where should we report that?
Comment 14 Daniel Bates 2016-12-07 20:32:47 PST
(In reply to comment #13)
> Where should we report that?

No need to report it again. We are internally tracking this in the radar provided in comment 7.
Comment 15 Manuel Perez 2016-12-09 10:34:38 PST
> Notice that the nightlies use system Safari. What version of Safari are you using?

I tested and reproduced the issue with Safari "Version 10.0.1 (12602.2.14.0.7, r208586)" and also with "Version 10.0.1 (12602.2.14.0.7, r209592)”.

> Are you saying that you were able to reproduce this issue with Safari version < 9.1 (601.5.17) and WebKit-SVN-r208586.dmg?

A teammate tested it on Safari Version 9.0 (11601.1.56) with Webkit Nightly r209601 and he could NOT reproduce the issue there.