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: components/copresence/handlers/directive_handler.h

Issue 865483005: Creating the audio_modem component (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Small fixes Created 5 years, 10 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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_COPRESENCE_HANDLERS_DIRECTIVE_HANDLER_H_ 5 #ifndef COMPONENTS_COPRESENCE_HANDLERS_DIRECTIVE_HANDLER_H_
6 #define COMPONENTS_COPRESENCE_HANDLERS_DIRECTIVE_HANDLER_H_ 6 #define COMPONENTS_COPRESENCE_HANDLERS_DIRECTIVE_HANDLER_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/macros.h" 10 #include "base/macros.h"
11 #include "components/copresence/public/copresence_constants.h" 11 #include "components/audio_modem/public/whispernet_client.h"
12 12
13 namespace copresence { 13 namespace copresence {
14 14
15 class Directive; 15 class Directive;
16 class WhispernetClient;
17 16
18 // The directive handler manages transmit and receive directives. 17 // The directive handler manages transmit and receive directives.
19 class DirectiveHandler { 18 class DirectiveHandler {
20 public: 19 public:
21 DirectiveHandler() {} 20 DirectiveHandler() {}
22 virtual ~DirectiveHandler() {} 21 virtual ~DirectiveHandler() {}
23 22
24 // Starts processing directives with the provided Whispernet client. 23 // Starts processing directives with the provided Whispernet delegate.
25 // Directives will be queued until this function is called. 24 // Directives will be queued until this function is called.
26 // |whispernet_client| is owned by the caller and must outlive the 25 // |whispernet_client| is owned by the caller and must outlive the
27 // DirectiveHandler. 26 // DirectiveHandler.
28 // |tokens_cb| is called for all audio tokens found in recorded audio. 27 // |tokens_cb| is called for all audio tokens found in recorded audio.
29 virtual void Start(WhispernetClient* whispernet_client, 28 // TODO(ckehoe): Have the Modem handle initialization of the client.
30 const TokensCallback& tokens_cb) = 0; 29 // Then none of our classes need this Init() mechanism.
30 virtual void Start(audio_modem::WhispernetClient* whispernet_client,
31 const audio_modem::TokensCallback& tokens_cb) = 0;
31 32
32 // Adds a directive to handle. 33 // Adds a directive to handle.
33 virtual void AddDirective(const Directive& directive) = 0; 34 virtual void AddDirective(const Directive& directive) = 0;
34 35
35 // Removes any directives associated with the given operation id. 36 // Removes any directives associated with the given operation id.
36 virtual void RemoveDirectives(const std::string& op_id) = 0; 37 virtual void RemoveDirectives(const std::string& op_id) = 0;
37 38
38 // TODO(rkc): Too many audio specific functions here. 39 // TODO(ckehoe): Move the Modem to be owned by CopresenceManager.
39 // Find a better way to get this information to the copresence manager. 40 // Then this will not need to be passed all the way down the tree.
40 virtual const std::string GetCurrentAudioToken(AudioType type) const = 0; 41 virtual const std::string
41 virtual bool IsAudioTokenHeard(AudioType type) const = 0; 42 GetCurrentAudioToken(audio_modem::AudioType type) const = 0;
43 virtual bool IsAudioTokenHeard(audio_modem::AudioType type) const = 0;
42 44
43 private: 45 private:
44 DISALLOW_COPY_AND_ASSIGN(DirectiveHandler); 46 DISALLOW_COPY_AND_ASSIGN(DirectiveHandler);
45 }; 47 };
46 48
47 } // namespace copresence 49 } // namespace copresence
48 50
49 #endif // COMPONENTS_COPRESENCE_HANDLERS_DIRECTIVE_HANDLER_H_ 51 #endif // COMPONENTS_COPRESENCE_HANDLERS_DIRECTIVE_HANDLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698