Unreviewed, rolling out r233112.
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Jul 2018 20:50:04 +0000 (20:50 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 3 Jul 2018 20:50:04 +0000 (20:50 +0000)
https://bugs.webkit.org/show_bug.cgi?id=187300

this revision is causing crashes on iOS 11 simulator
(Requested by Truitt on #webkit).

Reverted changeset:

"[Fullscreen] Restore ASSERT_NOT_REACHED() checks in exit
fullscreen handler after r231924"
https://bugs.webkit.org/show_bug.cgi?id=186945
https://trac.webkit.org/changeset/233112

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@233476 268f45cc-cd09-0410-ab3c-d52691b4dbfc

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

index b30fdd0..4de87a8 100644 (file)
@@ -1,3 +1,18 @@
+2018-07-03  Commit Queue  <commit-queue@webkit.org>
+
+        Unreviewed, rolling out r233112.
+        https://bugs.webkit.org/show_bug.cgi?id=187300
+
+        this revision is causing crashes on iOS 11 simulator
+        (Requested by Truitt on #webkit).
+
+        Reverted changeset:
+
+        "[Fullscreen] Restore ASSERT_NOT_REACHED() checks in exit
+        fullscreen handler after r231924"
+        https://bugs.webkit.org/show_bug.cgi?id=186945
+        https://trac.webkit.org/changeset/233112
+
 2018-07-03  Chris Dumez  <cdumez@apple.com>
 
         Implement support for Element.toggleAttribute
index ac3fef7..a755d1b 100644 (file)
@@ -1011,10 +1011,8 @@ void VideoFullscreenInterfaceAVKit::exitFullscreen(const IntRect& finalRect)
         }];
     } else if (m_currentMode.isFullscreen()) {
         [m_playerViewController exitFullScreenAnimated:YES completionHandler:[protectedThis = makeRefPtr(this), this] (BOOL success, NSError* error) mutable {
-            if (!success) {
+            if (!success)
                 WTFLogAlways("-[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error %s", [[error localizedDescription] UTF8String]);
-                ASSERT_NOT_REACHED();
-            }
 
             m_exitCompleted = true;
 
@@ -1049,10 +1047,8 @@ void VideoFullscreenInterfaceAVKit::cleanupFullscreen()
     if (m_currentMode.hasFullscreen()) {
         [[m_playerViewController view] layoutIfNeeded];
         [m_playerViewController exitFullScreenAnimated:NO completionHandler:[] (BOOL success, NSError* error) {
-            if (!success) {
+            if (!success)
                 WTFLogAlways("-[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error %s", [[error localizedDescription] UTF8String]);
-                ASSERT_NOT_REACHED();
-            }
         }];
     }
     
@@ -1794,10 +1790,8 @@ void VideoFullscreenInterfaceAVKit::doExitFullscreen()
 
 void VideoFullscreenInterfaceAVKit::exitFullscreenHandler(BOOL success, NSError* error)
 {
-    if (!success) {
+    if (!success)
         WTFLogAlways("-[AVPlayerViewController exitFullScreenAnimated:completionHandler:] failed with error %s", [[error localizedDescription] UTF8String]);
-        ASSERT_NOT_REACHED();
-    }
 
     LOG(Fullscreen, "VideoFullscreenInterfaceAVKit::didExitFullscreen(%p) - %d", this, success);