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

Unified Diff: ui/ozone/platform/dri/dri_surface_factory_unittest.cc

Issue 106633002: GBM Ozone implementation (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « ui/ozone/platform/dri/dri_surface_factory.cc ('k') | ui/ozone/platform/dri/dri_surface_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/ozone/platform/dri/dri_surface_factory_unittest.cc
diff --git a/ui/ozone/platform/dri/dri_surface_factory_unittest.cc b/ui/ozone/platform/dri/dri_surface_factory_unittest.cc
index 1622e7eba3ef1e704db398a389c6a7f8e0ab6697..4cdd1bf765b5f03aaf0a28649ca9fad7b54f1e71 100644
--- a/ui/ozone/platform/dri/dri_surface_factory_unittest.cc
+++ b/ui/ozone/platform/dri/dri_surface_factory_unittest.cc
@@ -18,6 +18,7 @@
#include "ui/ozone/platform/dri/screen_manager.h"
#include "ui/ozone/platform/dri/test/mock_dri_surface.h"
#include "ui/ozone/platform/dri/test/mock_dri_wrapper.h"
+#include "ui/ozone/platform/dri/test/mock_surface_generator.h"
namespace {
@@ -53,25 +54,17 @@ class MockDriSurfaceFactory : public ui::DriSurfaceFactory {
class MockScreenManager : public ui::ScreenManager {
public:
- MockScreenManager(ui::DriWrapper* dri)
- : ScreenManager(dri),
+ MockScreenManager(ui::DriWrapper* dri,
+ ui::ScanoutSurfaceGenerator* surface_generator)
+ : ScreenManager(dri, surface_generator),
dri_(dri) {}
virtual ~MockScreenManager() {}
- const std::vector<ui::MockDriSurface*>& get_surfaces() const {
- return surfaces_;
- }
-
// Normally we'd use DRM to figure out the controller configuration. But we
// can't use DRM in unit tests, so we just create a fake configuration.
virtual void ForceInitializationOfPrimaryDisplay() OVERRIDE {
ConfigureDisplayController(1, 2, kDefaultMode);
}
- virtual ui::DriSurface* CreateSurface(const gfx::Size& size) OVERRIDE {
- ui::MockDriSurface* surface = new ui::MockDriSurface(dri_, size);
- surfaces_.push_back(surface);
- return surface;
- }
private:
ui::DriWrapper* dri_; // Not owned.
@@ -91,6 +84,7 @@ class DriSurfaceFactoryTest : public testing::Test {
protected:
scoped_ptr<base::MessageLoop> message_loop_;
scoped_ptr<ui::MockDriWrapper> dri_;
+ scoped_ptr<ui::MockSurfaceGenerator> surface_generator_;
scoped_ptr<MockScreenManager> screen_manager_;
scoped_ptr<MockDriSurfaceFactory> factory_;
@@ -101,7 +95,9 @@ class DriSurfaceFactoryTest : public testing::Test {
void DriSurfaceFactoryTest::SetUp() {
message_loop_.reset(new base::MessageLoopForUI);
dri_.reset(new ui::MockDriWrapper(3));
- screen_manager_.reset(new MockScreenManager(dri_.get()));
+ surface_generator_.reset(new ui::MockSurfaceGenerator(dri_.get()));
+ screen_manager_.reset(new MockScreenManager(dri_.get(),
+ surface_generator_.get()));
factory_.reset(new MockDriSurfaceFactory(dri_.get(), screen_manager_.get()));
}
@@ -147,7 +143,7 @@ TEST_F(DriSurfaceFactoryTest, CheckNativeSurfaceContents) {
gfx::Rect(0, 0, kDefaultMode.hdisplay / 2, kDefaultMode.vdisplay / 2));
const std::vector<ui::DriBuffer*>& bitmaps =
- screen_manager_->get_surfaces()[0]->bitmaps();
+ surface_generator_->surfaces()[0]->bitmaps();
SkBitmap image;
bitmaps[1]->canvas()->readPixels(&image, 0, 0);
« no previous file with comments | « ui/ozone/platform/dri/dri_surface_factory.cc ('k') | ui/ozone/platform/dri/dri_surface_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698