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

Unified Diff: cc/output/gl_renderer_unittest.cc

Issue 132163004: Remove WebGraphicsContext3D getter from cc::ContextProvider (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 11 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: cc/output/gl_renderer_unittest.cc
diff --git a/cc/output/gl_renderer_unittest.cc b/cc/output/gl_renderer_unittest.cc
index b7f5c801938ae9616831126358d98bc9fe32b8c7..69c1cf0d364221237bc5a27139997c6657dc987c 100644
--- a/cc/output/gl_renderer_unittest.cc
+++ b/cc/output/gl_renderer_unittest.cc
@@ -179,8 +179,8 @@ class FakeRendererGL : public GLRenderer {
class GLRendererWithDefaultHarnessTest : public GLRendererTest {
protected:
GLRendererWithDefaultHarnessTest() {
- output_surface_ = FakeOutputSurface::Create3d(
- TestWebGraphicsContext3D::Create()).Pass();
+ output_surface_ =
+ FakeOutputSurface::Create3d(TestWebGraphicsContext3D::Create()).Pass();
CHECK(output_surface_->BindToClient(&output_surface_client_));
resource_provider_ = ResourceProvider::Create(
@@ -745,12 +745,8 @@ class VisibilityChangeIsLastCallTrackingContext
: last_call_was_set_visibility_(false) {}
// WebGraphicsContext3D methods.
- virtual void flush() {
- last_call_was_set_visibility_ = false;
- }
- virtual void deleteTexture(WebGLId) {
- last_call_was_set_visibility_ = false;
- }
+ virtual void flush() { last_call_was_set_visibility_ = false; }
+ virtual void deleteTexture(WebGLId) { last_call_was_set_visibility_ = false; }
virtual void deleteFramebuffer(WebGLId) {
last_call_was_set_visibility_ = false;
}
@@ -787,8 +783,8 @@ TEST_F(GLRendererTest, VisibilityChangeIsLastCall) {
base::Unretained(context)));
FakeOutputSurfaceClient output_surface_client;
- scoped_ptr<OutputSurface> output_surface(FakeOutputSurface::Create3d(
- provider));
+ scoped_ptr<OutputSurface> output_surface(
+ FakeOutputSurface::Create3d(provider));
CHECK(output_surface->BindToClient(&output_surface_client));
scoped_ptr<ResourceProvider> resource_provider(
@@ -1699,9 +1695,7 @@ TEST_F(GLRendererShaderTest, DrawSolidColorShader) {
class OutputSurfaceMockContext : public TestWebGraphicsContext3D {
public:
- OutputSurfaceMockContext() {
- test_capabilities_.post_sub_buffer = true;
- }
+ OutputSurfaceMockContext() { test_capabilities_.post_sub_buffer = true; }
// Specifically override methods even if they are unused (used in conjunction
// with StrictMock). We need to make sure that GLRenderer does not issue
@@ -1782,7 +1776,8 @@ class MockOutputSurfaceTest : public GLRendererTest {
OutputSurfaceMockContext* Context() {
return static_cast<OutputSurfaceMockContext*>(
- output_surface_.context_provider()->Context3d());
+ static_cast<TestContextProvider*>(
+ output_surface_.context_provider().get())->TestContext3d());
}
LayerTreeSettings settings_;

Powered by Google App Engine
This is Rietveld 408576698