Chromium Code Reviews| Index: third_party/WebKit/Source/core/layout/BidiRunForLine.cpp |
| diff --git a/third_party/WebKit/Source/core/layout/BidiRunForLine.cpp b/third_party/WebKit/Source/core/layout/BidiRunForLine.cpp |
| index 8b26465fdfd8dc99e8033211ce2eeae9f1407ec2..2a5d15d9c90c7d5bc2f4e3178467c638232f825a 100644 |
| --- a/third_party/WebKit/Source/core/layout/BidiRunForLine.cpp |
| +++ b/third_party/WebKit/Source/core/layout/BidiRunForLine.cpp |
| @@ -109,8 +109,8 @@ void constructBidiRunsForLine(InlineBidiResolver& topResolver, |
| bool isNewUBAParagraph) { |
| // FIXME: We should pass a BidiRunList into createBidiRunsForLine instead |
| // of the resolver owning the runs. |
| - ASSERT(&topResolver.runs() == &bidiRuns); |
| - ASSERT(topResolver.position() != endOfLine); |
| + DCHECK_EQ(&topResolver.runs(), &bidiRuns); |
| + DCHECK(topResolver.position() != endOfLine); |
|
tkent
2017/04/04 01:36:16
Use DCHECK_NE if it doesn't cause a compile failur
mrunal
2017/04/05 00:39:14
Not possible because of compile failure.
|
| LineLayoutItem currentRoot = topResolver.position().root(); |
| topResolver.createBidiRunsForLine(endOfLine, override, |
| previousLineBrokeCleanly); |
| @@ -133,7 +133,7 @@ void constructBidiRunsForLine(InlineBidiResolver& topResolver, |
| // nothing about LayoutObject). |
| LineLayoutItem isolatedInline = |
| highestContainingIsolateWithinRoot(startObj, currentRoot); |
| - ASSERT(isolatedInline); |
| + DCHECK(isolatedInline); |
| InlineBidiResolver isolatedResolver; |
| LineMidpointState& isolatedLineMidpointState = |
| @@ -171,7 +171,7 @@ void constructBidiRunsForLine(InlineBidiResolver& topResolver, |
| isolatedResolver.createBidiRunsForLine(endOfLine, NoVisualOverride, |
| previousLineBrokeCleanly); |
| - ASSERT(isolatedResolver.runs().runCount()); |
| + DCHECK(isolatedResolver.runs().runCount()); |
| if (isolatedResolver.runs().runCount()) |
| bidiRuns.replaceRunWithRuns(&isolatedRun.runToReplace, |
| isolatedResolver.runs()); |