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

Side by Side Diff: media/midi/midi_manager_android.h

Issue 2262043002: MidiScheduler binds to task runner of constructing thread (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years, 3 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
« no previous file with comments | « no previous file | media/midi/midi_manager_android.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 MEDIA_MIDI_MIDI_MANAGER_ANDROID_H_ 5 #ifndef MEDIA_MIDI_MIDI_MANAGER_ANDROID_H_
6 #define MEDIA_MIDI_MIDI_MANAGER_ANDROID_H_ 6 #define MEDIA_MIDI_MIDI_MANAGER_ANDROID_H_
7 7
8 #include <jni.h> 8 #include <jni.h>
9 #include <stddef.h> 9 #include <stddef.h>
10 #include <stdint.h> 10 #include <stdint.h>
11 11
12 #include <memory> 12 #include <memory>
13 #include <vector> 13 #include <vector>
14 14
15 #include "base/android/scoped_java_ref.h" 15 #include "base/android/scoped_java_ref.h"
16 #include "base/containers/hash_tables.h" 16 #include "base/containers/hash_tables.h"
17 #include "base/memory/scoped_vector.h" 17 #include "base/memory/scoped_vector.h"
18 #include "base/synchronization/lock.h"
18 #include "base/time/time.h" 19 #include "base/time/time.h"
19 #include "media/midi/midi_input_port_android.h" 20 #include "media/midi/midi_input_port_android.h"
20 #include "media/midi/midi_manager.h" 21 #include "media/midi/midi_manager.h"
21 #include "media/midi/midi_scheduler.h" 22 #include "media/midi/midi_scheduler.h"
22 23
23 namespace media { 24 namespace media {
24 namespace midi { 25 namespace midi {
25 26
26 class MidiDeviceAndroid; 27 class MidiDeviceAndroid;
27 class MidiOutputPortAndroid; 28 class MidiOutputPortAndroid;
28 29
29 // MidiManagerAndroid is a MidiManager subclass for Android M or newer. For 30 // MidiManagerAndroid is a MidiManager subclass for Android M or newer. For
30 // older android OSes, we use MidiManagerUsb. 31 // older android OSes, we use MidiManagerUsb.
31 class MidiManagerAndroid final : public MidiManager, 32 class MidiManagerAndroid final : public MidiManager,
32 public MidiInputPortAndroid::Delegate { 33 public MidiInputPortAndroid::Delegate {
33 public: 34 public:
34 MidiManagerAndroid(); 35 MidiManagerAndroid();
35 ~MidiManagerAndroid() override; 36 ~MidiManagerAndroid() override;
36 37
37 // MidiManager implementation. 38 // MidiManager implementation.
38 void StartInitialization() override; 39 void StartInitialization() override;
40 void Finalize() override;
39 void DispatchSendMidiData(MidiManagerClient* client, 41 void DispatchSendMidiData(MidiManagerClient* client,
40 uint32_t port_index, 42 uint32_t port_index,
41 const std::vector<uint8_t>& data, 43 const std::vector<uint8_t>& data,
42 double timestamp) override; 44 double timestamp) override;
43 45
44 // MidiInputPortAndroid::Delegate implementation. 46 // MidiInputPortAndroid::Delegate implementation.
45 void OnReceivedData(MidiInputPortAndroid*, 47 void OnReceivedData(MidiInputPortAndroid*,
46 const uint8_t* data, 48 const uint8_t* data,
47 size_t size, 49 size_t size,
48 base::TimeTicks timestamp) override; 50 base::TimeTicks timestamp) override;
(...skipping 25 matching lines...) Expand all
74 std::vector<MidiInputPortAndroid*> all_input_ports_; 76 std::vector<MidiInputPortAndroid*> all_input_ports_;
75 // A dictionary from a port to its index. 77 // A dictionary from a port to its index.
76 // input_port_to_index_[all_input_ports_[i]] == i for each valid |i|. 78 // input_port_to_index_[all_input_ports_[i]] == i for each valid |i|.
77 base::hash_map<MidiInputPortAndroid*, size_t> input_port_to_index_; 79 base::hash_map<MidiInputPortAndroid*, size_t> input_port_to_index_;
78 80
79 // Ditto for output ports. 81 // Ditto for output ports.
80 std::vector<MidiOutputPortAndroid*> all_output_ports_; 82 std::vector<MidiOutputPortAndroid*> all_output_ports_;
81 base::hash_map<MidiOutputPortAndroid*, size_t> output_port_to_index_; 83 base::hash_map<MidiOutputPortAndroid*, size_t> output_port_to_index_;
82 84
83 base::android::ScopedJavaGlobalRef<jobject> raw_manager_; 85 base::android::ScopedJavaGlobalRef<jobject> raw_manager_;
84 std::unique_ptr<MidiScheduler> scheduler_; 86
87 // Lock to ensure the MidiScheduler is being destructed only once in
88 // Finalize() on Chrome_IOThread.
89 base::Lock scheduler_lock_;
90 std::unique_ptr<MidiScheduler> scheduler_; // GUARDED_BY(scheduler_lock_)
85 }; 91 };
86 92
87 } // namespace midi 93 } // namespace midi
88 } // namespace media 94 } // namespace media
89 95
90 #endif // MEDIA_MIDI_MIDI_MANAGER_ANDROID_H_ 96 #endif // MEDIA_MIDI_MIDI_MANAGER_ANDROID_H_
OLDNEW
« no previous file with comments | « no previous file | media/midi/midi_manager_android.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698