Attachment 346646 Details for Bug 188351 – Patch

[patch] Patch

bug-188351-20180806131946.patch (text/plain), 1.62 KB, created by Jeremy Jones on 2018-08-06 13:19:46 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Jeremy Jones

Created: 2018-08-06 13:19:46 PDT

Size: 1.62 KB

patch

obsolete

>Subversion Revision: 234613
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 2ba87c3e3401319a9ef2f86da9c9be584deccffc..4726f4734dea7ab3966adba1250eec421ca0c259 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,16 @@
>+2018-08-06  Jeremy Jones  <jeremyj@apple.com>
>+
>+        UIWebView crashes while attempting to play youtube video on phone
>+        https://bugs.webkit.org/show_bug.cgi?id=188351
>+        rdar://problem/42489664
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        VideoFullscreenControllerContext must remove unretained reference to it from m_playbackModel during cleanup.
>+
>+        * platform/ios/WebVideoFullscreenControllerAVKit.mm:
>+        (VideoFullscreenControllerContext::didCleanupFullscreen):
>+
> 2018-08-06  Ryan Haddad  <ryanhaddad@apple.com>
> 
>         Unreviewed, rolling out r234569.
>diff --git a/Source/WebCore/platform/ios/WebVideoFullscreenControllerAVKit.mm b/Source/WebCore/platform/ios/WebVideoFullscreenControllerAVKit.mm
>index 3efbb17cfa649d31c5088035e0adfe0425299588..7ff0cf574d6fd4bf57eb52cfc92c0f44e9c204c7 100644
>--- a/Source/WebCore/platform/ios/WebVideoFullscreenControllerAVKit.mm
>+++ b/Source/WebCore/platform/ios/WebVideoFullscreenControllerAVKit.mm
>@@ -328,6 +328,7 @@ void VideoFullscreenControllerContext::didCleanupFullscreen()
>         m_fullscreenModel->setVideoFullscreenLayer(nil);
>         m_fullscreenModel->setVideoElement(nullptr);
>         m_playbackModel->setMediaElement(nullptr);
>+        m_playbackModel->removeClient(*this);
>         m_fullscreenModel->removeClient(*this);
>         m_fullscreenModel = nullptr;
>         m_videoElement = nullptr;

Flags:

ddkilzer: review+
jeremyj-wk: commit-queue-

Actions: View | Formatted Diff | Diff

Attachments on bug 188351: 346646 | 347513