Chromium Code Reviews| Index: chrome/browser/extensions/default_apps.cc |
| diff --git a/chrome/browser/extensions/default_apps.cc b/chrome/browser/extensions/default_apps.cc |
| index 2cc32289435e48d1243285e0de259e8ff594ec24..cf34173c068122d4210b41b71972785d4c255809 100644 |
| --- a/chrome/browser/extensions/default_apps.cc |
| +++ b/chrome/browser/extensions/default_apps.cc |
| @@ -14,10 +14,10 @@ |
| #include "chrome/browser/first_run/first_run.h" |
| #include "chrome/browser/profiles/profile.h" |
| #include "chrome/common/chrome_switches.h" |
| -#include "chrome/common/chrome_version_info.h" |
| #include "chrome/common/extensions/extension_constants.h" |
| #include "chrome/common/pref_names.h" |
| #include "components/pref_registry/pref_registry_syncable.h" |
| +#include "components/version_info/version_info.h" |
| #include "extensions/common/extension.h" |
| namespace { |
| @@ -76,9 +76,8 @@ bool Provider::ShouldInstallInProfile() { |
| // Only new installations and profiles get default apps. In theory the |
| // new profile checks should catch new installations, but that is not |
| // always the case (http:/crbug.com/145351). |
| - chrome::VersionInfo version_info; |
| - bool is_new_profile = |
| - profile_->WasCreatedByVersionOrLater(version_info.Version().c_str()); |
| + bool is_new_profile = profile_->WasCreatedByVersionOrLater( |
| + version_info::GetVersionNumber().c_str()); |
|
brettw
2015/07/31 21:13:16
Can you remove c_str() here? This just causes an u
|
| bool is_first_run = first_run::IsChromeFirstRun(); |
| if (!is_first_run && !is_new_profile) |
| install_apps = false; |