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

Side by Side Diff: chrome/browser/chromeos/arc/policy/arc_policy_bridge.cc

Issue 2599673005: arc: Use GET_INTERFACE_FOR_METHOD macro (Closed)
Patch Set: Addressed feedback Created 3 years, 11 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 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 "chrome/browser/chromeos/arc/policy/arc_policy_bridge.h" 5 #include "chrome/browser/chromeos/arc/policy/arc_policy_bridge.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <utility> 8 #include <utility>
9 9
10 #include "base/json/json_reader.h" 10 #include "base/json/json_reader.h"
(...skipping 267 matching lines...) Expand 10 before | Expand all | Expand 10 after
278 } 278 }
279 279
280 void ArcPolicyBridge::OnInstanceReady() { 280 void ArcPolicyBridge::OnInstanceReady() {
281 VLOG(1) << "ArcPolicyBridge::OnPolicyInstanceReady"; 281 VLOG(1) << "ArcPolicyBridge::OnPolicyInstanceReady";
282 if (policy_service_ == nullptr) { 282 if (policy_service_ == nullptr) {
283 InitializePolicyService(); 283 InitializePolicyService();
284 } 284 }
285 policy_service_->AddObserver(policy::POLICY_DOMAIN_CHROME, this); 285 policy_service_->AddObserver(policy::POLICY_DOMAIN_CHROME, this);
286 286
287 mojom::PolicyInstance* const policy_instance = 287 mojom::PolicyInstance* const policy_instance =
288 arc_bridge_service()->policy()->GetInstanceForMethod("Init"); 288 ARC_GET_INSTANCE_FOR_METHOD(arc_bridge_service()->policy(), Init);
289 DCHECK(policy_instance); 289 DCHECK(policy_instance);
290 policy_instance->Init(binding_.CreateInterfacePtrAndBind()); 290 policy_instance->Init(binding_.CreateInterfacePtrAndBind());
291 } 291 }
292 292
293 void ArcPolicyBridge::OnInstanceClosed() { 293 void ArcPolicyBridge::OnInstanceClosed() {
294 VLOG(1) << "ArcPolicyBridge::OnPolicyInstanceClosed"; 294 VLOG(1) << "ArcPolicyBridge::OnPolicyInstanceClosed";
295 policy_service_->RemoveObserver(policy::POLICY_DOMAIN_CHROME, this); 295 policy_service_->RemoveObserver(policy::POLICY_DOMAIN_CHROME, this);
296 policy_service_ = nullptr; 296 policy_service_ = nullptr;
297 } 297 }
298 298
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
360 VLOG(1) << "ArcPolicyBridge::ReportCompliance"; 360 VLOG(1) << "ArcPolicyBridge::ReportCompliance";
361 safe_json::SafeJsonParser::Parse( 361 safe_json::SafeJsonParser::Parse(
362 request, base::Bind(&OnReportComplianceParseSuccess, callback), 362 request, base::Bind(&OnReportComplianceParseSuccess, callback),
363 base::Bind(&OnReportComplianceParseFailure, callback)); 363 base::Bind(&OnReportComplianceParseFailure, callback));
364 } 364 }
365 365
366 void ArcPolicyBridge::OnPolicyUpdated(const policy::PolicyNamespace& ns, 366 void ArcPolicyBridge::OnPolicyUpdated(const policy::PolicyNamespace& ns,
367 const policy::PolicyMap& previous, 367 const policy::PolicyMap& previous,
368 const policy::PolicyMap& current) { 368 const policy::PolicyMap& current) {
369 VLOG(1) << "ArcPolicyBridge::OnPolicyUpdated"; 369 VLOG(1) << "ArcPolicyBridge::OnPolicyUpdated";
370 auto* instance = 370 auto* instance = ARC_GET_INSTANCE_FOR_METHOD(arc_bridge_service()->policy(),
371 arc_bridge_service()->policy()->GetInstanceForMethod("OnPolicyUpdated"); 371 OnPolicyUpdated);
372 if (!instance) 372 if (!instance)
373 return; 373 return;
374 instance->OnPolicyUpdated(); 374 instance->OnPolicyUpdated();
375 } 375 }
376 376
377 void ArcPolicyBridge::InitializePolicyService() { 377 void ArcPolicyBridge::InitializePolicyService() {
378 const user_manager::User* const primary_user = 378 const user_manager::User* const primary_user =
379 user_manager::UserManager::Get()->GetPrimaryUser(); 379 user_manager::UserManager::Get()->GetPrimaryUser();
380 Profile* const profile = 380 Profile* const profile =
381 chromeos::ProfileHelper::Get()->GetProfileByUser(primary_user); 381 chromeos::ProfileHelper::Get()->GetProfileByUser(primary_user);
382 auto* profile_policy_connector = 382 auto* profile_policy_connector =
383 policy::ProfilePolicyConnectorFactory::GetForBrowserContext(profile); 383 policy::ProfilePolicyConnectorFactory::GetForBrowserContext(profile);
384 policy_service_ = profile_policy_connector->policy_service(); 384 policy_service_ = profile_policy_connector->policy_service();
385 is_managed_ = profile_policy_connector->IsManaged(); 385 is_managed_ = profile_policy_connector->IsManaged();
386 } 386 }
387 387
388 } // namespace arc 388 } // namespace arc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698