cc: Add ResourceId validation checks at the time of AppendQuads.

cc/resources/resource_provider.cc

 // Copyright 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "cc/resources/resource_provider.h"
 
 #include <algorithm>
 #include <limits>
 
 #include "base/containers/hash_tables.h"
@@ skipping 2116 matching lines @@
 
   resource->read_lock_fence->Wait();
 }
 
 GLint ResourceProvider::GetActiveTextureUnit(GLES2Interface* gl) {
   GLint active_unit = 0;
   gl->GetIntegerv(GL_ACTIVE_TEXTURE, &active_unit);
   return active_unit;
 }
 
+void ResourceProvider::ValidateResource(ResourceId id) {
+  DCHECK(thread_checker_.CalledOnValidThread());
+  CHECK(id);
+  ResourceMap::iterator it = resources_.find(id);
+  CHECK(it != resources_.end());

[piman, 2015/04/07 00:13:39]

nit: all that is essentially (void)GetResource(id);

[danakj, 2015/04/07 00:15:00]

Ya I copied it from there. Heh. My random thought was that I would go back and
remove the CHECK(id) in GetResource, but maybe we'd wanna keep this function
around behind DCHECK_IS_ON() and it'd be nice to keep here. I will add a TODO on
the CHECK in GetResource.

+}
+
 GLES2Interface* ResourceProvider::ContextGL() const {
   ContextProvider* context_provider = output_surface_->context_provider();
   return context_provider ? context_provider->ContextGL() : NULL;
 }
 
 class GrContext* ResourceProvider::GrContext(bool worker_context) const {
   ContextProvider* context_provider =
       worker_context ? output_surface_->worker_context_provider()
                      : output_surface_->context_provider();
   return context_provider ? context_provider->GrContext() : NULL;
 }
 
 }  // namespace cc