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

Side by Side Diff: content/renderer/gpu/render_widget_compositor_unittest.cc

Issue 1586523004: Revert of RenderWidgetCompositor should be decoupled from RenderWidget. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "content/renderer/gpu/render_widget_compositor.h" 5 #include "content/renderer/gpu/render_widget_compositor.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/location.h" 9 #include "base/location.h"
10 #include "base/macros.h" 10 #include "base/macros.h"
(...skipping 44 matching lines...) Expand 10 before | Expand all | Expand 10 after
55 DISALLOW_COPY_AND_ASSIGN(TestRenderWidget); 55 DISALLOW_COPY_AND_ASSIGN(TestRenderWidget);
56 }; 56 };
57 57
58 int TestRenderWidget::next_routing_id_ = 0; 58 int TestRenderWidget::next_routing_id_ = 0;
59 59
60 class RenderWidgetCompositorTest : public testing::Test { 60 class RenderWidgetCompositorTest : public testing::Test {
61 public: 61 public:
62 RenderWidgetCompositorTest() 62 RenderWidgetCompositorTest()
63 : compositor_deps_(new FakeCompositorDependencies), 63 : compositor_deps_(new FakeCompositorDependencies),
64 render_widget_(new TestRenderWidget(compositor_deps_.get())), 64 render_widget_(new TestRenderWidget(compositor_deps_.get())),
65 render_widget_compositor_(RenderWidgetCompositor::Create( 65 render_widget_compositor_(
66 render_widget_.get(), 66 RenderWidgetCompositor::Create(render_widget_.get(),
67 1.f /* initial_device_scale_factor */, 67 compositor_deps_.get())) {}
68 compositor_deps_.get())) {}
69 ~RenderWidgetCompositorTest() override {} 68 ~RenderWidgetCompositorTest() override {}
70 69
71 protected: 70 protected:
72 base::MessageLoop loop_; 71 base::MessageLoop loop_;
73 MockRenderThread render_thread_; 72 MockRenderThread render_thread_;
74 scoped_ptr<FakeCompositorDependencies> compositor_deps_; 73 scoped_ptr<FakeCompositorDependencies> compositor_deps_;
75 scoped_refptr<TestRenderWidget> render_widget_; 74 scoped_refptr<TestRenderWidget> render_widget_;
76 scoped_ptr<RenderWidgetCompositor> render_widget_compositor_; 75 scoped_ptr<RenderWidgetCompositor> render_widget_compositor_;
77 76
78 private: 77 private:
(...skipping 158 matching lines...) Expand 10 before | Expand all | Expand 10 after
237 DISALLOW_COPY_AND_ASSIGN(RenderWidgetCompositorOutputSurface); 236 DISALLOW_COPY_AND_ASSIGN(RenderWidgetCompositorOutputSurface);
238 }; 237 };
239 238
240 class RenderWidgetCompositorOutputSurfaceTest : public testing::Test { 239 class RenderWidgetCompositorOutputSurfaceTest : public testing::Test {
241 public: 240 public:
242 RenderWidgetCompositorOutputSurfaceTest() 241 RenderWidgetCompositorOutputSurfaceTest()
243 : compositor_deps_(new FakeCompositorDependencies), 242 : compositor_deps_(new FakeCompositorDependencies),
244 render_widget_(new RenderWidgetOutputSurface(compositor_deps_.get())) { 243 render_widget_(new RenderWidgetOutputSurface(compositor_deps_.get())) {
245 render_widget_compositor_.reset(new RenderWidgetCompositorOutputSurface( 244 render_widget_compositor_.reset(new RenderWidgetCompositorOutputSurface(
246 render_widget_.get(), compositor_deps_.get())); 245 render_widget_.get(), compositor_deps_.get()));
247 render_widget_compositor_->Initialize( 246 render_widget_compositor_->Initialize();
248 1.f /* initial_device_scale_factor */);
249 render_widget_->SetCompositor(render_widget_compositor_.get()); 247 render_widget_->SetCompositor(render_widget_compositor_.get());
250 } 248 }
251 249
252 void RunTest(bool use_null_output_surface, 250 void RunTest(bool use_null_output_surface,
253 int num_failures_before_success, 251 int num_failures_before_success,
254 int expected_successes, 252 int expected_successes,
255 int expected_fallback_succeses) { 253 int expected_fallback_succeses) {
256 render_widget_compositor_->SetUp( 254 render_widget_compositor_->SetUp(
257 use_null_output_surface, num_failures_before_success, 255 use_null_output_surface, num_failures_before_success,
258 expected_successes, expected_fallback_succeses); 256 expected_successes, expected_fallback_succeses);
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
319 } 317 }
320 318
321 TEST_F(RenderWidgetCompositorOutputSurfaceTest, FallbackSuccessNormalSuccess) { 319 TEST_F(RenderWidgetCompositorOutputSurfaceTest, FallbackSuccessNormalSuccess) {
322 // The first success is a fallback, but the next should not be a fallback. 320 // The first success is a fallback, but the next should not be a fallback.
323 RunTest(false, RenderWidgetCompositor::OUTPUT_SURFACE_RETRIES_BEFORE_FALLBACK, 321 RunTest(false, RenderWidgetCompositor::OUTPUT_SURFACE_RETRIES_BEFORE_FALLBACK,
324 1, 1); 322 1, 1);
325 } 323 }
326 324
327 } // namespace 325 } // namespace
328 } // namespace content 326 } // namespace content
OLDNEW
« no previous file with comments | « content/renderer/gpu/render_widget_compositor_delegate.h ('k') | content/renderer/render_widget.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698