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

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

Issue 1587743002: RenderWidgetCompositor should be decoupled from RenderWidget. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed failure: Don't create an OutputSurface when closing 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_( 65 render_widget_compositor_(RenderWidgetCompositor::Create(
66 RenderWidgetCompositor::Create(render_widget_.get(), 66 render_widget_.get(),
67 compositor_deps_.get())) {} 67 1.f /* initial_device_scale_factor */,
68 compositor_deps_.get())) {}
68 ~RenderWidgetCompositorTest() override {} 69 ~RenderWidgetCompositorTest() override {}
69 70
70 protected: 71 protected:
71 base::MessageLoop loop_; 72 base::MessageLoop loop_;
72 MockRenderThread render_thread_; 73 MockRenderThread render_thread_;
73 scoped_ptr<FakeCompositorDependencies> compositor_deps_; 74 scoped_ptr<FakeCompositorDependencies> compositor_deps_;
74 scoped_refptr<TestRenderWidget> render_widget_; 75 scoped_refptr<TestRenderWidget> render_widget_;
75 scoped_ptr<RenderWidgetCompositor> render_widget_compositor_; 76 scoped_ptr<RenderWidgetCompositor> render_widget_compositor_;
76 77
77 private: 78 private:
(...skipping 158 matching lines...) Expand 10 before | Expand all | Expand 10 after
236 DISALLOW_COPY_AND_ASSIGN(RenderWidgetCompositorOutputSurface); 237 DISALLOW_COPY_AND_ASSIGN(RenderWidgetCompositorOutputSurface);
237 }; 238 };
238 239
239 class RenderWidgetCompositorOutputSurfaceTest : public testing::Test { 240 class RenderWidgetCompositorOutputSurfaceTest : public testing::Test {
240 public: 241 public:
241 RenderWidgetCompositorOutputSurfaceTest() 242 RenderWidgetCompositorOutputSurfaceTest()
242 : compositor_deps_(new FakeCompositorDependencies), 243 : compositor_deps_(new FakeCompositorDependencies),
243 render_widget_(new RenderWidgetOutputSurface(compositor_deps_.get())) { 244 render_widget_(new RenderWidgetOutputSurface(compositor_deps_.get())) {
244 render_widget_compositor_.reset(new RenderWidgetCompositorOutputSurface( 245 render_widget_compositor_.reset(new RenderWidgetCompositorOutputSurface(
245 render_widget_.get(), compositor_deps_.get())); 246 render_widget_.get(), compositor_deps_.get()));
246 render_widget_compositor_->Initialize(); 247 render_widget_compositor_->Initialize(
248 1.f /* initial_device_scale_factor */);
247 render_widget_->SetCompositor(render_widget_compositor_.get()); 249 render_widget_->SetCompositor(render_widget_compositor_.get());
248 } 250 }
249 251
250 void RunTest(bool use_null_output_surface, 252 void RunTest(bool use_null_output_surface,
251 int num_failures_before_success, 253 int num_failures_before_success,
252 int expected_successes, 254 int expected_successes,
253 int expected_fallback_succeses) { 255 int expected_fallback_succeses) {
254 render_widget_compositor_->SetUp( 256 render_widget_compositor_->SetUp(
255 use_null_output_surface, num_failures_before_success, 257 use_null_output_surface, num_failures_before_success,
256 expected_successes, expected_fallback_succeses); 258 expected_successes, expected_fallback_succeses);
(...skipping 60 matching lines...) Expand 10 before | Expand all | Expand 10 after
317 } 319 }
318 320
319 TEST_F(RenderWidgetCompositorOutputSurfaceTest, FallbackSuccessNormalSuccess) { 321 TEST_F(RenderWidgetCompositorOutputSurfaceTest, FallbackSuccessNormalSuccess) {
320 // The first success is a fallback, but the next should not be a fallback. 322 // The first success is a fallback, but the next should not be a fallback.
321 RunTest(false, RenderWidgetCompositor::OUTPUT_SURFACE_RETRIES_BEFORE_FALLBACK, 323 RunTest(false, RenderWidgetCompositor::OUTPUT_SURFACE_RETRIES_BEFORE_FALLBACK,
322 1, 1); 324 1, 1);
323 } 325 }
324 326
325 } // namespace 327 } // namespace
326 } // namespace content 328 } // 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