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

Unified Diff: Source/web/WebAXObject.cpp

Issue 1185343003: Implements the ability to get and set the caret position and the current selection from anywhere in… (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Moved layout tests to another CL. Created 5 years, 5 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
« no previous file with comments | « Source/modules/accessibility/AXObject.h ('k') | public/web/WebAXObject.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/web/WebAXObject.cpp
diff --git a/Source/web/WebAXObject.cpp b/Source/web/WebAXObject.cpp
index 79b2b4757f2251be5d66b71f65f21949d626e45c..683fb0c486c8a4e2d05aa06f55b282b9a661e47d 100644
--- a/Source/web/WebAXObject.cpp
+++ b/Source/web/WebAXObject.cpp
@@ -816,12 +816,47 @@ WebAXRole WebAXObject::role() const
return static_cast<WebAXRole>(m_private->roleValue());
}
+void WebAXObject::selection(WebAXObject& anchorObject, int& anchorOffset,
+ WebAXObject& focusObject, int& focusOffset) const
+{
+ if (isDetached()) {
+ anchorObject = WebAXObject();
+ anchorOffset = -1;
+ focusObject = WebAXObject();
+ focusOffset = -1;
+ return;
+ }
+
+ AXObject::AXRange axSelection = m_private->selection();
+ anchorObject = WebAXObject(axSelection.anchorObject);
+ anchorOffset = axSelection.anchorOffset;
+ focusObject = WebAXObject(axSelection.focusObject);
+ focusOffset = axSelection.focusOffset;
+ return;
+}
+
+void WebAXObject::setSelection(const WebAXObject& anchorObject, int anchorOffset,
+ const WebAXObject& focusObject, int focusOffset) const
+{
+ if (isDetached())
+ return;
+
+ AXObject::AXRange axSelection(anchorObject, anchorOffset,
+ focusObject, focusOffset);
+ m_private->setSelection(axSelection);
+ return;
+}
+
unsigned WebAXObject::selectionEnd() const
{
if (isDetached())
return 0;
- return m_private->selectedTextRange().start + m_private->selectedTextRange().length;
+ AXObject::AXRange axSelection = m_private->selectionUnderObject();
+ if (axSelection.focusOffset < 0)
+ return 0;
+
+ return axSelection.focusOffset;
}
unsigned WebAXObject::selectionStart() const
@@ -829,7 +864,11 @@ unsigned WebAXObject::selectionStart() const
if (isDetached())
return 0;
- return m_private->selectedTextRange().start;
+ AXObject::AXRange axSelection = m_private->selectionUnderObject();
+ if (axSelection.anchorOffset < 0)
+ return 0;
+
+ return axSelection.anchorOffset;
}
unsigned WebAXObject::selectionEndLineNumber() const
@@ -841,6 +880,7 @@ unsigned WebAXObject::selectionEndLineNumber() const
int lineNumber = m_private->lineForPosition(position);
if (lineNumber < 0)
return 0;
+
return lineNumber;
}
@@ -853,6 +893,7 @@ unsigned WebAXObject::selectionStartLineNumber() const
int lineNumber = m_private->lineForPosition(position);
if (lineNumber < 0)
return 0;
+
return lineNumber;
}
@@ -867,7 +908,7 @@ void WebAXObject::setSelectedTextRange(int selectionStart, int selectionEnd) con
if (isDetached())
return;
- m_private->setSelectedTextRange(AXObject::PlainTextRange(selectionStart, selectionEnd - selectionStart));
+ m_private->setSelection(AXObject::AXRange(selectionStart, selectionEnd));
}
void WebAXObject::setValue(WebString value) const
@@ -1489,17 +1530,19 @@ void WebAXObject::wordBoundaries(WebVector<int>& starts, WebVector<int>& ends) c
if (isDetached())
return;
- Vector<AXObject::PlainTextRange> words;
- m_private->wordBoundaries(words);
+ Vector<AXObject::AXRange> wordBoundaries;
+ m_private->wordBoundaries(wordBoundaries);
- WebVector<int> startsWebVector(words.size());
- WebVector<int> endsWebVector(words.size());
- for (size_t i = 0; i < words.size(); i++) {
- startsWebVector[i] = words[i].start;
- endsWebVector[i] = words[i].start + words[i].length;
+ WebVector<int> wordStartOffsets(wordBoundaries.size());
+ WebVector<int> wordEndOffsets(wordBoundaries.size());
+ for (size_t i = 0; i < wordBoundaries.size(); ++i) {
+ ASSERT(wordBoundaries[i].isSimple());
+ wordStartOffsets[i] = wordBoundaries[i].anchorOffset;
+ wordEndOffsets[i] = wordBoundaries[i].focusOffset;
}
- starts.swap(startsWebVector);
- ends.swap(endsWebVector);
+
+ starts.swap(wordStartOffsets);
+ ends.swap(wordEndOffsets);
}
bool WebAXObject::isScrollableContainer() const
« no previous file with comments | « Source/modules/accessibility/AXObject.h ('k') | public/web/WebAXObject.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698