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

Unified Diff: chrome/browser/chromeos/app_mode/kiosk_app_manager.cc

Issue 491403003: Update cached kiosk app crx from usb stick. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove the refcounted implementation KioskExternalUpdater. Created 6 years, 4 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/browser/chromeos/app_mode/kiosk_app_manager.cc
diff --git a/chrome/browser/chromeos/app_mode/kiosk_app_manager.cc b/chrome/browser/chromeos/app_mode/kiosk_app_manager.cc
index e4e7b0e53247c742d6bb9c586895afe3d0e80042..2921d50576561ab9eacf1cb24f81b1bff23e4775 100644
--- a/chrome/browser/chromeos/app_mode/kiosk_app_manager.cc
+++ b/chrome/browser/chromeos/app_mode/kiosk_app_manager.cc
@@ -9,6 +9,7 @@
#include "base/bind.h"
#include "base/files/file_path.h"
+#include "base/files/file_util.h"
#include "base/logging.h"
#include "base/path_service.h"
#include "base/prefs/pref_registry_simple.h"
@@ -20,6 +21,7 @@
#include "chrome/browser/chromeos/app_mode/kiosk_app_data.h"
#include "chrome/browser/chromeos/app_mode/kiosk_app_external_loader.h"
#include "chrome/browser/chromeos/app_mode/kiosk_app_manager_observer.h"
+#include "chrome/browser/chromeos/app_mode/kiosk_external_updater.h"
#include "chrome/browser/chromeos/policy/browser_policy_connector_chromeos.h"
#include "chrome/browser/chromeos/policy/device_local_account.h"
#include "chrome/browser/chromeos/settings/cros_settings.h"
@@ -74,6 +76,7 @@ const char KioskAppManager::kKeyApps[] = "apps";
const char KioskAppManager::kKeyAutoLoginState[] = "auto_login_state";
const char KioskAppManager::kIconCacheDir[] = "kiosk/icon";
const char KioskAppManager::kCrxCacheDir[] = "kiosk/crx";
+const char KioskAppManager::kCrxUnpackDir[] = "kiosk_unpack";
// static
static base::LazyInstance<KioskAppManager> instance = LAZY_INSTANCE_INITIALIZER;
@@ -370,6 +373,12 @@ bool KioskAppManager::HasCachedCrx(const std::string& app_id) const {
return GetCachedCrx(app_id, &crx_path, &version);
}
+bool KioskAppManager::GetCachedCrx(const std::string& app_id,
+ base::FilePath* file_path,
+ std::string* version) const {
+ return external_cache_->GetExtension(app_id, file_path, version);
+}
+
void KioskAppManager::AddObserver(KioskAppManagerObserver* observer) {
observers_.AddObserver(observer);
}
@@ -407,6 +416,11 @@ void KioskAppManager::UpdateExternalCache() {
UpdateAppData();
}
+void KioskAppManager::OnKioskAppCacheUpdated(const std::string& app_id) {
+ FOR_EACH_OBSERVER(
+ KioskAppManagerObserver, observers_, OnKioskAppCacheUpdated(app_id));
+}
+
KioskAppManager::KioskAppManager()
: ownership_established_(false), external_loader_created_(false) {
base::FilePath cache_dir;
@@ -431,10 +445,20 @@ KioskAppManager::KioskAppManager()
KioskAppManager::~KioskAppManager() {}
+void KioskAppManager::MonitorKioskExternalUpdate() {
+ base::FilePath cache_dir;
+ GetCrxCacheDir(&cache_dir);
+ base::FilePath unpack_dir;
+ GetCrxUnpackDir(&unpack_dir);
+ usb_stick_updater_.reset(new KioskExternalUpdater(
+ GetBackgroundTaskRunner(), cache_dir, unpack_dir));
+}
+
void KioskAppManager::CleanUp() {
local_accounts_subscription_.reset();
local_account_auto_login_id_subscription_.reset();
apps_.clear();
+ usb_stick_updater_.reset();
external_cache_.reset();
}
@@ -586,10 +610,10 @@ void KioskAppManager::GetCrxCacheDir(base::FilePath* cache_dir) {
*cache_dir = user_data_dir.AppendASCII(kCrxCacheDir);
}
-bool KioskAppManager::GetCachedCrx(const std::string& app_id,
- base::FilePath* file_path,
- std::string* version) const {
- return external_cache_->GetExtension(app_id, file_path, version);
+void KioskAppManager::GetCrxUnpackDir(base::FilePath* unpack_dir) {
+ base::FilePath temp_dir;
+ base::GetTempDir(&temp_dir);
+ *unpack_dir = temp_dir.AppendASCII(kCrxUnpackDir);
}
} // namespace chromeos

Powered by Google App Engine
This is Rietveld 408576698