Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2071)

Unified Diff: Source/platform/fonts/harfbuzz/HarfBuzzFace.cpp

Issue 619923003: Remove HarfBuzzFaceCoreText (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Rebased on top of eae's gyp changes Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/platform/blink_platform.gypi ('k') | Source/platform/fonts/harfbuzz/HarfBuzzFaceCoreText.mm » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/platform/fonts/harfbuzz/HarfBuzzFace.cpp
diff --git a/Source/platform/fonts/harfbuzz/HarfBuzzFace.cpp b/Source/platform/fonts/harfbuzz/HarfBuzzFace.cpp
index 64158c0b20e013e7ab887f223e240dc50224edf4..11afea385a0473f04b8a8d45e59aa2843c99fab8 100644
--- a/Source/platform/fonts/harfbuzz/HarfBuzzFace.cpp
+++ b/Source/platform/fonts/harfbuzz/HarfBuzzFace.cpp
@@ -31,9 +31,21 @@
#include "config.h"
#include "platform/fonts/harfbuzz/HarfBuzzFace.h"
-#include "platform/fonts/FontPlatformData.h"
#include "hb-ot.h"
#include "hb.h"
+#if OS(MACOSX)
+#include "hb-coretext.h"
+#endif
+#include "SkPaint.h"
+#include "SkPath.h"
+#include "SkPoint.h"
+#include "SkRect.h"
+#include "SkTypeface.h"
+#include "SkUtils.h"
+#include "platform/fonts/FontPlatformData.h"
+#include "platform/fonts/SimpleFontData.h"
+#include "platform/fonts/harfbuzz/HarfBuzzShaper.h"
+#include "wtf/HashMap.h"
namespace blink {
@@ -128,4 +140,177 @@ void HarfBuzzFace::setScriptForVerticalGlyphSubstitution(hb_buffer_t* buffer)
hb_buffer_set_script(buffer, m_scriptForVerticalText);
}
+struct HarfBuzzFontData {
+ HarfBuzzFontData(WTF::HashMap<uint32_t, uint16_t>* glyphCacheForFaceCacheEntry)
+ : m_glyphCacheForFaceCacheEntry(glyphCacheForFaceCacheEntry)
+ { }
+ SkPaint m_paint;
+ WTF::HashMap<uint32_t, uint16_t>* m_glyphCacheForFaceCacheEntry;
+};
+
+static hb_position_t SkiaScalarToHarfBuzzPosition(SkScalar value)
+{
+ return SkScalarToFixed(value);
+}
+
+static void SkiaGetGlyphWidthAndExtents(SkPaint* paint, hb_codepoint_t codepoint, hb_position_t* width, hb_glyph_extents_t* extents)
+{
+ ASSERT(codepoint <= 0xFFFF);
+ paint->setTextEncoding(SkPaint::kGlyphID_TextEncoding);
+
+ SkScalar skWidth;
+ SkRect skBounds;
+ uint16_t glyph = codepoint;
+
+ paint->getTextWidths(&glyph, sizeof(glyph), &skWidth, &skBounds);
+ if (width)
+ *width = SkiaScalarToHarfBuzzPosition(skWidth);
+ if (extents) {
+ // Invert y-axis because Skia is y-grows-down but we set up HarfBuzz to be y-grows-up.
+ extents->x_bearing = SkiaScalarToHarfBuzzPosition(skBounds.fLeft);
+ extents->y_bearing = SkiaScalarToHarfBuzzPosition(-skBounds.fTop);
+ extents->width = SkiaScalarToHarfBuzzPosition(skBounds.width());
+ extents->height = SkiaScalarToHarfBuzzPosition(-skBounds.height());
+ }
+}
+
+static hb_bool_t harfBuzzGetGlyph(hb_font_t* hbFont, void* fontData, hb_codepoint_t unicode, hb_codepoint_t variationSelector, hb_codepoint_t* glyph, void* userData)
+{
+ // Variation selectors not supported.
+ if (variationSelector)
+ return false;
+
+ HarfBuzzFontData* hbFontData = reinterpret_cast<HarfBuzzFontData*>(fontData);
+
+ WTF::HashMap<uint32_t, uint16_t>::AddResult result = hbFontData->m_glyphCacheForFaceCacheEntry->add(unicode, 0);
+ if (result.isNewEntry) {
+ SkPaint* paint = &hbFontData->m_paint;
+ paint->setTextEncoding(SkPaint::kUTF32_TextEncoding);
+ uint16_t glyph16;
+ paint->textToGlyphs(&unicode, sizeof(hb_codepoint_t), &glyph16);
+ result.storedValue->value = glyph16;
+ *glyph = glyph16;
+ }
+ *glyph = result.storedValue->value;
+ return !!*glyph;
+}
+
+static hb_position_t harfBuzzGetGlyphHorizontalAdvance(hb_font_t* hbFont, void* fontData, hb_codepoint_t glyph, void* userData)
+{
+ HarfBuzzFontData* hbFontData = reinterpret_cast<HarfBuzzFontData*>(fontData);
+ hb_position_t advance = 0;
+
+ SkiaGetGlyphWidthAndExtents(&hbFontData->m_paint, glyph, &advance, 0);
+ return advance;
+}
+
+static hb_bool_t harfBuzzGetGlyphHorizontalOrigin(hb_font_t* hbFont, void* fontData, hb_codepoint_t glyph, hb_position_t* x, hb_position_t* y, void* userData)
+{
+ // Just return true, following the way that HarfBuzz-FreeType
+ // implementation does.
+ return true;
+}
+
+static hb_position_t harfBuzzGetGlyphHorizontalKerning(hb_font_t*, void* fontData, hb_codepoint_t leftGlyph, hb_codepoint_t rightGlyph, void*)
+{
+ HarfBuzzFontData* hbFontData = reinterpret_cast<HarfBuzzFontData*>(fontData);
+ if (hbFontData->m_paint.isVerticalText()) {
+ // We don't support cross-stream kerning
+ return 0;
+ }
+
+ SkTypeface* typeface = hbFontData->m_paint.getTypeface();
+
+ const uint16_t glyphs[2] = { static_cast<uint16_t>(leftGlyph), static_cast<uint16_t>(rightGlyph) };
+ int32_t kerningAdjustments[1] = { 0 };
+
+ if (typeface->getKerningPairAdjustments(glyphs, 2, kerningAdjustments)) {
+ SkScalar upm = SkIntToScalar(typeface->getUnitsPerEm());
+ SkScalar size = hbFontData->m_paint.getTextSize();
+ return SkiaScalarToHarfBuzzPosition(SkScalarMulDiv(SkIntToScalar(kerningAdjustments[0]), size, upm));
+ }
+
+ return 0;
+}
+
+
+static hb_bool_t harfBuzzGetGlyphExtents(hb_font_t* hbFont, void* fontData, hb_codepoint_t glyph, hb_glyph_extents_t* extents, void* userData)
+{
+ HarfBuzzFontData* hbFontData = reinterpret_cast<HarfBuzzFontData*>(fontData);
+
+ SkiaGetGlyphWidthAndExtents(&hbFontData->m_paint, glyph, 0, extents);
+ return true;
+}
+
+static hb_font_funcs_t* harfBuzzSkiaGetFontFuncs()
+{
+ static hb_font_funcs_t* harfBuzzSkiaFontFuncs = 0;
+
+ // We don't set callback functions which we can't support.
+ // HarfBuzz will use the fallback implementation if they aren't set.
+ if (!harfBuzzSkiaFontFuncs) {
+ harfBuzzSkiaFontFuncs = hb_font_funcs_create();
+ hb_font_funcs_set_glyph_func(harfBuzzSkiaFontFuncs, harfBuzzGetGlyph, 0, 0);
+ hb_font_funcs_set_glyph_h_advance_func(harfBuzzSkiaFontFuncs, harfBuzzGetGlyphHorizontalAdvance, 0, 0);
+ hb_font_funcs_set_glyph_h_kerning_func(harfBuzzSkiaFontFuncs, harfBuzzGetGlyphHorizontalKerning, 0, 0);
+ hb_font_funcs_set_glyph_h_origin_func(harfBuzzSkiaFontFuncs, harfBuzzGetGlyphHorizontalOrigin, 0, 0);
+ hb_font_funcs_set_glyph_extents_func(harfBuzzSkiaFontFuncs, harfBuzzGetGlyphExtents, 0, 0);
+ hb_font_funcs_make_immutable(harfBuzzSkiaFontFuncs);
+ }
+ return harfBuzzSkiaFontFuncs;
+}
+
+#if !OS(MACOSX)
+static hb_blob_t* harfBuzzSkiaGetTable(hb_face_t* face, hb_tag_t tag, void* userData)
+{
+ SkTypeface* typeface = reinterpret_cast<SkTypeface*>(userData);
+
+ const size_t tableSize = typeface->getTableSize(tag);
+ if (!tableSize) {
+ return 0;
+ }
+
+ char* buffer = reinterpret_cast<char*>(fastMalloc(tableSize));
+ if (!buffer)
+ return 0;
+ size_t actualSize = typeface->getTableData(tag, 0, tableSize, buffer);
+ if (tableSize != actualSize) {
+ fastFree(buffer);
+ return 0;
+ }
+
+ return hb_blob_create(const_cast<char*>(buffer), tableSize, HB_MEMORY_MODE_WRITABLE, buffer, fastFree);
+}
+#endif
+
+static void destroyHarfBuzzFontData(void* userData)
+{
+ HarfBuzzFontData* hbFontData = reinterpret_cast<HarfBuzzFontData*>(userData);
+ delete hbFontData;
+}
+
+hb_face_t* HarfBuzzFace::createFace()
+{
+#if OS(MACOSX)
+ hb_face_t* face = hb_coretext_face_create(m_platformData->cgFont());
+#else
+ hb_face_t* face = hb_face_create_for_tables(harfBuzzSkiaGetTable, m_platformData->typeface(), 0);
+#endif
+ ASSERT(face);
+ return face;
+}
+
+hb_font_t* HarfBuzzFace::createFont()
+{
+ HarfBuzzFontData* hbFontData = new HarfBuzzFontData(m_glyphCacheForFaceCacheEntry);
+ m_platformData->setupPaint(&hbFontData->m_paint);
+ hb_font_t* font = hb_font_create(m_face);
+ hb_font_set_funcs(font, harfBuzzSkiaGetFontFuncs(), hbFontData, destroyHarfBuzzFontData);
+ float size = m_platformData->size();
+ int scale = SkiaScalarToHarfBuzzPosition(size);
+ hb_font_set_scale(font, scale, scale);
+ hb_font_make_immutable(font);
+ return font;
+}
+
} // namespace blink
« no previous file with comments | « Source/platform/blink_platform.gypi ('k') | Source/platform/fonts/harfbuzz/HarfBuzzFaceCoreText.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698