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

Side by Side Diff: components/policy/core/common/cloud/policy_builder.h

Issue 1056633004: Update {virtual,override} to follow C++11 style in components. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix logo tracker unittest. Created 5 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #ifndef COMPONENTS_POLICY_CORE_COMMON_CLOUD_POLICY_BUILDER_H_ 5 #ifndef COMPONENTS_POLICY_CORE_COMMON_CLOUD_POLICY_BUILDER_H_
6 #define COMPONENTS_POLICY_CORE_COMMON_CLOUD_POLICY_BUILDER_H_ 6 #define COMPONENTS_POLICY_CORE_COMMON_CLOUD_POLICY_BUILDER_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
114 114
115 DISALLOW_COPY_AND_ASSIGN(PolicyBuilder); 115 DISALLOW_COPY_AND_ASSIGN(PolicyBuilder);
116 }; 116 };
117 117
118 // Type-parameterized PolicyBuilder extension that allows for building policy 118 // Type-parameterized PolicyBuilder extension that allows for building policy
119 // blobs carrying protobuf payloads. 119 // blobs carrying protobuf payloads.
120 template<typename PayloadProto> 120 template<typename PayloadProto>
121 class TypedPolicyBuilder : public PolicyBuilder { 121 class TypedPolicyBuilder : public PolicyBuilder {
122 public: 122 public:
123 TypedPolicyBuilder(); 123 TypedPolicyBuilder();
124 virtual ~TypedPolicyBuilder() {} 124 ~TypedPolicyBuilder() override {}
125 125
126 // Returns a reference to the payload protobuf being built. 126 // Returns a reference to the payload protobuf being built.
127 PayloadProto& payload() { 127 PayloadProto& payload() {
128 if (!payload_.get()) 128 if (!payload_.get())
129 payload_.reset(new PayloadProto()); 129 payload_.reset(new PayloadProto());
130 return *payload_; 130 return *payload_;
131 } 131 }
132 void clear_payload() { 132 void clear_payload() {
133 payload_.reset(); 133 payload_.reset();
134 } 134 }
135 135
136 // PolicyBuilder: 136 // PolicyBuilder:
137 virtual void Build() override { 137 void Build() override {
138 if (payload_.get()) 138 if (payload_.get())
139 CHECK(payload_->SerializeToString(policy_data().mutable_policy_value())); 139 CHECK(payload_->SerializeToString(policy_data().mutable_policy_value()));
140 140
141 PolicyBuilder::Build(); 141 PolicyBuilder::Build();
142 } 142 }
143 143
144 private: 144 private:
145 scoped_ptr<PayloadProto> payload_; 145 scoped_ptr<PayloadProto> payload_;
146 146
147 DISALLOW_COPY_AND_ASSIGN(TypedPolicyBuilder); 147 DISALLOW_COPY_AND_ASSIGN(TypedPolicyBuilder);
148 }; 148 };
149 149
150 typedef TypedPolicyBuilder<enterprise_management::CloudPolicySettings> 150 typedef TypedPolicyBuilder<enterprise_management::CloudPolicySettings>
151 UserPolicyBuilder; 151 UserPolicyBuilder;
152 152
153 #if !defined(OS_ANDROID) && !defined(OS_IOS) 153 #if !defined(OS_ANDROID) && !defined(OS_IOS)
154 typedef TypedPolicyBuilder<enterprise_management::ExternalPolicyData> 154 typedef TypedPolicyBuilder<enterprise_management::ExternalPolicyData>
155 ComponentPolicyBuilder; 155 ComponentPolicyBuilder;
156 #endif 156 #endif
157 157
158 } // namespace policy 158 } // namespace policy
159 159
160 #endif // COMPONENTS_POLICY_CORE_COMMON_CLOUD_POLICY_BUILDER_H_ 160 #endif // COMPONENTS_POLICY_CORE_COMMON_CLOUD_POLICY_BUILDER_H_
OLDNEW
« no previous file with comments | « components/policy/core/common/async_policy_provider_unittest.cc ('k') | components/policy/core/common/policy_loader_win.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698