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

Unified Diff: ui/gfx/render_text_mac.mm

Issue 2756683002: Fix RenderTextMac. (Closed)
Patch Set: Created 3 years, 9 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 | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/gfx/render_text_mac.mm
diff --git a/ui/gfx/render_text_mac.mm b/ui/gfx/render_text_mac.mm
index 948a303ab320c29914d5ad9757acd9ea208f2a91..ec096bb52b37c0a652bb72f8baac62e5c55bb6ca 100644
--- a/ui/gfx/render_text_mac.mm
+++ b/ui/gfx/render_text_mac.mm
@@ -398,14 +398,6 @@ void RenderTextMac::ComputeRuns() {
run->width = run_width;
run->glyphs.resize(glyph_count);
CTRunGetGlyphs(ct_run, empty_cf_range, &run->glyphs[0]);
- // CTRunGetGlyphs() sometimes returns glyphs with value 65535 and zero
- // width (this has been observed at the beginning of a string containing
- // Arabic content). Passing these to Skia will trigger an assertion;
- // instead set their values to 0.
- for (size_t glyph = 0; glyph < glyph_count; glyph++) {
- if (run->glyphs[glyph] == 65535)
- run->glyphs[glyph] = 0;
- }
run->glyph_positions.resize(glyph_count);
const CGPoint* positions_ptr = CTRunGetPositionsPtr(ct_run);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698