cc: Use CheckedNumeric for resource size calculations.

cc/resources/resource.h

 // 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.
 
 #ifndef CC_RESOURCES_RESOURCE_H_
 #define CC_RESOURCES_RESOURCE_H_
 
+#include "base/numerics/safe_math.h"
 #include "cc/base/cc_export.h"
 #include "cc/resources/resource_provider.h"
 #include "ui/gfx/geometry/size.h"
 
 namespace cc {
 
 class CC_EXPORT Resource {
  public:
   Resource() : id_(0) {}
   Resource(unsigned id, const gfx::Size& size, ResourceFormat format)
       : id_(id),
         size_(size),
         format_(format) {}
 
   ResourceId id() const { return id_; }
   gfx::Size size() const { return size_; }
   ResourceFormat format() const { return format_; }
   size_t bytes() const;

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

we should remove this function.

 
-  inline static size_t MemorySizeBytes(const gfx::Size& size,
-                                       ResourceFormat format) {
-    DCHECK_EQ(0, (BitsPerPixel(format) * size.width() * size.height()) % 8);
-    // TODO(vmpstr): Make this function overflow safe. crbug.com/495867
-    return static_cast<size_t>(
-        (BitsPerPixel(format) * size.width() * size.height()) / 8);
+  // Return true if the call to UncheckedMemorySizeBytes would return a value
+  // that fits in a size_t.
+  static bool VerifySizeInBytes(const gfx::Size& size, ResourceFormat format) {
+    if (((BitsPerPixel(format) * size.width() * size.height()) % 8) != 0)

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

can you do this only if result.IsValid() and convert these to size_t's instead
of multiplying ints?

+      return false;
+    base::CheckedNumeric<size_t> result = BitsPerPixel(format);
+    result *= size.width();
+    result *= size.height();
+    result /= 8;
+    return result.IsValid();
+  }
+
+  inline static size_t UncheckedMemorySizeBytes(const gfx::Size& size,
+                                                ResourceFormat format) {
+    DCHECK(VerifySizeInBytes(size, format));
+    return static_cast<size_t>(BitsPerPixel(format)) * size.width() *
+           size.height() / 8;
   }
 
  protected:
   void set_id(ResourceId id) { id_ = id; }
   void set_dimensions(const gfx::Size& size, ResourceFormat format) {
     size_ = size;
     format_ = format;
   }
 
  private:
   ResourceId id_;
   gfx::Size size_;
   ResourceFormat format_;
 
   DISALLOW_COPY_AND_ASSIGN(Resource);
 };
 
 }  // namespace cc
 
 #endif  // CC_RESOURCES_RESOURCE_H_