Cleanup OriginIdentifierValueMap code.

chrome/browser/content_settings/content_settings_origin_identifier_value_map.h

 // Copyright (c) 2011 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 CHROME_BROWSER_CONTENT_SETTINGS_CONTENT_SETTINGS_ORIGIN_IDENTIFIER_VALUE_MAP_H_
 #define CHROME_BROWSER_CONTENT_SETTINGS_CONTENT_SETTINGS_ORIGIN_IDENTIFIER_VALUE_MAP_H_
 
 #include <list>
 #include <map>
 #include <string>
@@ skipping 78 matching lines @@
   // Deletes the map entry for the given |item_pattern|,
   // |top_level_frame_pattern|, |content_type|, |resource_identifier| tuple.
   void DeleteValue(
       const ContentSettingsPattern& item_pattern,
       const ContentSettingsPattern& top_level_frame_pattern,
       ContentSettingsType content_type,
       const ResourceIdentifier& resource_identifier);
 
   // Deletes the map entry at the passed position. The method returns the
   // position of the next entry in the map.
-  EntryList::iterator DeleteValue(EntryList::iterator entry);
+  EntryList::iterator erase(EntryList::iterator entry);

[markusheintz_, 2011/06/14 12:50:40]

Sorry I don't want to delete this one as it makes it easier to delete entires
while iterating over them. Otherwise I always have to create a list of entries
to delete.

 
   // Clears all map entries.
-  void Clear();
+  void clear();

[markusheintz_, 2011/06/14 12:50:40]

This one is used quite often :)

 
  private:
   // Finds the list entry for the given |item_pattern|,
   // |top_level_frame_pattern|, |content_type|, |resource_identifier| tuple and
   // returns the iterator of the list entry. If no entry is found for the passed
   // parameters then the end of list iterator is returned.
   EntryList::iterator FindEntry(
       const ContentSettingsPattern& item_pattern,
       const ContentSettingsPattern& top_level_frame_pattern,
       ContentSettingsType content_type,
       const ResourceIdentifier& resource_identifier);
 
   EntryList entries_;
 
   DISALLOW_COPY_AND_ASSIGN(OriginIdentifierValueMap);
 };
 
 // Compares two origin value map entries and tests if the item, top-level-frame
 // pattern pair of the first entry has a higher precedences then the pattern
 // pair of the second entry.
 bool operator>(const OriginIdentifierValueMap::Entry& first,
                const OriginIdentifierValueMap::Entry& second);
 
 }  // namespace content_settings
 
 #endif  // CHROME_BROWSER_CONTENT_SETTINGS_CONTENT_SETTINGS_ORIGIN_IDENTIFIER_VALUE_MAP_H_