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

Unified Diff: chrome/installer/setup/chrome_frame_quick_enable.cc

Issue 10912096: C++ reability review for gab. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix compile errors Created 8 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
« no previous file with comments | « no previous file | chrome/installer/setup/chrome_frame_ready_mode.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/installer/setup/chrome_frame_quick_enable.cc
diff --git a/chrome/installer/setup/chrome_frame_quick_enable.cc b/chrome/installer/setup/chrome_frame_quick_enable.cc
index f78038e217804e08fc2b7edab27aa4ed8b54762a..11eb578e9472cee185b1cade0e4149dfa6b03d91 100644
--- a/chrome/installer/setup/chrome_frame_quick_enable.cc
+++ b/chrome/installer/setup/chrome_frame_quick_enable.cc
@@ -103,7 +103,7 @@ InstallStatus ChromeFrameQuickEnable(const InstallationState& machine_state,
// This creates the uninstallation entry for GCF.
AddUninstallShortcutWorkItems(*installer_state, setup_path, new_version,
- item_list.get(), *cf);
+ *cf, item_list.get());
// Always set the "lang" value since quick-enable always happens in the
// context of an interactive session with a user.
AddVersionKeyWorkItems(installer_state->root_key(), cf->distribution(),
@@ -125,8 +125,8 @@ InstallStatus ChromeFrameQuickEnable(const InstallationState& machine_state,
// Add the items to remove the quick-enable-cf command from the registry.
AddQuickEnableChromeFrameWorkItems(
*installer_state, machine_state,
- &chrome_state->uninstall_command().GetProgram(),
- &chrome_state->version(),
+ chrome_state->uninstall_command().GetProgram(),
+ new_version,
item_list.get());
if (!item_list->Do()) {
« no previous file with comments | « no previous file | chrome/installer/setup/chrome_frame_ready_mode.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698