Title: [185761] trunk/Source/WebCore
- Revision
- 185761
- Author
- [email protected]
- Date
- 2015-06-19 12:10:51 -0700 (Fri, 19 Jun 2015)
Log Message
Fullscreen view should not update bounds of video when in PiP.
https://bugs.webkit.org/show_bug.cgi?id=146134
Patch by Jeremy Jones <[email protected]> on 2015-06-19
Reviewed by Darin Adler.
Don't update bounds on video layer when it is not a child.
* platform/ios/WebVideoFullscreenInterfaceAVKit.mm:
(-[WebAVVideoLayer setBounds:]):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (185760 => 185761)
--- trunk/Source/WebCore/ChangeLog 2015-06-19 19:10:31 UTC (rev 185760)
+++ trunk/Source/WebCore/ChangeLog 2015-06-19 19:10:51 UTC (rev 185761)
@@ -1,3 +1,15 @@
+2015-06-19 Jeremy Jones <[email protected]>
+
+ Fullscreen view should not update bounds of video when in PiP.
+ https://bugs.webkit.org/show_bug.cgi?id=146134
+
+ Reviewed by Darin Adler.
+
+ Don't update bounds on video layer when it is not a child.
+
+ * platform/ios/WebVideoFullscreenInterfaceAVKit.mm:
+ (-[WebAVVideoLayer setBounds:]):
+
2015-06-19 Zalan Bujtas <[email protected]>
RenderRubyText requires RenderRubyRun parent.
Modified: trunk/Source/WebCore/platform/ios/WebVideoFullscreenInterfaceAVKit.mm (185760 => 185761)
--- trunk/Source/WebCore/platform/ios/WebVideoFullscreenInterfaceAVKit.mm 2015-06-19 19:10:31 UTC (rev 185760)
+++ trunk/Source/WebCore/platform/ios/WebVideoFullscreenInterfaceAVKit.mm 2015-06-19 19:10:51 UTC (rev 185761)
@@ -693,8 +693,10 @@
{
[super setBounds:bounds];
- [_videoSublayer setPosition:CGPointMake(CGRectGetMidX(bounds), CGRectGetMidY(bounds))];
- [_videoSublayer setBounds:bounds];
+ if ([_videoSublayer superlayer] == self) {
+ [_videoSublayer setPosition:CGPointMake(CGRectGetMidX(bounds), CGRectGetMidY(bounds))];
+ [_videoSublayer setBounds:bounds];
+ }
}
- (void)setVideoLayerGravity:(AVVideoLayerGravity)videoLayerGravity
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
