[Webkit-unassigned] [Bug 25677] [GTK] Implement support for get_character_extents and get_range_extents

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 29 10:57:56 PDT 2010


chris fleizach <cfleizach at apple.com> changed:

           What    |Removed                     |Added
  Attachment #62755|review?                     |review-
               Flag|                            |

--- Comment #8 from chris fleizach <cfleizach at apple.com>  2010-07-29 10:57:56 PST ---
(From update of attachment 62755)
you could do here
if (!textLength) {
   Node* node = node();
   if (node) {
      RenderText* renderText = toRenderText(node->renderer());

 +          // Plain text a11y object first
don't use made up words like a11y. make sure to write full sentences for your comments

 +          if (isWebArea() || isGroup() || isLink() || isHeading()) {
this should be else if, no?

 +              textLength = textValue.length();
can't you do textLength = textUnderElement().length();

 +              // Check composite objects just in case
this comment should go above the if and should be a full sentence. you should also enumerate what "just in case" means

 +  #else
you also check isWebArea in the other locations. do you not want to do it here?

 +              // Check composite objects just in case
make sure to end comments with a period

 +      bool isTextObject =  isTextControl() || m_renderer->isText()
too much white space

 +  #endif
instead of doing this PLATFORM stuff and enumerating in each instance, i think you should have a platform specific method on AccessibilityObject, like

AccessibilityObject::allowsTextRanges() or something appropriate

then in your AccessibilityObjectGTK.cpp you can override. for the other platforms you can have the default impl live in AccessibilityObject and do what it does now

 +          rangeLength = textLength;
if the rangelength == 0 are you sure you want to reset to the text length?

i don't see renderObject being used anywhere

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list