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

Side by Side Diff: sync/internal_api/sync_manager_impl.cc

Issue 13947023: Merge 192503 "[Sync] Fix favicon updates to handle orphan nodes" (Closed) Base URL: svn://svn.chromium.org/chrome/branches/1453/src/
Patch Set: Created 7 years, 8 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/sync/glue/favicon_cache_unittest.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 "sync/internal_api/sync_manager_impl.h" 5 #include "sync/internal_api/sync_manager_impl.h"
6 6
7 #include <string> 7 #include <string>
8 8
9 #include "base/base64.h" 9 #include "base/base64.h"
10 #include "base/bind.h" 10 #include "base/bind.h"
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
114 // its own wait time logic for nudge. 114 // its own wait time logic for nudge.
115 CUSTOM, 115 CUSTOM,
116 }; 116 };
117 117
118 static NudgeDelayStrategy GetNudgeDelayStrategy(const ModelType& type) { 118 static NudgeDelayStrategy GetNudgeDelayStrategy(const ModelType& type) {
119 switch (type) { 119 switch (type) {
120 case AUTOFILL: 120 case AUTOFILL:
121 return ACCOMPANY_ONLY; 121 return ACCOMPANY_ONLY;
122 case PREFERENCES: 122 case PREFERENCES:
123 case SESSIONS: 123 case SESSIONS:
124 case FAVICON_IMAGES:
125 case FAVICON_TRACKING:
124 return CUSTOM; 126 return CUSTOM;
125 default: 127 default:
126 return IMMEDIATE; 128 return IMMEDIATE;
127 } 129 }
128 } 130 }
129 131
130 static TimeDelta GetNudgeDelayTimeDeltaFromType( 132 static TimeDelta GetNudgeDelayTimeDeltaFromType(
131 const NudgeDelayStrategy& delay_type, const ModelType& model_type, 133 const NudgeDelayStrategy& delay_type, const ModelType& model_type,
132 const SyncManagerImpl* core) { 134 const SyncManagerImpl* core) {
133 CHECK(core); 135 CHECK(core);
134 TimeDelta delay = TimeDelta::FromMilliseconds( 136 TimeDelta delay = TimeDelta::FromMilliseconds(
135 kDefaultNudgeDelayMilliseconds); 137 kDefaultNudgeDelayMilliseconds);
136 switch (delay_type) { 138 switch (delay_type) {
137 case IMMEDIATE: 139 case IMMEDIATE:
138 delay = TimeDelta::FromMilliseconds( 140 delay = TimeDelta::FromMilliseconds(
139 kDefaultNudgeDelayMilliseconds); 141 kDefaultNudgeDelayMilliseconds);
140 break; 142 break;
141 case ACCOMPANY_ONLY: 143 case ACCOMPANY_ONLY:
142 delay = TimeDelta::FromSeconds(kDefaultShortPollIntervalSeconds); 144 delay = TimeDelta::FromSeconds(kDefaultShortPollIntervalSeconds);
143 break; 145 break;
144 case CUSTOM: 146 case CUSTOM:
145 switch (model_type) { 147 switch (model_type) {
146 case PREFERENCES: 148 case PREFERENCES:
147 delay = TimeDelta::FromMilliseconds( 149 delay = TimeDelta::FromMilliseconds(
148 kPreferencesNudgeDelayMilliseconds); 150 kPreferencesNudgeDelayMilliseconds);
149 break; 151 break;
150 case SESSIONS: 152 case SESSIONS:
153 case FAVICON_IMAGES:
154 case FAVICON_TRACKING:
151 delay = core->scheduler()->GetSessionsCommitDelay(); 155 delay = core->scheduler()->GetSessionsCommitDelay();
152 break; 156 break;
153 default: 157 default:
154 NOTREACHED(); 158 NOTREACHED();
155 } 159 }
156 break; 160 break;
157 default: 161 default:
158 NOTREACHED(); 162 NOTREACHED();
159 } 163 }
160 return delay; 164 return delay;
(...skipping 1241 matching lines...) Expand 10 before | Expand all | Expand 10 after
1402 int SyncManagerImpl::GetDefaultNudgeDelay() { 1406 int SyncManagerImpl::GetDefaultNudgeDelay() {
1403 return kDefaultNudgeDelayMilliseconds; 1407 return kDefaultNudgeDelayMilliseconds;
1404 } 1408 }
1405 1409
1406 // static. 1410 // static.
1407 int SyncManagerImpl::GetPreferencesNudgeDelay() { 1411 int SyncManagerImpl::GetPreferencesNudgeDelay() {
1408 return kPreferencesNudgeDelayMilliseconds; 1412 return kPreferencesNudgeDelayMilliseconds;
1409 } 1413 }
1410 1414
1411 } // namespace syncer 1415 } // namespace syncer
OLDNEW
« no previous file with comments | « chrome/browser/sync/glue/favicon_cache_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698