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

Side by Side Diff: third_party/WebKit/Source/platform/fonts/Character.h

Issue 1541393003: Improve performance of Character::isCJKIdeographOrSymbol by using trie tree (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Cleanup, remove isCJKIdeograph, remove perf test code Created 4 years, 11 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) 2014 Google Inc. All rights reserved. 2 * Copyright (C) 2014 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
54 } 54 }
55 55
56 static inline bool isUnicodeVariationSelector(UChar32 character) 56 static inline bool isUnicodeVariationSelector(UChar32 character)
57 { 57 {
58 // http://www.unicode.org/Public/UCD/latest/ucd/StandardizedVariants.htm l 58 // http://www.unicode.org/Public/UCD/latest/ucd/StandardizedVariants.htm l
59 return isInRange(character, 0x180B, 0x180D) // MONGOLIAN FREE VARIATION SELECTOR ONE to THREE 59 return isInRange(character, 0x180B, 0x180D) // MONGOLIAN FREE VARIATION SELECTOR ONE to THREE
60 || isInRange(character, 0xFE00, 0xFE0F) // VARIATION SELECTOR-1 to 1 6 60 || isInRange(character, 0xFE00, 0xFE0F) // VARIATION SELECTOR-1 to 1 6
61 || isInRange(character, 0xE0100, 0xE01EF); // VARIATION SELECTOR-17 to 256 61 || isInRange(character, 0xE0100, 0xE01EF); // VARIATION SELECTOR-17 to 256
62 } 62 }
63 63
64 static bool isCJKIdeograph(UChar32);
65 static bool isCJKIdeographOrSymbol(UChar32); 64 static bool isCJKIdeographOrSymbol(UChar32);
66 65
67 static unsigned expansionOpportunityCount(const LChar*, size_t length, TextD irection, bool& isAfterExpansion, const TextJustify); 66 static unsigned expansionOpportunityCount(const LChar*, size_t length, TextD irection, bool& isAfterExpansion, const TextJustify);
68 static unsigned expansionOpportunityCount(const UChar*, size_t length, TextD irection, bool& isAfterExpansion, const TextJustify); 67 static unsigned expansionOpportunityCount(const UChar*, size_t length, TextD irection, bool& isAfterExpansion, const TextJustify);
69 68
70 static bool isUprightInMixedVertical(UChar32 character); 69 static bool isUprightInMixedVertical(UChar32 character);
71 70
72 static bool treatAsSpace(UChar c) 71 static bool treatAsSpace(UChar c)
73 { 72 {
74 return c == spaceCharacter 73 return c == spaceCharacter
(...skipping 49 matching lines...) Expand 10 before | Expand all | Expand 10 after
124 123
125 static String normalizeSpaces(const LChar*, unsigned length); 124 static String normalizeSpaces(const LChar*, unsigned length);
126 static String normalizeSpaces(const UChar*, unsigned length); 125 static String normalizeSpaces(const UChar*, unsigned length);
127 126
128 static bool isCommonOrInheritedScript(UChar32); 127 static bool isCommonOrInheritedScript(UChar32);
129 }; 128 };
130 129
131 } // namespace blink 130 } // namespace blink
132 131
133 #endif 132 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698