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

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

Issue 343673003: Ignore current command line in ProfileSyncServiceTest. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Use FILE_PATH_LITERAL. Created 6 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/sync/profile_sync_service_unittest.cc
diff --git a/chrome/browser/sync/profile_sync_service_unittest.cc b/chrome/browser/sync/profile_sync_service_unittest.cc
index 0e2a86efb718a489269854edb82ccdc42e53cf76..bd6150a98f9d783baa408983673ce94fab11d96e 100644
--- a/chrome/browser/sync/profile_sync_service_unittest.cc
+++ b/chrome/browser/sync/profile_sync_service_unittest.cc
@@ -586,17 +586,10 @@ TEST_F(ProfileSyncServiceTest, RollbackThenBackup) {
#endif
TEST_F(ProfileSyncServiceTest, GetSyncServiceURL) {
- CommandLine command_line(*CommandLine::ForCurrentProcess());
-
- // See that it defaults to a "dev" URL.
- //
- // Yes, we're hardcoding the URL here so this test will have to be updated
- // when/if the URL ever changes.
- EXPECT_EQ("https://clients4.google.com/chrome-sync/dev",
- ProfileSyncService::GetSyncServiceURL(command_line).spec());
-
// See that we can override the URL with a flag.
- command_line.AppendSwitchASCII("--sync-url", "https://foo/bar");
+ CommandLine command_line(
+ base::FilePath(base::FilePath(FILE_PATH_LITERAL("chrome.exe"))));
+ command_line.AppendSwitchASCII(switches::kSyncServiceURL, "https://foo/bar");
EXPECT_EQ("https://foo/bar",
ProfileSyncService::GetSyncServiceURL(command_line).spec());
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698