Move content::URLFetcher static functions to net::URLFetcher

chrome/chrome_browser.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.
 
 {
   'targets': [
     {
       'target_name': 'browser',
       'type': 'static_library',
@@ skipping 16 matching lines @@
         'debugger',
         'in_memory_url_index_cache_proto',
         'installer_util',
         'safe_browsing_proto',
         'safe_browsing_report_proto',
         'feedback_proto',
         'gdata_proto',
         'variations_seed_proto',
         '../build/temp_gyp/googleurl.gyp:googleurl',
         '../content/content.gyp:content_browser',
+        '../content/content.gyp:content_common',

[wtc, 2012/06/18 23:04:22]

What is this new dependency for?  Just curious.

[akalin, 2012/06/18 23:12:32]

template_url_fetcher.cc uses AssociateURLFetcherWIthRenderView, which is in
content_common.  It works fine without this line, probably because
content_browser pulls it in, but I think it's better to explicitly specify it.

Updated CL description

         '../crypto/crypto.gyp:crypto',
         '../media/media.gyp:media',
         '../net/net.gyp:net',
         '../ppapi/ppapi_internal.gyp:ppapi_proxy',  # For PpapiMsg_LoadPlugin
         '../printing/printing.gyp:printing',
         '../skia/skia.gyp:skia',
         # TODO(akalin): Depend only on syncapi_service from sync.
         '../sync/sync.gyp:syncapi_core',
         '../sync/sync.gyp:syncapi_service',
         '../third_party/adobe/flash/flash_player.gyp:flapper_version_h',
@@ skipping 5195 matching lines @@
         'browser/metrics/proto/study.proto',
       ],
       'variables': {
         'proto_in_dir': 'browser/metrics/proto',
         'proto_out_dir': 'chrome/browser/metrics/proto',
       },
       'includes': [ '../build/protoc.gypi' ]
     },
   ],
 }