[webkit-reviews] review denied: [Bug 25534] [GTK] Objects of ROLE_TABLE should implement the accessible table interface : [Attachment 41924] table 3
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Oct 27 05:53:29 PDT 2009
Xan Lopez <xan.lopez at gmail.com> has denied Joanmarie Diggs
<joanmarie.diggs at gmail.com>'s request for review:
Bug 25534: [GTK] Objects of ROLE_TABLE should implement the accessible table
interface
https://bugs.webkit.org/show_bug.cgi?id=25534
Attachment 41924: table 3
https://bugs.webkit.org/attachment.cgi?id=41924&action=review
------- Additional Comments from Xan Lopez <xan.lopez at gmail.com>
>+static const gchar* nameFromChildren(AccessibilityObject* object)
>+{
>+ if (object) {
>+ AccessibilityRenderObject::AccessibilityChildrenVector children =
object->children();
>+ // Currently, object->stringValue() should be an empty String. This
>+ // might not be the case down the road.
>+ String name = object->stringValue();
>+ for (unsigned i = 0; i < children.size(); ++i)
>+ name += children.at(i).get()->stringValue();
>+ return returnString(name);
>+ }
>+
>+ return NULL;
I think it's better to do a if (!object)\nreturn 0; at the beginning here.
>+static AtkObject* webkit_accessible_table_get_column_header(AtkTable* table,
gint column)
>+{
>+ // FIXME: This needs to be implemented.
>+ notImplemented();
>+ return NULL;
>+}
Same comment about NULL than in the other patches (sorry about that!).
r- for now, let's fix those two things.
More information about the webkit-reviews
mailing list