Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(86)

Unified Diff: chrome/browser/policy/policy_browsertest.cc

Issue 1217333002: Add a group policy for hardware acceleration. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/policy/policy_browsertest.cc
diff --git a/chrome/browser/policy/policy_browsertest.cc b/chrome/browser/policy/policy_browsertest.cc
index 8c20f2c763740abd8bdeb524b3c83507b1df758c..940b1c683b212d077a66be234bff35e482b7094f 100644
--- a/chrome/browser/policy/policy_browsertest.cc
+++ b/chrome/browser/policy/policy_browsertest.cc
@@ -3838,4 +3838,31 @@ IN_PROC_BROWSER_TEST_F(PolicyTest, NativeMessagingWhitelist) {
#endif // !defined(CHROME_OS)
+
+#if !defined(OS_CHROMEOS) && !defined(OS_ANDROID)
+// Sets the hardware acceleration mode policy before the browser is started.
+class HardwareAccelerationModePolicyTest : public PolicyTest {
+ public:
+ HardwareAccelerationModePolicyTest() {}
+
+ 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(HardwareAccelerationModePolicyTest,
Andrew T Wilson (Slow) 2015/07/16 09:59:55 What about the true case? Should we test that hard
Zhenyao Mo 2015/07/16 18:06:31 In the first CL I did test that, but unfortunately
+ HardwareAccelerationDisabled) {
+ // Verifies that hardware acceleration can be disabled with policy.
+ EXPECT_FALSE(
+ content::GpuDataManager::GetInstance()->GpuAccessAllowed(nullptr));
+}
+#endif // !defined(OS_CHROMEOS) && !defined(OS_ANDROID)
+
} // namespace policy

Powered by Google App Engine
This is Rietveld 408576698