summaryrefslogtreecommitdiffstats
path: root/apps/app_host/binaries_installer.cc
diff options
context:
space:
mode:
Diffstat (limited to 'apps/app_host/binaries_installer.cc')
-rw-r--r--apps/app_host/binaries_installer.cc23
1 files changed, 15 insertions, 8 deletions
diff --git a/apps/app_host/binaries_installer.cc b/apps/app_host/binaries_installer.cc
index ce72888..90a9b2b 100644
--- a/apps/app_host/binaries_installer.cc
+++ b/apps/app_host/binaries_installer.cc
@@ -177,11 +177,18 @@ HRESULT CreateAppBundle(IGoogleUpdate3* update3, IAppBundle** app_bundle) {
}
HRESULT SelectBinariesApValue(IGoogleUpdate3* update3,
- IAppBundle* app_bundle,
BSTR* ap_value) {
- HRESULT hr = GetAppHostApValue(update3, app_bundle, ap_value);
- if (SUCCEEDED(hr))
- return hr;
+ // TODO(erikwright): Uncomment this when we correctly propagate the AP value
+ // from the system-level binaries when quick-enabling the app host at
+ // user-level (http://crbug.com/178479).
+ // base::win::ScopedComPtr<IAppBundle> app_bundle;
+ // HRESULT hr = CreateAppBundle(update3, app_bundle.Receive());
+ // if (FAILED(hr))
+ // return hr;
+
+ // hr = GetAppHostApValue(update3, app_bundle, ap_value);
+ // if (SUCCEEDED(hr))
+ // return hr;
// TODO(erikwright): distinguish between AppHost not installed and an
// error in GetAppHostApValue.
@@ -294,13 +301,13 @@ HRESULT InstallBinaries() {
if (FAILED(hr))
return hr;
- base::win::ScopedComPtr<IAppBundle> app_bundle;
- hr = CreateAppBundle(update3, app_bundle.Receive());
+ base::win::ScopedBstr ap_value;
+ hr = SelectBinariesApValue(update3, ap_value.Receive());
if (FAILED(hr))
return hr;
- base::win::ScopedBstr ap_value;
- hr = SelectBinariesApValue(update3, app_bundle, ap_value.Receive());
+ base::win::ScopedComPtr<IAppBundle> app_bundle;
+ hr = CreateAppBundle(update3, app_bundle.Receive());
if (FAILED(hr))
return hr;