Commit c4ff6690 authored by tkent@chromium.org's avatar tkent@chromium.org

AX: remove unused AXObject::selectedText.

This CL removes the followings:
 - AXObject::selectedText
 - AXRenderObject::selectedText, which is an override of AXObject::selectedText
 - AXRenderObject::stringForRange and HTMLTextFormControlElement::selectedText,
   which are used only by AXRenderObject::selectedText.

BUG=none
TEST=none; no behavior changes

Review URL: https://codereview.chromium.org/137783006

git-svn-id: svn://svn.chromium.org/blink/trunk@165113 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent 6ee824c0
......@@ -514,7 +514,6 @@ public:
// Selected text.
virtual PlainTextRange selectedTextRange() const { return PlainTextRange(); }
virtual String selectedText() const { return String(); }
// Modify or take an action on an object.
virtual void increment() { }
......
......@@ -1571,24 +1571,6 @@ VisibleSelection AXRenderObject::selection() const
return m_renderer->frame()->selection().selection();
}
String AXRenderObject::selectedText() const
{
ASSERT(isTextControl());
if (isPasswordField())
return String(); // need to return something distinct from empty string
if (isNativeTextControl() && m_renderer->isTextControl()) {
HTMLTextFormControlElement* textControl = toRenderTextControl(m_renderer)->textFormControlElement();
return textControl->selectedText();
}
if (ariaRoleAttribute() == UnknownRole)
return String();
return stringForRange(ariaSelectedTextRange());
}
//
// Modify or take an action on an object.
//
......@@ -1804,23 +1786,6 @@ void AXRenderObject::lineBreaks(Vector<int>& lineBreaks) const
}
}
// A substring of the text associated with this accessibility object that is
// specified by the given character range.
String AXRenderObject::stringForRange(const PlainTextRange& range) const
{
if (!range.length)
return String();
if (!isTextControl())
return String();
String elementText = isPasswordField() ? String() : text();
if (range.start + range.length > elementText.length())
return String();
return elementText.substring(range.start, range.length);
}
//
// Private.
//
......
......@@ -181,7 +181,6 @@ protected:
// Selected text.
virtual PlainTextRange selectedTextRange() const OVERRIDE;
virtual String selectedText() const OVERRIDE;
// Modify or take an action on an object.
virtual void setSelectedTextRange(const PlainTextRange&) OVERRIDE;
......@@ -227,7 +226,6 @@ private:
bool inheritsPresentationalRole() const;
LayoutRect computeElementRect() const;
VisibleSelection selection() const;
String stringForRange(const PlainTextRange&) const;
int indexForVisiblePosition(const VisiblePosition&) const;
};
......
......@@ -182,13 +182,6 @@ void HTMLTextFormControlElement::select()
setSelectionRange(0, numeric_limits<int>::max(), SelectionHasNoDirection);
}
String HTMLTextFormControlElement::selectedText() const
{
if (!isTextFormControl())
return String();
return value().substring(selectionStart(), selectionEnd() - selectionStart());
}
void HTMLTextFormControlElement::dispatchFormControlChangeEvent()
{
if (m_textAsOfLastFormControlChangeEvent != value()) {
......
......@@ -70,7 +70,6 @@ public:
void setSelectionRange(int start, int end, const String& direction);
void setSelectionRange(int start, int end, TextFieldSelectionDirection = SelectionHasNoDirection);
PassRefPtr<Range> selection() const;
String selectedText() const;
virtual void dispatchFormControlChangeEvent() OVERRIDE FINAL;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment