Changeset 155810 in webkit for trunk/Source/WebCore/rendering/RenderVideo.cpp

Timestamp:

Sep 15, 2013, 7:12:11 AM (12 years ago)

Author:

[email protected]

Message:

Unreviewed, rolling out r155809.
​http://trac.webkit.org/changeset/155809
​https://bugs.webkit.org/show_bug.cgi?id=121388

Accidentally removed files, let's redo this.

File:

• trunk/Source/WebCore/rendering/RenderVideo.cpp (modified) (18 diffs)

trunk/Source/WebCore/rendering/RenderVideo.cpp

@@ -49 +49 @@
 using namespace HTMLNames;
 
-RenderVideo::RenderVideo(HTMLVideoElement& element)
-    : RenderMedia(element)
+RenderVideo::RenderVideo(HTMLVideoElement* video)
+    : RenderMedia(video)
 {
     setIntrinsicSize(calculateIntrinsicSize());
@@ -57 +57 @@
 RenderVideo::~RenderVideo()
 {
-    if (MediaPlayer* player = videoElement().player()) {
-        player->setVisible(false);
-        player->setFrameView(0);
+    if (MediaPlayer* p = mediaElement()->player()) {
+        p->setVisible(false);
+        p->setFrameView(0);
     }
 }
@@ -74 +74 @@
 void RenderVideo::intrinsicSizeChanged()
 {
-    if (videoElement().shouldDisplayPosterImage())
+    if (videoElement()->shouldDisplayPosterImage())
         RenderMedia::intrinsicSizeChanged();
     updateIntrinsicSize(); 
@@ -85 +85 @@
 
     // Never set the element size to zero when in a media document.
-    if (size.isEmpty() && document().isMediaDocument())
+    if (size.isEmpty() && element()->document().isMediaDocument())
         return;
 
@@ -98 +98 @@
 LayoutSize RenderVideo::calculateIntrinsicSize()
 {
+    HTMLVideoElement* video = videoElement();
+    
     // Spec text from 4.8.6
     //
@@ -107 +109 @@
     // of the video resource, if that is available; otherwise it is the intrinsic 
     // height of the poster frame, if that is available; otherwise it is 150 CSS pixels.
-    MediaPlayer* player = videoElement().player();
-    if (player && videoElement().readyState() >= HTMLVideoElement::HAVE_METADATA) {
+    MediaPlayer* player = mediaElement()->player();
+    if (player && video->readyState() >= HTMLVideoElement::HAVE_METADATA) {
         LayoutSize size = player->naturalSize();
         if (!size.isEmpty())
@@ -114 +116 @@
     }
 
-    if (videoElement().shouldDisplayPosterImage() && !m_cachedImageSize.isEmpty() && !imageResource()->errorOccurred())
+    if (video->shouldDisplayPosterImage() && !m_cachedImageSize.isEmpty() && !imageResource()->errorOccurred())
         return m_cachedImageSize;
 
     // When the natural size of the video is unavailable, we use the provided
     // width and height attributes of the video element as the intrinsic size until
-    // better values become available.
-    if (videoElement().hasAttribute(widthAttr) && videoElement().hasAttribute(heightAttr))
-        return LayoutSize(videoElement().width(), videoElement().height());
+    // better values become available. 
+    if (video->hasAttribute(widthAttr) && video->hasAttribute(heightAttr))
+        return LayoutSize(video->width(), video->height());
 
     // <video> in standalone media documents should not use the default 300x150
@@ -127 +129 @@
     // size to 300x1 the video will resize itself in these cases, and audio will
     // have the correct height (it needs to be > 0 for controls to render properly).
-    if (videoElement().document().isMediaDocument())
+    if (video->document().isMediaDocument())
         return LayoutSize(defaultSize().width(), 1);
 
@@ -140 +142 @@
     // even if we know the video intrinsic size but aren't able to draw video frames yet
     // (we don't want to scale the poster to the video size without keeping aspect ratio).
-    if (videoElement().shouldDisplayPosterImage())
+    if (videoElement()->shouldDisplayPosterImage())
         m_cachedImageSize = intrinsicSize();
 
@@ -152 +154 @@
     LayoutSize intrinsicSize = this->intrinsicSize();
 
-    if (videoElement().shouldDisplayPosterImage())
+    if (videoElement()->shouldDisplayPosterImage())
         intrinsicSize = m_cachedImageSize;
 
@@ -160 +162 @@
 bool RenderVideo::shouldDisplayVideo() const
 {
-    return !videoElement().shouldDisplayPosterImage();
+    return !videoElement()->shouldDisplayPosterImage();
 }
 
 void RenderVideo::paintReplaced(PaintInfo& paintInfo, const LayoutPoint& paintOffset)
 {
-    MediaPlayer* mediaPlayer = videoElement().player();
-    bool displayingPoster = videoElement().shouldDisplayPosterImage();
+    MediaPlayer* mediaPlayer = mediaElement()->player();
+    bool displayingPoster = videoElement()->shouldDisplayPosterImage();
 
     Page* page = frame().page();
@@ -210 +212 @@
 }
     
-HTMLVideoElement& RenderVideo::videoElement() const
-{
-    return toHTMLVideoElement(RenderMedia::mediaElement());
+HTMLVideoElement* RenderVideo::videoElement() const
+{
+    return toHTMLVideoElement(element()); 
 }
 
@@ -228 +230 @@
     updateIntrinsicSize();
 
-    MediaPlayer* mediaPlayer = videoElement().player();
+    MediaPlayer* mediaPlayer = mediaElement()->player();
     if (!mediaPlayer)
         return;
 
-    if (!videoElement().inActiveDocument()) {
+    if (!videoElement()->inActiveDocument()) {
         mediaPlayer->setVisible(false);
         return;
@@ -266 +268 @@
 bool RenderVideo::supportsAcceleratedRendering() const
 {
-    if (MediaPlayer* player = videoElement().player())
-        return player->supportsAcceleratedRendering();
+    MediaPlayer* p = mediaElement()->player();
+    if (p)
+        return p->supportsAcceleratedRendering();
+
     return false;
 }
@@ -273 +277 @@
 void RenderVideo::acceleratedRenderingStateChanged()
 {
-    if (MediaPlayer* player = videoElement().player())
-        player->acceleratedRenderingStateChanged();
+    MediaPlayer* p = mediaElement()->player();
+    if (p)
+        p->acceleratedRenderingStateChanged();
 }
 #endif  // USE(ACCELERATED_COMPOSITING)
@@ -280 +285 @@
 bool RenderVideo::requiresImmediateCompositing() const
 {
-    MediaPlayer* player = videoElement().player();
+    MediaPlayer* player = mediaElement()->player();
     return player && player->requiresImmediateCompositing();
 }
@@ -329 +334 @@
 bool RenderVideo::foregroundIsKnownToBeOpaqueInRect(const LayoutRect& localRect, unsigned maxDepthToTest) const
 {
-    if (videoElement().shouldDisplayPosterImage())
+    if (videoElement()->shouldDisplayPosterImage())
         return RenderImage::foregroundIsKnownToBeOpaqueInRect(localRect, maxDepthToTest);
 
@@ -335 +340 @@
         return false;
 
-    if (MediaPlayer* player = videoElement().player())
+    if (MediaPlayer* player = mediaElement()->player())
         return player->hasAvailableVideoFrame();