Bug 184848 - HTML String load cannot be prevented by responding 'Cancel' asynchronously in decidePolicyForNavigationAction
Summary: HTML String load cannot be prevented by responding 'Cancel' asynchronously in...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Page Loading (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords: InRadar
Depends on: 184209
Blocks:
  Show dependency treegraph
 
Reported: 2018-04-20 15:54 PDT by Chris Dumez
Modified: 2018-04-23 19:30 PDT (History)
10 users (show)

See Also:


Attachments
Patch (5.86 KB, patch)
2018-04-20 16:10 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Patch (8.74 KB, patch)
2018-04-20 16:59 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews124 for ios-simulator-wk2 (2.25 MB, application/zip)
2018-04-20 18:59 PDT, EWS Watchlist
no flags Details
Patch (11.61 KB, patch)
2018-04-20 21:13 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews103 for mac-sierra (2.34 MB, application/zip)
2018-04-20 22:18 PDT, EWS Watchlist
no flags Details
Archive of layout-test-results from ews116 for mac-sierra (3.13 MB, application/zip)
2018-04-20 23:27 PDT, EWS Watchlist
no flags Details
Patch (11.74 KB, patch)
2018-04-23 08:58 PDT, Chris Dumez
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2018-04-20 15:54:35 PDT
HTML String load cannot be prevented by responding 'Cancel' asynchronously in decidePolicyForNavigationAction.
Comment 1 Chris Dumez 2018-04-20 15:54:54 PDT
<rdar://problem/39145306>
Comment 2 Chris Dumez 2018-04-20 16:10:51 PDT
Created attachment 338483 [details]
Patch
Comment 3 Chris Dumez 2018-04-20 16:59:42 PDT
Created attachment 338493 [details]
Patch
Comment 4 EWS Watchlist 2018-04-20 18:59:49 PDT
Comment on attachment 338493 [details]
Patch

Attachment 338493 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/7390883

New failing tests:
fast/events/beforeunload-alert-user-interaction2.html
webarchive/loading/javascript-url-iframe-crash.html
fast/events/beforeunload-alert.html
Comment 5 EWS Watchlist 2018-04-20 18:59:51 PDT
Created attachment 338508 [details]
Archive of layout-test-results from ews124 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews124  Port: ios-simulator-wk2  Platform: Mac OS X 10.13.4
Comment 6 Chris Dumez 2018-04-20 21:13:44 PDT
Created attachment 338518 [details]
Patch
Comment 7 EWS Watchlist 2018-04-20 22:18:11 PDT
Comment on attachment 338518 [details]
Patch

Attachment 338518 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.webkit.org/results/7393468

New failing tests:
webarchive/loading/javascript-url-iframe-crash.html
Comment 8 EWS Watchlist 2018-04-20 22:18:12 PDT
Created attachment 338521 [details]
Archive of layout-test-results from ews103 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews103  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 9 EWS Watchlist 2018-04-20 23:27:17 PDT
Comment on attachment 338518 [details]
Patch

Attachment 338518 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/7393675

New failing tests:
webarchive/loading/javascript-url-iframe-crash.html
Comment 10 EWS Watchlist 2018-04-20 23:27:18 PDT
Created attachment 338523 [details]
Archive of layout-test-results from ews116 for mac-sierra

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews116  Port: mac-sierra  Platform: Mac OS X 10.12.6
Comment 11 Chris Dumez 2018-04-23 08:58:01 PDT
Created attachment 338589 [details]
Patch
Comment 12 Chris Dumez 2018-04-23 13:50:02 PDT
Comment on attachment 338589 [details]
Patch

Clearing flags on attachment: 338589

Committed r230919: <https://trac.webkit.org/changeset/230919>
Comment 13 Chris Dumez 2018-04-23 13:50:06 PDT
All reviewed patches have been landed.  Closing bug.
Comment 14 Ryan Haddad 2018-04-23 17:50:43 PDT
This change appears to have caused the flaky failures seen with the fast/events/beforeunload tests seen here:
https://build.webkit.org/results/Apple%20High%20Sierra%20Release%20WK2%20(Tests)/r230928%20(4164)/results.html
Comment 15 Chris Dumez 2018-04-23 18:44:48 PDT
(In reply to Ryan Haddad from comment #14)
> This change appears to have caused the flaky failures seen with the
> fast/events/beforeunload tests seen here:
> https://build.webkit.org/results/
> Apple%20High%20Sierra%20Release%20WK2%20(Tests)/r230928%20(4164)/results.html

Ok, looking now. The tests probably need updating.
Comment 16 Chris Dumez 2018-04-23 18:49:16 PDT
(In reply to Chris Dumez from comment #15)
> (In reply to Ryan Haddad from comment #14)
> > This change appears to have caused the flaky failures seen with the
> > fast/events/beforeunload tests seen here:
> > https://build.webkit.org/results/
> > Apple%20High%20Sierra%20Release%20WK2%20(Tests)/r230928%20(4164)/results.html
> 
> Ok, looking now. The tests probably need updating.

As I thought. I will update the tests shortly.
Comment 17 Chris Dumez 2018-04-23 19:30:55 PDT
(In reply to Chris Dumez from comment #16)
> (In reply to Chris Dumez from comment #15)
> > (In reply to Ryan Haddad from comment #14)
> > > This change appears to have caused the flaky failures seen with the
> > > fast/events/beforeunload tests seen here:
> > > https://build.webkit.org/results/
> > > Apple%20High%20Sierra%20Release%20WK2%20(Tests)/r230928%20(4164)/results.html
> > 
> > Ok, looking now. The tests probably need updating.
> 
> As I thought. I will update the tests shortly.

https://trac.webkit.org/r230939