Revert 51595 - Incorporating feedback from C++ readability review....

chrome/test/live_sync/live_sync_test.cc

 // Copyright (c) 2010 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 "chrome/test/live_sync/live_sync_test.h"
 
 #include "chrome/browser/profile.h"
 #include "chrome/browser/profile_manager.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_switches.h"
@@ skipping 30 matching lines @@
 void LiveSyncTest::TearDown() {
   // Allow the InProcessBrowserTest framework to perform its tear down.
   InProcessBrowserTest::TearDown();
 
   // Stop the local test sync server if one was used.
   if (started_local_test_server_)
     TearDownLocalTestServer();
 }
 
 // static
-Profile* LiveSyncTest::MakeProfile(const FilePath::StringType name) {
+Profile* LiveSyncTest::MakeProfile(
+    const FilePath::StringType name) {
   FilePath path;
   PathService::Get(chrome::DIR_USER_DATA, &path);
   return ProfileManager::CreateProfile(path.Append(name));
 }
 
 Profile* LiveSyncTest::GetProfile(int index) {
   EXPECT_FALSE(profiles_.empty()) << "SetupSync() has not yet been called.";
   EXPECT_TRUE((index >= 0) && (index < static_cast<int>(profiles_.size())))
       << "GetProfile(): Index is out of bounds.";
   return profiles_[index];
@@ skipping 85 matching lines @@
   CommandLine* cl = CommandLine::ForCurrentProcess();
   cl->AppendSwitchWithValue(switches::kSyncServiceURL,
       StringPrintf("http://%s:%d/chromiumsync", server_.kHostName,
           server_.kOKHTTPSPort));
 }
 
 void LiveSyncTest::TearDownLocalTestServer() {
   bool success = server_.Stop();
   ASSERT_TRUE(success);
 }