Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(194)

Side by Side Diff: chrome/browser/sync/profile_sync_service.cc

Issue 265843009: Restored an option to enable enhanced bookmarks experiment from Finch (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebased Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « chrome/browser/extensions/external_component_loader.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/sync/profile_sync_service.h" 5 #include "chrome/browser/sync/profile_sync_service.h"
6 6
7 #include <cstddef> 7 #include <cstddef>
8 #include <map> 8 #include <map>
9 #include <set> 9 #include <set>
10 #include <utility> 10 #include <utility>
(...skipping 1052 matching lines...) Expand 10 before | Expand all | Expand 10 after
1063 sync_driver::prefs::kEnhancedBookmarksExperimentEnabled); 1063 sync_driver::prefs::kEnhancedBookmarksExperimentEnabled);
1064 profile_->GetPrefs()->ClearPref( 1064 profile_->GetPrefs()->ClearPref(
1065 sync_driver::prefs::kEnhancedBookmarksExtensionId); 1065 sync_driver::prefs::kEnhancedBookmarksExtensionId);
1066 } 1066 }
1067 } 1067 }
1068 BookmarksExperimentState bookmarks_experiment_state = 1068 BookmarksExperimentState bookmarks_experiment_state =
1069 static_cast<BookmarksExperimentState>(profile_->GetPrefs()->GetInteger( 1069 static_cast<BookmarksExperimentState>(profile_->GetPrefs()->GetInteger(
1070 sync_driver::prefs::kEnhancedBookmarksExperimentEnabled)); 1070 sync_driver::prefs::kEnhancedBookmarksExperimentEnabled));
1071 // If bookmark experiment state was changed update about flags experiment. 1071 // If bookmark experiment state was changed update about flags experiment.
1072 if (bookmarks_experiment_state_before != bookmarks_experiment_state) { 1072 if (bookmarks_experiment_state_before != bookmarks_experiment_state) {
1073 UpdateBookmarksExperiment(g_browser_process->local_state(), 1073 ForceFinchBoomarkExperimentIfNeeded(g_browser_process->local_state(),
rlarocque 2014/05/09 21:43:56 s/Boomark/Bookmark ?
1074 bookmarks_experiment_state); 1074 bookmarks_experiment_state);
1075 } 1075 }
1076 1076
1077 // If this is a first time sync for a client, this will be called before 1077 // If this is a first time sync for a client, this will be called before
1078 // OnBackendInitialized() to ensure the new datatypes are available at sync 1078 // OnBackendInitialized() to ensure the new datatypes are available at sync
1079 // setup. As a result, the migrator won't exist yet. This is fine because for 1079 // setup. As a result, the migrator won't exist yet. This is fine because for
1080 // first time sync cases we're only concerned with making the datatype 1080 // first time sync cases we're only concerned with making the datatype
1081 // available. 1081 // available.
1082 if (migrator_.get() && 1082 if (migrator_.get() &&
1083 migrator_->state() != browser_sync::BackendMigrator::IDLE) { 1083 migrator_->state() != browser_sync::BackendMigrator::IDLE) {
1084 DVLOG(1) << "Dropping OnExperimentsChanged due to migrator busy."; 1084 DVLOG(1) << "Dropping OnExperimentsChanged due to migrator busy.";
(...skipping 1289 matching lines...) Expand 10 before | Expand all | Expand 10 after
2374 status.last_get_token_error = last_get_token_error_; 2374 status.last_get_token_error = last_get_token_error_;
2375 if (request_access_token_retry_timer_.IsRunning()) 2375 if (request_access_token_retry_timer_.IsRunning())
2376 status.next_token_request_time = next_token_request_time_; 2376 status.next_token_request_time = next_token_request_time_;
2377 return status; 2377 return status;
2378 } 2378 }
2379 2379
2380 void ProfileSyncService::OverrideNetworkResourcesForTest( 2380 void ProfileSyncService::OverrideNetworkResourcesForTest(
2381 scoped_ptr<syncer::NetworkResources> network_resources) { 2381 scoped_ptr<syncer::NetworkResources> network_resources) {
2382 network_resources_ = network_resources.Pass(); 2382 network_resources_ = network_resources.Pass();
2383 } 2383 }
OLDNEW
« no previous file with comments | « chrome/browser/extensions/external_component_loader.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698