Reduce footprint of registry controlled domain table

net/net.gyp

 # Copyright 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.
 
 {
   'variables': {
     'chromium_code': 1,
 
     'linux_link_kerberos%': 0,
     'use_tracing_cache_backend%': 0,
@@ skipping 26 matching lines @@
         'use_v8_in_net%': 1,
         'enable_built_in_dns%': 1,
       }],
     ],
   },
   'includes': [
     '../build/win_precompile.gypi',
   ],
   'targets': [
     {
+      'target_name': 'net_derived_sources',
+      'type': 'none',
+      'sources': [
+        'base/registry_controlled_domains/effective_tld_names.gperf',
+        'base/registry_controlled_domains/effective_tld_names_unittest1.gperf',
+        'base/registry_controlled_domains/effective_tld_names_unittest2.gperf',
+        'base/registry_controlled_domains/effective_tld_names_unittest3.gperf',
+        'base/registry_controlled_domains/effective_tld_names_unittest4.gperf',
+        'base/registry_controlled_domains/effective_tld_names_unittest5.gperf',
+        'base/registry_controlled_domains/effective_tld_names_unittest6.gperf',
+      ],
+      'rules': [
+        {
+          'rule_name': 'dafsa',
+          'extension': 'gperf',
+          'outputs': [
+            '<(SHARED_INTERMEDIATE_DIR)/net/<(RULE_INPUT_ROOT)-inc.cc',
+          ],
+          'inputs': [
+            'tools/tld_cleanup/make_dafsa.py',
+          ],
+          'action': [
+            'python',
+            'tools/tld_cleanup/make_dafsa.py',
+            '<(RULE_INPUT_PATH)',
+            '<(SHARED_INTERMEDIATE_DIR)/net/<(RULE_INPUT_ROOT)-inc.cc',
+          ],
+        },
+      ],
+    },
+    {
       'target_name': 'net',

[Ryan Sleevi, 2014/04/23 01:52:18]

You didn't seem to add <(SHARED_INTERMEDIATE_DIR)/net to the include_dirs here

However, see above

[Olle Liljenzin, 2014/04/24 09:30:00]

It looks like this include path is added to net by some other reason. I don't
know why. But I agree it should be added here to not add a dependency on
something that could change in the future.

       'type': '<(component)',
       'variables': { 'enable_wexit_time_destructors': 1, },
       'dependencies': [
         '../base/base.gyp:base',
         '../base/base.gyp:base_i18n',
         '../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations',
         '../crypto/crypto.gyp:crypto',
         '../sdch/sdch.gyp:sdch',
         '../third_party/icu/icu.gyp:icui18n',
         '../third_party/icu/icu.gyp:icuuc',
         '../third_party/zlib/zlib.gyp:zlib',
         '../url/url.gyp:url_lib',
         'net_resources',
+        'net_derived_sources',
       ],
       'sources': [
         'android/cert_verify_result_android.cc',
         'android/cert_verify_result_android.h',
         'android/cert_verify_status_android_list.h',
         'android/gurl_utils.cc',
         'android/gurl_utils.h',
         'android/android_private_key.cc',
         'android/android_private_key.h',
         'android/keystore.cc',
@@ skipping 1572 matching lines @@
         '../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations',
         '../crypto/crypto.gyp:crypto',
         '../testing/gmock.gyp:gmock',
         '../testing/gtest.gyp:gtest',
         '../third_party/zlib/zlib.gyp:zlib',
         '../url/url.gyp:url_lib',
         'http_server',
         'net',
         'net_test_support'
       ],
+      'include_dirs': [
+        '<(SHARED_INTERMEDIATE_DIR)/net',

[Ryan Sleevi, 2014/04/23 01:52:18]

Unnecessary, right?

[Olle Liljenzin, 2014/04/24 09:30:00]

Necessary unless there is a better way to add the include path to net_unittests.

+      ],
       'sources': [
         'android/keystore_unittest.cc',
         'android/network_change_notifier_android_unittest.cc',
         'base/address_list_unittest.cc',
         'base/address_tracker_linux_unittest.cc',
         'base/backoff_entry_unittest.cc',
         'base/data_url_unittest.cc',
         'base/directory_lister_unittest.cc',
         'base/dns_util_unittest.cc',
         'base/escape_unittest.cc',
@@ skipping 1560 matching lines @@
             'net_unittests.isolate',
           ],
           'sources': [
             'net_unittests.isolate',
           ],
         },
       ],
     }],
   ],
 }