Chromium Code Reviews| Index: chrome/browser/ui/startup/startup_browser_creator_win.cc |
| =================================================================== |
| --- chrome/browser/ui/startup/startup_browser_creator_win.cc (revision 208964) |
| +++ chrome/browser/ui/startup/startup_browser_creator_win.cc (working copy) |
| @@ -6,9 +6,7 @@ |
| #include "base/logging.h" |
| #include "base/win/metro.h" |
| -#include "chrome/browser/search_engines/template_url.h" |
| -#include "chrome/browser/search_engines/template_url_service.h" |
| -#include "chrome/browser/search_engines/template_url_service_factory.h" |
| +#include "chrome/browser/search_engines/util.h" |
| #include "chrome/browser/ui/browser_finder.h" |
| #include "chrome/browser/ui/browser_window.h" |
| #include "chrome/browser/ui/startup/startup_browser_creator_impl.h" |
| @@ -21,22 +19,11 @@ |
| string16 params; |
|
Jered
2013/06/28 21:57:28
"params" sounds confusingly plural.
Peter Kasting
2013/06/28 22:16:02
It can be plural, AFAICT, in that we can have mult
|
| base::win::MetroLaunchType launch_type = |
| base::win::GetMetroLaunchParams(¶ms); |
| - |
| if ((launch_type == base::win::METRO_PROTOCOL) || |
| - (launch_type == base::win::METRO_LAUNCH)) { |
| + (launch_type == base::win::METRO_LAUNCH)) |
| return GURL(params); |
| - } else if (launch_type == base::win::METRO_SEARCH) { |
| - const TemplateURL* default_provider = |
| - TemplateURLServiceFactory::GetForProfile(profile)-> |
| - GetDefaultSearchProvider(); |
| - if (default_provider) { |
| - const TemplateURLRef& search_url = default_provider->url_ref(); |
| - DCHECK(search_url.SupportsReplacement()); |
| - return GURL(search_url.ReplaceSearchTerms( |
| - TemplateURLRef::SearchTermsArgs(params))); |
| - } |
| - } |
| - return GURL(); |
| + return (launch_type == base::win::METRO_SEARCH) ? |
| + GetDefaultSearchURLForSearchTerms(profile, params) : GURL(); |
| } |
| } // namespace chrome |