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

Unified Diff: third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp

Issue 2614883007: Change computed style enums to be prefixed with 'k'. (Closed)
Patch Set: Rebase on ToT. Created 3 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp
diff --git a/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp b/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp
index e435441f99da0e38cb2ae9ef37a1d115faf26634..69bc7851635c2843cc2a421890729ff86b32017a 100644
--- a/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp
+++ b/third_party/WebKit/Source/core/editing/iterators/TextIterator.cpp
@@ -532,7 +532,7 @@ void TextIteratorAlgorithm<Strategy>::advance() {
}
static bool hasVisibleTextNode(LayoutText* layoutObject) {
- if (layoutObject->style()->visibility() == EVisibility::Visible)
+ if (layoutObject->style()->visibility() == EVisibility::kVisible)
return true;
if (!layoutObject->isTextFragment())
@@ -547,7 +547,7 @@ static bool hasVisibleTextNode(LayoutText* layoutObject) {
fragment->firstLetterPseudoElement()->layoutObject();
return pseudoElementLayoutObject &&
pseudoElementLayoutObject->style()->visibility() ==
- EVisibility::Visible;
+ EVisibility::kVisible;
}
template <typename Strategy>
@@ -592,7 +592,7 @@ bool TextIteratorAlgorithm<Strategy>::handleTextNode() {
return false;
}
}
- if (layoutObject->style()->visibility() != EVisibility::Visible &&
+ if (layoutObject->style()->visibility() != EVisibility::kVisible &&
!ignoresStyleVisibility())
return false;
int strLength = str.length();
@@ -616,7 +616,7 @@ bool TextIteratorAlgorithm<Strategy>::handleTextNode() {
if (!layoutObject->firstTextBox() && str.length() > 0 &&
!shouldHandleFirstLetter) {
- if (layoutObject->style()->visibility() != EVisibility::Visible &&
+ if (layoutObject->style()->visibility() != EVisibility::kVisible &&
!ignoresStyleVisibility())
return false;
m_lastTextNodeEndedWithCollapsedSpace =
@@ -698,7 +698,7 @@ void TextIteratorAlgorithm<Strategy>::handleTextBox() {
? m_firstLetterText
: toLayoutText(m_node->layoutObject());
- if (layoutObject->style()->visibility() != EVisibility::Visible &&
+ if (layoutObject->style()->visibility() != EVisibility::kVisible &&
!ignoresStyleVisibility()) {
m_textBox = nullptr;
} else {
@@ -767,7 +767,7 @@ void TextIteratorAlgorithm<Strategy>::handleTextBox() {
if (str[runStart] == '\n') {
// We need to preserve new lines in case of PreLine.
// See bug crbug.com/317365.
- if (layoutObject->style()->whiteSpace() == EWhiteSpace::PreLine)
+ if (layoutObject->style()->whiteSpace() == EWhiteSpace::kPreLine)
spliceBuffer('\n', m_node, 0, runStart, runStart);
else
spliceBuffer(spaceCharacter, m_node, 0, runStart, runStart + 1);
@@ -832,7 +832,7 @@ void TextIteratorAlgorithm<Strategy>::handleTextNodeFirstLetter(
return;
LayoutObject* pseudoLayoutObject = firstLetterElement->layoutObject();
- if (pseudoLayoutObject->style()->visibility() != EVisibility::Visible &&
+ if (pseudoLayoutObject->style()->visibility() != EVisibility::kVisible &&
!ignoresStyleVisibility())
return;
@@ -866,7 +866,7 @@ bool TextIteratorAlgorithm<Strategy>::handleReplacedElement() {
return false;
LayoutObject* layoutObject = m_node->layoutObject();
- if (layoutObject->style()->visibility() != EVisibility::Visible &&
+ if (layoutObject->style()->visibility() != EVisibility::kVisible &&
!ignoresStyleVisibility())
return false;
@@ -1083,7 +1083,7 @@ bool TextIteratorAlgorithm<Strategy>::shouldRepresentNodeOffsetZero() {
// unrendered content, we would create VisiblePositions on every call to this
// function without this check.
if (!m_node->layoutObject() ||
- m_node->layoutObject()->style()->visibility() != EVisibility::Visible ||
+ m_node->layoutObject()->style()->visibility() != EVisibility::kVisible ||
(m_node->layoutObject()->isLayoutBlockFlow() &&
!toLayoutBlock(m_node->layoutObject())->size().height() &&
!isHTMLBodyElement(*m_node)))

Powered by Google App Engine
This is Rietveld 408576698