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

Unified Diff: trunk/src/gpu/command_buffer/client/buffer_tracker_unittest.cc

Issue 216673006: Revert 260177 "By keeping track of transfer buffer usage (both s..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 9 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
Index: trunk/src/gpu/command_buffer/client/buffer_tracker_unittest.cc
===================================================================
--- trunk/src/gpu/command_buffer/client/buffer_tracker_unittest.cc (revision 260192)
+++ trunk/src/gpu/command_buffer/client/buffer_tracker_unittest.cc (working copy)
@@ -42,11 +42,6 @@
bool context_lost_;
};
-namespace {
-void EmptyPoll() {
-}
-}
-
class BufferTrackerTest : public testing::Test {
protected:
static const int32 kNumCommandEntries = 400;
@@ -58,7 +53,7 @@
helper_.reset(new GLES2CmdHelper(command_buffer_.get()));
helper_->Initialize(kCommandBufferSizeBytes);
mapped_memory_.reset(new MappedMemoryManager(
- helper_.get(), base::Bind(&EmptyPoll), MappedMemoryManager::kNoLimit));
+ helper_.get(), MappedMemoryManager::kNoLimit));
buffer_tracker_.reset(new BufferTracker(mapped_memory_.get()));
}
@@ -132,22 +127,5 @@
buffer_tracker_->RemoveBuffer(kId);
}
-TEST_F(BufferTrackerTest, Unmanage) {
- const GLuint kId = 123;
- const GLsizeiptr size = 64;
-
- BufferTracker::Buffer* buffer = buffer_tracker_->CreateBuffer(kId, size);
- ASSERT_TRUE(buffer != NULL);
- EXPECT_EQ(mapped_memory_->bytes_in_use(), static_cast<size_t>(size));
-
- void* mem = buffer->address();
- buffer_tracker_->Unmanage(buffer);
- buffer_tracker_->RemoveBuffer(kId);
- EXPECT_EQ(mapped_memory_->bytes_in_use(), static_cast<size_t>(size));
-
- mapped_memory_->Free(mem);
- EXPECT_EQ(mapped_memory_->bytes_in_use(), static_cast<size_t>(0));
-}
-
} // namespace gles2
} // namespace gpu
« no previous file with comments | « trunk/src/gpu/command_buffer/client/buffer_tracker.cc ('k') | trunk/src/gpu/command_buffer/client/cmd_buffer_helper.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698