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

Unified Diff: chrome_frame/ready_mode/internal/registry_ready_mode_state.cc

Issue 8013001: Remove the \servers build output directory and place Chrome Frame parts into the root output fold... (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: '' Created 9 years, 3 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
Index: chrome_frame/ready_mode/internal/registry_ready_mode_state.cc
===================================================================
--- chrome_frame/ready_mode/internal/registry_ready_mode_state.cc (revision 102100)
+++ chrome_frame/ready_mode/internal/registry_ready_mode_state.cc (working copy)
@@ -59,10 +59,9 @@
HANDLE LaunchCommandViaProcessLauncher(const std::wstring& command_field) {
HANDLE launched_process = NULL;
- scoped_ptr<CommandLine> command_line(
- chrome_launcher::CreateUpdateCommandLine(command_field));
-
- if (command_line != NULL) {
+ scoped_ptr<CommandLine> command_line;
+ if (chrome_launcher::CreateUpdateCommandLine(command_field, &command_line)) {
+ DCHECK(command_line != NULL);
base::LaunchOptions options;
options.start_hidden = true;
base::LaunchProcess(*command_line, options, &launched_process);

Powered by Google App Engine
This is Rietveld 408576698