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

Unified Diff: chrome/browser/sync/internal_api/sync_manager.cc

Issue 7669073: [Sync] Add support for enabling session sync remotely. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 9 years, 4 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/sync/internal_api/sync_manager.cc
diff --git a/chrome/browser/sync/internal_api/sync_manager.cc b/chrome/browser/sync/internal_api/sync_manager.cc
index c0b9cb35ff7e9c075139b36b90fff849a036970b..c931eb77fc77a628283e8f14144d58b333b26a3c 100644
--- a/chrome/browser/sync/internal_api/sync_manager.cc
+++ b/chrome/browser/sync/internal_api/sync_manager.cc
@@ -1968,10 +1968,19 @@ void SyncManager::RefreshEncryption() {
}
syncable::ModelTypeSet SyncManager::GetEncryptedDataTypes() const {
- sync_api::ReadTransaction trans(FROM_HERE, GetUserShare());
+ ReadTransaction trans(FROM_HERE, GetUserShare());
return GetEncryptedTypes(&trans);
}
+bool SyncManager::ShouldEnableSessions() const {
+ ReadTransaction trans(FROM_HERE, GetUserShare());
+ ReadNode node(&trans);
+ if (!node.InitByTagLookup(kNigoriTag)) {
+ return false;
tim (not reviewing) 2011/08/22 15:42:08 maybe a VLOG(SyncManager::ShouldEnableSessions can
Nicolas Zea 2011/08/22 22:50:15 Done.
+ }
+ return node.GetNigoriSpecifics().sync_tabs();
+}
+
bool SyncManager::HasUnsyncedItems() const {
sync_api::ReadTransaction trans(FROM_HERE, GetUserShare());
return (trans.GetWrappedTrans()->directory()->unsynced_entity_count() != 0);

Powered by Google App Engine
This is Rietveld 408576698