WebMIDI: Add MIDIOptions dictionary

Source/modules/webmidi/MIDIAccessInitializer.cpp

Index: Source/modules/webmidi/MIDIAccessInitializer.cpp
diff --git a/Source/modules/webmidi/MIDIAccessInitializer.cpp b/Source/modules/webmidi/MIDIAccessInitializer.cpp
index 7316d0288bbbc6b4b1244c559a42b57555feb5ca..11e502475ba12f324d3e15ce0d1702b24de51c31 100644
--- a/Source/modules/webmidi/MIDIAccessInitializer.cpp
+++ b/Source/modules/webmidi/MIDIAccessInitializer.cpp
@@ -17,11 +17,12 @@
 
 namespace blink {
 
-MIDIAccessInitializer::MIDIAccessInitializer(ScriptState* scriptState, const MIDIOptions& options)
+MIDIAccessInitializer::MIDIAccessInitializer(ScriptState* scriptState, const MIDIOptions* options)
     : ScriptPromiseResolver(scriptState)
-    , m_options(options)
     , m_sysexEnabled(false)
 {
+    if (options->hasSysex())

[Takashi Toyoshima, 2014/09/10 04:07:51]

hum... this change works, but now the variable name is curious.
Originally, m_options.sysex means that we are going to request a sysex
permission, and m_sysexEnabled means that the request is permitted by the
browser process.

So, maybe the better change is:

1. rename m_sysexEnabled to m_requestSysex
2. do not update the variable in setSysexEnabled(). now we do not need to hold
the result since we just reject() here.

Also, MIDIAccessInitializer::setSysexEnabled(bool enable) is still curious name
now. resolveSysexPermission(bool accept) or something like that. But this will
affect many module. So, I'm ok to keep the function name as is. I may handle it
later.

[bashi, 2014/09/10 08:04:38]

Renamed m_requestSysex and resolveSysexPermission(allowed) respectively.

+        m_sysexEnabled = options->sysex();
 }
 
 MIDIAccessInitializer::~MIDIAccessInitializer()
@@ -39,7 +40,7 @@ ScriptPromise MIDIAccessInitializer::start()
     ScriptPromise promise = this->promise();
     m_accessor = MIDIAccessor::create(this);
 
-    if (!m_options.sysex) {
+    if (!m_sysexEnabled) {
         m_accessor->startSession();
         return promise;
     }