Bug 409009 Table cells should have a table in their ancestry (additioal reviewer comment), r=evan.yan, a=beltzner
authorsurkov.alexander@gmail.com
Fri, 18 Jan 2008 16:03:43 -0800
changeset 10422 56bdea0329001315f40794edcc05e3d7fd5ffcf9
parent 10421 c77c234b2ba9f5fdcd23540c73ac7e346e24c9aa
child 10423 af3c4099a23078c1c518be0a2a088cb9d657877a
push id1
push userbsmedberg@mozilla.com
push dateThu, 20 Mar 2008 16:49:24 +0000
treeherdermozilla-central@61007906a1f8 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersevan.yan, beltzner
bugs409009
milestone1.9b3pre
Bug 409009 Table cells should have a table in their ancestry (additioal reviewer comment), r=evan.yan, a=beltzner
accessible/src/base/nsAccessibilityService.cpp
--- a/accessible/src/base/nsAccessibilityService.cpp
+++ b/accessible/src/base/nsAccessibilityService.cpp
@@ -1400,19 +1400,16 @@ NS_IMETHODIMP nsAccessibilityService::Ge
           frame->GetType() == nsAccessibilityAtoms::tableRowGroupFrame ||
           frame->GetType() == nsAccessibilityAtoms::tableRowFrame) {
 
         nsIContent *tableContent = content;
         while ((tableContent = tableContent->GetParent()) != nsnull) {
           nsIFrame *tableFrame = aPresShell->GetPrimaryFrameFor(tableContent);
           if (tableFrame &&
               tableFrame->GetType() == nsAccessibilityAtoms::tableOuterFrame) {
-            // Table that we're a descendant of is not styled as a table,
-            // and has no table accessible for an ancestor, or
-            // table that we're a descendant of is presentational
 
             nsCOMPtr<nsIDOMNode> tableNode(do_QueryInterface(tableContent));
             if (tableNode) {
               nsRoleMapEntry *tableRoleMapEntry =
                 nsAccUtils::GetRoleMapEntry(tableNode);
               if (tableRoleMapEntry &&
                   tableRoleMapEntry != &nsARIAMap::gLandmarkRoleMap)
                 tryTagNameOrFrame = PR_FALSE;