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

Side by Side Diff: cc/output/renderer_pixeltest.cc

Issue 92703003: Support videos with JPEG color range in GPU YUV convert path. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@yuv_v2
Patch Set: jpeg_color_range->has_jpeg_color_range Created 7 years 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
OLDNEW
1 // Copyright 2012 The Chromium Authors. All rights reserved. 1 // Copyright 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 "base/message_loop/message_loop.h" 5 #include "base/message_loop/message_loop.h"
6 #include "cc/layers/append_quads_data.h" 6 #include "cc/layers/append_quads_data.h"
7 #include "cc/output/gl_renderer.h" 7 #include "cc/output/gl_renderer.h"
8 #include "cc/quads/draw_quad.h" 8 #include "cc/quads/draw_quad.h"
9 #include "cc/quads/picture_draw_quad.h" 9 #include "cc/quads/picture_draw_quad.h"
10 #include "cc/quads/texture_draw_quad.h" 10 #include "cc/quads/texture_draw_quad.h"
(...skipping 449 matching lines...) Expand 10 before | Expand all | Expand 10 after
460 gfx::Vector2d()); 460 gfx::Vector2d());
461 resource_provider_->SetPixels(v_resource, v_plane.get(), uv_rect, uv_rect, 461 resource_provider_->SetPixels(v_resource, v_plane.get(), uv_rect, uv_rect,
462 gfx::Vector2d()); 462 gfx::Vector2d());
463 if (with_alpha) { 463 if (with_alpha) {
464 resource_provider_->SetPixels(a_resource, a_plane.get(), rect, rect, 464 resource_provider_->SetPixels(a_resource, a_plane.get(), rect, rect,
465 gfx::Vector2d()); 465 gfx::Vector2d());
466 } 466 }
467 467
468 scoped_ptr<YUVVideoDrawQuad> yuv_quad = cc::YUVVideoDrawQuad::Create(); 468 scoped_ptr<YUVVideoDrawQuad> yuv_quad = cc::YUVVideoDrawQuad::Create();
469 yuv_quad->SetNew(shared_state, rect, opaque_rect, gfx::Size(), 469 yuv_quad->SetNew(shared_state, rect, opaque_rect, gfx::Size(),
470 y_resource, u_resource, v_resource, a_resource); 470 y_resource, u_resource, v_resource, a_resource, false);
danakj 2013/12/03 21:35:34 use a temp variable here to give the boolean liter
rileya (GONE FROM CHROMIUM) 2013/12/03 23:33:41 Made into a param actually, in order to add the te
471 return yuv_quad.Pass(); 471 return yuv_quad.Pass();
472 } 472 }
473 }; 473 };
474 474
475 TEST_F(VideoGLRendererPixelTest, SimpleYUVRect) { 475 TEST_F(VideoGLRendererPixelTest, SimpleYUVRect) {
danakj 2013/12/03 21:35:34 Can you make a test like this that uses your new c
476 gfx::Rect rect(this->device_viewport_size_); 476 gfx::Rect rect(this->device_viewport_size_);
477 477
478 RenderPass::Id id(1, 1); 478 RenderPass::Id id(1, 1);
479 scoped_ptr<RenderPass> pass = CreateTestRootRenderPass(id, rect); 479 scoped_ptr<RenderPass> pass = CreateTestRootRenderPass(id, rect);
480 480
481 scoped_ptr<SharedQuadState> shared_state = 481 scoped_ptr<SharedQuadState> shared_state =
482 CreateTestSharedQuadState(gfx::Transform(), rect); 482 CreateTestSharedQuadState(gfx::Transform(), rect);
483 483
484 scoped_ptr<YUVVideoDrawQuad> yuv_quad = 484 scoped_ptr<YUVVideoDrawQuad> yuv_quad =
485 CreateTestYUVVideoDrawQuad(shared_state.get(), false, false); 485 CreateTestYUVVideoDrawQuad(shared_state.get(), false, false);
(...skipping 1589 matching lines...) Expand 10 before | Expand all | Expand 10 after
2075 &pass_list, 2075 &pass_list,
2076 PixelTest::NoOffscreenContext, 2076 PixelTest::NoOffscreenContext,
2077 base::FilePath(FILE_PATH_LITERAL("wrap_mode_repeat.png")), 2077 base::FilePath(FILE_PATH_LITERAL("wrap_mode_repeat.png")),
2078 FuzzyPixelOffByOneComparator(true))); 2078 FuzzyPixelOffByOneComparator(true)));
2079 } 2079 }
2080 2080
2081 #endif // !defined(OS_ANDROID) 2081 #endif // !defined(OS_ANDROID)
2082 2082
2083 } // namespace 2083 } // namespace
2084 } // namespace cc 2084 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698