Oilpan: cancel MIDI permission request when prefinalizing

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 7fff2affa7b652bf8ba6a71304a0d224c7b5a4fb..ffa49a1ac4e2362eeac0240dad2b45ff1762b04f 100644
--- a/Source/modules/webmidi/MIDIAccessInitializer.cpp
+++ b/Source/modules/webmidi/MIDIAccessInitializer.cpp
@@ -27,7 +27,10 @@ MIDIAccessInitializer::MIDIAccessInitializer(ScriptState* scriptState, const MID
 
 MIDIAccessInitializer::~MIDIAccessInitializer()
 {

[haraken, 2014/12/18 14:42:46]

What happens if MIDIAccessInitializer gets destructed before
ActiveDOMObject::stop() is called?

[sof, 2014/12/18 14:53:54]

A permissions request will be made with the controller as part of
MIDIAccessInitializer start()ing. At that point the ActiveDOMObject is in a
started or suspended state, right? I assume that it is not possible for this
ActiveDOMObject to be finalized before a stop() is issued by the execution
context.

[haraken, 2014/12/18 15:02:28]

I'm wondering if it's possible that the MIDIAccessInitializer object dies before
the ExecutionContext stops. For example:

(1) MIDIAccessInitializer is created.
(2) The last reference to the MIDIAccessInitializer is gone.
(3) GC collects the MIDIAccessInitializer.
(4) ExecutionContext stops.

?

[sof, 2014/12/18 15:17:57]

If that's a concern, I can rephrase this to be a reg/unreg protocol between the
controller and this object instead, when there is time. Using a controller-weak
reference, or something.

[haraken, 2014/12/18 15:44:02]

Yeah, I guess a more straightforward approach would be to have MIDIController
have a weak reference to the MIDIAccessInitializer so that the MIDIController
can call cancelSysexPermissionRequest() when the MIDIAccessInitializer is going
to be destructed.

In your approach, it's still not clear to me who keeps the MIDIAccessInitializer
object alive until the ExecutionContext stops.

-    // It is safe to cancel a request which is already finished or canceld.
+}
+
+void MIDIAccessInitializer::cancel()
+{
     Document* document = toDocument(executionContext());
     ASSERT(document);
     MIDIController* controller = MIDIController::from(document->frame());
@@ -35,6 +38,13 @@ MIDIAccessInitializer::~MIDIAccessInitializer()
         controller->cancelSysexPermissionRequest(this);
 }
 
+void MIDIAccessInitializer::stop()
+{
+    // It is safe to cancel a request which is already finished or canceld.

[haraken, 2014/12/18 14:42:46]

canceled

+    cancel();
+    ScriptPromiseResolver::stop();
+}
+
 ScriptPromise MIDIAccessInitializer::start()
 {
     ScriptPromise promise = this->promise();