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

Side by Side Diff: Source/platform/fonts/Font.cpp

Issue 622653002: Force ComplexPath if any part of the node requires it (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * (C) 2000 Dirk Mueller (mueller@kde.org) 4 * (C) 2000 Dirk Mueller (mueller@kde.org)
5 * Copyright (C) 2003, 2006, 2010, 2011 Apple Inc. All rights reserved. 5 * Copyright (C) 2003, 2006, 2010, 2011 Apple Inc. All rights reserved.
6 * 6 *
7 * This library is free software; you can redistribute it and/or 7 * This library is free software; you can redistribute it and/or
8 * modify it under the terms of the GNU Library General Public 8 * modify it under the terms of the GNU Library General Public
9 * License as published by the Free Software Foundation; either 9 * License as published by the Free Software Foundation; either
10 * version 2 of the License, or (at your option) any later version. 10 * version 2 of the License, or (at your option) any later version.
(...skipping 280 matching lines...) Expand 10 before | Expand all | Expand 10 after
291 291
292 if (run.length() > 1 && fontDescription().typesettingFeatures()) 292 if (run.length() > 1 && fontDescription().typesettingFeatures())
293 return ComplexPath; 293 return ComplexPath;
294 294
295 if (!run.characterScanForCodePath()) 295 if (!run.characterScanForCodePath())
296 return SimplePath; 296 return SimplePath;
297 297
298 if (run.is8Bit()) 298 if (run.is8Bit())
299 return SimplePath; 299 return SimplePath;
300 300
301 if (run.useComplexCodePath())
leviw_travelin_and_unemployed 2014/10/01 20:33:14 Are you sure you want this check here? Could a Ren
eae 2014/10/01 21:16:43 Yes but then it would be due to a CSS setting in w
leviw_travelin_and_unemployed 2014/10/01 21:37:28 I'm not sure I understand. The case I'm thinking i
302 return ComplexPath;
303
301 // Start from 0 since drawing and highlighting also measure the characters b efore run->from. 304 // Start from 0 since drawing and highlighting also measure the characters b efore run->from.
302 return Character::characterRangeCodePath(run.characters16(), run.length()); 305 return Character::characterRangeCodePath(run.characters16(), run.length());
303 } 306 }
304 307
305 void Font::willUseFontData(UChar32 character) const 308 void Font::willUseFontData(UChar32 character) const
306 { 309 {
307 const FontFamily& family = fontDescription().family(); 310 const FontFamily& family = fontDescription().family();
308 if (m_fontFallbackList && m_fontFallbackList->fontSelector() && !family.fami lyIsEmpty()) 311 if (m_fontFallbackList && m_fontFallbackList->fontSelector() && !family.fami lyIsEmpty())
309 m_fontFallbackList->fontSelector()->willUseFontData(fontDescription(), f amily.family(), character); 312 m_fontFallbackList->fontSelector()->willUseFontData(fontDescription(), f amily.family(), character);
310 } 313 }
(...skipping 585 matching lines...) Expand 10 before | Expand all | Expand 10 after
896 if (delta <= 0) 899 if (delta <= 0)
897 break; 900 break;
898 } 901 }
899 } 902 }
900 } 903 }
901 904
902 return offset; 905 return offset;
903 } 906 }
904 907
905 } // namespace blink 908 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698