Enable gpu crash tests in content

content/browser/gpu/gpu_crash_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 "base/command_line.h"
+#include "base/memory/scoped_ptr.h"
 #include "base/path_service.h"
 #include "base/utf_string_conversions.h"
-#include "chrome/browser/ui/browser.h"
-#include "chrome/browser/ui/browser_commands.h"
-#include "chrome/common/chrome_paths.h"
-#include "chrome/common/chrome_switches.h"
-#include "chrome/common/url_constants.h"
-#include "chrome/test/base/in_process_browser_test.h"
-#include "chrome/test/base/test_launcher_utils.h"
-#include "chrome/test/base/ui_test_utils.h"
+#include "content/public/common/content_paths.h"
+#include "content/public/common/url_constants.h"
 #include "content/public/test/browser_test_utils.h"
+#include "content/shell/shell.h"
+#include "content/test/content_browser_test.h"
+#include "content/test/content_browser_test_utils.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/gl/gl_implementation.h"
 
 namespace {
 
-void SimulateGPUCrash(Browser* browser) {
-  LOG(ERROR) << "SimulateGPUCrash, before NavigateToURLWithDisposition";
-  ui_test_utils::NavigateToURLWithDisposition(browser,
-      GURL(chrome::kChromeUIGpuCrashURL), NEW_FOREGROUND_TAB,
-      ui_test_utils::BROWSER_TEST_NONE);
-  chrome::SelectPreviousTab(browser);
-  LOG(ERROR) << "SimulateGPUCrash, after CloseTab";
+// SimulateGPUCrash owns the passed 'shell', this should be

[jam, 2012/10/01 16:10:47]

this comment is confusing, what do you mean by owns it? this function doesn't
delete it...

[yongsheng, 2012/10/11 02:08:47]

this's obselete. i'll remove it.

+// other than the main one
+void SimulateGPUCrash(content::Shell* s) {
+  LOG(ERROR) << "SimulateGPUCrash, before NavigateToURL";
+  s->LoadURL(GURL(chrome::kChromeUIGpuCrashURL));
+  LOG(ERROR) << "SimulateGPUCrash, after NavigateToURL";
 }
 
 } // namespace
 
-class GPUCrashTest : public InProcessBrowserTest {
+namespace content {
+class GPUCrashTest : public ContentBrowserTest {
  protected:
   virtual void SetUpCommandLine(CommandLine* command_line) {
     // GPU tests require gpu acceleration.
     // We do not care which GL backend is used.
     command_line->AppendSwitchASCII(switches::kUseGL, "any");
   }
   virtual void SetUpInProcessBrowserTestFixture() {
     FilePath test_dir;
-    ASSERT_TRUE(PathService::Get(chrome::DIR_TEST_DATA, &test_dir));
+    ASSERT_TRUE(PathService::Get(DIR_TEST_DATA, &test_dir));
     gpu_test_dir_ = test_dir.AppendASCII("gpu");
   }
   FilePath gpu_test_dir_;
 };
 
 // Currently Kill times out on GPU bots: http://crbug.com/101513
-IN_PROC_BROWSER_TEST_F(GPUCrashTest, DISABLED_Kill) {
-  content::DOMMessageQueue message_queue;
+IN_PROC_BROWSER_TEST_F(GPUCrashTest, MANUAL_DISABLED_Kill) {
+  DOMMessageQueue message_queue;
 
-  ui_test_utils::NavigateToURL(
-      browser(),
-      content::GetFileUrlWithQuery(
+  NavigateToURL(
+      shell(),
+      GetFileUrlWithQuery(
           gpu_test_dir_.AppendASCII("webgl.html"), "query=kill"));
-  SimulateGPUCrash(browser());
+  scoped_ptr<Shell> shell(CreateBrowser());
+  SimulateGPUCrash(shell.get());

[jam, 2012/10/01 16:10:47]

the test used to kill the browser window that had the url loaded. now you're
creating a new window that doesn't have that url loaded. why the change?

[yongsheng, 2012/10/11 02:08:47]

the scenario is that previously browser in chrome have many tabs and here it
created a new tab to load the gpu crash url.
Here shell doesn't have this. actually one shell is mapped to one tab in browser
in chrome. so I make this change.

[jam, 2012/10/17 15:17:15]

I'm not following what you're saying. Yes in browser_tests we use tabs while in
content_browsertests we use windows instead. But in the previous test, the url
was loaded in the same tab. Why isn't the new test reusing the same window?

 
   std::string m;
   ASSERT_TRUE(message_queue.WaitForMessage(&m));
   EXPECT_EQ("\"SUCCESS\"", m);
 }
 
 
-IN_PROC_BROWSER_TEST_F(GPUCrashTest, WebkitLoseContext) {
-  content::DOMMessageQueue message_queue;
+IN_PROC_BROWSER_TEST_F(GPUCrashTest, MANUAL_WebkitLoseContext) {
+  DOMMessageQueue message_queue;
 
-  ui_test_utils::NavigateToURL(
-      browser(),
-      content::GetFileUrlWithQuery(
+  NavigateToURL(
+      shell(),
+      GetFileUrlWithQuery(
           gpu_test_dir_.AppendASCII("webgl.html"),
           "query=WEBGL_lose_context"));
 
   std::string m;
   ASSERT_TRUE(message_queue.WaitForMessage(&m));
   EXPECT_EQ("\"SUCCESS\"", m);
 }
+
+} // namespace content