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

Side by Side Diff: components/sync_sessions/favicon_cache.cc

Issue 2343463003: [Sync] Fix namespaces for the sync_sessions component. (Closed)
Patch Set: Fix gn. Created 4 years, 3 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
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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 "components/sync_sessions/favicon_cache.h" 5 #include "components/sync_sessions/favicon_cache.h"
6 6
7 #include <utility> 7 #include <utility>
8 8
9 #include "base/location.h" 9 #include "base/location.h"
10 #include "base/macros.h" 10 #include "base/macros.h"
11 #include "base/metrics/histogram_macros.h" 11 #include "base/metrics/histogram_macros.h"
12 #include "base/single_thread_task_runner.h" 12 #include "base/single_thread_task_runner.h"
13 #include "base/threading/thread_task_runner_handle.h" 13 #include "base/threading/thread_task_runner_handle.h"
14 #include "components/favicon/core/favicon_service.h" 14 #include "components/favicon/core/favicon_service.h"
15 #include "components/history/core/browser/history_service.h" 15 #include "components/history/core/browser/history_service.h"
16 #include "components/history/core/browser/history_types.h" 16 #include "components/history/core/browser/history_types.h"
17 #include "components/sync/api/time.h" 17 #include "components/sync/api/time.h"
18 #include "components/sync/protocol/favicon_image_specifics.pb.h" 18 #include "components/sync/protocol/favicon_image_specifics.pb.h"
19 #include "components/sync/protocol/favicon_tracking_specifics.pb.h" 19 #include "components/sync/protocol/favicon_tracking_specifics.pb.h"
20 #include "components/sync/protocol/sync.pb.h" 20 #include "components/sync/protocol/sync.pb.h"
21 #include "ui/gfx/favicon_size.h" 21 #include "ui/gfx/favicon_size.h"
22 22
23 namespace browser_sync { 23 namespace sync_sessions {
24 24
25 // Synced favicon storage and tracking. 25 // Synced favicon storage and tracking.
26 // Note: we don't use the favicon service for storing these because these 26 // Note: we don't use the favicon service for storing these because these
27 // favicons are not necessarily associated with any local navigation, and 27 // favicons are not necessarily associated with any local navigation, and
28 // hence would not work with the current expiration logic. We have custom 28 // hence would not work with the current expiration logic. We have custom
29 // expiration logic based on visit time/bookmark status/etc. 29 // expiration logic based on visit time/bookmark status/etc.
30 // See crbug.com/122890. 30 // See crbug.com/122890.
31 struct SyncedFaviconInfo { 31 struct SyncedFaviconInfo {
32 explicit SyncedFaviconInfo(const GURL& favicon_url) 32 explicit SyncedFaviconInfo(const GURL& favicon_url)
33 : favicon_url(favicon_url), 33 : favicon_url(favicon_url),
(...skipping 1006 matching lines...) Expand 10 before | Expand all | Expand 10 after
1040 if (favicon_images_sync_processor_.get()) { 1040 if (favicon_images_sync_processor_.get()) {
1041 favicon_images_sync_processor_->ProcessSyncChanges(FROM_HERE, 1041 favicon_images_sync_processor_->ProcessSyncChanges(FROM_HERE,
1042 image_deletions); 1042 image_deletions);
1043 } 1043 }
1044 if (favicon_tracking_sync_processor_.get()) { 1044 if (favicon_tracking_sync_processor_.get()) {
1045 favicon_tracking_sync_processor_->ProcessSyncChanges(FROM_HERE, 1045 favicon_tracking_sync_processor_->ProcessSyncChanges(FROM_HERE,
1046 tracking_deletions); 1046 tracking_deletions);
1047 } 1047 }
1048 } 1048 }
1049 1049
1050 } // namespace browser_sync 1050 } // namespace sync_sessions
OLDNEW
« no previous file with comments | « components/sync_sessions/favicon_cache.h ('k') | components/sync_sessions/favicon_cache_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698