| Index: chrome/browser/ui/webui/sync_promo/sync_promo_ui.cc
|
| diff --git a/chrome/browser/ui/webui/sync_promo/sync_promo_ui.cc b/chrome/browser/ui/webui/sync_promo/sync_promo_ui.cc
|
| index 659d0e7bf8ea89a1bdf8e1cf169f3630cb11c343..8a8d2297f6f0ff020d9685a2792f4b883d5ceb6b 100644
|
| --- a/chrome/browser/ui/webui/sync_promo/sync_promo_ui.cc
|
| +++ b/chrome/browser/ui/webui/sync_promo/sync_promo_ui.cc
|
| @@ -173,11 +173,8 @@ void SyncPromoUI::RegisterUserPrefs(PrefService* prefs) {
|
|
|
| // static
|
| bool SyncPromoUI::ShouldShowSyncPromoAtStartup(Profile* profile,
|
| - bool is_new_profile,
|
| - bool* promo_suppressed) {
|
| + bool is_new_profile) {
|
| DCHECK(profile);
|
| - DCHECK(promo_suppressed);
|
| - *promo_suppressed = false;
|
|
|
| if (!ShouldShowSyncPromo(profile))
|
| return false;
|
| @@ -203,19 +200,6 @@ bool SyncPromoUI::ShouldShowSyncPromoAtStartup(Profile* profile,
|
| if (show_count >= kSyncPromoShowAtStartupMaximum)
|
| return false;
|
|
|
| - // If the current install is part of trial then let the trial determine if we
|
| - // should show the promo or not.
|
| - switch (sync_promo_trial::GetStartupOverrideForCurrentTrial()) {
|
| - case sync_promo_trial::STARTUP_OVERRIDE_NONE:
|
| - // No override so simply continue.
|
| - break;
|
| - case sync_promo_trial::STARTUP_OVERRIDE_SHOW:
|
| - return true;
|
| - case sync_promo_trial::STARTUP_OVERRIDE_HIDE:
|
| - *promo_suppressed = true;
|
| - return false;
|
| - }
|
| -
|
| // This pref can be set in the master preferences file to allow or disallow
|
| // showing the sync promo at startup.
|
| if (prefs->HasPrefPath(prefs::kSyncPromoShowOnFirstRunAllowed))
|
| @@ -298,25 +282,3 @@ std::string SyncPromoUI::GetSourceForSyncPromoURL(const GURL& url) {
|
| return GetValueForKeyInQuery(url, kSyncPromoQueryKeySource, &value) ?
|
| value : std::string();
|
| }
|
| -
|
| -// static
|
| -SyncPromoUI::Version SyncPromoUI::GetSyncPromoVersion() {
|
| - Version version;
|
| - if (sync_promo_trial::GetSyncPromoVersionForCurrentTrial(&version)) {
|
| - // Currently the sync promo dialog has two problems. First, it's not modal
|
| - // so the user can interact with other browser windows. Second, it uses
|
| - // a nested message loop that can cause the sync promo page not to render.
|
| - // To work around these problems the sync promo dialog is only shown for
|
| - // the first profile. TODO(sail): Fix these issues if the sync promo dialog
|
| - // is more widely deployed.
|
| - ProfileInfoCache& cache =
|
| - g_browser_process->profile_manager()->GetProfileInfoCache();
|
| - if (cache.GetNumberOfProfiles() > 1 &&
|
| - version == SyncPromoUI::VERSION_DIALOG) {
|
| - return SyncPromoUI::VERSION_SIMPLE;
|
| - }
|
| - return version;
|
| - }
|
| -
|
| - return VERSION_SIMPLE;
|
| -}
|
|
|