Use the WebTestProxy for layout tests in content_shell

content/content_renderer.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.
 
 {
   'dependencies': [
     '../jingle/jingle.gyp:jingle_glue',
     '../net/net.gyp:net',
     '../ppapi/ppapi_internal.gyp:ppapi_host',
     '../ppapi/ppapi_internal.gyp:ppapi_proxy',
@@ skipping 252 matching lines @@
     'renderer/web_ui_bindings.cc',
     'renderer/web_ui_bindings.h',
     'renderer/webplugin_delegate_proxy.cc',
     'renderer/webplugin_delegate_proxy.h',
     'renderer/websharedworker_proxy.cc',
     'renderer/websharedworker_proxy.h',
     'renderer/websharedworkerrepository_impl.cc',
     'renderer/websharedworkerrepository_impl.h',
   ],
   'conditions': [
+    ['component=="shared_library" and OS!="ios"', {
+      'sources': [
+        'public/test/content_layouttest_support.h',
+        'test/content_layouttest_support.cc',

[jam, 2012/11/08 20:37:06]

why is this dependency there on shared builds?

[jochen (gone - plz use gerrit), 2012/11/08 20:41:08]

The alternative is to content_export RenderViewImpl and all of its base classes.
Given that shared builds are a development-only thing that seemed like the
better alternative.

In WebKit, we do the same with the webkit_test_support target. In a shared
library build, we just build the test stuff into the webkit.dll

[jam, 2012/11/08 20:49:24]

would this only be RenderViewImpl and RenderWidget?

even if we have to export the other interfaces, there aren't that many
we put in CONTENT_EXPORT all over the code already to support dev-only shared
builds, so not sure why this would be different? I ask because there's a price
for this complication of the gypi files as well.

+      ],
+      'dependencies': [
+        '<(webkit_src_dir)/Tools/DumpRenderTree/DumpRenderTree.gyp/DumpRenderTree.gyp:TestRunner',
+      ],
+    }],
     ['notifications==0', {
       'sources!': [
         'renderer/notification_provider.cc',
         'renderer/active_notification_tracker.cc',
       ],
     }],
     ['input_speech==0', {
       'sources!': [
         'renderer/input_tag_speech_dispatcher.cc',
         'renderer/input_tag_speech_dispatcher.h',
@@ skipping 84 matching lines @@
     }],
   ],
   'target_conditions': [
     ['OS=="android"', {
       'sources/': [
         ['include', '^renderer/render_view_linux\\.cc$'],
       ],
     }],
   ],
 }