Title: [154455] trunk/Source/WebCore
- Revision
- 154455
- Author
- [email protected]
- Date
- 2013-08-22 11:48:22 -0700 (Thu, 22 Aug 2013)
Log Message
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):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (154454 => 154455)
--- trunk/Source/WebCore/ChangeLog 2013-08-22 16:59:45 UTC (rev 154454)
+++ trunk/Source/WebCore/ChangeLog 2013-08-22 18:48:22 UTC (rev 154455)
@@ -1,3 +1,16 @@
+2013-08-21 Simon Fraser <[email protected]>
+
+ 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 <[email protected]>
Remove accidental cruft from r154449.
Modified: trunk/Source/WebCore/rendering/RenderVideo.cpp (154454 => 154455)
--- trunk/Source/WebCore/rendering/RenderVideo.cpp 2013-08-22 16:59:45 UTC (rev 154454)
+++ trunk/Source/WebCore/rendering/RenderVideo.cpp 2013-08-22 18:48:22 UTC (rev 154455)
@@ -343,7 +343,13 @@
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
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
