Flaky Test: media/media-source/media-source-page-cache.html
https://bugs.webkit.org/show_bug.cgi?id=202775
Reviewed by Eric Carlson.
SourceBufferList should never prevent entering the page cache, even if it has
pending events in its queue. Its queue is a MainThreadGenericEventQueue, which
is Page Cache-aware and will properly suspend the firing of events when needed.
No new tests, covered by flaky test which should no longer be.
* Modules/mediasource/SourceBufferList.cpp:
(WebCore::SourceBufferList::canSuspendForDocumentSuspension const):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@250976 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
index dd61c58..b2ea0f6 100644
--- a/Source/WebCore/ChangeLog
+++ b/Source/WebCore/ChangeLog
@@ -1,3 +1,19 @@
+2019-10-10 Chris Dumez <cdumez@apple.com>
+
+ Flaky Test: media/media-source/media-source-page-cache.html
+ https://bugs.webkit.org/show_bug.cgi?id=202775
+
+ Reviewed by Eric Carlson.
+
+ SourceBufferList should never prevent entering the page cache, even if it has
+ pending events in its queue. Its queue is a MainThreadGenericEventQueue, which
+ is Page Cache-aware and will properly suspend the firing of events when needed.
+
+ No new tests, covered by flaky test which should no longer be.
+
+ * Modules/mediasource/SourceBufferList.cpp:
+ (WebCore::SourceBufferList::canSuspendForDocumentSuspension const):
+
2019-10-10 Sihui Liu <sihui_liu@apple.com>
Add a unit test for StorageQuotaManager
diff --git a/Source/WebCore/Modules/mediasource/SourceBufferList.cpp b/Source/WebCore/Modules/mediasource/SourceBufferList.cpp
index b4d82c4..49a7c3b 100644
--- a/Source/WebCore/Modules/mediasource/SourceBufferList.cpp
+++ b/Source/WebCore/Modules/mediasource/SourceBufferList.cpp
@@ -103,7 +103,7 @@
bool SourceBufferList::canSuspendForDocumentSuspension() const
{
- return !m_asyncEventQueue->hasPendingEvents();
+ return true;
}
const char* SourceBufferList::activeDOMObjectName() const
