[iOS/Mac/GN] The framework_bundle template should not force dependencies to link it.

build/config/ios/rules.gni

 # Copyright 2015 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/ios/ios_sdk.gni")
 import("//build/config/mac/rules.gni")
 
 # TODO(crbug.com/297668): refactor this template to extract common behaviour
 # between OS X and iOS bundle generation, then create a generic "app" template
 # that forward to "executable" on all platform except iOS/OS X.
@@ skipping 204 matching lines @@
     sources = get_target_outputs(":$_compile_xib")
 
     outputs = [
       "{{bundle_resources_dir}}/{{source_file_part}}",
     ]
   }
 }
 
 # Template to package a shared library into an iOS framework bundle.
 #
+# This template provides two targets to control whether the framework is
+# merely built when targets depend on it, or whether it is linked as well:
+# "$target_name" and "$target_name+link".
+#
+# See the //build/config/mac/rules.gni:framework_bundle for a discussion
+# and examples.
+#
 # Arguments
 #
 #     output_name:
 #         (optional) string, name of the generated framework without the
 #         .framework suffix. If omitted, defaults to target_name.
 #
 #     framework_version:
 #         (optional) string, version of the framework. Typically this is a
 #         single letter, like "A". If omitted, the Versions/ subdirectory
 #         structure will not be created, and build output will go directly
@@ skipping 90 matching lines @@
 
     group(_framework_headers_target) {
       deps = [
         ":$_compile_headers_map_target",
         ":$_copy_public_headers_target",
         ":$_create_module_map_target",
       ]
     }
   }
 
+  _framework_public_config = _target_name + "_ios_public_config"
+  config(_framework_public_config) {
+    visibility = [ ":$_framework_public_config" ]
+    if (defined(_public_headers)) {
+      common_flags = [ "-F" + rebase_path("$root_out_dir/.", root_out_dir) ]
+      cflags_objc = common_flags
+      cflags_objcc = common_flags
+    }
+
+    # The link settings are inherited from the framework_bundle config.
+  }
+
   framework_bundle(_framework_target) {
     forward_variables_from(invoker,
                            "*",
                            [
                              "output_name",
                              "public_headers",
                              "visibility",
                            ])
     output_name = _output_name
 
+    if (!defined(public_configs)) {
+      public_configs = []
+    }
+    public_configs += [ ":$_framework_public_config" ]
+
     if (defined(_public_headers)) {
-      visibility = [ ":$_target_name" ]
+      visibility = [
+        ":$_target_name",
+        ":$_target_name+link",
+      ]
       configs += [ ":$_headers_map_config" ]
 
       if (!defined(deps)) {
         deps = []
       }
       deps += [ ":$_framework_headers_target" ]
     } else {
-      forward_variables_from(invoker, [ "visibility" ])
+      if (defined(invoker.visibility)) {
+        visibility = invoker.visibility
+        visibility += [ ":$_target_name+link" ]
+      }
     }
   }
 
   if (defined(_public_headers)) {
     group(_target_name) {
       forward_variables_from(invoker,
                              [
                                "testonly",
-                               "visibility",
                                "public_configs",
                              ])
 
+      if (defined(invoker.visibility)) {
+        visibility = invoker.visibility
+        visibility += [ ":$_target_name+link" ]
+      }
+
       public_deps = [
         ":$_framework_target",
       ]
     }
+
+    group(_target_name + "+link") {
+      forward_variables_from(invoker,
+                             [
+                               "testonly",
+                               "visibility",
+                             ])
+      public_deps = [
+        ":$_framework_target+link",
+      ]
+    }
   }
 }