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

Side by Side Diff: third_party/WebKit/Source/core/layout/line/InlineTextBox.h

Issue 2748103015: Replace ASSERT with DCHECK in core/layout/<sub dirs> (Closed)
Patch Set: Rebase with latest 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 * (C) 1999 Lars Knoll (knoll@kde.org) 2 * (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 2000 Dirk Mueller (mueller@kde.org) 3 * (C) 2000 Dirk Mueller (mueller@kde.org)
4 * Copyright (C) 2004, 2005, 2006, 2009, 2010, 2011 Apple Inc. 4 * Copyright (C) 2004, 2005, 2006, 2009, 2010, 2011 Apple Inc.
5 * All rights reserved. 5 * 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 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
64 return LineLayoutText(InlineBox::getLineLayoutItem()); 64 return LineLayoutText(InlineBox::getLineLayoutItem());
65 } 65 }
66 66
67 void destroy() final; 67 void destroy() final;
68 68
69 InlineTextBox* prevTextBox() const { return m_prevTextBox; } 69 InlineTextBox* prevTextBox() const { return m_prevTextBox; }
70 InlineTextBox* nextTextBox() const { return m_nextTextBox; } 70 InlineTextBox* nextTextBox() const { return m_nextTextBox; }
71 void setNextTextBox(InlineTextBox* n) { m_nextTextBox = n; } 71 void setNextTextBox(InlineTextBox* n) { m_nextTextBox = n; }
72 void setPreviousTextBox(InlineTextBox* p) { m_prevTextBox = p; } 72 void setPreviousTextBox(InlineTextBox* p) { m_prevTextBox = p; }
73 73
74 // FIXME: These accessors should ASSERT(!isDirty()). See 74 // FIXME: These accessors should DCHECK(!isDirty()). See
75 // https://bugs.webkit.org/show_bug.cgi?id=97264 75 // https://bugs.webkit.org/show_bug.cgi?id=97264
76 unsigned start() const { return m_start; } 76 unsigned start() const { return m_start; }
77 unsigned end() const { return m_len ? m_start + m_len - 1 : m_start; } 77 unsigned end() const { return m_len ? m_start + m_len - 1 : m_start; }
78 unsigned len() const { return m_len; } 78 unsigned len() const { return m_len; }
79 79
80 void offsetRun(int delta); 80 void offsetRun(int delta);
81 81
82 unsigned short truncation() const { return m_truncation; } 82 unsigned short truncation() const { return m_truncation; }
83 83
84 void markDirty() final; 84 void markDirty() final;
(...skipping 153 matching lines...) Expand 10 before | Expand all | Expand 10 after
238 (expansion() && nextLeafChild() ? TextRun::AllowTrailingExpansion 238 (expansion() && nextLeafChild() ? TextRun::AllowTrailingExpansion
239 : TextRun::ForbidTrailingExpansion); 239 : TextRun::ForbidTrailingExpansion);
240 } 240 }
241 }; 241 };
242 242
243 DEFINE_INLINE_BOX_TYPE_CASTS(InlineTextBox); 243 DEFINE_INLINE_BOX_TYPE_CASTS(InlineTextBox);
244 244
245 } // namespace blink 245 } // namespace blink
246 246
247 #endif // InlineTextBox_h 247 #endif // InlineTextBox_h
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698