[orca] Fix for bgo#618863 - Table structural navigation not working with th element in Firefox 3.6

[Joanmarie Diggs <joanied src gnome org>]

commit d6222691a6546cbfb6b220f54e2ea131a90a40cd
Author: Joanmarie Diggs <joanmarie diggs gmail com>
Date:   Mon May 17 08:35:10 2010 -0400

    Fix for bgo#618863 - Table structural navigation not working with th element in Firefox 3.6

 src/orca/structural_navigation.py |   45 ++++++++++++++++++++++++++++--------
 1 files changed, 35 insertions(+), 10 deletions(-)
---
diff --git a/src/orca/structural_navigation.py b/src/orca/structural_navigation.py
index b5b0ba9..53d13bc 100644
--- a/src/orca/structural_navigation.py
+++ b/src/orca/structural_navigation.py
@@ -1373,10 +1373,11 @@ class StructuralNavigation:
         - obj: the accessible object of interest.
         """
 
-        if obj and obj.getRole() != pyatspi.ROLE_TABLE_CELL:
+        cellRoles = [pyatspi.ROLE_TABLE_CELL, pyatspi.ROLE_COLUMN_HEADER]
+        if obj and not obj.getRole() in cellRoles:
             document = self._getDocument()
             obj = self._script.utilities.ancestorWithRole(
-                obj, [pyatspi.ROLE_TABLE_CELL], [document.getRole()])
+                obj, cellRoles, [document.getRole()])
         return obj
 
     def getTableForCell(self, obj):
@@ -1399,6 +1400,9 @@ class StructuralNavigation:
         - obj: the accessible table cell to examime
         """
 
+        if obj and obj.getRole() == pyatspi.ROLE_COLUMN_HEADER and obj.name:
+            return False
+
         text = self._script.utilities.displayedText(obj)
         if text and len(text.strip()) and text != obj.name:
             return False
@@ -1474,8 +1478,13 @@ class StructuralNavigation:
         if not obj or (obj.getRole() != pyatspi.ROLE_TABLE_CELL):
             return
 
+        parentTable = self.getTableForCell(obj)
+        try:
+            table = parentTable.queryTable()
+        except:
+            return
+
         [row, col] = self.getCellCoordinates(obj)
-        table = obj.parent.queryTable()
         rowspan = table.getRowExtentAt(row, col)
         colspan = table.getColumnExtentAt(row, col)
         spanString = ""
@@ -1548,8 +1557,9 @@ class StructuralNavigation:
         if not obj:
             return rowHeaders
 
+        parentTable = self.getTableForCell(obj)
         try:
-            table = obj.parent.queryTable()
+            table = parentTable.queryTable()
         except:
             pass
         else:
@@ -1595,8 +1605,9 @@ class StructuralNavigation:
         if not obj:
             return columnHeaders
 
+        parentTable = self.getTableForCell(obj)
         try:
-            table = obj.parent.queryTable()
+            table = parentTable.queryTable()
         except:
             pass
         else:
@@ -1639,7 +1650,12 @@ class StructuralNavigation:
         """
 
         if obj and obj.getRole() == pyatspi.ROLE_TABLE_CELL:
-            table = obj.parent.queryTable()
+            parentTable = self.getTableForCell(obj)
+            try:
+                table = parentTable.queryTable()
+            except:
+                return True
+
             index = self._script.utilities.cellIndex(obj)
             row = table.getRowAtIndex(index)
             for col in xrange(table.nColumns):
@@ -1658,7 +1674,12 @@ class StructuralNavigation:
         """
 
         if obj and obj.getRole() == pyatspi.ROLE_TABLE_CELL:
-            table = obj.parent.queryTable()
+            parentTable = self.getTableForCell(obj)
+            try:
+                table = parentTable.queryTable()
+            except:
+                return True
+
             index = self._script.utilities.cellIndex(obj)
             col = table.getColumnAtIndex(index)
             for row in xrange(table.nRows):
@@ -1679,7 +1700,8 @@ class StructuralNavigation:
             return False
 
         elif obj.getRole() in [pyatspi.ROLE_TABLE_COLUMN_HEADER,
-                               pyatspi.ROLE_TABLE_ROW_HEADER]:
+                               pyatspi.ROLE_TABLE_ROW_HEADER,
+                               pyatspi.ROLE_COLUMN_HEADER]:
             return True
 
         else:
@@ -1689,6 +1711,8 @@ class StructuralNavigation:
                     if attribute == "tag:TH":
                         return True
 
+        return False
+
     def _getHeadingLevel(self, obj):
         """Determines the heading level of the given object.  A value
         of 0 means there is no heading level.
@@ -3331,7 +3355,7 @@ class StructuralNavigation:
           the criteria (e.g. the level of a heading).
         """
 
-        role = [pyatspi.ROLE_TABLE_CELL]
+        role = [pyatspi.ROLE_TABLE_CELL, pyatspi.ROLE_COLUMN_HEADER]
         return MatchCriteria(collection, roles=role)
 
     def _tableCellPredicate(self, obj, arg=None):
@@ -3344,7 +3368,8 @@ class StructuralNavigation:
           the criteria (e.g. the level of a heading).
         """
 
-        return (obj and obj.getRole() == pyatspi.ROLE_TABLE_CELL)
+        return (obj and obj.getRole() in [pyatspi.ROLE_COLUMN_HEADER,
+                                          pyatspi.ROLE_TABLE_CELL])
 
     def _tableCellPresentation(self, cell, arg):
         """Presents the table cell or indicates that one was not found.