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

Side by Side Diff: components/policy/core/common/schema_registry.cc

Issue 2481233004: Rename SchemaRegistry::SetReady to SetDomainReady (Closed)
Patch Set: Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "components/policy/core/common/schema_registry.h" 5 #include "components/policy/core/common/schema_registry.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 8
9 namespace policy { 9 namespace policy {
10 10
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
59 } 59 }
60 60
61 bool SchemaRegistry::IsReady() const { 61 bool SchemaRegistry::IsReady() const {
62 for (int i = 0; i < POLICY_DOMAIN_SIZE; ++i) { 62 for (int i = 0; i < POLICY_DOMAIN_SIZE; ++i) {
63 if (!domains_ready_[i]) 63 if (!domains_ready_[i])
64 return false; 64 return false;
65 } 65 }
66 return true; 66 return true;
67 } 67 }
68 68
69 void SchemaRegistry::SetReady(PolicyDomain domain) { 69 void SchemaRegistry::SetDomainReady(PolicyDomain domain) {
70 if (domains_ready_[domain]) 70 if (domains_ready_[domain])
71 return; 71 return;
72 domains_ready_[domain] = true; 72 domains_ready_[domain] = true;
73 if (IsReady()) { 73 if (IsReady()) {
74 for (auto& observer : observers_) 74 for (auto& observer : observers_)
75 observer.OnSchemaRegistryReady(); 75 observer.OnSchemaRegistryReady();
76 } 76 }
77 } 77 }
78 78
79 void SchemaRegistry::SetAllDomainsReady() { 79 void SchemaRegistry::SetAllDomainsReady() {
80 for (int i = 0; i < POLICY_DOMAIN_SIZE; ++i) 80 for (int i = 0; i < POLICY_DOMAIN_SIZE; ++i)
81 SetReady(static_cast<PolicyDomain>(i)); 81 SetDomainReady(static_cast<PolicyDomain>(i));
82 } 82 }
83 83
84 void SchemaRegistry::SetExtensionsDomainsReady() { 84 void SchemaRegistry::SetExtensionsDomainsReady() {
85 SetReady(POLICY_DOMAIN_EXTENSIONS); 85 SetDomainReady(POLICY_DOMAIN_EXTENSIONS);
86 SetReady(POLICY_DOMAIN_SIGNIN_EXTENSIONS); 86 SetDomainReady(POLICY_DOMAIN_SIGNIN_EXTENSIONS);
87 } 87 }
88 88
89 void SchemaRegistry::AddObserver(Observer* observer) { 89 void SchemaRegistry::AddObserver(Observer* observer) {
90 observers_.AddObserver(observer); 90 observers_.AddObserver(observer);
91 } 91 }
92 92
93 void SchemaRegistry::RemoveObserver(Observer* observer) { 93 void SchemaRegistry::RemoveObserver(Observer* observer) {
94 observers_.RemoveObserver(observer); 94 observers_.RemoveObserver(observer);
95 } 95 }
96 96
(...skipping 136 matching lines...) Expand 10 before | Expand all | Expand 10 after
233 void ForwardingSchemaRegistry::OnSchemaRegistryShuttingDown( 233 void ForwardingSchemaRegistry::OnSchemaRegistryShuttingDown(
234 SchemaRegistry* registry) { 234 SchemaRegistry* registry) {
235 DCHECK_EQ(wrapped_, registry); 235 DCHECK_EQ(wrapped_, registry);
236 wrapped_->RemoveObserver(this); 236 wrapped_->RemoveObserver(this);
237 wrapped_->RemoveInternalObserver(this); 237 wrapped_->RemoveInternalObserver(this);
238 wrapped_ = NULL; 238 wrapped_ = NULL;
239 // Keep serving the same |schema_map_|. 239 // Keep serving the same |schema_map_|.
240 } 240 }
241 241
242 } // namespace policy 242 } // namespace policy
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698