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

Unified Diff: third_party/WebKit/Source/core/editing/VisibleUnits.cpp

Issue 2181953004: Changed ComputedStyle's EVisibility to be an enum class and fixed naming (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@_move_computedstyle_around
Patch Set: Rebase Created 4 years, 4 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/VisibleUnits.cpp
diff --git a/third_party/WebKit/Source/core/editing/VisibleUnits.cpp b/third_party/WebKit/Source/core/editing/VisibleUnits.cpp
index 30dc1f5d6bb8878d5b6727191fe61abb7c8524dd..ce8de63a7a44d8c6dc5bb73fcd88e4da110cfd16 100644
--- a/third_party/WebKit/Source/core/editing/VisibleUnits.cpp
+++ b/third_party/WebKit/Source/core/editing/VisibleUnits.cpp
@@ -1558,7 +1558,7 @@ PositionTemplate<Strategy> startOfParagraphAlgorithm(const PositionTemplate<Stra
continue;
}
const ComputedStyle& style = layoutItem.styleRef();
- if (style.visibility() != VISIBLE) {
+ if (style.visibility() != EVisibility::Visible) {
prevousNodeIterator = Strategy::previousPostOrder(*prevousNodeIterator, startBlock);
continue;
}
@@ -1649,7 +1649,7 @@ static PositionTemplate<Strategy> endOfParagraphAlgorithm(const PositionTemplate
continue;
}
const ComputedStyle& style = layoutObject->styleRef();
- if (style.visibility() != VISIBLE) {
+ if (style.visibility() != EVisibility::Visible) {
nextNodeItreator = Strategy::next(*nextNodeItreator, startBlock);
continue;
}
@@ -2457,7 +2457,7 @@ static PositionTemplate<Strategy> mostBackwardCaretPosition(const PositionTempla
// skip position in non-laid out or invisible node
LayoutObject* layoutObject = associatedLayoutObjectOf(*currentNode, currentPos.offsetInLeafNode());
- if (!layoutObject || layoutObject->style()->visibility() != VISIBLE)
+ if (!layoutObject || layoutObject->style()->visibility() != EVisibility::Visible)
continue;
if (rule == CanCrossEditingBoundary && boundaryCrossed) {
@@ -2505,7 +2505,7 @@ static PositionTemplate<Strategy> mostBackwardCaretPosition(const PositionTempla
// |Text| node with :first-letter.
DCHECK_GE(currentPos.offsetInLeafNode(), 1);
LayoutObject* firstLetterLayoutObject = toLayoutTextFragment(layoutObject)->firstLetterPseudoElement()->layoutObject();
- if (firstLetterLayoutObject && firstLetterLayoutObject->style()->visibility() == VISIBLE)
+ if (firstLetterLayoutObject && firstLetterLayoutObject->style()->visibility() == EVisibility::Visible)
return currentPos.computePosition();
}
continue;
@@ -2608,7 +2608,7 @@ PositionTemplate<Strategy> mostForwardCaretPosition(const PositionTemplate<Strat
// skip position in non-laid out or invisible node
LayoutObject* layoutObject = associatedLayoutObjectOf(*currentNode, currentPos.offsetInLeafNode());
- if (!layoutObject || layoutObject->style()->visibility() != VISIBLE)
+ if (!layoutObject || layoutObject->style()->visibility() != EVisibility::Visible)
continue;
if (rule == CanCrossEditingBoundary && boundaryCrossed) {
@@ -2726,7 +2726,7 @@ static bool isVisuallyEquivalentCandidateAlgorithm(const PositionTemplate<Strate
if (!layoutObject)
return false;
- if (layoutObject->style()->visibility() != VISIBLE)
+ if (layoutObject->style()->visibility() != EVisibility::Visible)
return false;
if (layoutObject->isBR()) {

Powered by Google App Engine
This is Rietveld 408576698