Attachment 349012 Details for Bug 188786 – Patch for landing

[patch] Patch for landing

bug-188786-20180906121138.patch (text/plain), 1.76 KB, created by Xabier Rodríguez Calvar on 2018-09-06 03:11:39 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Xabier Rodríguez Calvar

Created: 2018-09-06 03:11:39 PDT

Size: 1.76 KB

patch

obsolete

>Subversion Revision: 235610
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index a7a62ea856b5c0cc44ec9f6f3c9be67993949f4d..d2f27f698d564e4b4d5ad517e1d5f7c5cd97252d 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,16 @@
>+2018-08-21  Xabier Rodriguez Calvar  <calvaris@igalia.com>
>+
>+        [GStreamer] Extra ASSERTs at MainThreadNotifier
>+        https://bugs.webkit.org/show_bug.cgi?id=188786
>+
>+        Reviewed by Carlos Garcia Campos.
>+
>+        We add a couple of checks in debug mode: there is only one bit on
>+        per notification and ensure a notifier was invalidated before
>+        being destroyed.
>+
>+        * platform/graphics/gstreamer/MainThreadNotifier.h:
>+
> 2018-09-04  Yacine Bandou  <yacine.bandou_ext@softathome.com>
> 
>         [EME] Add the WebM initData support in ClearKey CDM
>diff --git a/Source/WebCore/platform/graphics/gstreamer/MainThreadNotifier.h b/Source/WebCore/platform/graphics/gstreamer/MainThreadNotifier.h
>index 3752dc275e719578ebf19596f4fa6bf2fc88f9b5..5eada639b99293e2bba3fd17c81e28638c55ff7d 100644
>--- a/Source/WebCore/platform/graphics/gstreamer/MainThreadNotifier.h
>+++ b/Source/WebCore/platform/graphics/gstreamer/MainThreadNotifier.h
>@@ -35,10 +35,17 @@ public:
>         return adoptRef(*new MainThreadNotifier());
>     }
> 
>+    ~MainThreadNotifier()
>+    {
>+        ASSERT(!m_isValid.load());
>+    }
>+
>     template<typename F>
>     void notify(T notificationType, F&& callbackFunctor)
>     {
>         ASSERT(m_isValid.load());
>+        // Assert that there is only one bit on at a time.
>+        ASSERT(!(static_cast<int>(notificationType) & (static_cast<int>(notificationType) - 1)));
>         if (isMainThread()) {
>             removePendingNotification(notificationType);
>             callbackFunctor();

Actions: View | Formatted Diff | Diff

Attachments on bug 188786: 347630 | 349012