Index: cc/layers/texture_layer_unittest.cc |
diff --git a/cc/layers/texture_layer_unittest.cc b/cc/layers/texture_layer_unittest.cc |
index 4e2f67577adf3d17bfe663e265e8ed8fc5147c6e..f2dbb335c6757eb8c4b99b98f0d9428a8a097764 100644 |
--- a/cc/layers/texture_layer_unittest.cc |
+++ b/cc/layers/texture_layer_unittest.cc |
@@ -101,7 +101,7 @@ class MockMailboxCallback { |
uint32 sync_point, |
bool lost_resource)); |
MOCK_METHOD3(Release2, |
- void(base::SharedMemory* shared_memory, |
+ void(SharedBitmap* shared_bitmap, |
uint32 sync_point, |
bool lost_resource)); |
MOCK_METHOD4(ReleaseImpl, |
@@ -110,19 +110,18 @@ class MockMailboxCallback { |
bool lost_resource, |
BlockingTaskRunner* main_thread_task_runner)); |
MOCK_METHOD4(ReleaseImpl2, |
- void(base::SharedMemory* shared_memory, |
+ void(SharedBitmap* shared_bitmap, |
uint32 sync_point, |
bool lost_resource, |
BlockingTaskRunner* main_thread_task_runner)); |
}; |
struct CommonMailboxObjects { |
- CommonMailboxObjects() |
+ explicit CommonMailboxObjects(SharedBitmapManager* manager) |
: mailbox_name1_(MailboxFromChar('1')), |
mailbox_name2_(MailboxFromChar('2')), |
sync_point1_(1), |
- sync_point2_(2), |
- shared_memory_(new base::SharedMemory) { |
+ sync_point2_(2) { |
release_mailbox1_ = base::Bind(&MockMailboxCallback::Release, |
base::Unretained(&mock_callback_), |
mailbox_name1_); |
@@ -140,14 +139,15 @@ struct CommonMailboxObjects { |
mailbox1_ = TextureMailbox(mailbox_name1_, arbitrary_target1, sync_point1_); |
mailbox2_ = TextureMailbox(mailbox_name2_, arbitrary_target2, sync_point2_); |
gfx::Size size(128, 128); |
- EXPECT_TRUE(shared_memory_->CreateAndMapAnonymous(4 * size.GetArea())); |
- release_mailbox3_ = base::Bind(&MockMailboxCallback::Release2, |
- base::Unretained(&mock_callback_), |
- shared_memory_.get()); |
- release_mailbox3_impl_ = base::Bind(&MockMailboxCallback::ReleaseImpl2, |
- base::Unretained(&mock_callback_), |
- shared_memory_.get()); |
- mailbox3_ = TextureMailbox(shared_memory_.get(), size); |
+ shared_bitmap_ = manager->AllocateSharedBitmap(size); |
+ DCHECK(shared_bitmap_); |
+ release_mailbox3_ = |
+ base::Bind(&MockMailboxCallback::Release2, |
+ base::Unretained(&mock_callback_), shared_bitmap_.get()); |
+ release_mailbox3_impl_ = |
+ base::Bind(&MockMailboxCallback::ReleaseImpl2, |
+ base::Unretained(&mock_callback_), shared_bitmap_.get()); |
+ mailbox3_ = TextureMailbox(shared_bitmap_.get(), size); |
} |
gpu::Mailbox mailbox_name1_; |
@@ -164,7 +164,7 @@ struct CommonMailboxObjects { |
TextureMailbox mailbox3_; |
uint32 sync_point1_; |
uint32 sync_point2_; |
- scoped_ptr<base::SharedMemory> shared_memory_; |
+ scoped_ptr<SharedBitmap> shared_bitmap_; |
}; |
class TextureLayerTest : public testing::Test { |
@@ -172,7 +172,8 @@ class TextureLayerTest : public testing::Test { |
TextureLayerTest() |
: fake_client_( |
FakeLayerTreeHostClient(FakeLayerTreeHostClient::DIRECT_3D)), |
- host_impl_(&proxy_, &shared_bitmap_manager_) {} |
+ host_impl_(&proxy_, &shared_bitmap_manager_), |
+ test_data_(&shared_bitmap_manager_) {} |
protected: |
virtual void SetUp() { |
@@ -195,6 +196,7 @@ class TextureLayerTest : public testing::Test { |
FakeLayerTreeHostClient fake_client_; |
TestSharedBitmapManager shared_bitmap_manager_; |
FakeLayerTreeHostImpl host_impl_; |
+ CommonMailboxObjects test_data_; |
}; |
TEST_F(TextureLayerTest, CheckPropertyChangeCausesCorrectBehavior) { |
@@ -309,8 +311,6 @@ class TextureLayerWithMailboxTest : public TextureLayerTest { |
false)).Times(1); |
TextureLayerTest::TearDown(); |
} |
- |
- CommonMailboxObjects test_data_; |
}; |
TEST_F(TextureLayerWithMailboxTest, ReplaceMailboxOnMainThreadBeforeCommit) { |
@@ -359,9 +359,7 @@ TEST_F(TextureLayerWithMailboxTest, ReplaceMailboxOnMainThreadBeforeCommit) { |
EXPECT_CALL(*layer_tree_host_, SetNeedsCommit()).Times(AtLeast(1)); |
EXPECT_CALL(test_data_.mock_callback_, |
- Release2(test_data_.shared_memory_.get(), |
- 0, false)) |
- .Times(1); |
+ Release2(test_data_.shared_bitmap_.get(), 0, false)).Times(1); |
test_layer->SetTextureMailbox(TextureMailbox(), nullptr); |
Mock::VerifyAndClearExpectations(layer_tree_host_.get()); |
Mock::VerifyAndClearExpectations(&test_data_.mock_callback_); |
@@ -452,7 +450,6 @@ class TextureLayerMailboxHolderTest : public TextureLayerTest { |
main_ref_; |
base::Thread main_thread_; |
scoped_ptr<BlockingTaskRunner> main_thread_task_runner_; |
- CommonMailboxObjects test_data_; |
}; |
TEST_F(TextureLayerMailboxHolderTest, TwoCompositors_BothReleaseThenMain) { |
@@ -946,7 +943,6 @@ class TextureLayerImplWithMailboxTest : public TextureLayerTest { |
return will_draw; |
} |
- CommonMailboxObjects test_data_; |
FakeLayerTreeHostClient fake_client_; |
}; |
@@ -958,7 +954,7 @@ TEST_F(TextureLayerImplWithMailboxTest, TestWillDraw) { |
ReleaseImpl(test_data_.mailbox_name1_, test_data_.sync_point1_, false, _)) |
.Times(AnyNumber()); |
EXPECT_CALL(test_data_.mock_callback_, |
- ReleaseImpl2(test_data_.shared_memory_.get(), 0, false, _)) |
+ ReleaseImpl2(test_data_.shared_bitmap_.get(), 0, false, _)) |
.Times(AnyNumber()); |
// Hardware mode. |
{ |