Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(424)

Unified Diff: webkit/glue/media/video_renderer_impl.cc

Issue 7792041: roll skia to 2193 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « DEPS ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webkit/glue/media/video_renderer_impl.cc
===================================================================
--- webkit/glue/media/video_renderer_impl.cc (revision 98831)
+++ webkit/glue/media/video_renderer_impl.cc (working copy)
@@ -11,18 +11,6 @@
#include "third_party/skia/include/core/SkDevice.h"
#include "webkit/glue/webmediaplayer_proxy.h"
-// Transform destination rect to local coordinates.
-static void TransformToSkIRect(const SkMatrix& matrix,
- const gfx::Rect& src_rect,
- SkIRect* dest_rect) {
- SkRect transformed_rect;
- SkRect skia_dest_rect;
- skia_dest_rect.iset(src_rect.x(), src_rect.y(),
- src_rect.right(), src_rect.bottom());
- matrix.mapRect(&transformed_rect, skia_dest_rect);
- transformed_rect.round(dest_rect);
-}
-
namespace webkit_glue {
VideoRendererImpl::VideoRendererImpl(bool pts_logging)
@@ -127,23 +115,10 @@
SkScalarNearlyZero(total_matrix.getSkewY()) &&
total_matrix.getScaleX() > 0 &&
total_matrix.getScaleY() > 0) {
- // Get the properties of the SkDevice and the clip rect.
SkDevice* device = canvas->getDevice();
-
- // Get the boundary of the device.
- SkIRect device_rect;
- device->getBounds(&device_rect);
-
- // Get the pixel config of the device.
const SkBitmap::Config config = device->config();
- // Get the total clip rect associated with the canvas.
- const SkRegion& total_clip = canvas->getTotalClip();
- SkIRect dest_irect;
- TransformToSkIRect(canvas->getTotalMatrix(), dest_rect, &dest_irect);
-
- if (config == SkBitmap::kARGB_8888_Config && device->isOpaque() &&
- device_rect.contains(total_clip.getBounds())) {
+ if (config == SkBitmap::kARGB_8888_Config && device->isOpaque()) {
return true;
}
}
« no previous file with comments | « DEPS ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698