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

Unified Diff: chrome/browser/chromeos/login/ownership_service.cc

Issue 8727037: Signed settings refactoring: Proper caching and more tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased to ToT and removed some debug output left. Created 9 years, 1 month 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/login/ownership_service.cc
diff --git a/chrome/browser/chromeos/login/ownership_service.cc b/chrome/browser/chromeos/login/ownership_service.cc
index ffdcf571ed728889d3c23613075cdd89b7334e6a..184452ff93eab1eacbe5509e9902f9aa9f33865f 100644
--- a/chrome/browser/chromeos/login/ownership_service.cc
+++ b/chrome/browser/chromeos/login/ownership_service.cc
@@ -30,7 +30,6 @@ OwnershipService* OwnershipService::GetSharedInstance() {
OwnershipService::OwnershipService()
: manager_(new OwnerManager),
utils_(OwnerKeyUtils::Create()),
- policy_(NULL),
ownership_status_(OWNERSHIP_UNKNOWN) {
notification_registrar_.Add(
this,
@@ -57,19 +56,6 @@ void OwnershipService::Prewarm() {
}
}
-void OwnershipService::set_cached_policy(const em::PolicyData& pol) {
- policy_.reset(pol.New());
- policy_->CheckTypeAndMergeFrom(pol);
-}
-
-bool OwnershipService::has_cached_policy() {
- return policy_.get();
-}
-
-const em::PolicyData& OwnershipService::cached_policy() {
- return *(policy_.get());
-}
-
bool OwnershipService::IsAlreadyOwned() {
return file_util::PathExists(utils_->GetOwnerKeyFilePath());
}

Powered by Google App Engine
This is Rietveld 408576698