Rename CommandLine::GetCommandLineString().

chrome/browser/web_applications/web_app.cc

 // Copyright (c) 2011 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/browser/web_applications/web_app.h"
 
 #if defined(OS_WIN)
 #include <shlobj.h>
 #endif  // defined(OS_WIN)
 
@@ skipping 312 matching lines @@
     NOTREACHED();
     return false;
   }
 
   // Working directory.
   FilePath chrome_folder = chrome_exe.DirName();
 
   CommandLine cmd_line =
      ShellIntegration::CommandLineArgsForLauncher(shortcut_info_.url,
                                                   shortcut_info_.extension_id);
-  // TODO(evan): we rely on the fact that command_line_string() is
-  // properly quoted for a Windows command line.  The method on
-  // CommandLine should probably be renamed to better reflect that
-  // fact.

[Evan Martin, 2011/07/15 21:05:16]

This TODO still exists, then.

[msw, 2011/07/18 20:59:33]

Done.

-  std::wstring wide_switches(cmd_line.command_line_string());
+  std::wstring wide_switches(cmd_line.GetCommandLineString());
 
   // Sanitize description
   if (shortcut_info_.description.length() >= MAX_PATH)
     shortcut_info_.description.resize(MAX_PATH - 1);
 
   // Generates app id from web app url and profile path.
   std::string app_name =
       web_app::GenerateApplicationNameFromInfo(shortcut_info_);
   std::wstring app_id = ShellIntegration::GetAppId(
       UTF8ToWide(app_name), profile_path_);
@@ skipping 182 matching lines @@
 
 #if defined(TOOLKIT_USES_GTK)
 std::string GetWMClassFromAppName(std::string app_name) {
   file_util::ReplaceIllegalCharactersInPath(&app_name, '_');
   TrimString(app_name, "_", &app_name);
   return app_name;
 }
 #endif
 
 }  // namespace web_app