Title: [199015] trunk/Source/WebCore
Revision
199015
Author
[email protected]
Date
2016-04-04 12:41:29 -0700 (Mon, 04 Apr 2016)

Log Message

Addressing post-review feedback on r198970
https://bugs.webkit.org/show_bug.cgi?id=156123

Unreviewed.

* platform/text/BidiResolver.h:
(WebCore::BidiCharacterRun::~BidiCharacterRun):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (199014 => 199015)


--- trunk/Source/WebCore/ChangeLog	2016-04-04 19:39:10 UTC (rev 199014)
+++ trunk/Source/WebCore/ChangeLog	2016-04-04 19:41:29 UTC (rev 199015)
@@ -1,5 +1,15 @@
 2016-04-04  Myles C. Maxfield  <[email protected]>
 
+        Addressing post-review feedback on r198970
+        https://bugs.webkit.org/show_bug.cgi?id=156123
+
+        Unreviewed.
+
+        * platform/text/BidiResolver.h:
+        (WebCore::BidiCharacterRun::~BidiCharacterRun):
+
+2016-04-04  Myles C. Maxfield  <[email protected]>
+
         [SVG -> OTF Converter] Crash when trying to re-convert a previously-failed font conversion
         https://bugs.webkit.org/show_bug.cgi?id=156175
         <rdar://problem/25130547>

Modified: trunk/Source/WebCore/platform/text/BidiResolver.h (199014 => 199015)


--- trunk/Source/WebCore/platform/text/BidiResolver.h	2016-04-04 19:39:10 UTC (rev 199014)
+++ trunk/Source/WebCore/platform/text/BidiResolver.h	2016-04-04 19:41:29 UTC (rev 199015)
@@ -176,6 +176,14 @@
         }
     }
 
+    ~BidiCharacterRun()
+    {
+        // Delete the linked list in a loop to prevent destructor recursion.
+        auto next = WTFMove(m_next);
+        while (next)
+            next = WTFMove(next->m_next);
+    }
+
     int start() const { return m_start; }
     int stop() const { return m_stop; }
     unsigned char level() const { return m_level; }
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to