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

Unified Diff: media/renderers/skcanvas_video_renderer.cc

Issue 2697663002: Clean up naming of paint-related identifiers (Closed)
Patch Set: 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
Index: media/renderers/skcanvas_video_renderer.cc
diff --git a/media/renderers/skcanvas_video_renderer.cc b/media/renderers/skcanvas_video_renderer.cc
index 5f76455e94c5447a411a13b466c5fe7b3e3764d6..239081bc4d773782e4cd5f6cc7683c21d0ca5676 100644
--- a/media/renderers/skcanvas_video_renderer.cc
+++ b/media/renderers/skcanvas_video_renderer.cc
@@ -341,11 +341,11 @@ SkCanvasVideoRenderer::~SkCanvasVideoRenderer() {
void SkCanvasVideoRenderer::Paint(const scoped_refptr<VideoFrame>& video_frame,
cc::PaintCanvas* canvas,
const gfx::RectF& dest_rect,
- cc::PaintFlags& paint,
+ cc::PaintFlags& flags,
VideoRotation video_rotation,
const Context3D& context_3d) {
DCHECK(thread_checker_.CalledOnValidThread());
- if (paint.getAlpha() == 0) {
+ if (flags.getAlpha() == 0) {
return;
}
@@ -359,7 +359,7 @@ void SkCanvasVideoRenderer::Paint(const scoped_refptr<VideoFrame>& video_frame,
video_frame->format() == media::PIXEL_FORMAT_Y16 ||
video_frame->HasTextures())) {
cc::PaintFlags black_with_alpha_flags;
- black_with_alpha_flags.setAlpha(paint.getAlpha());
+ black_with_alpha_flags.setAlpha(flags.getAlpha());
canvas->drawRect(dest, black_with_alpha_flags);
canvas->flush();
return;
@@ -370,9 +370,9 @@ void SkCanvasVideoRenderer::Paint(const scoped_refptr<VideoFrame>& video_frame,
return;
cc::PaintFlags video_flags;
- video_flags.setAlpha(paint.getAlpha());
- video_flags.setBlendMode(paint.getBlendMode());
- video_flags.setFilterQuality(paint.getFilterQuality());
+ video_flags.setAlpha(flags.getAlpha());
+ video_flags.setBlendMode(flags.getBlendMode());
+ video_flags.setFilterQuality(flags.getFilterQuality());
const bool need_rotation = video_rotation != VIDEO_ROTATION_0;
const bool need_scaling =
« media/renderers/skcanvas_video_renderer.h ('K') | « media/renderers/skcanvas_video_renderer.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698