Make an AddAll in ListModel.

ui/base/models/list_model.h

Index: ui/base/models/list_model.h
diff --git a/ui/base/models/list_model.h b/ui/base/models/list_model.h
index 9b4440d9a020d2db41d8bfb7adade37fe2ec173e..3b8459c3ce5bc71993463ef201df32e40434c823 100644
--- a/ui/base/models/list_model.h
+++ b/ui/base/models/list_model.h
@@ -34,6 +34,19 @@ class ListModel {
     AddAt(item_count(), item);
   }
 
+  // Add a vector of items to the end of the model. This triggers one
+  // notification after adding all items.
+  void AddAll(ScopedVector<ItemType> new_items) {
+    size_t count = item_count();

[xiyuan, 2013/08/06 03:42:07]

nit: const size_t

[calamity, 2013/08/07 07:19:40]

Done.

+    std::vector<ItemType*> new_items_released;
+    new_items.release(&new_items_released);
+    items_.insert(items_.end(),
+                  new_items_released.begin(),
+                  new_items_released.end());

[xiyuan, 2013/08/06 03:42:07]

How about getting rid of |new_items_released| and do:

items_.insert(items_.end(), new_items.begin(), new_items.end());
items_.weak_clear();

[calamity, 2013/08/07 07:19:40]

Done.

+    NotifyItemsAdded(count, new_items_released.size());
+  }
+
+

[xiyuan, 2013/08/06 03:42:07]

nit: nuke one empty line

[calamity, 2013/08/07 07:19:40]

Done.

   // Removes an item at given |index| from the model. Note the removed item
   // is NOT deleted and it's up to the caller to delete it.
   ItemType* RemoveAt(size_t index) {