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

Side by Side Diff: media/renderers/skcanvas_video_renderer_unittest.cc

Issue 2317173002: Remove a un-necessary initialization line (Closed)
Patch Set: Created 4 years, 3 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <stdint.h> 5 #include <stdint.h>
6 6
7 #include "base/macros.h" 7 #include "base/macros.h"
8 #include "base/message_loop/message_loop.h" 8 #include "base/message_loop/message_loop.h"
9 #include "gpu/GLES2/gl2extchromium.h" 9 #include "gpu/GLES2/gl2extchromium.h"
10 #include "gpu/command_buffer/client/gles2_interface_stub.h" 10 #include "gpu/command_buffer/client/gles2_interface_stub.h"
(...skipping 224 matching lines...) Expand 10 before | Expand all | Expand 10 after
235 break; 235 break;
236 case kGreen: 236 case kGreen:
237 media::FillYUV(video_frame.get(), 149, 43, 21); 237 media::FillYUV(video_frame.get(), 149, 43, 21);
238 break; 238 break;
239 case kBlue: 239 case kBlue:
240 media::FillYUV(video_frame.get(), 29, 255, 107); 240 media::FillYUV(video_frame.get(), 29, 255, 107);
241 break; 241 break;
242 } 242 }
243 SkPaint paint; 243 SkPaint paint;
244 paint.setXfermodeMode(mode); 244 paint.setXfermodeMode(mode);
245 paint.setAlpha(0xFF);
246 paint.setFilterQuality(kLow_SkFilterQuality); 245 paint.setFilterQuality(kLow_SkFilterQuality);
247 renderer_.Paint(video_frame, canvas, dest_rect, paint, video_rotation, 246 renderer_.Paint(video_frame, canvas, dest_rect, paint, video_rotation,
248 Context3D()); 247 Context3D());
249 } 248 }
250 249
251 void SkCanvasVideoRendererTest::Copy( 250 void SkCanvasVideoRendererTest::Copy(
252 const scoped_refptr<VideoFrame>& video_frame, 251 const scoped_refptr<VideoFrame>& video_frame,
253 SkCanvas* canvas) { 252 SkCanvas* canvas) {
254 renderer_.Copy(video_frame, canvas, Context3D()); 253 renderer_.Copy(video_frame, canvas, Context3D());
255 } 254 }
(...skipping 285 matching lines...) Expand 10 before | Expand all | Expand 10 after
541 auto video_frame = VideoFrame::WrapNativeTextures( 540 auto video_frame = VideoFrame::WrapNativeTextures(
542 PIXEL_FORMAT_UYVY, holders, base::Bind(MailboxHoldersReleased), size, 541 PIXEL_FORMAT_UYVY, holders, base::Bind(MailboxHoldersReleased), size,
543 gfx::Rect(size), size, kNoTimestamp); 542 gfx::Rect(size), size, kNoTimestamp);
544 543
545 SkPaint paint; 544 SkPaint paint;
546 paint.setFilterQuality(kLow_SkFilterQuality); 545 paint.setFilterQuality(kLow_SkFilterQuality);
547 renderer_.Paint(video_frame, &canvas, kNaturalRect, paint, VIDEO_ROTATION_90, 546 renderer_.Paint(video_frame, &canvas, kNaturalRect, paint, VIDEO_ROTATION_90,
548 context_3d); 547 context_3d);
549 } 548 }
550 } // namespace media 549 } // namespace media
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698