Change picker model API to allow less notification messages

chrome/browser/ui/intents/web_intent_picker_model.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.
 
 #ifndef CHROME_BROWSER_UI_INTENTS_WEB_INTENT_PICKER_MODEL_H_
 #define CHROME_BROWSER_UI_INTENTS_WEB_INTENT_PICKER_MODEL_H_
 
 #include <vector>
 
 #include "base/basictypes.h"
@@ skipping 90 matching lines @@
 
   // Return the intent service that uses |url| as its service url, or NULL.
   const InstalledService* GetInstalledServiceWithURL(const GURL& url) const;
 
   // Return the number of intent services in the picker.
   size_t GetInstalledServiceCount() const;
 
   // Update the favicon for the intent service at |index| to |image|.
   void UpdateFaviconAt(size_t index, const gfx::Image& image);
 
-  // Add a new suggested extension with |id|, |title| and |average_rating| to
-  // the picker.
-  void AddSuggestedExtension(const string16& title,
-                             const string16& id,
-                             double average_rating);
-
-  // Remove a suggested extension from the picker at |index|.
-  void RemoveSuggestedExtensionAt(size_t index);
+  // Add a list of suggested extensions to the model.
+  void AddSuggestedExtensions(
+      const std::vector<SuggestedExtension>& suggestions);
 
   // Return the suggested extension at |index|.
   const SuggestedExtension& GetSuggestedExtensionAt(size_t index) const;
 
   // Return the number of suggested extensions to be displayed.
   size_t GetSuggestedExtensionCount() const;
 
   // Return the text to use in the "Get more suggestions" link. Returns UTF8.
   // Will return an empty string if the link should not be shown.
   string16 GetSuggestionsLinkText() const;
@@ skipping 14 matching lines @@
 
  private:
   // Delete all elements in |installed_services_| and |suggested_extensions_|.
   // Note that this method does not reset the observer.
   void DestroyAll();
 
   // A vector of all installed services in the picker. Each installed service
   // is owned by this model.
   std::vector<InstalledService*> installed_services_;
 
-  // A vector of all suggested extensions in the picker. Each element is owned
-  // by this model.
-  std::vector<SuggestedExtension*> suggested_extensions_;
+  // A vector of all suggested extensions in the picker.
+  std::vector<SuggestedExtension> suggested_extensions_;
 
   // The observer to send notifications to, or NULL if none. Not owned.
   WebIntentPickerModelObserver* observer_;
 
   // The url of the intent service that is being displayed inline, or
   // GURL::EmptyGURL() if none.
   GURL inline_disposition_url_;
 
   // A cached copy of the action that instantiated the picker.
   string16 action_;
 
   // A cached copy of the type that instantiated the picker.
   string16 type_;
 
   // The non-empty url of the default service if the WebIntentsRegistry
   // finds a default service matching the intent being dispatched.
   GURL default_service_url_;
 
   DISALLOW_COPY_AND_ASSIGN(WebIntentPickerModel);
 };
 
 #endif  // CHROME_BROWSER_UI_INTENTS_WEB_INTENT_PICKER_MODEL_H_