Bug 171610 - REGRESSION (r215951): LayoutTest media/modern-media-controls/placard-support/placard-support-pip.html is a flaky crash
Summary: REGRESSION (r215951): LayoutTest media/modern-media-controls/placard-support/...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jeremy Jones
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2017-05-03 11:17 PDT by Ryan Haddad
Modified: 2017-05-05 08:49 PDT (History)
6 users (show)

See Also:


Attachments
Crashlog (101.92 KB, text/plain)
2017-05-03 11:17 PDT, Ryan Haddad
no flags Details
Patch (2.57 KB, patch)
2017-05-04 20:07 PDT, Jeremy Jones
no flags Details | Formatted Diff | Diff
Patch (4.62 KB, patch)
2017-05-04 20:16 PDT, Jeremy Jones
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Comment 1 Ryan Haddad 2017-05-03 11:18:54 PDT
The crash is beneath WebKit in AVPlayer.

Application Specific Information:
CRASHING TEST: media/modern-media-controls/placard-support/placard-support-pip.html
*** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'An instance 0x7fe5a8655d60 of class AVPlayer was deallocated while key value observers were still registered with it. Current observation info: <NSKeyValueObservationInfo 0x7fe5a80ddce0> (
<NSKeyValueObservance 0x7fe5a795a600: Observer: 0x7fe5a7bd8e50, Key path: rate, Options: <New: NO, Old: NO, Prior: NO> Context: 0x0, Property: 0x7fe5a7eecee0>
)'
terminating with uncaught exception of type NSException
abort() called
Comment 2 Ryan Haddad 2017-05-03 13:01:36 PDT
Maybe this has something to do with the "workaround" that was removed from media/modern-media-controls/pip-support/pip-support-click.html in https://trac.webkit.org/changeset/215951/webkit
Comment 3 Radar WebKit Bug Importer 2017-05-03 17:02:13 PDT
<rdar://problem/31976820>
Comment 4 Ryan Haddad 2017-05-04 10:20:45 PDT
Skipped media/modern-media-controls/pip-support/pip-support-click.html to see if that stops the crashes in https://trac.webkit.org/r216191
Comment 5 Jeremy Jones 2017-05-04 17:23:37 PDT
(In reply to Ryan Haddad from comment #4)
> Skipped media/modern-media-controls/pip-support/pip-support-click.html to
> see if that stops the crashes in https://trac.webkit.org/r216191

I'm able to report the problem by doing 
run-webkit-tests LayoutTests/media/modern-media-controls/placard-support/placard-support-pip.html LayoutTests/media/modern-media-controls/pip-support/pip-support-click.html -1 --child-processes=1
Comment 6 Jeremy Jones 2017-05-04 20:07:06 PDT
Created attachment 309132 [details]
Patch
Comment 7 Jeremy Jones 2017-05-04 20:16:35 PDT
Created attachment 309133 [details]
Patch
Comment 8 WebKit Commit Bot 2017-05-05 08:49:57 PDT
Comment on attachment 309133 [details]
Patch

Clearing flags on attachment: 309133

Committed r216245: <http://trac.webkit.org/changeset/216245>
Comment 9 WebKit Commit Bot 2017-05-05 08:49:58 PDT
All reviewed patches have been landed.  Closing bug.