Bug 168703 - Resource Load Statistics: Figure out why release build tests are crashing
Summary: Resource Load Statistics: Figure out why release build tests are crashing
Status: RESOLVED CONFIGURATION CHANGED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: John Wilander
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-02-21 20:08 PST by John Wilander
Modified: 2017-02-22 12:41 PST (History)
6 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description John Wilander 2017-02-21 20:08:48 PST
The tests added in https://bugs.webkit.org/show_bug.cgi?id=168347 crash on Sierra 64-bit release builds. However, they don't on my machine.

We should skip the test cases while we investigate.
Comment 1 John Wilander 2017-02-21 20:32:39 PST
Test cases skipped as of http://trac.webkit.org/changeset/212800.
Comment 2 Radar WebKit Bug Importer 2017-02-21 20:37:23 PST
<rdar://problem/30646710>
Comment 3 Brent Fulgham 2017-02-21 22:59:05 PST
I am able to reproduce the problems locally with a release build. Let me know if you need any assistance debugging.
Comment 4 John Wilander 2017-02-21 23:53:53 PST
(In reply to comment #3)
> I am able to reproduce the problems locally with a release build. Let me
> know if you need any assistance debugging.

That's good news since I did try myself before I filed this. Let's look at it tomorrow.
Comment 5 Ryan Haddad 2017-02-22 08:48:02 PST
LayoutTest http/tests/security/contentSecurityPolicy/1.1/frame-ancestors/frame-ancestors-nested-cross-in-cross-self-block.html also seems to be affected by this crash:

https://build.webkit.org/results/Apple%20Sierra%20Release%20WK2%20(Tests)/r212828%20(3805)/results.html
Comment 6 Ryan Haddad 2017-02-22 08:56:33 PST
Also seen with http/tests/media/video-error-abort.html here: https://build.webkit.org/results/Apple%20Sierra%20Debug%20WK2%20(Tests)/r212823%20(2205)/results.html
Comment 7 Brent Fulgham 2017-02-22 09:13:12 PST
I also hit this with:

http/tests/plugins/geturlnotify-from-npp-destroystream.html
Comment 8 Alexey Proskuryakov 2017-02-22 12:41:20 PST
These needs to be done, but no need to track in a bug since the patch got rolled out.