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

Unified Diff: trunk/src/chrome/browser/chromeos/policy/device_local_account_policy_service.h

Issue 343103005: Revert 278518 "Wire up component cloud policy to device local ac..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 6 years, 6 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: trunk/src/chrome/browser/chromeos/policy/device_local_account_policy_service.h
===================================================================
--- trunk/src/chrome/browser/chromeos/policy/device_local_account_policy_service.h (revision 278641)
+++ trunk/src/chrome/browser/chromeos/policy/device_local_account_policy_service.h (working copy)
@@ -11,7 +11,6 @@
#include "base/basictypes.h"
#include "base/compiler_specific.h"
-#include "base/files/file_path.h"
#include "base/memory/ref_counted.h"
#include "base/memory/scoped_ptr.h"
#include "base/memory/weak_ptr.h"
@@ -49,7 +48,6 @@
// |task_runner| is the runner for policy refresh tasks.
DeviceLocalAccountPolicyBroker(
const DeviceLocalAccount& account,
- const base::FilePath& component_policy_cache_path,
scoped_ptr<DeviceLocalAccountPolicyStore> store,
scoped_refptr<DeviceLocalAccountExternalDataManager>
external_data_manager,
@@ -88,15 +86,9 @@
// empty string if the policy is not present.
std::string GetDisplayName() const;
- // Returns a directory where component policy for this account can be cached.
- // The DeviceLocalAccountPolicyService takes care of cleaning up caches of
- // accounts that have been removed.
- base::FilePath GetComponentPolicyCachePath() const;
-
private:
const std::string account_id_;
const std::string user_id_;
- const base::FilePath component_policy_cache_path_;
const scoped_ptr<DeviceLocalAccountPolicyStore> store_;
scoped_refptr<DeviceLocalAccountExternalDataManager> external_data_manager_;
scoped_refptr<chromeos::DeviceLocalAccountExternalPolicyLoader>
@@ -122,9 +114,6 @@
// The list of accounts has been updated.
virtual void OnDeviceLocalAccountsChanged() = 0;
-
- // The given |broker| is about to be destroyed.
- virtual void OnBrokerShutdown(DeviceLocalAccountPolicyBroker* broker) {}
};
DeviceLocalAccountPolicyService(
@@ -153,8 +142,6 @@
// |user_id|.
bool IsPolicyAvailableForUser(const std::string& user_id);
- scoped_refptr<net::URLRequestContextGetter> request_context() const;
-
void AddObserver(Observer* observer);
void RemoveObserver(Observer* observer);
@@ -242,10 +229,6 @@
const scoped_ptr<chromeos::CrosSettings::ObserverSubscription>
local_accounts_subscription_;
- // Path to the directory that contains the cached policies for components
- // for device local accounts.
- base::FilePath component_policy_cache_root_;
-
base::WeakPtrFactory<DeviceLocalAccountPolicyService> weak_factory_;
DISALLOW_COPY_AND_ASSIGN(DeviceLocalAccountPolicyService);

Powered by Google App Engine
This is Rietveld 408576698