cc: Use CheckedNumeric for resource size calculations.

cc/trees/layer_tree_host.cc

 // Copyright 2011 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/trees/layer_tree_host.h"
 
 #include <algorithm>
 #include <stack>
 #include <string>
 
@@ skipping 915 matching lines @@
     const RenderSurfaceLayerList& update_list) {
   size_t readback_bytes = 0;
   size_t contents_texture_bytes = 0;
 
   // Start iteration at 1 to skip the root surface as it does not have a texture
   // cost.
   for (size_t i = 1; i < update_list.size(); ++i) {
     Layer* render_surface_layer = update_list.at(i);
     RenderSurface* render_surface = render_surface_layer->render_surface();
 
-    size_t bytes =
-        Resource::MemorySizeBytes(render_surface->content_rect().size(),
-                                  RGBA_8888);
+    CHECK(Resource::VerifySizeInBytes(render_surface->content_rect().size(),

[vmpstr, 2015/06/04 20:52:59]

I wasn't sure about this function, so I think it's better to CHECK than not? Let
me know.

[danakj, 2015/06/04 20:58:44]

RenderSurface content rects are bounded by the max texture size which is small.

+                                      RGBA_8888));
+    size_t bytes = Resource::UncheckedMemorySizeBytes(
+        render_surface->content_rect().size(), RGBA_8888);
     contents_texture_bytes += bytes;
 
     if (render_surface_layer->background_filters().IsEmpty() &&
         render_surface_layer->uses_default_blend_mode())
       continue;
 
     if (!readback_bytes) {
-      readback_bytes = Resource::MemorySizeBytes(device_viewport_size_,
-                                                 RGBA_8888);
+      CHECK(Resource::VerifySizeInBytes(device_viewport_size_, RGBA_8888));

[danakj, 2015/06/04 20:58:44]

This is correct to check about I think it comes from outside cc and is somewhat
opaque to us. How about adding a Resource::CheckedMemorySizeBytes that does
ValueOrDie()?

+      readback_bytes =
+          Resource::UncheckedMemorySizeBytes(device_viewport_size_, RGBA_8888);
     }
   }
   return readback_bytes + contents_texture_bytes;
 }
 
 void LayerTreeHost::PaintMasksForRenderSurface(Layer* render_surface_layer,
                                                ResourceUpdateQueue* queue,
                                                bool* did_paint_content,
                                                bool* need_more_updates) {
   // Note: Masks and replicas only exist for layers that own render surfaces. If
@@ skipping 338 matching lines @@
     const BeginFrameArgs& args) const {
   client_->SendBeginFramesToChildren(args);
 }
 
 void LayerTreeHost::SetAuthoritativeVSyncInterval(
     const base::TimeDelta& interval) {
   proxy_->SetAuthoritativeVSyncInterval(interval);
 }
 
 }  // namespace cc