Add the first app_shell_browsertest test.

apps/shell/app_shell.gyp

 # Copyright 2014 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.
 
 {
   'variables': {
     'chromium_code': 1,
   },
   'targets': [
     {
@@ skipping 147 matching lines @@
         '<(DEPTH)/content/content_shell_and_tests.gyp:content_browser_test_support',
         '<(DEPTH)/content/content_shell_and_tests.gyp:test_support_content',
         '<(DEPTH)/testing/gtest.gyp:gtest',
       ],
       'defines': [
         'HAS_OUT_OF_PROC_TEST_RUNNER',
       ],
       'msvs_large_pdb': 1,
       'sources': [
         # TODO(yoz): Refactor once we have a second test target.
-        '../test/app_shell_test.h',
-        '../test/app_shell_test.cc',
-        '../test/apps_test_launcher_delegate.cc',
-        '../test/apps_test_launcher_delegate.h',
-        '../test/apps_tests_main.cc',
+        'browser/shell_browsertest.cc',
+        'test/app_shell_test.h',

[James Cook, 2014/04/08 16:03:49]

I'm OK with these names and I like that the app_shell_browsertests code is in a
file named app_shell. However, did you consider naming them all just shell_*? 
That's what I've done with most of the files in apps/shell. I was relying on the
//apps path and apps:: namespace to differentiate from other Shell* classes.

[Yoyo Zhou, 2014/04/08 21:55:21]

Ok, will just call them that.

+        'test/app_shell_test.cc',
+        'test/app_shell_test_launcher_delegate.cc',
+        'test/app_shell_test_launcher_delegate.h',
+        'test/app_shell_tests_main.cc',
       ],
     },
   ],  # targets
 }