Bug 173835

Summary: Invalidate WebVideoFullscreenManager when WebPage is destroyed.
Product: WebKit Reporter: Jeremy Jones <jeremyj-wk>
Component: New BugsAssignee: Jeremy Jones <jeremyj-wk>
Status: RESOLVED FIXED    
Severity: Normal CC: buildbot, commit-queue, eric.carlson, jer.noble, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
jer.noble: review+, buildbot: commit-queue-
Archive of layout-test-results from ews123 for ios-simulator-wk2
none
Patch for landing. none

Description Jeremy Jones 2017-06-26 09:14:13 PDT
Invalidate WebVideoFullscreenManager when WebPage is destroyed.
Comment 1 Jeremy Jones 2017-06-26 09:17:51 PDT
Created attachment 313843 [details]
Patch
Comment 2 Jeremy Jones 2017-06-26 09:18:33 PDT
rdar://problem/32969161
Comment 3 Jeremy Jones 2017-06-26 10:36:44 PDT
Investigating media/controls/ipad/close-page-with-picture-in-picture-video-assertion-failure.html crash to see if it is related.
Comment 4 Build Bot 2017-06-26 10:44:49 PDT
Comment on attachment 313843 [details]
Patch

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

New failing tests:
media/controls/ipad/close-page-with-picture-in-picture-video-assertion-failure.html
Comment 5 Build Bot 2017-06-26 10:44:50 PDT
Created attachment 313850 [details]
Archive of layout-test-results from ews123 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews123  Port: ios-simulator-wk2  Platform: Mac OS X 10.12.5
Comment 6 Jeremy Jones 2017-06-26 13:54:19 PDT
Created attachment 313867 [details]
Patch for landing.
Comment 7 WebKit Commit Bot 2017-06-26 15:36:49 PDT
Comment on attachment 313867 [details]
Patch for landing.

Clearing flags on attachment: 313867

Committed r218822: <http://trac.webkit.org/changeset/218822>