Add test to each OptionsPage to open that page

chrome/chrome_tests.gypi

 # Copyright (c) 2011 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' : {
     'pyautolib_sources': [
       'app/chrome_command_ids.h',
       'app/chrome_dll_resource.h',
       'common/automation_constants.h',
@@ skipping 2601 matching lines @@
         'renderer/safe_browsing/phishing_classifier_delegate_browsertest.cc',
         'renderer/safe_browsing/phishing_dom_feature_extractor_browsertest.cc',
         'renderer/safe_browsing/phishing_thumbnailer_browsertest.cc',
         'renderer/safe_browsing/render_view_fake_resources_test.cc',
         'renderer/safe_browsing/render_view_fake_resources_test.h',
         'renderer/translate_helper_browsertest.cc',
         'test/automation/dom_automation_browsertest.cc',
         'test/data/webui/assertions-inl.h',
         'test/data/webui/assertions.js',
         'test/data/webui/print_preview.js',
         'test/data/webui/options.js',

[James Hawkins, 2011/07/26 01:26:24]

mkdir test/data/webui/options

[Sheridan Rawlins, 2011/07/26 01:37:17]

I can do that, but, know that the generated file wouldn't have options/ in it as
there's no way to get the dirname in the rule (see
http://code.google.com/p/gyp/issues/detail?id=200).
The right thing might be doable by refactoring things under options/ to a
separate rule/library and linking against it but it seems extreme for something
that a gyp fix can solve.  I can take a look at the gyp patch and offer it up
for commit, but I think it's be higher priority to tackle http://crbug.com/87362
than shaving this yak.

-SCR

On 2011/07/26 01:26:24, James Hawkins wrote:

[Sheridan Rawlins, 2011/08/03 16:06:42]

Can this wait until crbug.com/90907 is addressed (to allow source files to be in
any directory for the generation rule)?

+        'test/data/webui/options_page_advanced.js',
+        'test/data/webui/options_page_autofill.js',
+        'test/data/webui/options_page_browser.js',
+        'test/data/webui/options_page_certificates.js',
+        'test/data/webui/options_page_content.js',
+        'test/data/webui/options_page_contentexceptions.js',
+        'test/data/webui/options_page_cookies.js',
+        'test/data/webui/options_page_fonts.js',
+        'test/data/webui/options_page_languages.js',
+        'test/data/webui/options_page_passwords.js',
+        'test/data/webui/options_page_personal.js',
+        'test/data/webui/options_page_searchengines.js',
         'test/gpu/gpu_browsertest.cc',
         'test/in_process_browser_test_browsertest.cc',
         'test/out_of_proc_test_runner.cc',
         'test/render_view_test.cc',
         'test/render_view_test.h',
         # TODO(craig): Rename this and run from base_unittests when the test
         # is safe to run there. See http://crbug.com/78722 for details.
         '../base/files/file_path_watcher_browsertest.cc',
         '../content/browser/child_process_security_policy_browsertest.cc',
         '../content/browser/device_orientation/device_orientation_browsertest.cc',
@@ skipping 1721 matching lines @@
               # Use outputs of this action as inputs for the main target build.
               # Seems as a misnomer but makes this happy on Linux (scons).
               'process_outputs_as_sources': 1,
             },
           ],  # 'actions'
         },
       ]
     }],  # 'coverage!=0'
   ],  # 'conditions'
 }