Title: [201661] trunk/Source/WebCore
- Revision
- 201661
- Author
- adac...@apple.com
- Date
- 2016-06-03 14:59:07 -0700 (Fri, 03 Jun 2016)
Log Message
REGRESSION(r201474): Subtitles are not resizing in other fullscreen modes
https://bugs.webkit.org/show_bug.cgi?id=158355
Reviewed by Eric Carlson.
* platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
(WebCore::MediaPlayerPrivateAVFoundationObjC::syncTextTrackBounds):
Set the text track representation layer's frame based on the active video layer.
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (201660 => 201661)
--- trunk/Source/WebCore/ChangeLog 2016-06-03 21:42:03 UTC (rev 201660)
+++ trunk/Source/WebCore/ChangeLog 2016-06-03 21:59:07 UTC (rev 201661)
@@ -1,3 +1,14 @@
+2016-06-03 Ada Chan <adac...@apple.com>
+
+ REGRESSION(r201474): Subtitles are not resizing in other fullscreen modes
+ https://bugs.webkit.org/show_bug.cgi?id=158355
+
+ Reviewed by Eric Carlson.
+
+ * platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
+ (WebCore::MediaPlayerPrivateAVFoundationObjC::syncTextTrackBounds):
+ Set the text track representation layer's frame based on the active video layer.
+
2016-06-03 Brady Eidson <beid...@apple.com>
Miscellaneous IDBBindingUtilities cleanup.
Modified: trunk/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm (201660 => 201661)
--- trunk/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm 2016-06-03 21:42:03 UTC (rev 201660)
+++ trunk/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm 2016-06-03 21:59:07 UTC (rev 201661)
@@ -2181,7 +2181,8 @@
return;
FloatRect videoFullscreenFrame = m_videoFullscreenLayerManager->videoFullscreenFrame();
- CGRect textFrame = m_videoLayer ? [m_videoLayer videoRect] : CGRectMake(0, 0, videoFullscreenFrame.width(), videoFullscreenFrame.height());
+ auto activeLayer = m_secondaryVideoLayer.get() ?: m_videoLayer.get();
+ CGRect textFrame = activeLayer ? [activeLayer videoRect] : CGRectMake(0, 0, videoFullscreenFrame.width(), videoFullscreenFrame.height());
[m_textTrackRepresentationLayer setFrame:textFrame];
#endif
}
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes