diff options
| author | gman@chromium.org <gman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-18 02:40:03 +0000 |
|---|---|---|
| committer | gman@chromium.org <gman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-04-18 02:40:03 +0000 |
| commit | 5bd6feee0b7185ee2b07f245f7a89a74f8f2ef94 (patch) | |
| tree | 5d7f3c4740b71581314ab00493350a9afb9b74f4 /gpu | |
| parent | ef405f66b8eeb3777f9306c97086d002390de7bc (diff) | |
| download | chromium_src-5bd6feee0b7185ee2b07f245f7a89a74f8f2ef94.zip chromium_src-5bd6feee0b7185ee2b07f245f7a89a74f8f2ef94.tar.gz chromium_src-5bd6feee0b7185ee2b07f245f7a89a74f8f2ef94.tar.bz2 | |
Add a few more DCHECKs to the GPU code.
Now taht these managers require all the stuff they are
managing to be freed before being freed themselves
I think these checks are valid and will hopefully
prevent regressions.
TEST=none
BUG=none
R=apatrick@chromium.org
Review URL: http://codereview.chromium.org/10105024
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@132728 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'gpu')
| -rw-r--r-- | gpu/command_buffer/service/renderbuffer_manager.cc | 2 | ||||
| -rw-r--r-- | gpu/command_buffer/service/texture_manager.cc | 4 |
2 files changed, 6 insertions, 0 deletions
diff --git a/gpu/command_buffer/service/renderbuffer_manager.cc b/gpu/command_buffer/service/renderbuffer_manager.cc index 06d732f..8a37ec6 100644 --- a/gpu/command_buffer/service/renderbuffer_manager.cc +++ b/gpu/command_buffer/service/renderbuffer_manager.cc @@ -27,6 +27,8 @@ RenderbufferManager::~RenderbufferManager() { // If this triggers, that means something is keeping a reference to // a RenderbufferInfo belonging to this. CHECK_EQ(renderbuffer_info_count_, 0u); + + DCHECK_EQ(0, num_uncleared_renderbuffers_); } size_t RenderbufferManager::RenderbufferInfo::EstimatedSize() { diff --git a/gpu/command_buffer/service/texture_manager.cc b/gpu/command_buffer/service/texture_manager.cc index 055c2ab..241fb3e 100644 --- a/gpu/command_buffer/service/texture_manager.cc +++ b/gpu/command_buffer/service/texture_manager.cc @@ -62,6 +62,10 @@ TextureManager::~TextureManager() { // If this triggers, that means something is keeping a reference to // a TextureInfo belonging to this. CHECK_EQ(texture_info_count_, 0u); + + DCHECK_EQ(0, num_unrenderable_textures_); + DCHECK_EQ(0, num_unsafe_textures_); + DCHECK_EQ(0, num_uncleared_mips_); } void TextureManager::Destroy(bool have_context) { |