Revert of [Mac/GN] Fix rebuilds when changing framework_version of a mac_framework_bundle.

build/config/mac/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/mac/base_rules.gni")
 
 # Generates Info.plist files for Mac apps and frameworks.
 #
 # Arguments
 #
@@ skipping 240 matching lines @@
   if (defined(invoker.output_name)) {
     _output_name = invoker.output_name
   }
 
   # If the framework is unversioned, the final _target_name will be the
   # create_bundle(_framework_target), otherwise an action with the name
   # _target_name will depends on the the create_bundle() in order to prepare
   # the versioned directory structure.
   _framework_target = _target_name
   _framework_name = _output_name + ".framework"
-  _framework_base_dir = "$root_out_dir/$_framework_name"
+  _framework_root_dir = "$root_out_dir/$_framework_name"
   if (defined(invoker.framework_version) && invoker.framework_version != "") {
     _framework_version = invoker.framework_version
-    _framework_root_dir = _framework_base_dir + "/Versions/$_framework_version"
+    _framework_root_dir += "/Versions/$_framework_version"
     _framework_target = _target_name + "_create_bundle"
-  } else {
-    _framework_root_dir = _framework_base_dir
   }
 
   _link_shared_library_target = target_name + "_shared_library"
   _shared_library_bundle_data = target_name + "_shared_library_bundle_data"
 
   shared_library(_link_shared_library_target) {
     forward_variables_from(invoker,
                            "*",
                            [
                              "assert_no_deps",
@@ skipping 19 matching lines @@
       "$target_out_dir/$_link_shared_library_target/$_output_name",
     ]
     outputs = [
       "{{bundle_executable_dir}}/$_output_name",
     ]
     public_deps = [
       ":$_link_shared_library_target",
     ]
   }
 
-  # Clean the entire framework if the framework_version changes.
-  _version_arg = ""
-  if (defined(_framework_version)) {
-    _version_arg = _framework_version
-  }
-  _version_file = "$target_out_dir/${target_name}_version"
-  exec_script("//build/config/mac/prepare_framework_version.py",
-              [
-                rebase_path(_version_file),
-                rebase_path(_framework_base_dir),
-                "'$_version_arg'",
-              ])
-
   _framework_public_config = _target_name + "_public_config"
   config(_framework_public_config) {
     # TODO(sdefresne): should we have a framework_dirs similar to lib_dirs
     # and include_dirs to avoid duplicate values on the command-line.
     visibility = [ ":$_framework_target" ]
     ldflags = [
       "-F",
       rebase_path("$root_out_dir/.", root_build_dir),
     ]
     lib_dirs = [ root_out_dir ]
@@ skipping 317 matching lines @@
                            ])
     if (!defined(deps)) {
       deps = []
     }
     deps += [ ":$_loadable_module_bundle_data" ]
 
     bundle_root_dir = "$root_out_dir/$_output_name.plugin/Contents"
     bundle_executable_dir = "$bundle_root_dir/MacOS"
   }
 }