Fix possible multiple calls to MidiManager::StartInitialization() in StartSession()

media/midi/midi_manager.h

Index: media/midi/midi_manager.h
diff --git a/media/midi/midi_manager.h b/media/midi/midi_manager.h
index 9d0896a3703bd833ab46509ce4e108e61867d27b..7e595d6d8415a15438953f57c4e928d4c8adeaba 100644
--- a/media/midi/midi_manager.h
+++ b/media/midi/midi_manager.h
@@ -190,8 +190,14 @@ class MIDI_EXPORT MidiManager {
   // order to invoke CompleteStartSession() on the thread.
   scoped_refptr<base::SingleThreadTaskRunner> session_thread_runner_;
 
-  // Keeps true if platform dependent initialization is already completed.
-  bool initialized_;
+  enum class InitializationState {

[Takashi Toyoshima, 2016/10/24 10:07:29]

https://google.github.io/styleguide/cppguide.html#Declaration_Order

Can you define this enum class immediately after the private section starting at
177?

[Shao-Chuan Lee, 2016/10/24 10:09:38]

Done.

+    NOT_STARTED,
+    STARTED,
+    COMPLETED,
+  };
+
+  // Tracks platform dependent initialization state.
+  InitializationState initialization_state_;
 
   // Keeps false until Finalize() is called.
   bool finalized_;
@@ -205,7 +211,7 @@ class MIDI_EXPORT MidiManager {
   MidiPortInfoList output_ports_;
 
   // Protects access to |clients_|, |pending_clients_|,
-  // |session_thread_runner_|, |initialized_|, |finalize_|, |result_|,
+  // |session_thread_runner_|, |initialization_state_|, |finalize_|, |result_|,
   // |input_ports_| and |output_ports_|.
   base::Lock lock_;