Avoid referencing an HTMLMediaElement if is currently being deleted

Source/core/html/HTMLMediaElement.cpp

Index: Source/core/html/HTMLMediaElement.cpp
diff --git a/Source/core/html/HTMLMediaElement.cpp b/Source/core/html/HTMLMediaElement.cpp
index 15007ba151486fe30206665094af261aae130263..9a095f5585d974b8b4960a1e6a76720a928d9c62 100644
--- a/Source/core/html/HTMLMediaElement.cpp
+++ b/Source/core/html/HTMLMediaElement.cpp
@@ -290,6 +290,9 @@ HTMLMediaElement::HTMLMediaElement(const QualifiedName& tagName, Document* docum
 HTMLMediaElement::~HTMLMediaElement()
 {
     LOG(Media, "HTMLMediaElement::~HTMLMediaElement");
+
+    m_asyncEventQueue.close();

[aarya, 2013/06/03 16:47:26]

This line was a compile failure, that is why i uploaded the build fix.

[vcarbune.chromium, 2013/06/03 17:24:22]

Argh, sorry about that, I didn't realize you uploaded another patch. I'll fix it
and dcommit.

+
     setShouldDelayLoadEvent(false);
     if (m_textTracks)
         m_textTracks->clearOwner();
@@ -542,9 +545,8 @@ void HTMLMediaElement::scheduleEvent(const AtomicString& eventName)
 #if LOG_MEDIA_EVENTS
     LOG(Media, "HTMLMediaElement::scheduleEvent - scheduling '%s'", eventName.string().ascii().data());
 #endif
-    RefPtr<Event> event = Event::create(eventName, false, true);
-    event->setTarget(this);
 
+    RefPtr<Event> event = Event::create(eventName, false, true);
     m_asyncEventQueue->enqueueEvent(event.release());
 }
 
@@ -905,8 +907,6 @@ static bool eventTimeCueCompare(const std::pair<double, TextTrackCue*>& a,
 
 void HTMLMediaElement::updateActiveTextTrackCues(double movieTime)
 {
-    LOG(Media, "HTMLMediaElement::updateActiveTextTrackCues");
-
     // 4.8.10.8 Playing the media resource
 
     //  If the current playback position changes while the steps are running,
@@ -915,6 +915,8 @@ void HTMLMediaElement::updateActiveTextTrackCues(double movieTime)
     if (ignoreTrackDisplayUpdateRequests())
         return;
 
+    LOG(Media, "HTMLMediaElement::updateActiveTextTrackCues");
+
     // 1 - Let current cues be a list of cues, initialized to contain all the
     // cues of all the hidden, showing, or showing by default text tracks of the
     // media element (not the disabled ones) whose start times are less than or
@@ -3508,6 +3510,8 @@ void HTMLMediaElement::stop()
 
     stopPeriodicTimers();
     cancelPendingEventsAndCallbacks();
+
+    m_asyncEventQueue->close();
 }
 
 void HTMLMediaElement::suspend(ReasonForSuspension why)