Rename LayoutTextFragment::m_end to m_length

third_party/WebKit/Source/core/layout/LayoutTextFragment.h

Index: third_party/WebKit/Source/core/layout/LayoutTextFragment.h
diff --git a/third_party/WebKit/Source/core/layout/LayoutTextFragment.h b/third_party/WebKit/Source/core/layout/LayoutTextFragment.h
index fe4c6f56fd7b75bfc69c92221d8baea1c4c3c121..c9126f52febe63c8a69eb8bafa14b28a664291fd 100644
--- a/third_party/WebKit/Source/core/layout/LayoutTextFragment.h
+++ b/third_party/WebKit/Source/core/layout/LayoutTextFragment.h
@@ -44,7 +44,7 @@ public:
     bool canBeSelectionLeaf() const override { return node() && node()->hasEditableStyle(); }
 
     unsigned start() const { return m_start; }
-    unsigned end() const { return m_end; }
+    unsigned fragmentLength() const { return m_length; }

[dsinclair, 2015/10/26 13:19:34]

This should probably be m_fragmentLength to be consistent.

[dsinclair, 2015/10/26 13:19:34]

Is this never called outside this class? Can we make it private?

[Srirama, 2015/10/26 13:40:30]

Done.

[Srirama, 2015/10/26 13:40:30]

This is not called outside, though start() is used outside this class. So should
we keep it here itself together with start() for better readability?

 
     unsigned textStartOffset() const override { return start(); }
 
@@ -83,7 +83,7 @@ private:
     void updateHitTestResult(HitTestResult&, const LayoutPoint&) override;
 
     unsigned m_start;
-    unsigned m_end;
+    unsigned m_length;
     bool m_isRemainingTextLayoutObject;
     RefPtr<StringImpl> m_contentString;
     // Reference back to FirstLetterPseudoElement; cleared by FirstLetterPseudoElement::detach() if