Add apps_browsertests target (currently running 0 tests).

apps/shell/app/shell_main_delegate.cc

Index: apps/shell/app/shell_main_delegate.cc
diff --git a/apps/shell/app/shell_main_delegate.cc b/apps/shell/app/shell_main_delegate.cc
index 13706cda47a78edcf9960e4953b1c0c3a6a0e880..d9f9904559637dd3aa5a0f3fd19bafb079bb909c 100644
--- a/apps/shell/app/shell_main_delegate.cc
+++ b/apps/shell/app/shell_main_delegate.cc
@@ -64,6 +64,26 @@ void ShellMainDelegate::PreSandboxStartup() {
     InitializeResourceBundle();
 }
 
+int ShellMainDelegate::RunProcess(

[jam, 2014/03/20 16:58:13]

I'm assuming you copied this from content/shell? That app overrides this method
because it does a bunch of custom things when running layout tests (see
ShellBrowserMain).

But for app_shell, this code is just a duplicate of what the standard
BrowserMain does in content\browser\browser_main. so you shouldn't need to
override this method.

[Yoyo Zhou, 2014/03/20 21:17:30]

Ok. I deleted it for now. As I was trying to get tests to run, I had to make
numerous changes like this, and I'm not sure which of them were extraneous.

See James's comment below about closing the window/app - I'm not sure where in
the test framework that should be done.

+    const std::string& process_type,
+    const content::MainFunctionParams& main_function_params) {
+  if (!process_type.empty())

[James Cook, 2014/03/20 00:00:35]

nit: Maybe comment that empty process_type means browser process and that's the
only one we support?

+    return -1;
+
+  scoped_ptr<content::BrowserMainRunner> browser_main_runner;
+  browser_main_runner.reset(content::BrowserMainRunner::Create());

[James Cook, 2014/03/20 00:00:35]

nit: Maybe collapse with line above?

scoped_ptr<content::BrowserMainRunner> runner(
  content::BrowserMainRunner::Create());

+  int exit_code = browser_main_runner->Initialize(main_function_params);
+  DCHECK_LT(exit_code, 0)
+      << "BrowserMainRunner::Initialize failed in apps::ShellMainDelegate";
+  if (exit_code >= 0)
+    return exit_code;
+
+  browser_main_runner->Run();
+  // TODO(yoz): close windows?

[James Cook, 2014/03/20 00:00:35]

Not for this CL, but I suspect we'll need to explicitly close the window and/or
app somehow.

+  browser_main_runner->Shutdown();
+  return 0;
+}
+
 content::ContentBrowserClient* ShellMainDelegate::CreateContentBrowserClient() {
   browser_client_.reset(new apps::ShellContentBrowserClient);
   return browser_client_.get();