Replace forward_dependent_configs with public_deps

chrome/browser/BUILD.gn

 # Copyright 2014 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.
 
 import("//build/config/crypto.gni")
 import("//build/config/features.gni")
 import("//build/config/ui.gni")
 import("//components/nacl/nacl_defines.gni")
 # //build/config/android/rules.gni imports //tools/grit/grit_rule.gni, which
 # produces a conflict for the "grit" template so we have to only include one.
@@ skipping 15 matching lines @@
 
 static_library("browser") {
   configs += [ "//build/config/compiler:wexit_time_destructors" ]
   defines = []
   sources = []
   libs = []
   ldflags = []
 
   # iOS/non-iOS shared deps. New dependencies should generally be added in the
   # non-iOS deps below.
+  public_deps = [
+    "//components/autofill/core/browser",
+    "//content/public/browser",
+    "//sql",
+    "//sync",
+  ]
   deps = [
     "//base/allocator",
     "//chrome:extra_resources",
     "//chrome:resources",
     "//chrome:strings",
     "//chrome/app:generated_resources_map",
     "//chrome/app/resources:platform_locale_settings",
     "//chrome/app/theme:theme_resources",
     "//chrome/browser/history:in_memory_url_index_cache_proto",
     "//chrome/browser/net:cert_logger_proto",
@@ skipping 71 matching lines @@
     "//ui/base",
     "//ui/events",
     "//ui/gfx",
     "//ui/gfx/geometry",
     "//ui/message_center",
     "//ui/shell_dialogs",
     "//ui/strings",
     "//ui/resources",
   ]
 
-  forward_dependent_configs_from = [
-    "//components/autofill/core/browser",
-    "//content/public/browser",
-    "//sql",
-    "//sync",
-  ]
-
   sources += rebase_path(gypi_values.chrome_browser_undo_sources,
                          ".", "//chrome")
 
   if (!is_ios) {
     sources += rebase_path(gypi_values.chrome_browser_non_ios_sources,
                            ".", "//chrome")
     #TODO(GYP) Add these sources:
     #        # These files are generated by GRIT.
     #        '<(grit_out_dir)/grit/component_extension_resources_map.cc',
     #        '<(grit_out_dir)/grit/theme_resources_map.cc',
@@ skipping 92 matching lines @@
   if (!toolkit_views) {
     deps -= [ "//components/native_app_window" ]
   }
   if (cld2_data_source == "component") {
     sources += [
       "component_updater/cld_component_installer.cc",
       "component_updater/cld_component_installer.h",
     ]
   }
   if (enable_extensions) {
+    public_deps += [ "//chrome/browser/extensions" ]
     deps += [
-      "//chrome/browser/extensions",
       "//chrome/browser/sync_file_system/drive_backend:sync_file_system_drive_proto",
       "//chrome/common/extensions/api",
       "//chrome/common/extensions/api:api_registration",
     ]
-    forward_dependent_configs_from += [ "//chrome/browser/extensions" ]
     sources += rebase_path(gypi_values.chrome_browser_extensions_sources,
                            ".", "//chrome")
   }
   if (enable_background) {
     sources += rebase_path(gypi_values.chrome_browser_background_sources,
                            ".", "//chrome")
     if (!use_aura || is_win) {
       sources -= [ "background/background_mode_manager_aura.cc" ]
     }
   }
@@ skipping 251 matching lines @@
   # TODO(GYP)
   # Temporary fix to break the browser target into smaller chunks so it
   # will link with goma builds.
   #["OS=="win" and chromium_win_pch==0", {
   #  "msvs_shard": 4,
   #}],
 
   if (is_win) {
     sources += rebase_path(gypi_values.chrome_browser_win_sources,
                            ".", "//chrome")
+    public_deps += [
+      "//ui/views",
+      "//ui/views/controls/webview",
+    ]
     deps += [
       "//chrome:version_header",
       "//chrome/installer/util:strings",
       "//third_party/iaccessible2",
       "//third_party/isimpledom",
       "//third_party/wtl",
-      "//ui/views",
-      "//ui/views/controls/webview",
       #"chrome_process_finder",  TODO(GYP)
       #"../chrome_elf/chrome_elf.gyp:chrome_elf",  TODO(GYP)
       #"../chrome_elf/chrome_elf.gyp:chrome_elf_constants",  TODO(GYP)
       #"../chrome_elf/chrome_elf.gyp:dll_hash",  TODO(GYP)
       #"../google_update/google_update.gyp:google_update",  TODO(GYP)
       #"../ui/metro_viewer/metro_viewer.gyp:metro_viewer_messages",  TODO(GYP)
       #"../win8/win8.gyp:metro_viewer",  TODO(GYP)
     ]
-    forward_dependent_configs_from += [
-      "//ui/views",
-      "//ui/views/controls/webview",
-    ]
   } else {
     # Non-Windows.
     sources += rebase_path(gypi_values.chrome_browser_non_win_sources,
                            ".", "//chrome")
     if (toolkit_views) {
       deps += [
       "//ui/views",
       "//ui/views/controls/webview",
       ]
       # TODO(GYP) why is this needed? Very susicious.
@@ skipping 214 matching lines @@
     "ssl/ssl_client_auth_requestor_mock.cc",
     "ssl/ssl_client_auth_requestor_mock.h",
     "sync/glue/local_device_info_provider_mock.cc",
     "sync/glue/local_device_info_provider_mock.h",
     "sync/profile_sync_components_factory_mock.cc",
     "sync/profile_sync_components_factory_mock.h",
     "sync/profile_sync_service_mock.cc",
     "sync/profile_sync_service_mock.h",
   ]
 
+  public_deps = [
+    ":browser",
+    "//chrome/browser/ui:test_support",
+  ]
   deps = [
-    ":browser",
     "//base:prefs_test_support",
     "//chrome/browser",
-    "//chrome/browser/ui:test_support",
     "//chrome/common",
     "//chrome/common/safe_browsing:proto",
     "//components/invalidation",
     "//components/invalidation:test_support",
     "//components/password_manager/core/browser:test_support",
     "//components/search_engines:test_support",
     "//content/test:test_support",
     "//chrome/app/theme:theme_resources",
     "//net:test_support",
     "//skia",
     "//testing/gmock",
     "//testing/gtest",
     "//ui/gfx",
   ]
 
-  forward_dependent_configs_from = [
-    ":browser",
-    "//chrome/browser/ui:test_support",
-  ]
-
   if (!is_ios) {
     deps += [
       "//components/sessions:test_support",
       "//google_apis:test_support",
     ]
   }
 
   if (enable_extensions) {
     deps += [
       "//extensions:test_support",
@@ skipping 49 matching lines @@
     ]
     configs += [ "//build/config/linux:dbus" ]
     #deps += [ "//chromeos:test_support" ]  TODO(GYP)
   }
 
   if (enable_configuration_policy) {
     sources += [
       "policy/test/local_policy_test_server.cc",
       "policy/test/local_policy_test_server.h",
     ]
-    policy_deps = [
+    public_deps += [
       "//components/policy:policy_component_test_support",
       "//components/policy:test_support",
     ]
-    deps += policy_deps
-    forward_dependent_configs_from += policy_deps
   }
 
   if (safe_browsing_mode == 1) {
     sources += [
       "extensions/fake_safe_browsing_database_manager.cc",
       "extensions/fake_safe_browsing_database_manager.h",
     ]
   }
 
   if (enable_extensions) {
@@ skipping 44 matching lines @@
     ]
   }
 
   if (enable_wifi_bootstrapping) {
     sources += [
       "local_discovery/wifi/mock_wifi_manager.cc",
       "local_discovery/wifi/mock_wifi_manager.h",
     ]
   }
 }