Remove PolicyMap::Set old version

Bug: 1092469
Change-Id: I4a405c8642bbb78a59bd3681178e372cc2a56cc0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2348772
Commit-Queue: Aya Elsayed <ayaelattar@chromium.org>
Reviewed-by: Maksim Ivanov <emaxx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#797325}

components/policy/core/common/policy_map.cc

@@ -212,19 +212,6 @@ PolicyMap::Entry* PolicyMap::GetMutableUntrusted(const std::string& policy) {
   return entry != map_.end() ? &entry->second : nullptr;
 }
 
-void PolicyMap::Set(
-    const std::string& policy,
-    PolicyLevel level,
-    PolicyScope scope,
-    PolicySource source,
-    std::unique_ptr<base::Value> value,
-    std::unique_ptr<ExternalDataFetcher> external_data_fetcher) {
-  Entry entry(level, scope, source,
-              value ? base::make_optional(std::move(*value)) : base::nullopt,
-              std::move(external_data_fetcher));
-  Set(policy, std::move(entry));
-}
-
 void PolicyMap::Set(
     const std::string& policy,
     PolicyLevel level,

components/policy/core/common/policy_map.h

@@ -143,16 +143,6 @@ class POLICY_EXPORT PolicyMap {
 
   // Overwrites any existing information stored in the map for the key |policy|.
   // Resets the error for that policy to the empty string.
-  // DEPRECATED: Use the other version that takes base::Optional<base::Value>
-  // below.
-  // TODO(crbug.com/1092469): Migrate the existing usages and delete this
-  // method.
-  void Set(const std::string& policy,
-           PolicyLevel level,
-           PolicyScope scope,
-           PolicySource source,
-           std::unique_ptr<base::Value> value,
-           std::unique_ptr<ExternalDataFetcher> external_data_fetcher);
   void Set(const std::string& policy,
            PolicyLevel level,
            PolicyScope scope,