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

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

Issue 2461463002: [Sync] Replacing NULL with nullptr/null throughout sync code. (Closed)
Patch Set: Reverted PROFILE_nullptr mistake. Created 4 years, 1 month 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/metrics/histogram_macros.h" 10 #include "base/metrics/histogram_macros.h"
(...skipping 40 matching lines...) Expand 10 before | Expand all | Expand 10 after
51 DISALLOW_COPY_AND_ASSIGN(SyncedFaviconInfo); 51 DISALLOW_COPY_AND_ASSIGN(SyncedFaviconInfo);
52 }; 52 };
53 53
54 // Information for handling local favicon updates. Used in 54 // Information for handling local favicon updates. Used in
55 // OnFaviconDataAvailable. 55 // OnFaviconDataAvailable.
56 struct LocalFaviconUpdateInfo { 56 struct LocalFaviconUpdateInfo {
57 LocalFaviconUpdateInfo() 57 LocalFaviconUpdateInfo()
58 : new_image(false), 58 : new_image(false),
59 new_tracking(false), 59 new_tracking(false),
60 image_needs_rewrite(false), 60 image_needs_rewrite(false),
61 favicon_info(NULL) {} 61 favicon_info(nullptr) {}
62 62
63 bool new_image; 63 bool new_image;
64 bool new_tracking; 64 bool new_tracking;
65 bool image_needs_rewrite; 65 bool image_needs_rewrite;
66 SyncedFaviconInfo* favicon_info; 66 SyncedFaviconInfo* favicon_info;
67 }; 67 };
68 68
69 namespace { 69 namespace {
70 70
71 // Maximum number of favicons to keep in memory (0 means no limit). 71 // Maximum number of favicons to keep in memory (0 means no limit).
(...skipping 967 matching lines...) Expand 10 before | Expand all | Expand 10 after
1039 favicon_images_sync_processor_->ProcessSyncChanges(FROM_HERE, 1039 favicon_images_sync_processor_->ProcessSyncChanges(FROM_HERE,
1040 image_deletions); 1040 image_deletions);
1041 } 1041 }
1042 if (favicon_tracking_sync_processor_.get()) { 1042 if (favicon_tracking_sync_processor_.get()) {
1043 favicon_tracking_sync_processor_->ProcessSyncChanges(FROM_HERE, 1043 favicon_tracking_sync_processor_->ProcessSyncChanges(FROM_HERE,
1044 tracking_deletions); 1044 tracking_deletions);
1045 } 1045 }
1046 } 1046 }
1047 1047
1048 } // namespace sync_sessions 1048 } // namespace sync_sessions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698