Handling default service in the web intents picker controller.

chrome/browser/ui/intents/web_intent_picker_model.h

Index: chrome/browser/ui/intents/web_intent_picker_model.h
diff --git a/chrome/browser/ui/intents/web_intent_picker_model.h b/chrome/browser/ui/intents/web_intent_picker_model.h
index fda5ef3ee634a5f6e08ee598613ef8862429b4ed..1d5a207e52ca5ba3dcf99edeb12b81387748faa1 100644
--- a/chrome/browser/ui/intents/web_intent_picker_model.h
+++ b/chrome/browser/ui/intents/web_intent_picker_model.h
@@ -83,6 +83,12 @@ class WebIntentPickerModel {
 
   const string16& mimetype() { return mimetype_; }
 
+  void set_default_service_url(const std::string& default_url) {
+    default_service_url_ = default_url;

[groby-ooo-7-16, 2012/05/01 15:28:33]

See previous comment on keeping an InstalledService* instead.

+  }
+
+  const std::string& default_service_url() { return default_service_url_; }
+
   // Add a new installed service with |title|, |url| and |disposition| to the
   // picker.
   void AddInstalledService(const string16& title,
@@ -164,6 +170,10 @@ class WebIntentPickerModel {
   // A cached copy of the mimetype that instantiated the picker.
   string16 mimetype_;
 
+  // The non-empty url of the default service if the WebIntentsRegistry
+  // finds a default service matching the intent being dispatched.
+  std::string default_service_url_;

[groby-ooo-7-16, 2012/05/01 15:28:33]

If it's an URL, shouldn't it be a GURL, not std::string?

[Greg Billock, 2012/05/01 19:17:54]

Done.

+
   DISALLOW_COPY_AND_ASSIGN(WebIntentPickerModel);
 };