Title: [295439] trunk/Source/WebCore/platform/graphics
Revision
295439
Author
[email protected]
Date
2022-06-09 16:12:18 -0700 (Thu, 09 Jun 2022)

Log Message

[iOS] iPhone fullscreen captions are sometimes duplicated
https://bugs.webkit.org/show_bug.cgi?id=241464
rdar://94261522

Reviewed by Jer Noble.

Clear the text track representation when the video layer is cleared.

* Source/WebCore/platform/graphics/avfoundation/objc/VideoLayerManagerObjC.mm:
(WebCore::VideoLayerManagerObjC::didDestroyVideoLayer): Clear the text track representation.

* Source/WebCore/platform/graphics/cocoa/TextTrackRepresentationCocoa.mm:
(WebCore::TextTrackRepresentationCocoa::TextTrackRepresentationCocoa): Name the text
track representation layer to aid debugging in the future.

Canonical link: https://commits.webkit.org/251445@main

Modified Paths

Diff

Modified: trunk/Source/WebCore/platform/graphics/avfoundation/objc/VideoLayerManagerObjC.mm (295438 => 295439)


--- trunk/Source/WebCore/platform/graphics/avfoundation/objc/VideoLayerManagerObjC.mm	2022-06-09 21:45:40 UTC (rev 295438)
+++ trunk/Source/WebCore/platform/graphics/avfoundation/objc/VideoLayerManagerObjC.mm	2022-06-09 23:12:18 UTC (rev 295439)
@@ -91,6 +91,9 @@
 {
     ALWAYS_LOG(LOGIDENTIFIER);
 
+#if ENABLE(VIDEO_PRESENTATION_MODE)
+    setTextTrackRepresentationLayer(nil);
+#endif
     [m_videoLayer removeFromSuperlayer];
 
     m_videoInlineLayer = nil;

Modified: trunk/Source/WebCore/platform/graphics/cocoa/TextTrackRepresentationCocoa.mm (295438 => 295439)


--- trunk/Source/WebCore/platform/graphics/cocoa/TextTrackRepresentationCocoa.mm	2022-06-09 21:45:40 UTC (rev 295438)
+++ trunk/Source/WebCore/platform/graphics/cocoa/TextTrackRepresentationCocoa.mm	2022-06-09 23:12:18 UTC (rev 295439)
@@ -119,6 +119,8 @@
 {
     [m_layer setDelegate:m_delegate.get()];
     [m_layer setContentsGravity:kCAGravityBottom];
+
+    [m_layer setName:@"TextTrackRepresentation"];
 }
 
 TextTrackRepresentationCocoa::~TextTrackRepresentationCocoa()
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to