Use AttachmentUploadImpl instead of FakeAttachmentUploader.

chrome/browser/sync/profile_sync_service_unittest.cc

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 ddc9a9e44ff604c99e72e8bd3c5438afeb72468c..0e2a86efb718a489269854edb82ccdc42e53cf76 100644
--- a/chrome/browser/sync/profile_sync_service_unittest.cc
+++ b/chrome/browser/sync/profile_sync_service_unittest.cc
@@ -196,7 +196,7 @@ class ProfileSyncServiceTest : public ::testing::Test {
     service_.reset(new ProfileSyncService(
         components_factory_,
         profile_,
-        new ManagedUserSigninManagerWrapper(profile_, signin),
+        make_scoped_ptr(new ManagedUserSigninManagerWrapper(profile_, signin)),
         oauth2_token_service,
         behavior));
     service_->SetClearingBrowseringDataForTesting(
@@ -585,5 +585,21 @@ 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");
+  EXPECT_EQ("https://foo/bar",
+            ProfileSyncService::GetSyncServiceURL(command_line).spec());
+}
+
 }  // namespace
 }  // namespace browser_sync