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

Unified Diff: media/renderers/skcanvas_video_renderer.cc

Issue 2706453003: switch to new ImageGenerator API (Closed)
Patch Set: fix typo Created 3 years, 10 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 | « content/renderer/render_thread_impl.cc ('k') | skia/config/SkUserConfig.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: media/renderers/skcanvas_video_renderer.cc
diff --git a/media/renderers/skcanvas_video_renderer.cc b/media/renderers/skcanvas_video_renderer.cc
index e59db6f65b2f90526e6c39ef25f1e800686c0d75..82a99ca8cf5e1e496fd9ce6b34243215045171e0 100644
--- a/media/renderers/skcanvas_video_renderer.cc
+++ b/media/renderers/skcanvas_video_renderer.cc
@@ -8,6 +8,7 @@
#include <limits>
#include "base/macros.h"
+#include "base/memory/ptr_util.h"
#include "cc/paint/paint_canvas.h"
#include "cc/paint/paint_flags.h"
#include "gpu/GLES2/gl2extchromium.h"
@@ -938,8 +939,8 @@ bool SkCanvasVideoRenderer::UpdateLastImage(
NewSkImageFromVideoFrameNative(video_frame.get(), context_3d);
}
} else {
- auto* video_generator = new VideoImageGenerator(video_frame);
- last_image_ = SkImage::MakeFromGenerator(video_generator);
+ last_image_ = SkImage::MakeFromGenerator(
+ base::MakeUnique<VideoImageGenerator>(video_frame));
}
CorrectLastImageDimensions(gfx::RectToSkIRect(video_frame->visible_rect()));
if (!last_image_) // Couldn't create the SkImage.
« no previous file with comments | « content/renderer/render_thread_impl.cc ('k') | skia/config/SkUserConfig.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698