Title: [237508] releases/WebKitGTK/webkit-2.22/Source/WebCore
- Revision
- 237508
- Author
- ape...@igalia.com
- Date
- 2018-10-28 06:42:39 -0700 (Sun, 28 Oct 2018)
Log Message
Merged r236985 - [WPE][GTK] Complex text crashes with harfbuzz 1.8.8
https://bugs.webkit.org/show_bug.cgi?id=190409
Reviewed by Žan Doberšek.
We discovered that harfbuzz 1.8.8 breaks WebKit by changing the ownership semantics of
hb_icu_get_unicode_funcs() from (transfer full) to (transfer none). Our code was actually
expecting (transfer none), so it's not immediately clear to me why it's crashing now, but
doesn't matter as Behdad recommends removing this line of code because it hasn't been
been needed for many years.
This should be covered by all our complex text tests if the bots were upgraded to the newer
harfbuzz.
* platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.cpp:
(WebCore::ComplexTextController::collectComplexTextRunsForCharacters):
Modified Paths
Diff
Modified: releases/WebKitGTK/webkit-2.22/Source/WebCore/ChangeLog (237507 => 237508)
--- releases/WebKitGTK/webkit-2.22/Source/WebCore/ChangeLog 2018-10-28 13:42:33 UTC (rev 237507)
+++ releases/WebKitGTK/webkit-2.22/Source/WebCore/ChangeLog 2018-10-28 13:42:39 UTC (rev 237508)
@@ -1,3 +1,22 @@
+2018-10-09 Michael Catanzaro <mcatanz...@igalia.com>
+
+ [WPE][GTK] Complex text crashes with harfbuzz 1.8.8
+ https://bugs.webkit.org/show_bug.cgi?id=190409
+
+ Reviewed by Žan Doberšek.
+
+ We discovered that harfbuzz 1.8.8 breaks WebKit by changing the ownership semantics of
+ hb_icu_get_unicode_funcs() from (transfer full) to (transfer none). Our code was actually
+ expecting (transfer none), so it's not immediately clear to me why it's crashing now, but
+ doesn't matter as Behdad recommends removing this line of code because it hasn't been
+ been needed for many years.
+
+ This should be covered by all our complex text tests if the bots were upgraded to the newer
+ harfbuzz.
+
+ * platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.cpp:
+ (WebCore::ComplexTextController::collectComplexTextRunsForCharacters):
+
2018-10-09 Philippe Normand <pnorm...@igalia.com>
[GStreamer] Stealing cross-origin video pixel with HLS
Modified: releases/WebKitGTK/webkit-2.22/Source/WebCore/platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.cpp (237507 => 237508)
--- releases/WebKitGTK/webkit-2.22/Source/WebCore/platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.cpp 2018-10-28 13:42:33 UTC (rev 237507)
+++ releases/WebKitGTK/webkit-2.22/Source/WebCore/platform/graphics/harfbuzz/ComplexTextControllerHarfBuzz.cpp 2018-10-28 13:42:39 UTC (rev 237508)
@@ -193,7 +193,6 @@
const auto& fontPlatformData = font->platformData();
auto features = fontFeatures(m_font, fontPlatformData.orientation());
HbUniquePtr<hb_buffer_t> buffer(hb_buffer_create());
- hb_buffer_set_unicode_funcs(buffer.get(), hb_icu_get_unicode_funcs());
for (unsigned i = 0; i < runCount; ++i) {
auto& run = runList[m_run.rtl() ? runCount - i - 1 : i];
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes
