UI Resource Manager

cc/trees/layer_tree_host_impl_unittest.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_impl.h"
 
 #include <cmath>
 
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 6223 matching lines @@
       0, ManagedMemoryPolicy::CUTOFF_ALLOW_NOTHING);
   host_impl_->SetMemoryPolicy(policy2, false);
   EXPECT_EQ(actual_policy, host_impl_->ActualManagedMemoryPolicy());
   EXPECT_EQ(policy1.bytes_limit_when_visible, current_limit_bytes_);
   EXPECT_EQ(visible_cutoff_value, current_priority_cutoff_value_);
   host_impl_->SetVisible(false);
   EXPECT_EQ(policy1.bytes_limit_when_not_visible, current_limit_bytes_);
   EXPECT_EQ(not_visible_cutoff_value, current_priority_cutoff_value_);
 }
 
+TEST_F(LayerTreeHostImplTest, UIResourceManagement) {

[aelias_OOO_until_Jul13, 2013/07/31 21:00:30]

Add a test case for two creations with the same UIResourceId.

[powei, 2013/07/31 21:29:39]

Done.

+  scoped_ptr<TestWebGraphicsContext3D> context =
+      TestWebGraphicsContext3D::Create();
+  TestWebGraphicsContext3D* context3d = context.get();
+  scoped_ptr<OutputSurface> output_surface = FakeOutputSurface::Create3d(
+      context.PassAs<WebKit::WebGraphicsContext3D>()).PassAs<OutputSurface>();
+  host_impl_->InitializeRenderer(output_surface.Pass());
+
+  EXPECT_EQ(0u, context3d->NumTextures());
+
+  UIResourceId ui_resource_id = 1;
+  scoped_refptr<UIResourceBitmap> bitmap =
+      UIResourceBitmap::Create(new uint8_t[1],
+                               UIResourceBitmap::RGBA8,
+                               gfx::Size(1, 1));
+  host_impl_->CreateUIResource(ui_resource_id, bitmap);
+  EXPECT_EQ(1u, context3d->NumTextures());
+  ResourceProvider::ResourceId id =
+      host_impl_->ResourceIdForUIResource(ui_resource_id);
+  EXPECT_NE(0u, id);
+  EXPECT_EQ(ResourceProvider::GLTexture,
+            host_impl_->resource_provider()->GetResourceType(id));
+
+  // Should return zero for invalid UIResourceId.  Number of textures should
+  // not change.
+  EXPECT_EQ(0u, host_impl_->ResourceIdForUIResource(-1));
+  EXPECT_EQ(1u, context3d->NumTextures());
+
+  host_impl_->DeleteUIResource(ui_resource_id);

[enne (OOO), 2013/07/31 22:02:40]

Could you delete an invalid resource id here too?

[powei, 2013/08/01 00:05:10]

Done.

+  EXPECT_EQ(0u, host_impl_->ResourceIdForUIResource(ui_resource_id));
+  EXPECT_EQ(0u, context3d->NumTextures());
+
+  // Should not change state for multiple deletion on one UIResourceId
+  host_impl_->DeleteUIResource(ui_resource_id);
+  EXPECT_EQ(0u, context3d->NumTextures());
+}
+
 }  // namespace
 }  // namespace cc