Fix log spew for "_comment..." policies.

components/policy/core/common/policy_map.h

 // 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.
 
 #ifndef COMPONENTS_POLICY_CORE_COMMON_POLICY_MAP_H_
 #define COMPONENTS_POLICY_CORE_COMMON_POLICY_MAP_H_
 
 #include <stddef.h>
 
 #include <map>
@@ skipping 60 matching lines @@
            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, Entry entry);
 
   // Erase the given |policy|, if it exists in this map.
   void Erase(const std::string& policy);
 
+  // Erase all entries for which |filter| returns true.
+  void EraseMatching(const base::Callback<bool(const const_iterator)>& filter);
+
   // Erase all entries for which |filter| returns false.
   void EraseNonmatching(
       const base::Callback<bool(const const_iterator)>& filter);
 
   // Swaps the internal representation of |this| with |other|.
   void Swap(PolicyMap* other);
 
   // |this| becomes a copy of |other|. Any existing policies are dropped.
   void CopyFrom(const PolicyMap& other);
 
@@ skipping 27 matching lines @@
 
   const_iterator begin() const;
   const_iterator end() const;
   void Clear();
 
  private:
   // Helper function for Equals().
   static bool MapEntryEquals(const PolicyMapType::value_type& a,
                              const PolicyMapType::value_type& b);
 
+  // Erase all entries for which |filter| returns |deletion_value|.
+  void EraseGeneric(const base::Callback<bool(const const_iterator)>& filter,

[pastarmovj, 2016/10/18 16:20:11]

nit: I don't like this name much but I can't come up with anything better either
so I will just leave it here as a comment if you can come up with something less
"generic" :)

[Thiemo Nagel, 2016/10/18 16:31:50]

I wasn't happy with FilterGeneric() either. Using FilterErase() instead.

+                    bool deletion_value);
+
   PolicyMapType map_;
 
   DISALLOW_COPY_AND_ASSIGN(PolicyMap);
 };
 
 }  // namespace policy
 
 #endif  // COMPONENTS_POLICY_CORE_COMMON_POLICY_MAP_H_