Title: [228166] branches/safari-605-branch/Source/WebCore
Revision
228166
Author
[email protected]
Date
2018-02-06 07:16:34 -0800 (Tue, 06 Feb 2018)

Log Message

Cherry-pick r228119. rdar://problem/37264495

Modified Paths

Diff

Modified: branches/safari-605-branch/Source/WebCore/ChangeLog (228165 => 228166)


--- branches/safari-605-branch/Source/WebCore/ChangeLog	2018-02-06 15:16:31 UTC (rev 228165)
+++ branches/safari-605-branch/Source/WebCore/ChangeLog	2018-02-06 15:16:34 UTC (rev 228166)
@@ -1,5 +1,23 @@
 2018-02-06  Jason Marcell  <[email protected]>
 
+        Cherry-pick r228119. rdar://problem/37264495
+
+    2018-02-05  John Wilander  <[email protected]>
+
+            Build fix for r228115, simple naming issue succeeded —> success.
+            https://bugs.webkit.org/show_bug.cgi?id=182507
+            <rdar://problem/37248566>
+
+            Reviewed by Eric Carlson.
+
+            No new tests. Build fix.
+
+            * platform/ios/VideoFullscreenInterfaceAVKit.mm:
+            (VideoFullscreenInterfaceAVKit::enterFullscreenHandler):
+                succeeded —> success
+
+2018-02-06  Jason Marcell  <[email protected]>
+
         Cherry-pick r228115. rdar://problem/37264495
 
     2018-02-05  Eric Carlson  <[email protected]>

Modified: branches/safari-605-branch/Source/WebCore/platform/ios/VideoFullscreenInterfaceAVKit.mm (228165 => 228166)


--- branches/safari-605-branch/Source/WebCore/platform/ios/VideoFullscreenInterfaceAVKit.mm	2018-02-06 15:16:31 UTC (rev 228165)
+++ branches/safari-605-branch/Source/WebCore/platform/ios/VideoFullscreenInterfaceAVKit.mm	2018-02-06 15:16:34 UTC (rev 228166)
@@ -1586,7 +1586,7 @@
 
 void VideoFullscreenInterfaceAVKit::enterFullscreenHandler(BOOL success, NSError* error)
 {
-    if (!succeeded) {
+    if (!success) {
         WTFLogAlways("-[AVPlayerViewController enterFullScreenAnimated:completionHandler:] failed with error %s", [[error localizedDescription] UTF8String]);
         ASSERT_NOT_REACHED();
         m_videoFullscreenModel->requestFullscreenMode(HTMLMediaElementEnums::VideoFullscreenModeNone, YES);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to