Fix DCHECK, memory leak, and refactor PasswordStore to use CancelableRequest

base/memory/scoped_vector.h

Index: base/memory/scoped_vector.h
diff --git a/base/memory/scoped_vector.h b/base/memory/scoped_vector.h
index aec43757cd123c14a009b7201b4eff14974e2786..6e0cf05594687a4f45223186b81d8c064e35a20d 100644
--- a/base/memory/scoped_vector.h
+++ b/base/memory/scoped_vector.h
@@ -62,6 +62,12 @@ class ScopedVector {
     return v.insert(position, x);
   }
 
+  // Lets the ScopedVector take ownership of elements in [first,last).
+  template<typename InputIterator>
+  void insert(iterator position, InputIterator first, InputIterator last) {
+    v.insert(position, first, last);
+  }
+
   iterator erase(iterator position) {
     delete *position;
     return v.erase(position);