cc: Fix BytesPerPixel issue and refactor.

cc/resources/resource_util.h

+// Copyright 2015 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_UTIL_H_
+#define CC_RESOURCES_RESOURCE_UTIL_H_
+
+#include "cc/base/cc_export.h"
+#include "cc/resources/resource_format.h"
+#include "ui/gfx/geometry/size.h"
+
+namespace cc {
+
+class CC_EXPORT ResourceUtil {
+ public:
+  // Return true if the call to UncheckedMemorySizeBytes would return a value
+  // that fits in a size_t.

[prashant.n, 2015/06/25 15:23:04]

These are just the functions moved out resource_provider.h + one new function
added for aligned data.

When grouped all these functions together, IMO we should have some different
function names than these.

reveman@, vmpstr@, wdyt?

I also think of adding one more function for BytesPerRow? Should we add it?

+  static bool VerifySizeInBytes(const gfx::Size& size, ResourceFormat format);
+  static size_t CheckedMemorySizeBytes(const gfx::Size& size,
+                                       ResourceFormat format);
+  static size_t UncheckedMemorySizeBytes(const gfx::Size& size,
+                                         ResourceFormat format);
+  static size_t UncheckedMemorySizeBytesAligned(const gfx::Size& size,
+                                                ResourceFormat format);
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(ResourceUtil);
+};
+
+}  // namespace cc
+
+#endif  // CC_RESOURCES_RESOURCE_UTIL_H_