[SCM] WebKit Debian packaging branch, webkit-1.2, updated. upstream/1.1.90-6072-g9a69373
eric at webkit.org
eric at webkit.org
Wed Apr 7 23:16:36 UTC 2010
The following commit has been merged in the webkit-1.2 branch:
commit f62e3d8c3b6c7cb4fcb1b2ed1e4d9d8643871f72
Author: eric at webkit.org <eric at webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date: Fri Oct 30 17:35:12 2009 +0000
2009-10-30 Alexander Pavlov <apavlov at chromium.org>
Reviewed by Timothy Hatcher.
Fix Web Inspector crash on the errors/warnings counter click
RenderObject::createVisiblePosition(const Position& position)
understands "null Positions", so we can construct such a Position manually.
https://bugs.webkit.org/show_bug.cgi?id=30499
* rendering/RenderBox.cpp:
(WebCore::RenderBox::positionForPoint):
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@50342 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/WebCore/ChangeLog b/WebCore/ChangeLog
index 129b691..27fd1c9 100644
--- a/WebCore/ChangeLog
+++ b/WebCore/ChangeLog
@@ -19,6 +19,30 @@
https://bugs.webkit.org/show_bug.cgi?id=25534
[GTK] Objects of ROLE_TABLE should implement the accessible table interface
+ Third part of the implementation of AtkTable.
+
+ * accessibility/gtk/AccessibilityObjectWrapperAtk.cpp:
+ (nameFromChildren):
+ (webkit_accessible_get_name):
+ New convenience function to construct an object's name using the name(s) of
+ any children it has.
+
+ (atk_table_interface_init):
+ (webkit_accessible_table_get_column_description):
+ (webkit_accessible_table_get_row_description):
+ Implemented.
+
+ (webkit_accessible_table_get_column_header):
+ Stub function added so that webkit_accessible_table_get_column_description
+ could be implemented in the meantime.
+
+2009-10-30 Joanmarie Diggs <joanmarie.diggs at gmail.com>
+
+ Reviewed by Xan Lopez.
+
+ https://bugs.webkit.org/show_bug.cgi?id=25534
+ [GTK] Objects of ROLE_TABLE should implement the accessible table interface
+
Second part of the implementation of AtkTable.
* accessibility/gtk/AccessibilityObjectWrapperAtk.cpp:
diff --git a/WebCore/accessibility/gtk/AccessibilityObjectWrapperAtk.cpp b/WebCore/accessibility/gtk/AccessibilityObjectWrapperAtk.cpp
index 2721359..2ea8336 100644
--- a/WebCore/accessibility/gtk/AccessibilityObjectWrapperAtk.cpp
+++ b/WebCore/accessibility/gtk/AccessibilityObjectWrapperAtk.cpp
@@ -137,21 +137,27 @@ static AccessibilityObject* core(AtkTable* table)
return core(ATK_OBJECT(table));
}
+static const gchar* nameFromChildren(AccessibilityObject* object)
+{
+ if (!object)
+ return 0;
+
+ 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);
+}
+
static const gchar* webkit_accessible_get_name(AtkObject* object)
{
AccessibilityObject* coreObject = core(object);
if (coreObject->isControl()) {
AccessibilityRenderObject* renderObject = static_cast<AccessibilityRenderObject*>(coreObject);
AccessibilityObject* label = renderObject->correspondingLabelForControlElement();
- if (label) {
- AccessibilityRenderObject::AccessibilityChildrenVector children = label->children();
- // Currently, label->stringValue() should be an empty String. This
- // might not be the case down the road.
- String name = label->stringValue();
- for (unsigned i = 0; i < children.size(); ++i)
- name += children.at(i).get()->stringValue();
- return returnString(name);
- }
+ if (label)
+ return returnString(nameFromChildren(label));
}
return returnString(coreObject->stringValue());
}
@@ -1327,6 +1333,13 @@ static gint webkit_accessible_table_get_row_extent_at(AtkTable* table, gint row,
return 0;
}
+static AtkObject* webkit_accessible_table_get_column_header(AtkTable* table, gint column)
+{
+ // FIXME: This needs to be implemented.
+ notImplemented();
+ return 0;
+}
+
static AtkObject* webkit_accessible_table_get_row_header(AtkTable* table, gint row)
{
AccessibilityObject* accTable = core(table);
@@ -1358,6 +1371,24 @@ static AtkObject* webkit_accessible_table_get_caption(AtkTable* table)
return 0;
}
+static const gchar* webkit_accessible_table_get_column_description(AtkTable* table, gint column)
+{
+ AtkObject* columnHeader = atk_table_get_column_header(table, column);
+ if (columnHeader)
+ return returnString(nameFromChildren(core(columnHeader)));
+
+ return 0;
+}
+
+static const gchar* webkit_accessible_table_get_row_description(AtkTable* table, gint row)
+{
+ AtkObject* rowHeader = atk_table_get_row_header(table, row);
+ if (rowHeader)
+ return returnString(nameFromChildren(core(rowHeader)));
+
+ return 0;
+}
+
static void atk_table_interface_init(AtkTableIface* iface)
{
iface->ref_at = webkit_accessible_table_ref_at;
@@ -1368,8 +1399,11 @@ static void atk_table_interface_init(AtkTableIface* iface)
iface->get_n_rows = webkit_accessible_table_get_n_rows;
iface->get_column_extent_at = webkit_accessible_table_get_column_extent_at;
iface->get_row_extent_at = webkit_accessible_table_get_row_extent_at;
+ iface->get_column_header = webkit_accessible_table_get_column_header;
iface->get_row_header = webkit_accessible_table_get_row_header;
iface->get_caption = webkit_accessible_table_get_caption;
+ iface->get_column_description = webkit_accessible_table_get_column_description;
+ iface->get_row_description = webkit_accessible_table_get_row_description;
}
static const GInterfaceInfo AtkInterfacesInitFunctions[] = {
--
WebKit Debian packaging
More information about the Pkg-webkit-commits
mailing list