Re-enable extension GPU features

chrome/browser/extensions/extension_browsertests_misc.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 "base/file_util.h"
 #include "base/memory/ref_counted.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/browser/extensions/autoupdate_interceptor.h"
 #include "chrome/browser/extensions/extension_apitest.h"
 #include "chrome/browser/extensions/extension_browsertest.h"
@@ skipping 111 matching lines @@
   ASSERT_TRUE(LoadExtension(
       test_data_dir_.AppendASCII("good").AppendASCII("Extensions")
                     .AppendASCII("behllobkkfkfnphdnhnkndlbkcpglgmj")
                     .AppendASCII("1.0.0.0")));
 
   ExtensionProcessManager* manager =
         browser()->profile()->GetExtensionProcessManager();
   ExtensionHost* host = FindHostWithPath(manager, "/backgroundpage.html", 1);
   WebPreferences prefs =
       host->render_view_host()->GetDelegate()->GetWebkitPrefs();
-  ASSERT_FALSE(prefs.experimental_webgl_enabled);
-  ASSERT_FALSE(prefs.accelerated_compositing_enabled);
-  ASSERT_FALSE(prefs.accelerated_2d_canvas_enabled);
+  ASSERT_TRUE(prefs.experimental_webgl_enabled);
+  ASSERT_TRUE(prefs.accelerated_compositing_enabled);
+  ASSERT_TRUE(prefs.accelerated_2d_canvas_enabled);
 }
 
 IN_PROC_BROWSER_TEST_F(ExtensionBrowserTest, PageActionCrash25562) {
   ASSERT_TRUE(test_server()->Start());
 
   CommandLine::ForCurrentProcess()->AppendSwitch(
       switches::kAllowLegacyExtensionManifests);
 
   // This page action will not show an icon, since it doesn't specify one but
   // is included here to test for a crash (http://crbug.com/25562).
@@ skipping 545 matching lines @@
   ExtensionProcessManager* manager =
       browser()->profile()->GetExtensionProcessManager();
   ExtensionHost* host = FindHostWithPath(manager, "/bg.html", 1);
 
   bool result = false;
   ASSERT_TRUE(ui_test_utils::ExecuteJavaScriptAndExtractBool(
       host->render_view_host(), L"", L"testLastError()", &result));
   EXPECT_TRUE(result);
 }
 
+// Tests to make sure we can use Canvas2D from a background page
+IN_PROC_BROWSER_TEST_F(ExtensionBrowserTest, Canvas2D) {

[Aaron Boodman, 2012/03/09 01:00:06]

Did you see the note at the bottom of the file (don't blame you if you didn't,
the file is huge)? You should put this test in
chrome/browser/extensions/background_page_apitest.cc. See
background_scripts_apitest.cc for an example.

[Josh Horwich, 2012/03/09 01:21:56]

Didn't see the note.

Do you mean app_background_page_apitest.cc or do you mean creating a new file
(background_page_apitest.cc) for this one test?

Either way I can switch it over to using the techniques used in that example,
thank you for the pointer.  

[Josh Horwich, 2012/03/09 02:26:25]

Done.

+  ASSERT_TRUE(LoadExtension(
+      test_data_dir_.AppendASCII("canvas")));
+
+  // Get the ExtensionHost that is hosting our background page.
+  ExtensionProcessManager* manager =
+      browser()->profile()->GetExtensionProcessManager();
+  ExtensionHost* host = FindHostWithPath(manager, "/background.html", 1);
+
+  bool result = false;
+  ASSERT_TRUE(ui_test_utils::ExecuteJavaScriptAndExtractBool(
+      host->render_view_host(), L"", L"test()", &result));
+  EXPECT_TRUE(result);
+}
+
 // Helper function for common code shared by the 3 WindowOpen tests below.
 static void WindowOpenHelper(Browser* browser, const GURL& start_url,
                              const std::string& newtab_url,
                              WebContents** newtab_result) {
   ui_test_utils::NavigateToURL(browser, start_url);
 
   ui_test_utils::WindowedNotificationObserver observer(
       content::NOTIFICATION_LOAD_STOP,
       content::NotificationService::AllSources());
   ASSERT_TRUE(ui_test_utils::ExecuteJavaScript(
@@ skipping 214 matching lines @@
 
   EXPECT_EQ(extension->GetResourceURL("options.html"),
             tab_strip->GetTabContentsAt(1)->web_contents()->GetURL());
 }
 
 //==============================================================================
 // STOP! Please do not add any more random-ass tests here. Create new files for
 // your tests grouped by functionality. Also, you should strongly consider using
 // ExtensionAPITest if possible.
 //==============================================================================