| Index: gpu/command_buffer/client/fenced_allocator_test.cc
|
| diff --git a/gpu/command_buffer/client/fenced_allocator_test.cc b/gpu/command_buffer/client/fenced_allocator_test.cc
|
| index d5b3437fd5ac6f167bd2311066033056a78b72f4..94faf0ca917eb8cb7012a71570a1160f501aa6f5 100644
|
| --- a/gpu/command_buffer/client/fenced_allocator_test.cc
|
| +++ b/gpu/command_buffer/client/fenced_allocator_test.cc
|
| @@ -80,11 +80,6 @@ class BaseFencedAllocatorTest : public testing::Test {
|
| const unsigned int BaseFencedAllocatorTest::kBufferSize;
|
| #endif
|
|
|
| -namespace {
|
| -void EmptyPoll() {
|
| -}
|
| -}
|
| -
|
| // Test fixture for FencedAllocator test - Creates a FencedAllocator, using a
|
| // CommandBufferHelper with a mock AsyncAPIInterface for its interface (calling
|
| // it directly, not through the RPC mechanism), making sure Noops are ignored
|
| @@ -93,9 +88,7 @@ class FencedAllocatorTest : public BaseFencedAllocatorTest {
|
| protected:
|
| void SetUp() override {
|
| BaseFencedAllocatorTest::SetUp();
|
| - allocator_.reset(new FencedAllocator(kBufferSize,
|
| - helper_.get(),
|
| - base::Bind(&EmptyPoll)));
|
| + allocator_.reset(new FencedAllocator(kBufferSize, helper_.get()));
|
| }
|
|
|
| void TearDown() override {
|
| @@ -390,63 +383,6 @@ TEST_F(FencedAllocatorTest, TestGetLargestFreeOrPendingSize) {
|
| EXPECT_EQ(kBufferSize, allocator_->GetLargestFreeSize());
|
| }
|
|
|
| -class FencedAllocatorPollTest : public BaseFencedAllocatorTest {
|
| - public:
|
| - static const unsigned int kAllocSize = 128;
|
| -
|
| - MOCK_METHOD0(MockedPoll, void());
|
| -
|
| - protected:
|
| - virtual void TearDown() {
|
| - // If the GpuScheduler posts any tasks, this forces them to run.
|
| - base::MessageLoop::current()->RunUntilIdle();
|
| -
|
| - BaseFencedAllocatorTest::TearDown();
|
| - }
|
| -};
|
| -
|
| -TEST_F(FencedAllocatorPollTest, TestPoll) {
|
| - scoped_ptr<FencedAllocator> allocator(
|
| - new FencedAllocator(kBufferSize,
|
| - helper_.get(),
|
| - base::Bind(&FencedAllocatorPollTest::MockedPoll,
|
| - base::Unretained(this))));
|
| -
|
| - FencedAllocator::Offset mem1 = allocator->Alloc(kAllocSize);
|
| - FencedAllocator::Offset mem2 = allocator->Alloc(kAllocSize);
|
| - EXPECT_NE(mem1, FencedAllocator::kInvalidOffset);
|
| - EXPECT_NE(mem2, FencedAllocator::kInvalidOffset);
|
| - EXPECT_TRUE(allocator->CheckConsistency());
|
| - EXPECT_EQ(allocator->bytes_in_use(), kAllocSize * 2);
|
| -
|
| - // Check that no-op Poll doesn't affect the state.
|
| - EXPECT_CALL(*this, MockedPoll()).RetiresOnSaturation();
|
| - allocator->FreeUnused();
|
| - EXPECT_TRUE(allocator->CheckConsistency());
|
| - EXPECT_EQ(allocator->bytes_in_use(), kAllocSize * 2);
|
| -
|
| - // Check that freeing in Poll works.
|
| - base::Closure free_mem1_closure =
|
| - base::Bind(&FencedAllocator::Free,
|
| - base::Unretained(allocator.get()),
|
| - mem1);
|
| - EXPECT_CALL(*this, MockedPoll())
|
| - .WillOnce(InvokeWithoutArgs(&free_mem1_closure, &base::Closure::Run))
|
| - .RetiresOnSaturation();
|
| - allocator->FreeUnused();
|
| - EXPECT_TRUE(allocator->CheckConsistency());
|
| - EXPECT_EQ(allocator->bytes_in_use(), kAllocSize * 1);
|
| -
|
| - // Check that freeing still works.
|
| - EXPECT_CALL(*this, MockedPoll()).RetiresOnSaturation();
|
| - allocator->Free(mem2);
|
| - allocator->FreeUnused();
|
| - EXPECT_TRUE(allocator->CheckConsistency());
|
| - EXPECT_EQ(allocator->bytes_in_use(), 0u);
|
| -
|
| - allocator.reset();
|
| -}
|
| -
|
| // Test fixture for FencedAllocatorWrapper test - Creates a
|
| // FencedAllocatorWrapper, using a CommandBufferHelper with a mock
|
| // AsyncAPIInterface for its interface (calling it directly, not through the
|
| @@ -464,7 +400,6 @@ class FencedAllocatorWrapperTest : public BaseFencedAllocatorTest {
|
| kBufferSize, kAllocAlignment)));
|
| allocator_.reset(new FencedAllocatorWrapper(kBufferSize,
|
| helper_.get(),
|
| - base::Bind(&EmptyPoll),
|
| buffer_.get()));
|
| }
|
|
|
|
|