Use AttachmentUploadImpl instead of FakeAttachmentUploader.

chrome/browser/sync/profile_sync_service_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "base/basictypes.h"
 #include "base/command_line.h"
 #include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/run_loop.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 178 matching lines @@
   void CreateService(ProfileSyncServiceStartBehavior behavior) {
     SigninManagerBase* signin =
         SigninManagerFactory::GetForProfile(profile_);
     signin->SetAuthenticatedUsername("test");
     ProfileOAuth2TokenService* oauth2_token_service =
         ProfileOAuth2TokenServiceFactory::GetForProfile(profile_);
     components_factory_ = new StrictMock<ProfileSyncComponentsFactoryMock>();
     service_.reset(new ProfileSyncService(
         components_factory_,
         profile_,
-        new ManagedUserSigninManagerWrapper(profile_, signin),
+        make_scoped_ptr(new ManagedUserSigninManagerWrapper(profile_, signin)),
         oauth2_token_service,
         behavior));
     service_->SetClearingBrowseringDataForTesting(
         base::Bind(&ProfileSyncServiceTest::ClearBrowsingDataCallback,
                    base::Unretained(this)));
   }
 
 #if defined(OS_WIN) || defined(OS_MACOSX) || (defined(OS_LINUX) && !defined(OS_CHROMEOS))
   void CreateServiceWithoutSignIn() {
     CreateService(browser_sync::MANUAL_START);
@@ skipping 368 matching lines @@
   // First sync time is erased once backup starts.
   EXPECT_TRUE(sync_prefs.GetFirstSyncTime().is_null());
 
   EXPECT_EQ(3u, delete_dir_param.size());
   EXPECT_FALSE(delete_dir_param[0]);
   EXPECT_FALSE(delete_dir_param[1]);
   EXPECT_TRUE(delete_dir_param[2]);
 }
 #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