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

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

Issue 2388973002: [Sync] Removing duplicate includes, part 2. (Closed)
Patch Set: Update for Max's comments. Created 4 years, 2 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 <stddef.h>
8 #include <stdint.h>
9
10 #include "base/macros.h"
11 #include "base/memory/ptr_util.h" 7 #include "base/memory/ptr_util.h"
12 #include "base/message_loop/message_loop.h" 8 #include "base/message_loop/message_loop.h"
13 #include "base/run_loop.h" 9 #include "base/run_loop.h"
14 #include "base/strings/string_number_conversions.h" 10 #include "base/strings/string_number_conversions.h"
15 #include "base/strings/string_util.h" 11 #include "base/strings/string_util.h"
16 #include "base/strings/stringprintf.h" 12 #include "base/strings/stringprintf.h"
17 #include "base/time/time.h" 13 #include "base/time/time.h"
18 #include "components/sync/api/attachments/attachment_id.h" 14 #include "components/sync/api/attachments/attachment_id.h"
19 #include "components/sync/api/sync_change_processor_wrapper_for_test.h" 15 #include "components/sync/api/sync_change_processor_wrapper_for_test.h"
20 #include "components/sync/api/sync_error_factory_mock.h" 16 #include "components/sync/api/sync_error_factory_mock.h"
(...skipping 1908 matching lines...) Expand 10 before | Expand all | Expand 10 after
1929 EXPECT_EQ(changes[4].change_type(), syncer::SyncChange::ACTION_ADD); 1925 EXPECT_EQ(changes[4].change_type(), syncer::SyncChange::ACTION_ADD);
1930 EXPECT_EQ(changes[4].sync_data().GetDataType(), syncer::FAVICON_TRACKING); 1926 EXPECT_EQ(changes[4].sync_data().GetDataType(), syncer::FAVICON_TRACKING);
1931 EXPECT_EQ(kMaxSyncFavicons, GetFaviconId(changes[4])); 1927 EXPECT_EQ(kMaxSyncFavicons, GetFaviconId(changes[4]));
1932 // Expire tracking for favicon[0]. 1928 // Expire tracking for favicon[0].
1933 EXPECT_EQ(changes[5].change_type(), syncer::SyncChange::ACTION_DELETE); 1929 EXPECT_EQ(changes[5].change_type(), syncer::SyncChange::ACTION_DELETE);
1934 EXPECT_EQ(changes[5].sync_data().GetDataType(), syncer::FAVICON_TRACKING); 1930 EXPECT_EQ(changes[5].sync_data().GetDataType(), syncer::FAVICON_TRACKING);
1935 EXPECT_EQ(0, GetFaviconId(changes[5])); 1931 EXPECT_EQ(0, GetFaviconId(changes[5]));
1936 } 1932 }
1937 1933
1938 } // namespace sync_sessions 1934 } // namespace sync_sessions
OLDNEW
« no previous file with comments | « components/sync/syncable/parent_child_index_unittest.cc ('k') | components/sync_sessions/lost_navigations_recorder.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698