UI Resource Manager

cc/resources/ui_resource_manager.cc

+// Copyright 2013 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/ui_resource_manager.h"
+
+namespace cc {
+
+scoped_refptr<UIResourceBitmap>
+UIResourceBitmap::Create(void* pixels,
+                         UIResourceFormat format,
+                         gfx::Size size) {
+  scoped_refptr<UIResourceBitmap> ret = new UIResourceBitmap();
+  ret->pixels_ = pixels;
+  ret->format_ = format;
+  ret->size_ = size;
+
+  return ret;
+}
+
+UIResourceBitmap::~UIResourceBitmap() {
+  // this class has ownership of the pixels
+  if (format_ == RGBA8)
+    delete [] reinterpret_cast<uint8_t*>(pixels_);

[enne (OOO), 2013/07/09 17:57:48]

Just make this a scoped_array<uint8_t>.

[powei, 2013/07/10 17:47:21]

Can you elaborate on this?  Are we assume that pixels is always going to be
uint8_t?  Here I only assume that RGBA8 bitmaps are stored as uint8_t.  

Also, scoped_array is not found in base/memory/scoped_ptr.h, but it is found in
googleurl/base/scoped_ptr.h, and that header contains an identical definition
for scoped_ptr, which conflicts with base/memory/ref_counted.h.  Would you point
me to how this is usually resolved?

[tfarina, 2013/07/10 23:30:45]

this is spelled today as scoped_ptr<uint8_t[]>, see dcheng's patches.

+  // handle other formats?

[enne (OOO), 2013/07/09 17:57:48]

You're probably going to need to handle BGRA or swizzle this elsewhere.

+}
+
+
+}  // namespace cc
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+
+