Bug 187156 - REGRESSION (233281): fast/dom/location-new-window-no-crash.html and some other tests are timing out
Summary: REGRESSION (233281): fast/dom/location-new-window-no-crash.html and some othe...
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: Nobody
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2018-06-28 14:22 PDT by Antti Koivisto
Modified: 2018-06-29 17:36 PDT (History)
9 users (show)

See Also:


Attachments
patch (2.40 KB, patch)
2018-06-28 14:27 PDT, Antti Koivisto
no flags Details | Formatted Diff | Diff
patch (2.41 KB, patch)
2018-06-28 14:31 PDT, Antti Koivisto
zalan: review+
Details | Formatted Diff | Diff
patch (2.52 KB, patch)
2018-06-28 14:59 PDT, Antti Koivisto
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews204 for win-future (12.81 MB, application/zip)
2018-06-28 16:41 PDT, EWS Watchlist
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Antti Koivisto 2018-06-28 14:22:28 PDT
Tests Timeout:
fast/dom/location-new-window-no-crash.html
http/tests/security/javascriptURL/xss-ALLOWED-from-javascript-url-window-open.html	
storage/indexeddb/modern/blob-cursor.html
storage/indexeddb/modern/blob-svg-image.html
Comment 1 Antti Koivisto 2018-06-28 14:27:26 PDT
Created attachment 343852 [details]
patch
Comment 2 EWS Watchlist 2018-06-28 14:29:39 PDT
Attachment 343852 [details] did not pass style-queue:


ERROR: Source/WebCore/style/StyleTreeResolver.cpp:582:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebCore/style/StyleTreeResolver.cpp:584:  Extra space before ( in function call  [whitespace/parens] [4]
Total errors found: 2 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Antti Koivisto 2018-06-28 14:31:41 PDT
Created attachment 343854 [details]
patch
Comment 4 EWS Watchlist 2018-06-28 14:34:03 PDT
Attachment 343854 [details] did not pass style-queue:


ERROR: Source/WebCore/style/StyleTreeResolver.cpp:582:  Extra space before ( in function call  [whitespace/parens] [4]
ERROR: Source/WebCore/style/StyleTreeResolver.cpp:584:  Extra space before ( in function call  [whitespace/parens] [4]
Total errors found: 2 in 2 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 5 Antti Koivisto 2018-06-28 14:59:41 PDT
Created attachment 343856 [details]
patch
Comment 6 EWS Watchlist 2018-06-28 16:41:42 PDT
Comment on attachment 343856 [details]
patch

Attachment 343856 [details] did not pass win-ews (win):
Output: https://webkit-queues.webkit.org/results/8375321

New failing tests:
http/tests/security/contentSecurityPolicy/userAgentShadowDOM/allow-video.html
Comment 7 EWS Watchlist 2018-06-28 16:41:54 PDT
Created attachment 343870 [details]
Archive of layout-test-results from ews204 for win-future

The attached test failures were seen while running run-webkit-tests on the win-ews.
Bot: ews204  Port: win-future  Platform: CYGWIN_NT-6.1-2.9.0-0.318-5-3-x86_64-64bit
Comment 8 Ryan Haddad 2018-06-28 16:43:16 PDT
(In reply to Build Bot from comment #6)
> Comment on attachment 343856 [details]
> patch
> 
> Attachment 343856 [details] did not pass win-ews (win):
> Output: https://webkit-queues.webkit.org/results/8375321
> 
> New failing tests:
> http/tests/security/contentSecurityPolicy/userAgentShadowDOM/allow-video.html
Looks unrelated to this patch.
Comment 9 WebKit Commit Bot 2018-06-28 16:58:03 PDT
Comment on attachment 343856 [details]
patch

Clearing flags on attachment: 343856

Committed r233341: <https://trac.webkit.org/changeset/233341>
Comment 10 WebKit Commit Bot 2018-06-28 16:58:04 PDT
All reviewed patches have been landed.  Closing bug.
Comment 11 Radar WebKit Bug Importer 2018-06-28 17:00:07 PDT
<rdar://problem/41608990>
Comment 12 Darin Adler 2018-06-29 17:36:25 PDT
Comment on attachment 343856 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=343856&action=review

> Source/WebCore/style/StyleTreeResolver.cpp:633
> +        for (size_t i = 0; i < queue.size(); ++i) {
> +            if (auto* page = queue[i]->page())
> +                page->setMemoryCacheClientCallsEnabled(true);
> +        }

Why not a modern for loop?