Commit ed839845 authored by simon.fraser@apple.com's avatar simon.fraser@apple.com
Browse files

Reloading this video test shows garbage briefly

https://bugs.webkit.org/show_bug.cgi?id=119377

Reviewed by Anders Carlson.

RenderVideo should not claim that it's foreground is opaque
unless it has a video frame to display.

* rendering/RenderVideo.cpp:
(WebCore::RenderVideo::foregroundIsKnownToBeOpaqueInRect):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@154455 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 03ddc910
2013-08-21 Simon Fraser <simon.fraser@apple.com>
Reloading this video test shows garbage briefly
https://bugs.webkit.org/show_bug.cgi?id=119377
Reviewed by Anders Carlson.
RenderVideo should not claim that it's foreground is opaque
unless it has a video frame to display.
* rendering/RenderVideo.cpp:
(WebCore::RenderVideo::foregroundIsKnownToBeOpaqueInRect):
2013-08-22 Andreas Kling <akling@apple.com>
 
Remove accidental cruft from r154449.
......@@ -343,7 +343,13 @@ bool RenderVideo::foregroundIsKnownToBeOpaqueInRect(const LayoutRect& localRect,
if (videoElement()->shouldDisplayPosterImage())
return RenderImage::foregroundIsKnownToBeOpaqueInRect(localRect, maxDepthToTest);
return videoBox().contains(enclosingIntRect(localRect));
if (!videoBox().contains(enclosingIntRect(localRect)))
return false;
if (MediaPlayer* player = mediaElement()->player())
return player->hasAvailableVideoFrame();
return false;
}
} // namespace WebCore
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment