Handling default service in the web intents picker controller.

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_
 #pragma once
 
 #include <vector>
 
@@ skipping 65 matching lines @@
   }
 
   void set_action(const string16& action) { action_ = action; }
 
   const string16& action() { return action_; }
 
   void set_mimetype(const string16& mimetype) { mimetype_ = mimetype; }
 
   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,
                            const GURL& url,
                            Disposition disposition);
 
   // Remove an installed service from the picker at |index|.
   void RemoveInstalledServiceAt(size_t index);
 
   // Remove all installed services from the picker, and resets to not
@@ skipping 61 matching lines @@
   // 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 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);
 };
 
 #endif  // CHROME_BROWSER_UI_INTENTS_WEB_INTENT_PICKER_MODEL_H_