Changeset 228406 in webkit

Timestamp:

Feb 12, 2018 5:00:08 PM (6 years ago)

Author:

jmarcell@apple.com

Message:

Cherry-pick r228229. rdar://problem/37475107

Location:

branches/safari-605-branch/Source/WebCore

Files:

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

branches/safari-605-branch/Source/WebCore/ChangeLog

@@ +1 @@
+2018-02-12  Jason Marcell  <jmarcell@apple.com>
+
+        Cherry-pick r228229. rdar://problem/37475107
+
+    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-12  Jason Marcell  <jmarcell@apple.com>
 

branches/safari-605-branch/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();
             }
         }];