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

Side by Side Diff: chrome/browser/chromeos/arc/arc_policy_bridge.h

Issue 2133503002: arc: Revamp the ArcBridgeService interface (Closed) Base URL: https://chromium.googlesource.com/a/chromium/src.git@master
Patch Set: Fix ui_arc_unittests Created 4 years, 5 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 #ifndef CHROME_BROWSER_CHROMEOS_ARC_ARC_POLICY_BRIDGE_H_ 5 #ifndef CHROME_BROWSER_CHROMEOS_ARC_ARC_POLICY_BRIDGE_H_
6 #define CHROME_BROWSER_CHROMEOS_ARC_ARC_POLICY_BRIDGE_H_ 6 #define CHROME_BROWSER_CHROMEOS_ARC_ARC_POLICY_BRIDGE_H_
7 7
8 #include "base/macros.h" 8 #include "base/macros.h"
9 #include "components/arc/arc_bridge_service.h" 9 #include "components/arc/arc_bridge_service.h"
10 #include "components/arc/arc_service.h" 10 #include "components/arc/arc_service.h"
11 #include "components/policy/core/common/policy_service.h" 11 #include "components/policy/core/common/policy_service.h"
12 #include "mojo/public/cpp/bindings/binding.h" 12 #include "mojo/public/cpp/bindings/binding.h"
13 13
14 namespace policy { 14 namespace policy {
15 class PolicyMap; 15 class PolicyMap;
16 } // namespace policy 16 } // namespace policy
17 17
18 namespace arc { 18 namespace arc {
19 19
20 // Constants for the ARC certs sync mode are defined in the policy, please keep 20 // Constants for the ARC certs sync mode are defined in the policy, please keep
21 // its in sync. 21 // its in sync.
22 enum ArcCertsSyncMode : int32_t { 22 enum ArcCertsSyncMode : int32_t {
23 // Certificates sync is disabled. 23 // Certificates sync is disabled.
24 SYNC_DISABLED = 0, 24 SYNC_DISABLED = 0,
25 // Copy of CA certificates is enabled. 25 // Copy of CA certificates is enabled.
26 COPY_CA_CERTS = 1 26 COPY_CA_CERTS = 1
27 }; 27 };
28 28
29 class ArcPolicyBridge : public ArcService, 29 class ArcPolicyBridge
30 public ArcBridgeService::Observer, 30 : public ArcService,
31 public mojom::PolicyHost, 31 public ArcBridgeService::InstanceObserver<mojom::PolicyInstance>,
32 public policy::PolicyService::Observer { 32 public mojom::PolicyHost,
33 public policy::PolicyService::Observer {
33 public: 34 public:
34 explicit ArcPolicyBridge(ArcBridgeService* bridge_service); 35 explicit ArcPolicyBridge(ArcBridgeService* bridge_service);
35 ArcPolicyBridge(ArcBridgeService* bridge_service, 36 ArcPolicyBridge(ArcBridgeService* bridge_service,
36 policy::PolicyService* policy_service); 37 policy::PolicyService* policy_service);
37 ~ArcPolicyBridge() override; 38 ~ArcPolicyBridge() override;
38 39
39 void OverrideIsManagedForTesting(bool is_managed); 40 void OverrideIsManagedForTesting(bool is_managed);
40 41
41 // ArcBridgeService::Observer overrides. 42 // ArcBridgeService::InstanceObserver<mojom::PolicyInstance> overrides.
42 void OnPolicyInstanceReady() override; 43 void OnInstanceReady(mojom::PolicyInstance* policy_instance,
43 void OnPolicyInstanceClosed() override; 44 uint32_t version) override;
45 void OnInstanceClosed(mojom::PolicyInstance*) override;
44 46
45 // PolicyHost overrides. 47 // PolicyHost overrides.
46 void GetPolicies(const GetPoliciesCallback& callback) override; 48 void GetPolicies(const GetPoliciesCallback& callback) override;
47 49
48 // PolicyService::Observer overrides. 50 // PolicyService::Observer overrides.
49 void OnPolicyUpdated(const policy::PolicyNamespace& ns, 51 void OnPolicyUpdated(const policy::PolicyNamespace& ns,
50 const policy::PolicyMap& previous, 52 const policy::PolicyMap& previous,
51 const policy::PolicyMap& current) override; 53 const policy::PolicyMap& current) override;
52 54
53 private: 55 private:
54 void InitializePolicyService(); 56 void InitializePolicyService();
55 57
56 mojo::Binding<PolicyHost> binding_; 58 mojo::Binding<PolicyHost> binding_;
57 policy::PolicyService* policy_service_ = nullptr; 59 policy::PolicyService* policy_service_ = nullptr;
58 bool is_managed_ = false; 60 bool is_managed_ = false;
59 61
60 DISALLOW_COPY_AND_ASSIGN(ArcPolicyBridge); 62 DISALLOW_COPY_AND_ASSIGN(ArcPolicyBridge);
61 }; 63 };
62 64
63 } // namespace arc 65 } // namespace arc
64 66
65 #endif // CHROME_BROWSER_CHROMEOS_ARC_ARC_POLICY_BRIDGE_H_ 67 #endif // CHROME_BROWSER_CHROMEOS_ARC_ARC_POLICY_BRIDGE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698