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

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

Issue 2465803002: Get rid of VisibleSelection::expandUsingGranularity() (Closed)
Patch Set: 2016-10-31T17:16:07 Created 4 years, 2 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/VisibleSelection.cpp
diff --git a/third_party/WebKit/Source/core/editing/VisibleSelection.cpp b/third_party/WebKit/Source/core/editing/VisibleSelection.cpp
index 04527dafb5f0fbe0f1a08f0301217b13ff6895e5..440180ed1f0829ad486fbab9aded519ed351494f 100644
--- a/third_party/WebKit/Source/core/editing/VisibleSelection.cpp
+++ b/third_party/WebKit/Source/core/editing/VisibleSelection.cpp
@@ -59,7 +59,7 @@ VisibleSelectionTemplate<Strategy>::VisibleSelectionTemplate(
m_isDirectional(selection.isDirectional()),
m_granularity(selection.granularity()),
m_hasTrailingWhitespace(selection.hasTrailingWhitespace()) {
- validate();
+ validate(m_granularity);
}
template <typename Strategy>
@@ -207,14 +207,6 @@ VisibleSelectionTemplate<Strategy>::toNormalizedEphemeralRange() const {
}
template <typename Strategy>
-void VisibleSelectionTemplate<Strategy>::expandUsingGranularity(
- TextGranularity granularity) {
- if (isNone())
- return;
- validate(granularity);
-}
-
-template <typename Strategy>
static EphemeralRangeTemplate<Strategy> makeSearchRange(
const PositionTemplate<Strategy>& pos) {
Node* node = pos.anchorNode();

Powered by Google App Engine
This is Rietveld 408576698