Chromium Code Reviews| Index: Source/platform/text/BidiResolver.h |
| diff --git a/Source/platform/text/BidiResolver.h b/Source/platform/text/BidiResolver.h |
| index 68aeec1027e7aaab2495df8329bc239e00806de1..c25705290ccf60d1af212d8729d572a7b74c10d4 100644 |
| --- a/Source/platform/text/BidiResolver.h |
| +++ b/Source/platform/text/BidiResolver.h |
| @@ -219,7 +219,7 @@ public: |
| void embed(WTF::Unicode::Direction, BidiEmbeddingSource); |
| bool commitExplicitEmbedding(); |
| - void createBidiRunsForLine(const Iterator& end, VisualDirectionOverride = NoVisualOverride, bool hardLineBreak = false); |
| + void createBidiRunsForLine(const Iterator& end, VisualDirectionOverride = NoVisualOverride, bool hardLineBreak = false, bool reorderRuns = true); |
|
leviw_travelin_and_unemployed
2014/01/09 00:49:54
Boolean params make me sad. It'd be great if we co
|
| BidiRunList<Run>& runs() { return m_runs; } |
| @@ -635,7 +635,7 @@ TextDirection BidiResolver<Iterator, Run>::determineParagraphDirectionality(bool |
| } |
| template <class Iterator, class Run> |
| -void BidiResolver<Iterator, Run>::createBidiRunsForLine(const Iterator& end, VisualDirectionOverride override, bool hardLineBreak) |
| +void BidiResolver<Iterator, Run>::createBidiRunsForLine(const Iterator& end, VisualDirectionOverride override, bool hardLineBreak, bool reorderRuns) |
| { |
| using namespace WTF::Unicode; |
| @@ -1034,7 +1034,8 @@ void BidiResolver<Iterator, Run>::createBidiRunsForLine(const Iterator& end, Vis |
| } |
| m_runs.setLogicallyLastRun(m_runs.lastRun()); |
| - reorderRunsFromLevels(); |
| + if (reorderRuns) |
| + reorderRunsFromLevels(); |
| m_endOfRunAtEndOfLine = Iterator(); |
| m_endOfLine = Iterator(); |