Some extra testing for GpuDataManager.

content/browser/gpu/gpu_data_manager_impl_unittest.cc

Index: content/browser/gpu/gpu_data_manager_impl_unittest.cc
===================================================================
--- content/browser/gpu/gpu_data_manager_impl_unittest.cc	(revision 157852)
+++ content/browser/gpu/gpu_data_manager_impl_unittest.cc	(working copy)
@@ -166,11 +166,32 @@
 
   manager->BlacklistCard();
   EXPECT_FALSE(manager->GpuAccessAllowed());
+  EXPECT_EQ(content::GPU_FEATURE_TYPE_ALL,
+            manager->GetBlacklistedFeatures());
 
+  delete manager;
+}
+
+TEST_F(GpuDataManagerImplTest, SoftwareRendering) {
+  GpuDataManagerImpl* manager = new GpuDataManagerImpl();
+  ASSERT_TRUE(manager);
+  EXPECT_EQ(0, manager->GetBlacklistedFeatures());
+  EXPECT_TRUE(manager->GpuAccessAllowed());
+
+  EXPECT_FALSE(manager->ShouldUseSoftwareRendering());
+
+  manager->BlacklistCard();

[jbauman, 2012/09/21 21:02:38]

It might also be nice to add a test (in a separate patch) where we register the
swiftshader path first, then blacklist the card, and ensure that also works.

+  EXPECT_FALSE(manager->GpuAccessAllowed());
+  EXPECT_FALSE(manager->ShouldUseSoftwareRendering());
+
   // If software rendering is enabled, even if we blacklist GPU,
   // GPU process is still allowed.
-  manager->software_rendering_ = true;
+  const FilePath test_path(FILE_PATH_LITERAL("AnyPath"));
+  manager->RegisterSwiftShaderPath(test_path);
+  EXPECT_TRUE(manager->ShouldUseSoftwareRendering());
   EXPECT_TRUE(manager->GpuAccessAllowed());
+  EXPECT_EQ(content::GPU_FEATURE_TYPE_ACCELERATED_2D_CANVAS,
+            manager->GetBlacklistedFeatures());
 
   delete manager;
 }