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

Unified Diff: chrome/browser/sync/sessions/sessions_sync_manager_unittest.cc

Issue 1220963005: Update base::StartsWith calls to new form (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@starts_with
Patch Set: Created 5 years, 6 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/sessions/sessions_sync_manager_unittest.cc
diff --git a/chrome/browser/sync/sessions/sessions_sync_manager_unittest.cc b/chrome/browser/sync/sessions/sessions_sync_manager_unittest.cc
index e011d0d15c085f300c459ee20e2291a976845819..e8240e9ca57db791381fe2dd72a4f8de8ded6297 100644
--- a/chrome/browser/sync/sessions/sessions_sync_manager_unittest.cc
+++ b/chrome/browser/sync/sessions/sessions_sync_manager_unittest.cc
@@ -1045,8 +1045,9 @@ TEST_F(SessionsSyncManagerTest, MergeWithLocalAndForeignTabs) {
for (int i = 1; i < 3; i++) {
EXPECT_TRUE(output[i].IsValid());
const SyncData data(output[i].sync_data());
- EXPECT_TRUE(base::StartsWithASCII(syncer::SyncDataLocal(data).GetTag(),
- manager()->current_machine_tag(), true));
+ EXPECT_TRUE(base::StartsWith(syncer::SyncDataLocal(data).GetTag(),
+ manager()->current_machine_tag(),
+ base::CompareCase::SENSITIVE));
const sync_pb::SessionSpecifics& specifics(data.GetSpecifics().session());
EXPECT_EQ(manager()->current_machine_tag(), specifics.session_tag());
}
@@ -1479,15 +1480,17 @@ TEST_F(SessionsSyncManagerTest, AssociateWindowsDontReloadTabs) {
EXPECT_EQ(3U, out.size()); // Tab add, update, and header update.
EXPECT_TRUE(
- base::StartsWithASCII(syncer::SyncDataLocal(out[0].sync_data()).GetTag(),
- manager()->current_machine_tag(), true));
+ base::StartsWith(syncer::SyncDataLocal(out[0].sync_data()).GetTag(),
+ manager()->current_machine_tag(),
+ base::CompareCase::SENSITIVE));
EXPECT_EQ(manager()->current_machine_tag(),
out[0].sync_data().GetSpecifics().session().session_tag());
EXPECT_EQ(SyncChange::ACTION_ADD, out[0].change_type());
EXPECT_TRUE(
- base::StartsWithASCII(syncer::SyncDataLocal(out[1].sync_data()).GetTag(),
- manager()->current_machine_tag(), true));
+ base::StartsWith(syncer::SyncDataLocal(out[1].sync_data()).GetTag(),
+ manager()->current_machine_tag(),
+ base::CompareCase::SENSITIVE));
EXPECT_EQ(manager()->current_machine_tag(),
out[1].sync_data().GetSpecifics().session().session_tag());
EXPECT_TRUE(out[1].sync_data().GetSpecifics().session().has_tab());
@@ -1541,8 +1544,9 @@ TEST_F(SessionsSyncManagerTest, OnLocalTabModified) {
SCOPED_TRACE(i);
EXPECT_TRUE(out[i].IsValid());
const SyncData data(out[i].sync_data());
- EXPECT_TRUE(base::StartsWithASCII(syncer::SyncDataLocal(data).GetTag(),
- manager()->current_machine_tag(), true));
+ EXPECT_TRUE(base::StartsWith(syncer::SyncDataLocal(data).GetTag(),
+ manager()->current_machine_tag(),
+ base::CompareCase::SENSITIVE));
const sync_pb::SessionSpecifics& specifics(data.GetSpecifics().session());
EXPECT_EQ(manager()->current_machine_tag(), specifics.session_tag());
if (i % 6 == 0) {
@@ -1649,8 +1653,9 @@ TEST_F(SessionsSyncManagerTest, MergeLocalSessionExistingTabs) {
for (int i = 1; i < 5; i++) {
EXPECT_TRUE(out[i].IsValid());
const SyncData data(out[i].sync_data());
- EXPECT_TRUE(base::StartsWithASCII(syncer::SyncDataLocal(data).GetTag(),
- manager()->current_machine_tag(), true));
+ EXPECT_TRUE(base::StartsWith(syncer::SyncDataLocal(data).GetTag(),
+ manager()->current_machine_tag(),
+ base::CompareCase::SENSITIVE));
const sync_pb::SessionSpecifics& specifics(data.GetSpecifics().session());
EXPECT_EQ(manager()->current_machine_tag(), specifics.session_tag());
if (i % 2 == 1) {

Powered by Google App Engine
This is Rietveld 408576698