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

Side by Side Diff: third_party/WebKit/Source/platform/text/BidiCharacterRun.h

Issue 2811453002: Replace ASSERT, ASSERT_NOT_REACHED, and RELEASE_ASSERT in platform/text (Closed)
Patch Set: Replace ASSERT, ASSERT_NOT_REACHED, and RELEASE_ASSERT in platform/text Created 3 years, 8 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) 2000 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 2000 Lars Knoll (knoll@kde.org)
3 * Copyright (C) 2003, 2004, 2006, 2007, 2008 Apple Inc. All right reserved. 3 * Copyright (C) 2003, 2004, 2006, 2007, 2008 Apple Inc. All right reserved.
4 * 4 *
5 * This library is free software; you can redistribute it and/or 5 * This library is free software; you can redistribute it and/or
6 * modify it under the terms of the GNU Library General Public 6 * modify it under the terms of the GNU Library General Public
7 * License as published by the Free Software Foundation; either 7 * License as published by the Free Software Foundation; either
8 * version 2 of the License, or (at your option) any later version. 8 * version 2 of the License, or (at your option) any later version.
9 * 9 *
10 * This library is distributed in the hope that it will be useful, 10 * This library is distributed in the hope that it will be useful,
(...skipping 21 matching lines...) Expand all
32 unsigned char level, 32 unsigned char level,
33 int start, 33 int start,
34 int stop, 34 int stop,
35 WTF::Unicode::CharDirection dir, 35 WTF::Unicode::CharDirection dir,
36 WTF::Unicode::CharDirection overrideDir) 36 WTF::Unicode::CharDirection overrideDir)
37 : m_override(override), 37 : m_override(override),
38 m_level(level), 38 m_level(level),
39 m_next(0), 39 m_next(0),
40 m_start(start), 40 m_start(start),
41 m_stop(stop) { 41 m_stop(stop) {
42 ASSERT(m_start <= m_stop); 42 DCHECK_LE(m_start, m_stop);
43 if (dir == WTF::Unicode::OtherNeutral) 43 if (dir == WTF::Unicode::OtherNeutral)
44 dir = overrideDir; 44 dir = overrideDir;
45 45
46 m_level = level; 46 m_level = level;
47 47
48 // add level of run (cases I1 & I2) 48 // add level of run (cases I1 & I2)
49 if (m_level % 2) { 49 if (m_level % 2) {
50 if (dir == WTF::Unicode::LeftToRight || 50 if (dir == WTF::Unicode::LeftToRight ||
51 dir == WTF::Unicode::ArabicNumber || 51 dir == WTF::Unicode::ArabicNumber ||
52 dir == WTF::Unicode::EuropeanNumber) 52 dir == WTF::Unicode::EuropeanNumber)
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
95 bool m_hasHyphen : 1; 95 bool m_hasHyphen : 1;
96 unsigned char m_level; 96 unsigned char m_level;
97 BidiCharacterRun* m_next; 97 BidiCharacterRun* m_next;
98 int m_start; 98 int m_start;
99 int m_stop; 99 int m_stop;
100 }; 100 };
101 101
102 } // namespace blink 102 } // namespace blink
103 103
104 #endif // BidiCharacterRun_h 104 #endif // BidiCharacterRun_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698