Title: [259081] trunk/Source/WebCore
Revision
259081
Author
cdu...@apple.com
Date
2020-03-26 14:36:35 -0700 (Thu, 26 Mar 2020)

Log Message

[StressGC] ASSERTION FAILED: m_wrapper under WebCore::HTMLMediaElement::dispatchEvent
https://bugs.webkit.org/show_bug.cgi?id=209616
<rdar://problem/60541294>

Reviewed by Saam Barati.

HTMLMediaElement::hasPendingActivity() should return true if there are pending tasks on
m_playbackTargetIsWirelessQueue since the tasks we enqueue there dispatch events.

No new tests, covered by media/modern-media-controls/placard-support/placard-support-airplay.html.

* html/HTMLMediaElement.cpp:
(WebCore::HTMLMediaElement::hasPendingActivity const):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (259080 => 259081)


--- trunk/Source/WebCore/ChangeLog	2020-03-26 21:33:33 UTC (rev 259080)
+++ trunk/Source/WebCore/ChangeLog	2020-03-26 21:36:35 UTC (rev 259081)
@@ -1,5 +1,21 @@
 2020-03-26  Chris Dumez  <cdu...@apple.com>
 
+        [StressGC] ASSERTION FAILED: m_wrapper under WebCore::HTMLMediaElement::dispatchEvent
+        https://bugs.webkit.org/show_bug.cgi?id=209616
+        <rdar://problem/60541294>
+
+        Reviewed by Saam Barati.
+
+        HTMLMediaElement::hasPendingActivity() should return true if there are pending tasks on
+        m_playbackTargetIsWirelessQueue since the tasks we enqueue there dispatch events.
+
+        No new tests, covered by media/modern-media-controls/placard-support/placard-support-airplay.html.
+
+        * html/HTMLMediaElement.cpp:
+        (WebCore::HTMLMediaElement::hasPendingActivity const):
+
+2020-03-26  Chris Dumez  <cdu...@apple.com>
+
         [ Mac wk1] ASSERTION FAILED: m_wrapper under WebCore::XMLHttpRequestUpload::dispatchProgressEvent
         https://bugs.webkit.org/show_bug.cgi?id=209560
         <rdar://problem/60887773>

Modified: trunk/Source/WebCore/html/HTMLMediaElement.cpp (259080 => 259081)


--- trunk/Source/WebCore/html/HTMLMediaElement.cpp	2020-03-26 21:33:33 UTC (rev 259080)
+++ trunk/Source/WebCore/html/HTMLMediaElement.cpp	2020-03-26 21:36:35 UTC (rev 259081)
@@ -5764,7 +5764,7 @@
 
 bool HTMLMediaElement::hasPendingActivity() const
 {
-    return (hasAudio() && isPlaying()) || m_asyncEventQueue->hasPendingEvents() || m_creatingControls;
+    return (hasAudio() && isPlaying()) || m_asyncEventQueue->hasPendingEvents() || m_playbackTargetIsWirelessQueue.hasPendingTasks() || m_creatingControls;
 }
 
 void HTMLMediaElement::mediaVolumeDidChange()
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to