cc: Some more cleanup and removing TODOs for validating resources.

cc/resources/resource_provider.cc

Index: cc/resources/resource_provider.cc
diff --git a/cc/resources/resource_provider.cc b/cc/resources/resource_provider.cc
index a336d759463e8c57302019fb08970a7e8bedd0ef..97d0ca0636a732f0c75ac88962ddc9d3721d8967 100644
--- a/cc/resources/resource_provider.cc
+++ b/cc/resources/resource_provider.cc
@@ -863,8 +863,7 @@ ResourceProvider::Resource* ResourceProvider::InsertResource(
 
 ResourceProvider::Resource* ResourceProvider::GetResource(ResourceId id) {
   DCHECK(thread_checker_.CalledOnValidThread());
-  // TODO(danakj): crbug.com/455931
-  CHECK(id);
+  DCHECK(id);
   ResourceMap::iterator it = resources_.find(id);
   CHECK(it != resources_.end());

[piman, 2015/04/23 22:55:10]

I wonder if this still triggers now. If not we could probably DCHECK instead.

[danakj, 2015/04/23 22:57:12]

We'll just crash later on *it->second, so ya we could make this DCHECK I guess.

   return &it->second;
@@ -2044,7 +2043,7 @@ GLint ResourceProvider::GetActiveTextureUnit(GLES2Interface* gl) {
   return active_unit;
 }
 
-void ResourceProvider::ValidateResource(ResourceId id) {
+void ResourceProvider::ValidateResource(ResourceId id) const {
   DCHECK(thread_checker_.CalledOnValidThread());
   DCHECK(id);
   DCHECK(resources_.find(id) != resources_.end());