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

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

Issue 13666003: [Sync] Fix favicon updates to handle orphan nodes (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address comments 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 104 matching lines...) Expand 10 before | Expand all | Expand 10 after
115 // its own wait time logic for nudge. 115 // its own wait time logic for nudge.
116 CUSTOM, 116 CUSTOM,
117 }; 117 };
118 118
119 static NudgeDelayStrategy GetNudgeDelayStrategy(const ModelType& type) { 119 static NudgeDelayStrategy GetNudgeDelayStrategy(const ModelType& type) {
120 switch (type) { 120 switch (type) {
121 case AUTOFILL: 121 case AUTOFILL:
122 return ACCOMPANY_ONLY; 122 return ACCOMPANY_ONLY;
123 case PREFERENCES: 123 case PREFERENCES:
124 case SESSIONS: 124 case SESSIONS:
125 case FAVICON_IMAGES:
126 case FAVICON_TRACKING:
125 return CUSTOM; 127 return CUSTOM;
126 default: 128 default:
127 return IMMEDIATE; 129 return IMMEDIATE;
128 } 130 }
129 } 131 }
130 132
131 static TimeDelta GetNudgeDelayTimeDeltaFromType( 133 static TimeDelta GetNudgeDelayTimeDeltaFromType(
132 const NudgeDelayStrategy& delay_type, const ModelType& model_type, 134 const NudgeDelayStrategy& delay_type, const ModelType& model_type,
133 const SyncManagerImpl* core) { 135 const SyncManagerImpl* core) {
134 CHECK(core); 136 CHECK(core);
135 TimeDelta delay = TimeDelta::FromMilliseconds( 137 TimeDelta delay = TimeDelta::FromMilliseconds(
136 kDefaultNudgeDelayMilliseconds); 138 kDefaultNudgeDelayMilliseconds);
137 switch (delay_type) { 139 switch (delay_type) {
138 case IMMEDIATE: 140 case IMMEDIATE:
139 delay = TimeDelta::FromMilliseconds( 141 delay = TimeDelta::FromMilliseconds(
140 kDefaultNudgeDelayMilliseconds); 142 kDefaultNudgeDelayMilliseconds);
141 break; 143 break;
142 case ACCOMPANY_ONLY: 144 case ACCOMPANY_ONLY:
143 delay = TimeDelta::FromSeconds(kDefaultShortPollIntervalSeconds); 145 delay = TimeDelta::FromSeconds(kDefaultShortPollIntervalSeconds);
144 break; 146 break;
145 case CUSTOM: 147 case CUSTOM:
146 switch (model_type) { 148 switch (model_type) {
147 case PREFERENCES: 149 case PREFERENCES:
148 delay = TimeDelta::FromMilliseconds( 150 delay = TimeDelta::FromMilliseconds(
149 kPreferencesNudgeDelayMilliseconds); 151 kPreferencesNudgeDelayMilliseconds);
150 break; 152 break;
151 case SESSIONS: 153 case SESSIONS:
154 case FAVICON_IMAGES:
155 case FAVICON_TRACKING:
152 delay = core->scheduler()->GetSessionsCommitDelay(); 156 delay = core->scheduler()->GetSessionsCommitDelay();
153 break; 157 break;
154 default: 158 default:
155 NOTREACHED(); 159 NOTREACHED();
156 } 160 }
157 break; 161 break;
158 default: 162 default:
159 NOTREACHED(); 163 NOTREACHED();
160 } 164 }
161 return delay; 165 return delay;
(...skipping 1234 matching lines...) Expand 10 before | Expand all | Expand 10 after
1396 int SyncManagerImpl::GetDefaultNudgeDelay() { 1400 int SyncManagerImpl::GetDefaultNudgeDelay() {
1397 return kDefaultNudgeDelayMilliseconds; 1401 return kDefaultNudgeDelayMilliseconds;
1398 } 1402 }
1399 1403
1400 // static. 1404 // static.
1401 int SyncManagerImpl::GetPreferencesNudgeDelay() { 1405 int SyncManagerImpl::GetPreferencesNudgeDelay() {
1402 return kPreferencesNudgeDelayMilliseconds; 1406 return kPreferencesNudgeDelayMilliseconds;
1403 } 1407 }
1404 1408
1405 } // namespace syncer 1409 } // 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