Changeset 228229 in webkit

Timestamp:

Feb 7, 2018 10:13:52 AM (6 years ago)

Author:

Matt Lewis

Message:

Removed the assertions from VideoFullscreenInterfaceAVKit.mm temporarily to stop the resulting crashes during debugging.
​https://bugs.webkit.org/show_bug.cgi?id=182527

Unreviewed build fix.

• platform/ios/VideoFullscreenInterfaceAVKit.mm:

(VideoFullscreenInterfaceAVKit::exitFullscreen):
(VideoFullscreenInterfaceAVKit::cleanupFullscreen):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/ios/VideoFullscreenInterfaceAVKit.mm (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2018-02-06  Matt Lewis  <jlewis3@apple.com>
+
+        Removed the assertions from VideoFullscreenInterfaceAVKit.mm temporarily to stop the resulting crashes during debugging.
+        https://bugs.webkit.org/show_bug.cgi?id=182527
+
+        Unreviewed build fix.
+
+        * platform/ios/VideoFullscreenInterfaceAVKit.mm:
+        (VideoFullscreenInterfaceAVKit::exitFullscreen):
+        (VideoFullscreenInterfaceAVKit::cleanupFullscreen):
+
 2018-02-07  Zalan Bujtas  <zalan@apple.com>
 

trunk/Source/WebCore/platform/ios/VideoFullscreenInterfaceAVKit.mm

@@ -777 +777 @@
             if (!success) {
                 WTFLogAlways("-[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error %s", [[error localizedDescription] UTF8String]);
-                ASSERT_NOT_REACHED();
             }
 
@@ -815 +814 @@
             if (!success) {
                 WTFLogAlways("-[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error %s", [[error localizedDescription] UTF8String]);
-                ASSERT_NOT_REACHED();
             }
         }];