Move test headers from content\test to content\public\test. This way we can enforce that internal c…

chrome/chrome_tests.gypi

 # 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.
 {
   'variables' : {
     'pyautolib_sources': [
       'app/chrome_command_ids.h',
       'app/chrome_dll_resource.h',
       'common/automation_constants.h',
       'common/pref_names.cc',
@@ skipping 3000 matching lines @@
         '../content/browser/plugin_data_remover_impl_browsertest.cc',
         '../content/browser/plugin_service_impl_browsertest.cc',
         '../content/browser/renderer_host/render_process_host_browsertest.cc',
         '../content/browser/renderer_host/render_view_host_browsertest.cc',
         '../content/browser/renderer_host/render_view_host_manager_browsertest.cc',
         '../content/browser/renderer_host/resource_dispatcher_host_browsertest.cc',
         '../content/browser/session_history_browsertest.cc',
         '../content/browser/speech/speech_recognition_browsertest.cc',
         '../content/browser/webkit_browsertest.cc',
         '../content/browser/worker_host/test/worker_browsertest.cc',
+        '../content/public/test/render_widget_browsertest.h',

[Paweł Hajdan Jr., 2012/06/05 09:47:24]

nit: Wasn't the convention to have public header files only in content targets?
Up to you of course.

[jam, 2012/06/05 15:57:54]

I think these files need to be here since we only have one browser_test target?
Once I have content_browsertest I'll move these out

         '../content/renderer/mouse_lock_dispatcher_browsertest.cc',
         '../content/renderer/render_view_browsertest.cc',
         '../content/renderer/render_view_browsertest_mac.mm',
         '../content/test/gpu/test_switches.cc',
         '../content/test/gpu/test_switches.h',
         '../content/test/layout_browsertest.cc',
         '../content/test/layout_browsertest.h',
         '../content/test/render_widget_browsertest.cc',
-        '../content/test/render_widget_browsertest.h',
         '../content/test/test_launcher.cc',
         '../content/test/test_launcher.h',
       ],
       'rules': [
         {
           'rule_name': 'js2webui',
           'extension': 'js',
           'msvs_external_rule': 1,
           'inputs': [
             '<(gypv8sh)',
@@ skipping 1612 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'
 }