OLD | NEW |
---|---|
(Empty) | |
1 // Copyright 2017 The Chromium Authors. All rights reserved. | |
2 // Use of this source code is governed by a BSD-style license that can be | |
3 // found in the LICENSE file. | |
4 | |
5 #ifndef CHROMEOS_DBUS_MEDIA_ANALYTICS_CLIENT_H_ | |
6 #define CHROMEOS_DBUS_MEDIA_ANALYTICS_CLIENT_H_ | |
7 | |
8 #include "base/callback.h" | |
9 #include "base/macros.h" | |
10 #include "chromeos/chromeos_export.h" | |
11 #include "chromeos/dbus/dbus_client.h" | |
12 #include "chromeos/media_perception/media_perception.pb.h" | |
13 | |
14 namespace chromeos { | |
15 | |
16 // MediaAnalyticsClient is used to communicate with a media analytics process | |
17 // running outside of Chrome. | |
18 class CHROMEOS_EXPORT MediaAnalyticsClient : public DBusClient { | |
19 public: | |
20 // Callback type for a State proto message received from the media analytics | |
21 // process. | |
22 using StateCallback = | |
23 base::Callback<void(bool succeeded, const mri::State& state)>; | |
24 // Handler type for signal received from media analytics process. Contains a | |
tbarzic
2017/05/05 21:10:42
The "byte array with a serialized MediaPercetion p
Luke Sorenson
2017/05/08 19:06:08
Done.
| |
25 // byte array with a serialized MediaPerception proto. | |
26 using MediaPerceptionSignalHandler = | |
27 base::Callback<void(const mri::MediaPerception& media_perception)>; | |
28 // Callback type for a Diagnostics proto message received from the media | |
29 // analytics process. | |
30 using DiagnosticsCallback = | |
31 base::Callback<void(bool succeeded, const mri::Diagnostics& diagnostics)>; | |
32 | |
33 ~MediaAnalyticsClient() override; | |
34 | |
35 // API for getting and setting the state of the media analytics process over | |
36 // D-Bus using a State proto. | |
37 virtual void State(const mri::State& state, | |
tbarzic
2017/05/05 21:10:42
can we have separate methods for getting and setti
Luke Sorenson
2017/05/08 19:06:08
From the perspective of the D-Bus comms, it's simp
tbarzic
2017/05/08 23:04:55
Wouldn't you need only one service that can provid
Luke Sorenson
2017/05/09 17:39:45
Okay, I see what you're saying. Done.
| |
38 const StateCallback& callback) = 0; | |
39 | |
40 // Register event handler for the MediaPerception protos received as signal | |
41 // from the media analytics process. | |
42 virtual void SetMediaPerceptionSignalHandler( | |
43 const MediaPerceptionSignalHandler& handler) = 0; | |
44 | |
45 virtual void UnsetMediaPerceptionSignalHandler() = 0; | |
tbarzic
2017/05/08 23:04:55
suggestion: Clear/Reset instead of Unset
Luke Sorenson
2017/05/09 17:39:45
Done.
| |
46 | |
47 // API for getting diagnostic information from the media analytics process | |
48 // over D-Bus as a Diagnostics proto message. | |
49 virtual void GetDiagnostics(const DiagnosticsCallback& callback) = 0; | |
50 | |
51 // Factory function, creates new instance and returns ownership. | |
52 // For normal usage, access the singleton via DbusThreadManager::Get(). | |
53 static MediaAnalyticsClient* Create(); | |
54 | |
55 protected: | |
56 MediaAnalyticsClient(); | |
57 | |
58 private: | |
59 DISALLOW_COPY_AND_ASSIGN(MediaAnalyticsClient); | |
60 }; | |
61 | |
62 } // namespace chromeos | |
63 | |
64 #endif // CHROMEOS_DBUS_MEDIA_ANALYTICS_CLIENT_H_ | |
OLD | NEW |