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

Unified Diff: chrome/installer/setup/app_launcher_installer.h

Issue 1982653002: Add best-effort/allow rollback flags on WorkItem. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix for official builds Created 4 years, 7 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 | « chrome/chrome_installer.gypi ('k') | chrome/installer/setup/app_launcher_installer.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/installer/setup/app_launcher_installer.h
diff --git a/chrome/installer/setup/app_launcher_installer.h b/chrome/installer/setup/app_launcher_installer.h
index 132005e072e779e14bbf11d45e6b89a91a776816..262195af8a393121f2fddca8eca184894534b11e 100644
--- a/chrome/installer/setup/app_launcher_installer.h
+++ b/chrome/installer/setup/app_launcher_installer.h
@@ -11,13 +11,10 @@
#ifndef CHROME_INSTALLER_SETUP_APP_LAUNCHER_INSTALLER_H_
#define CHROME_INSTALLER_SETUP_APP_LAUNCHER_INSTALLER_H_
-#if defined(GOOGLE_CHROME_BUILD)
grt (UTC plus 2) 2016/05/16 18:55:01 why remove this?
fdoray 2016/05/17 17:18:48 To catch build errors before code reaches internal
-
#include <windows.h>
namespace base {
class FilePath;
-class Version;
} // namespace base
class WorkItemList;
@@ -26,15 +23,6 @@ namespace installer {
class InstallerState;
-// Create Version key for a App Launcher (if not already present) and sets the
-// new product version as the last step. We need this for App Launcher in
-// order for shadow registry key to work.
-void AddAppLauncherVersionKeyWorkItems(
grt (UTC plus 2) 2016/05/16 18:55:01 why mix the WI refactor with removing this code?
fdoray 2016/05/17 17:18:48 created a separate CL https://codereview.chromium.
- HKEY root,
- const base::Version& new_version,
- bool add_language_identifier,
- WorkItemList* list);
-
// Remove App Launcher's registry key, so it is in sync with Google Chrome's.
// Note: The key is added by App Launcher in SetDidRunForNDayActiveStats().
void RemoveAppLauncherVersionKey(HKEY reg_root);
@@ -44,15 +32,6 @@ void AddRemoveLegacyAppHostExeWorkItems(const base::FilePath& target_path,
const base::FilePath& temp_path,
WorkItemList* list);
-// Adds work item to unconditionally remove legacy app commands like
-// "install-application", "install-extension", and
-// "quick-enable-application-host".
-void AddRemoveLegacyAppCommandsWorkItems(
- const InstallerState& installer_state,
- WorkItemList* work_item_list);
-
} // namespace installer
-#endif // defined(GOOGLE_CHROME_BUILD)
-
#endif // CHROME_INSTALLER_SETUP_APP_LAUNCHER_INSTALLER_H_
« no previous file with comments | « chrome/chrome_installer.gypi ('k') | chrome/installer/setup/app_launcher_installer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698