Title: [154924] trunk/Source/WebCore
- Revision
- 154924
- Author
- [email protected]
- Date
- 2013-08-30 18:10:45 -0700 (Fri, 30 Aug 2013)
Log Message
[Windows] Build fix after r154921.
AVFoundationCF uses an internal wrapper object around the C-API. The call point
in the original patch was inside the wrapper, which did not (yet) provide
a method implementation.
* platform/graphics/avfoundation/cf/MediaPlayerPrivateAVFoundationCF.cpp:
(WebCore::MediaPlayerPrivateAVFoundationCF::updateVideoLayerGravity):
(WebCore::AVFWrapper::updateVideoLayerGravity):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (154923 => 154924)
--- trunk/Source/WebCore/ChangeLog 2013-08-31 00:24:54 UTC (rev 154923)
+++ trunk/Source/WebCore/ChangeLog 2013-08-31 01:10:45 UTC (rev 154924)
@@ -1,3 +1,15 @@
+2013-08-30 Brent Fulgham <[email protected]>
+
+ [Windows] Build fix after r154921.
+
+ AVFoundationCF uses an internal wrapper object around the C-API. The call point
+ in the original patch was inside the wrapper, which did not (yet) provide
+ a method implementation.
+
+ * platform/graphics/avfoundation/cf/MediaPlayerPrivateAVFoundationCF.cpp:
+ (WebCore::MediaPlayerPrivateAVFoundationCF::updateVideoLayerGravity):
+ (WebCore::AVFWrapper::updateVideoLayerGravity):
+
2013-08-30 Simon Fraser <[email protected]>
Video with object-fit: cover can spill outside the box
Modified: trunk/Source/WebCore/platform/graphics/avfoundation/cf/MediaPlayerPrivateAVFoundationCF.cpp (154923 => 154924)
--- trunk/Source/WebCore/platform/graphics/avfoundation/cf/MediaPlayerPrivateAVFoundationCF.cpp 2013-08-31 00:24:54 UTC (rev 154923)
+++ trunk/Source/WebCore/platform/graphics/avfoundation/cf/MediaPlayerPrivateAVFoundationCF.cpp 2013-08-31 01:10:45 UTC (rev 154924)
@@ -104,6 +104,7 @@
void beginLoadingMetadata();
void seekToTime(float);
+ void updateVideoLayerGravity();
void setCurrentTrack(InbandTextTrackPrivateAVF*);
InbandTextTrackPrivateAVF* currentTrack() const { return m_currentTrack; }
@@ -334,7 +335,10 @@
void MediaPlayerPrivateAVFoundationCF::updateVideoLayerGravity()
{
- // We should call AVCFPlayerLayerSetVideoGravity() here, but it is not yet implemented.
+ ASSERT(supportsAcceleratedRendering());
+
+ if (m_avfWrapper)
+ m_avfWrapper->updateVideoLayerGravity();
}
bool MediaPlayerPrivateAVFoundationCF::hasLayerRenderer() const
@@ -1676,6 +1680,12 @@
}
#endif
+void AVFWrapper::updateVideoLayerGravity()
+{
+ // We should call AVCFPlayerLayerSetVideoGravity() here, but it is not yet implemented.
+ // FIXME: <rdar://problem/14884340>
+}
+
void LayerClient::platformCALayerLayoutSublayersOfLayer(PlatformCALayer* wrapperLayer)
{
ASSERT(m_parent);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
