Add a helper function and update test results with ax role of table header.

content/browser/accessibility/browser_accessibility_cocoa.mm

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <execinfo.h>
 
 #import "content/browser/accessibility/browser_accessibility_cocoa.h"
 
 #include <map>
 
@@ skipping 215 matching lines @@
     int id = uniqueCellIds[i];
     BrowserAccessibility* cell =
         browserAccessibility_->manager()->GetFromID(id);
     if (cell && cell->GetRole() == ui::AX_ROLE_COLUMN_HEADER)
       [ret addObject:cell->ToBrowserAccessibilityCocoa()];
   }
   return ret;
 }
 
 - (NSValue*)columnIndexRange {
-  if ([self internalRole] != ui::AX_ROLE_CELL)
+  if ([self internalRole] != ui::AX_ROLE_CELL &&

[dmazzoni, 2015/01/08 18:11:25]

Please add a helper function for this, like IsCellOrTableHeaderRole()

[je_julie(Not used), 2015/01/09 16:47:37]

Done

+      [self internalRole] != ui::AX_ROLE_COLUMN_HEADER &&
+      [self internalRole] != ui::AX_ROLE_ROW_HEADER)
     return nil;
 
   int column = -1;
   int colspan = -1;
   browserAccessibility_->GetIntAttribute(
       ui::AX_ATTR_TABLE_CELL_COLUMN_INDEX, &column);
   browserAccessibility_->GetIntAttribute(
       ui::AX_ATTR_TABLE_CELL_COLUMN_SPAN, &colspan);
   if (column >= 0 && colspan >= 1)
     return [NSValue valueWithRange:NSMakeRange(column, colspan)];
@@ skipping 412 matching lines @@
     int id = uniqueCellIds[i];
     BrowserAccessibility* cell =
         browserAccessibility_->manager()->GetFromID(id);
     if (cell && cell->GetRole() == ui::AX_ROLE_ROW_HEADER)
       [ret addObject:cell->ToBrowserAccessibilityCocoa()];
   }
   return ret;
 }
 
 - (NSValue*)rowIndexRange {
-  if ([self internalRole] != ui::AX_ROLE_CELL)
+  if ([self internalRole] != ui::AX_ROLE_CELL &&
+      [self internalRole] != ui::AX_ROLE_COLUMN_HEADER &&
+      [self internalRole] != ui::AX_ROLE_ROW_HEADER)
     return nil;
 
   int row = -1;
   int rowspan = -1;
   browserAccessibility_->GetIntAttribute(
       ui::AX_ATTR_TABLE_CELL_ROW_INDEX, &row);
   browserAccessibility_->GetIntAttribute(
       ui::AX_ATTR_TABLE_CELL_ROW_SPAN, &rowspan);
   if (row >= 0 && rowspan >= 1)
     return [NSValue valueWithRange:NSMakeRange(row, rowspan)];
@@ skipping 387 matching lines @@
         continue;
       }
       if (rowIndex < row)
         continue;
       if (rowIndex > row)
         break;
       for (size_t j = 0;
            j < child->PlatformChildCount();
            ++j) {
         BrowserAccessibility* cell = child->PlatformGetChild(j);
-        if (cell->GetRole() != ui::AX_ROLE_CELL)
+        if ([self internalRole] != ui::AX_ROLE_CELL &&
+            [self internalRole] != ui::AX_ROLE_COLUMN_HEADER &&
+            [self internalRole] != ui::AX_ROLE_ROW_HEADER)
           continue;
         int colIndex;
         if (!cell->GetIntAttribute(
                 ui::AX_ATTR_TABLE_CELL_COLUMN_INDEX,
                 &colIndex)) {
           continue;
         }
         if (colIndex == column)
           return cell->ToBrowserAccessibilityCocoa();
         if (colIndex > column)
@@ skipping 444 matching lines @@
     return [super hash];
   return browserAccessibility_->GetId();
 }
 
 - (BOOL)accessibilityShouldUseUniqueId {
   return YES;
 }
 
 @end