ui/android: Fix Resource meta-data sharing with ResourceManager.

ui/android/resources/resource.cc

Index: ui/android/resources/resource.cc
diff --git a/ui/android/resources/resource.cc b/ui/android/resources/resource.cc
new file mode 100644
index 0000000000000000000000000000000000000000..91fb44f14a45a22bbad569cfbe6d15f04e19bc06
--- /dev/null
+++ b/ui/android/resources/resource.cc
@@ -0,0 +1,32 @@
+// Copyright 2017 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 "ui/android/resources/resource.h"
+
+#include "base/memory/ptr_util.h"
+#include "base/trace_event/memory_usage_estimator.h"
+
+namespace ui {
+
+Resource::Resource() : Resource(Type::BITMAP) {}
+
+Resource::Resource(Type type) : type_(type) {}

[mdjones, 2017/03/10 18:12:00]

Shouldn't a resource class know what type it is?

[Khushal, 2017/03/10 19:19:24]

Is this about the UNKNOWN case? The type is stored on Resource so a user can
safely downcast it.

[mdjones, 2017/03/13 16:47:06]

I meant that each class should have it's resource type hard-coded. An argument
can be made either way though.

[Khushal, 2017/03/13 21:03:25]

Discussed offline. We are setting the type of a resource in the sub-class.

+
+Resource::~Resource() = default;
+
+void Resource::SetUIResource(std::unique_ptr<cc::ScopedUIResource> ui_resource,
+                             const gfx::Size& size) {
+  ui_resource_ = std::move(ui_resource);
+  size_ = size;
+}
+
+std::unique_ptr<Resource> Resource::CreateForCopy() {
+  return base::MakeUnique<Resource>();
+}
+
+size_t Resource::EstimateMemoryUsage() const {
+  return base::trace_event::EstimateMemoryUsage(ui_resource_);
+}
+
+}  // namespace ui