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

Side by Side Diff: content/browser/compositor/software_browser_compositor_output_surface_unittest.cc

Issue 277713002: ui/compositor: move the browser compositor thread to the ContextFactory (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase, fix includes Created 6 years, 7 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 | Annotate | Revision Log
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 "base/message_loop/message_loop.h" 5 #include "base/message_loop/message_loop.h"
6 #include "cc/output/compositor_frame.h" 6 #include "cc/output/compositor_frame.h"
7 #include "content/browser/compositor/browser_compositor_output_surface_proxy.h" 7 #include "content/browser/compositor/browser_compositor_output_surface_proxy.h"
8 #include "content/browser/compositor/software_browser_compositor_output_surface. h" 8 #include "content/browser/compositor/software_browser_compositor_output_surface. h"
9 #include "testing/gtest/include/gtest/gtest.h" 9 #include "testing/gtest/include/gtest/gtest.h"
10 #include "ui/compositor/compositor.h" 10 #include "ui/compositor/compositor.h"
(...skipping 73 matching lines...) Expand 10 before | Expand all | Expand 10 after
84 // compositor to use. 84 // compositor to use.
85 message_loop_.reset(new base::MessageLoopForUI); 85 message_loop_.reset(new base::MessageLoopForUI);
86 } 86 }
87 87
88 SoftwareBrowserCompositorOutputSurfaceTest:: 88 SoftwareBrowserCompositorOutputSurfaceTest::
89 ~SoftwareBrowserCompositorOutputSurfaceTest() {} 89 ~SoftwareBrowserCompositorOutputSurfaceTest() {}
90 90
91 void SoftwareBrowserCompositorOutputSurfaceTest::SetUp() { 91 void SoftwareBrowserCompositorOutputSurfaceTest::SetUp() {
92 bool enable_pixel_output = false; 92 bool enable_pixel_output = false;
93 ui::InitializeContextFactoryForTests(enable_pixel_output); 93 ui::InitializeContextFactoryForTests(enable_pixel_output);
94 ui::Compositor::Initialize();
95 94
96 compositor_.reset(new ui::Compositor(gfx::kNullAcceleratedWidget)); 95 compositor_.reset(new ui::Compositor(gfx::kNullAcceleratedWidget));
97 surface_proxy_ = 96 surface_proxy_ =
98 new content::BrowserCompositorOutputSurfaceProxy(&surface_map_); 97 new content::BrowserCompositorOutputSurfaceProxy(&surface_map_);
99 } 98 }
100 99
101 void SoftwareBrowserCompositorOutputSurfaceTest::TearDown() { 100 void SoftwareBrowserCompositorOutputSurfaceTest::TearDown() {
102 output_surface_.reset(); 101 output_surface_.reset();
103 compositor_.reset(); 102 compositor_.reset();
104 103
105 EXPECT_TRUE(surface_map_.IsEmpty()); 104 EXPECT_TRUE(surface_map_.IsEmpty());
106 105
107 surface_map_.Clear(); 106 surface_map_.Clear();
108 ui::TerminateContextFactoryForTests(); 107 ui::TerminateContextFactoryForTests();
109 ui::Compositor::Terminate();
110 } 108 }
111 109
112 scoped_ptr<content::BrowserCompositorOutputSurface> 110 scoped_ptr<content::BrowserCompositorOutputSurface>
113 SoftwareBrowserCompositorOutputSurfaceTest::CreateSurface( 111 SoftwareBrowserCompositorOutputSurfaceTest::CreateSurface(
114 scoped_ptr<cc::SoftwareOutputDevice> device) { 112 scoped_ptr<cc::SoftwareOutputDevice> device) {
115 return scoped_ptr<content::BrowserCompositorOutputSurface>( 113 return scoped_ptr<content::BrowserCompositorOutputSurface>(
116 new content::SoftwareBrowserCompositorOutputSurface( 114 new content::SoftwareBrowserCompositorOutputSurface(
117 surface_proxy_, 115 surface_proxy_,
118 device.Pass(), 116 device.Pass(),
119 1, 117 1,
(...skipping 19 matching lines...) Expand all
139 137
140 FakeVSyncProvider* vsync_provider = static_cast<FakeVSyncProvider*>( 138 FakeVSyncProvider* vsync_provider = static_cast<FakeVSyncProvider*>(
141 output_surface_->software_device()->GetVSyncProvider()); 139 output_surface_->software_device()->GetVSyncProvider());
142 EXPECT_EQ(0, vsync_provider->call_count()); 140 EXPECT_EQ(0, vsync_provider->call_count());
143 141
144 cc::CompositorFrame frame; 142 cc::CompositorFrame frame;
145 output_surface_->SwapBuffers(&frame); 143 output_surface_->SwapBuffers(&frame);
146 144
147 EXPECT_EQ(1, vsync_provider->call_count()); 145 EXPECT_EQ(1, vsync_provider->call_count());
148 } 146 }
OLDNEW
« no previous file with comments | « content/browser/compositor/reflector_impl.cc ('k') | content/browser/compositor/software_output_device_ozone_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698