Removing base::DictionaryValue::key_iterator.

base/values.h

 // Copyright (c) 2012 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.
 
 // This file specifies a recursive data storage class called Value intended for
 // storing setting and other persistable data.  It includes the ability to
 // specify (recursive) lists and dictionaries, so it's fairly expressive.
 // However, the API is optimized for the common case, namely storing a
 // hierarchical tree of simple values.  Given a DictionaryValue root, you can
 // easily do things like:
@@ skipping 318 matching lines @@
   // Merge |dictionary| into this dictionary. This is done recursively, i.e. any
   // sub-dictionaries will be merged as well. In case of key collisions, the
   // passed in dictionary takes precedence and data already present will be
   // replaced. Values within |dictionary| are deep-copied, so |dictionary| may
   // be freed any time after this call.
   void MergeDictionary(const DictionaryValue* dictionary);
 
   // Swaps contents with the |other| dictionary.
   virtual void Swap(DictionaryValue* other);
 
-  // This class provides an iterator for the keys in the dictionary.
-  // It can't be used to modify the dictionary.
-  //
-  // YOU SHOULD ALWAYS USE THE XXXWithoutPathExpansion() APIs WITH THESE, NOT
-  // THE NORMAL XXX() APIs.  This makes sure things will work correctly if any
-  // keys have '.'s in them.
-  class BASE_EXPORT key_iterator
-      : private std::iterator<std::input_iterator_tag, const std::string> {
-   public:
-    explicit key_iterator(ValueMap::const_iterator itr);
-    // Not explicit, because this is a copy constructor.
-    key_iterator(const key_iterator& rhs);
-    key_iterator operator++() {
-      ++itr_;
-      return *this;
-    }
-    const std::string& operator*() { return itr_->first; }
-    bool operator!=(const key_iterator& other) { return itr_ != other.itr_; }
-    bool operator==(const key_iterator& other) { return itr_ == other.itr_; }
-
-   private:
-    ValueMap::const_iterator itr_;
-  };
-
-  key_iterator begin_keys() const { return key_iterator(dictionary_.begin()); }
-  key_iterator end_keys() const { return key_iterator(dictionary_.end()); }
-
   // This class provides an iterator over both keys and values in the
   // dictionary.  It can't be used to modify the dictionary.
   class BASE_EXPORT Iterator {
    public:
     explicit Iterator(const DictionaryValue& target);
 
     // DEPRECATED: use !IsAtEnd() instead.
     bool HasNext() const { return it_ != target_.dictionary_.end(); }
 
     bool IsAtEnd() const { return it_ == target_.dictionary_.end(); }
@@ skipping 151 matching lines @@
 
 }  // namespace base
 
 // http://crbug.com/88666
 using base::DictionaryValue;
 using base::ListValue;
 using base::StringValue;
 using base::Value;
 
 #endif  // BASE_VALUES_H_