Add a group policy for hardware acceleration.

chrome/browser/policy/policy_browsertest.cc

 // Copyright (c) 2012 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 <algorithm>
 #include <string>
 #include <vector>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 3819 matching lines @@
 
   PrefService* prefs = browser()->profile()->GetPrefs();
   EXPECT_TRUE(extensions::MessageService::IsNativeMessagingHostAllowed(
       prefs, "host.name"));
   EXPECT_FALSE(extensions::MessageService::IsNativeMessagingHostAllowed(
       prefs, "other.host.name"));
 }
 
 #endif  // !defined(CHROME_OS)
 
+// Sets the hardware acceleration mode policy before the browser is started.
+class HardwareAccelerationModePolicyTest : public PolicyTest {
+ public:
+  HardwareAccelerationModePolicyTest() {}
+  ~HardwareAccelerationModePolicyTest() override {}

[bartfab (slow), 2015/07/01 09:16:54]

Nit: No need to override the destructor here.

[Zhenyao Mo, 2015/07/01 17:56:07]

Done.

+
+  void SetUpInProcessBrowserTestFixture() override {
+    PolicyTest::SetUpInProcessBrowserTestFixture();
+    PolicyMap policies;
+    policies.Set(key::kHardwareAccelerationModeEnabled,
+                 POLICY_LEVEL_MANDATORY,
+                 POLICY_SCOPE_USER,
+                 new base::FundamentalValue(false),
+                 NULL);
+    provider_.UpdateChromePolicy(policies);
+  }
+};
+
+IN_PROC_BROWSER_TEST_F(PolicyTest, HardwareAccelerationEnabledByDefault) {
+  EXPECT_TRUE(content::GpuDataManager::GetInstance()->GpuAccessAllowed(NULL));

[bartfab (slow), 2015/07/01 09:16:54]

1: Nit: s/NULL/nullptr/
2: PolicyTest.Disable3DAPIs seems to assume that hardware acceleration may not
always be available in tests (e.g. unavailable when running under Xvfb?). Please
make sure the test actually works under Xvfb and on bots.

[Zhenyao Mo, 2015/07/01 17:56:07]

Although it seems to run OK on all try bots, but I agree this is fragile,
because we might just block the GPUs on future bots through software rendering
list.  Remove this test.

+}
+
+IN_PROC_BROWSER_TEST_F(HardwareAccelerationModePolicyTest,
+                       HardwareAccelerationDisabled) {
+  // Verifies that hardware acceleration can be disabled with policy.
+  EXPECT_FALSE(content::GpuDataManager::GetInstance()->GpuAccessAllowed(NULL));
+}
+
 }  // namespace policy