aura: Add glib dependency when appropriate.

content/content_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.
 
 {
   'targets': [
     {
       'target_name': 'test_support_content',
       'type': 'static_library',
       'defines!': ['CONTENT_IMPLEMENTATION'],
@@ skipping 28 matching lines @@
         'browser/geolocation/arbitrator_dependency_factories_for_test.cc',
         'browser/geolocation/arbitrator_dependency_factories_for_test.h',
         'browser/geolocation/fake_access_token_store.cc',
         'browser/geolocation/fake_access_token_store.h',
         'browser/geolocation/mock_location_provider.cc',
         'browser/geolocation/mock_location_provider.h',
         'browser/mock_content_browser_client.cc',
         'browser/mock_content_browser_client.h',
         'browser/mock_resource_context.cc',
         'browser/mock_resource_context.h',
-        'browser/net/url_request_abort_on_end_job.cc',
-        'browser/net/url_request_abort_on_end_job.h',
+        'browser/net/url_request_abort_on_end_job.cc',
+        'browser/net/url_request_abort_on_end_job.h',
         'browser/renderer_host/dummy_resource_handler.cc',
         'browser/renderer_host/dummy_resource_handler.h',
         'browser/renderer_host/media/mock_media_observer.cc',
         'browser/renderer_host/media/mock_media_observer.h',
         'browser/renderer_host/test_backing_store.cc',
         'browser/renderer_host/test_backing_store.h',
         'browser/renderer_host/test_render_view_host.cc',
         'browser/renderer_host/test_render_view_host.h',
         'browser/renderer_host/mock_render_process_host.cc',
         'browser/renderer_host/mock_render_process_host.h',
@@ skipping 47 matching lines @@
         '../webkit/appcache/appcache_test_helper.h',
         '../webkit/quota/mock_special_storage_policy.cc',
         '../webkit/quota/mock_special_storage_policy.h',
       ],
       'conditions': [
         ['toolkit_uses_gtk == 1', {
           'dependencies': [
             '../build/linux/system.gyp:gtk',
           ],
         }],
+        ['use_glib == 1', {

[sky, 2011/11/18 17:26:22]

I'm very surprised we need this here. Shouldn't someone be depending upon base
which has this dependency for us? I'm sure I don't understand how dependencies
in gyp.

[sadrul, 2011/11/18 17:31:04]

Indeed. I am surprised we need this too. And I am also surprised that the FYI
bots are not failing (perhaps it will show up with a clobber build?)

The 'test_support_content' target does not depend on 'base'. I tried adding the
dependency since that seemed more appropriate, but that causes other gyp errors
that I can't make any sense of. So I went with the easy way out and did the same
thing that's done for gtk above.

+          'dependencies': [
+            '../build/linux/system.gyp:glib',
+          ],
+        }],
       ],
     },
     {
       'target_name': 'content_unittests',
       'type': 'executable',
       'defines!': ['CONTENT_IMPLEMENTATION'],
       'dependencies': [
         'content_browser',
         'content_gpu',
         'content_plugin',
@@ skipping 249 matching lines @@
           ],
           'sources': [
             'common/gpu/media/omx_video_decode_accelerator_unittest.cc',
           ],
         }
       ],
     },
    ],
   ],
 }