Title: [198370] trunk/Source/WebCore
Revision
198370
Author
[email protected]
Date
2016-03-17 18:12:47 -0700 (Thu, 17 Mar 2016)

Log Message

Improve some metadata tests
https://bugs.webkit.org/show_bug.cgi?id=155616

Patch by Eric Carlson <[email protected]> on 2016-03-17
Reviewed by Saam Barati.

* html/track/DataCue.cpp:
(WebCore::DataCue::DataCue):
(WebCore::DataCue::setData):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (198369 => 198370)


--- trunk/Source/WebCore/ChangeLog	2016-03-18 01:02:32 UTC (rev 198369)
+++ trunk/Source/WebCore/ChangeLog	2016-03-18 01:12:47 UTC (rev 198370)
@@ -1,3 +1,14 @@
+2016-03-17  Eric Carlson  <[email protected]>
+
+        Improve some metadata tests
+        https://bugs.webkit.org/show_bug.cgi?id=155616
+
+        Reviewed by Saam Barati.
+
+        * html/track/DataCue.cpp:
+        (WebCore::DataCue::DataCue):
+        (WebCore::DataCue::setData):
+
 2016-03-17  Myles C. Maxfield  <[email protected]>
 
         [RTL Scrollbars] Position: absolute divs are covered by vertical scrollbar

Modified: trunk/Source/WebCore/html/track/DataCue.cpp (198369 => 198370)


--- trunk/Source/WebCore/html/track/DataCue.cpp	2016-03-18 01:02:32 UTC (rev 198369)
+++ trunk/Source/WebCore/html/track/DataCue.cpp	2016-03-18 01:12:47 UTC (rev 198370)
@@ -62,6 +62,8 @@
     , m_type(type)
     , m_value(value)
 {
+    if (m_value)
+        JSC::gcProtect(m_value);
 }
 #endif
 
@@ -95,6 +97,8 @@
 
 #if ENABLE(DATACUE_VALUE)
     m_platformValue = nullptr;
+    if (m_value)
+        JSC::gcUnprotect(m_value);
     m_value = JSC::JSValue();
 #endif
 
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to