Add a constructor that takes only the "interesting" args (basically the args...

Add a constructor that takes only the "interesting" args (basically the args that aren't predictably defaulted). This will allow the cleanup a lot of tests and probably some rel code as well.


Review URL: https://chromiumcodereview.appspot.com/10855065

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@151150 0039d316-1c4b-4281-b951-d872f2087c98

chrome/browser/intents/default_web_intent_service.cc

@@ -9,6 +9,14 @@ DefaultWebIntentService::DefaultWebIntentService()
     user_date(-1),
     suppression(0) {}
 
+DefaultWebIntentService::DefaultWebIntentService(
+    const string16& srv_action,
+    const string16& srv_type,
+    const std::string& srv_service_url)
+    : action(srv_action), type(srv_type),
+      url_pattern(URLPattern::SCHEME_ALL, URLPattern::kAllUrlsPattern),
+      user_date(-1), suppression(0), service_url(srv_service_url) {}
+
 DefaultWebIntentService::~DefaultWebIntentService() {}
 
 bool DefaultWebIntentService::operator==(

chrome/browser/intents/default_web_intent_service.h

@@ -22,6 +22,7 @@
 struct DefaultWebIntentService {
   string16 action;
   string16 type;
+
   URLPattern url_pattern;
 
   // |user_date| holds the offset time when a user set the default.
@@ -35,6 +36,10 @@ struct DefaultWebIntentService {
   std::string service_url;
 
   DefaultWebIntentService();
+  DefaultWebIntentService(
+      const string16& srv_action,
+      const string16& srv_type,
+      const std::string& srv_service_url);
   ~DefaultWebIntentService();
 
   bool operator==(const DefaultWebIntentService& other) const;