Calls BrowserAccessibility::IsCellOrTableHeaderRole() with a correct instance.

content/browser/accessibility/browser_accessibility.cc

 // 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 "content/browser/accessibility/browser_accessibility.h"
 
 #include <algorithm>
 
 #include "base/logging.h"
 #include "base/strings/string_number_conversions.h"
@@ skipping 666 matching lines @@
   if (EqualsASCII(value, "mixed"))
     *is_mixed = true;
 
   return false;  // Not set
 }
 
 bool BrowserAccessibility::HasState(ui::AXState state_enum) const {
   return (GetState() >> state_enum) & 1;
 }
 
-bool BrowserAccessibility::IsCellOrTableHeaderRole() const {
-  return (GetRole() == ui::AX_ROLE_CELL ||
-          GetRole() == ui::AX_ROLE_COLUMN_HEADER ||
-          GetRole() == ui::AX_ROLE_ROW_HEADER);
+bool BrowserAccessibility::IsCellOrTableHeaderRole(int32 role) const {

[dmazzoni, 2015/03/11 16:31:41]

I don't think it's necessary to add this argument - see comment below.

+  return (role == ui::AX_ROLE_CELL ||
+          role == ui::AX_ROLE_COLUMN_HEADER ||
+          role == ui::AX_ROLE_ROW_HEADER);
 }
 
 bool BrowserAccessibility::IsEditableText() const {
   // These roles don't have readonly set, but they're not editable text.
   if (GetRole() == ui::AX_ROLE_SCROLL_AREA ||
       GetRole() == ui::AX_ROLE_COLUMN ||
       GetRole() == ui::AX_ROLE_TABLE_HEADER_CONTAINER) {
     return false;
   }
 
@@ skipping 80 matching lines @@
       }
       need_to_offset_web_area = true;
     }
     parent = parent->GetParentForBoundsCalculation();
   }
 
   return bounds;
 }
 
 }  // namespace content