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

Unified Diff: chrome/browser/sync/profile_sync_service_session_unittest.cc

Issue 10125002: [Sync] Add per-navigation timestamps/unique ids to tab sync. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Address comments Created 8 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/sync/profile_sync_service_session_unittest.cc
diff --git a/chrome/browser/sync/profile_sync_service_session_unittest.cc b/chrome/browser/sync/profile_sync_service_session_unittest.cc
index 2991b0dd002b874176b057292ae01c307fccb062..fcb0ea7c5c5c6d2ff763684bb9876f9656c6519f 100644
--- a/chrome/browser/sync/profile_sync_service_session_unittest.cc
+++ b/chrome/browser/sync/profile_sync_service_session_unittest.cc
@@ -97,7 +97,6 @@ void BuildTabSpecifics(const std::string& tag, int window_id, int tab_id,
tab->set_pinned(true);
tab->set_extension_app_id("app_id");
sync_pb::TabNavigation* navigation = tab->add_navigation();
- navigation->set_index(12);
navigation->set_virtual_url("http://foo/1");
navigation->set_referrer("referrer");
navigation->set_title("title");
@@ -140,7 +139,6 @@ void VerifySyncedSession(
ASSERT_TRUE(tab->pinned);
ASSERT_EQ("app_id", tab->extension_app_id);
ASSERT_EQ(1U, tab->navigations.size());
- ASSERT_EQ(12, tab->navigations[0].index());
ASSERT_EQ(tab->navigations[0].virtual_url(), GURL("http://foo/1"));
ASSERT_EQ(tab->navigations[0].referrer().url, GURL("referrer"));
ASSERT_EQ(tab->navigations[0].title(), string16(ASCIIToUTF16("title")));
« no previous file with comments | « chrome/browser/sync/glue/synced_session_tracker_unittest.cc ('k') | sync/protocol/proto_value_conversions.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698