Changeset 233925 in webkit

Timestamp:

Jul 18, 2018 12:44:32 PM (6 years ago)

Author:

jer.noble@apple.com

Message:

Dissociate the VideoFullscreenInterface from its VideoFullscreenModel before removing it from the manager
​https://bugs.webkit.org/show_bug.cgi?id=187775
<rdar://problem/42343229>

Reviewed by Jon Lee.

• UIProcess/Cocoa/VideoFullscreenManagerProxy.mm:

(WebKit::VideoFullscreenManagerProxy::didCleanupFullscreen):

Location:

trunk/Source/WebKit

Files:

• ChangeLog (modified) (1 diff)
• UIProcess/Cocoa/VideoFullscreenManagerProxy.mm (modified) (1 diff)

trunk/Source/WebKit/ChangeLog

@@ +1 @@
+2018-07-18  Jer Noble  <jer.noble@apple.com>
+
+        Dissociate the VideoFullscreenInterface from its VideoFullscreenModel before removing it from the manager
+        https://bugs.webkit.org/show_bug.cgi?id=187775
+        <rdar://problem/42343229>
+
+        Reviewed by Jon Lee.
+
+        * UIProcess/Cocoa/VideoFullscreenManagerProxy.mm:
+        (WebKit::VideoFullscreenManagerProxy::didCleanupFullscreen):
+
 2018-07-18  Wenson Hsieh  <wenson_hsieh@apple.com>
 

trunk/Source/WebKit/UIProcess/Cocoa/VideoFullscreenManagerProxy.mm

@@ -610 +610 @@
 
     interface->setMode(HTMLMediaElementEnums::VideoFullscreenModeNone);
+    interface->setVideoFullscreenModel(nullptr);
     removeClientForContext(contextId);
 }