Remove dependency on --gc-sections for most Android debug targets.

content/content_tests.gypi

 # Copyright (c) 2013 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.
 
 {
   'targets': [
     {
       'target_name': 'layouttest_support_content',
       'type': 'static_library',
       'conditions': [
@@ skipping 37 matching lines @@
         '../testing/gtest.gyp:gtest',
         '../ui/accessibility/accessibility.gyp:ax_gen',
         '../ui/base/ui_base.gyp:ui_base',
         '../ui/events/events.gyp:dom4_keycode_converter',
         '../ui/events/events.gyp:events_base',
         '../ui/events/events.gyp:events_test_support',
         '../ui/gfx/gfx.gyp:gfx',
         '../ui/gfx/gfx.gyp:gfx_geometry',
         '../ui/gfx/gfx.gyp:gfx_test_support',
         '../ui/resources/ui_resources.gyp:ui_resources',
+        '../v8/tools/gyp/v8.gyp:v8',
         '../url/url.gyp:url_lib',
+        '../webkit/common/webkit_common.gyp:webkit_common',
         'browser/speech/proto/speech_proto.gyp:speech_proto',
         'content.gyp:content_app_both',
         'content.gyp:content_browser',
         'content.gyp:content_common',
       ],
       'include_dirs': [
         '..',
       ],
       'sources': [
         'public/test/async_file_test_helper.cc',
@@ skipping 125 matching lines @@
         'test/test_web_contents.cc',
         'test/test_web_contents.h',
         'test/test_web_contents_view.cc',
         'test/test_web_contents_view.h',
         'test/web_gesture_curve_mock.cc',
         'test/web_gesture_curve_mock.h',
         'test/weburl_loader_mock.cc',
         'test/weburl_loader_mock.h',
         'test/weburl_loader_mock_factory.cc',
         'test/weburl_loader_mock_factory.h',
+        # Required by test_file_system_context.cc
+        '../webkit/browser/quota/mock_special_storage_policy.cc',
+        '../webkit/browser/quota/mock_special_storage_policy.h',

[Nico, 2014/04/11 20:36:59]

This looks really icky. It looks like two other targets do this already, but
these files should really be in some test support target that the 2 (now 3)
targets should depend on.

[Primiano Tucci (use gerrit), 2014/04/12 10:26:00]

Hmm, I am not completely sure I got the sense of "do this already". It looks to
me that mock_special_storage_policy.cc is included only here and nowhere else
(after my removal from test_support_common).
You are referring to the general fact of including ../webkit/..cc files in the
various targets here, right?
Would you feel better grouping them in a new static_library target (something
like 'webkit_content_test_support') and have the targets depending on them?
(provided that they don't create *more magic cycles*)

[Nico, 2014/04/14 15:23:07]

Yes.
I think that'd be better (I feel that in general, each source file should be in
exactly one target), but it doesn't have to be in this CL.

[jam, 2014/04/14 16:18:06]

note: i would ignore this for now, these files are moving to cntent soon, and
that'll solve this problem

       ],
       'conditions': [
+        ['enable_plugins==0', {
+          'sources!': [
+            'test/ppapi_unittest.cc',
+          ],
+        }],
+        ['input_speech==0', {
+          'sources!': [
+            'test/mock_google_streaming_server.cc',
+          ],
+        }],
         ['OS == "ios"', {
           'sources/': [
             # iOS only needs a small portion of content; exclude all the
             # implementation, and re-include what is used.
             ['exclude', '\\.(cc|mm)$'],
             ['include', '_ios\\.(cc|mm)$'],
             ['include', '^public/test/content_test_suite_base\\.cc$'],
             ['include', '^public/test/mock_notification_observer\\.cc$'],
             ['include', '^public/test/mock_resource_context\\.cc$'],
             ['include', '^public/test/test_browser_thread\\.cc$'],
@@ skipping 428 matching lines @@
         '../webkit/browser/fileapi/sandbox_directory_database_unittest.cc',
         '../webkit/browser/fileapi/sandbox_isolated_origin_database_unittest.cc',
         '../webkit/browser/fileapi/sandbox_origin_database_unittest.cc',
         '../webkit/browser/fileapi/sandbox_prioritized_origin_database_unittest.cc',
         '../webkit/browser/fileapi/timed_task_helper_unittest.cc',
         '../webkit/child/multipart_response_delegate_unittest.cc',
         '../webkit/common/blob/shareable_file_reference_unittest.cc',
         '../webkit/common/database/database_connections_unittest.cc',
         '../webkit/common/database/database_identifier_unittest.cc',
         '../webkit/common/fileapi/file_system_util_unittest.cc',
-        '../webkit/browser/quota/mock_special_storage_policy.cc',
-        '../webkit/browser/quota/mock_special_storage_policy.h',
         '../webkit/browser/quota/mock_storage_client.cc',
         '../webkit/browser/quota/mock_storage_client.h',
         '../webkit/browser/quota/quota_database_unittest.cc',
         '../webkit/browser/quota/quota_manager_unittest.cc',
         '../webkit/browser/quota/quota_temporary_storage_evictor_unittest.cc',
         '../webkit/browser/quota/storage_monitor_unittest.cc',
         '../webkit/browser/quota/usage_tracker_unittest.cc',
       ],
       'conditions': [
         ['OS == "ios"', {
@@ skipping 21 matching lines @@
             '../jingle/jingle.gyp:jingle_glue_test_util',
             '../media/media.gyp:media_test_support',
             '../media/media.gyp:shared_memory_support',
             '../third_party/WebKit/public/blink.gyp:blink',
             '../third_party/icu/icu.gyp:icui18n',
             '../third_party/icu/icu.gyp:icuuc',
             '../third_party/leveldatabase/leveldatabase.gyp:leveldatabase',
             '../third_party/libjingle/libjingle.gyp:libjingle',
             '../ui/compositor/compositor.gyp:compositor_test_support',
             '../ui/gl/gl.gyp:gl',
-            '../v8/tools/gyp/v8.gyp:v8',
-            '../webkit/common/webkit_common.gyp:webkit_common',
             '../webkit/child/webkit_child.gyp:webkit_child',
             '../webkit/storage_browser.gyp:webkit_storage_browser',
             '../webkit/storage_common.gyp:webkit_storage_common',
           ],
         }],
         ['enable_plugins==0', {
           'sources/': [
             ['exclude', '^browser/renderer_host/pepper/'],
             ['exclude', '^child/npapi/'],
             ['exclude', '^renderer/npapi/'],
@@ skipping 547 matching lines @@
           'target_name': 'content_gl_tests',
           'type': '<(gtest_target_type)',
           'dependencies': [
             'content.gyp:content_common',
             'test_support_content',
             '../base/base.gyp:test_support_base',
             '../gpu/gpu.gyp:command_buffer_common',
             '../testing/gtest.gyp:gtest',
             '../third_party/WebKit/public/blink.gyp:blink',
             '../ui/base/ui_base.gyp:ui_base',
+            '../v8/tools/gyp/v8.gyp:v8',
             '../ui/gfx/gfx.gyp:gfx',
             '../ui/gfx/gfx.gyp:gfx_geometry',
             '../ui/gl/gl.gyp:gl',
-            # The following two dependencies provide the missing
-            # symbol HeapProfilerStart in Linux component builds.
+            # The following dependency provides the missing symbol
+            # HeapProfilerStart in Linux component builds.
             '../webkit/child/webkit_child.gyp:webkit_child',
           ],
           'include_dirs': [
             '..',
           ],
           'sources': [
             'common/gpu/client/gl_helper_unittest.cc',
             'common/gpu/client/gpu_in_process_context_tests.cc',
           ],
           'conditions': [
@@ skipping 507 matching lines @@
                   'files': ['<(PRODUCT_DIR)/libnpapi_test_plugin.so'],
                 },
               ],
             }],
           ],
         },
       ],
     }],
   ],
 }