Bommels05/Mozilla
Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

bug 395923. The whole page is repeated in a single node following the ordered list with 8 items on this page. Correct fix that doesn't change role for links with onclicks. r=surkov, a=dsicore

Browse Source 
git-svn-id: svn://10.0.0.236/trunk@237363 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
aaronleventhal%moonset.net 2007-10-06 16:52:04 +00:00
parent e7b958c780
commit 0749ea7e96
1 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
mozilla/accessible/src/base/nsBaseWidgetAccessible.cpp
View File

@ -226,11 +226,7 @@ void nsLinkableAccessible::CacheActionContent()
for (nsCOMPtr<nsIContent> walkUpContent(do_QueryInterface(mDOMNode));
walkUpContent;
walkUpContent = walkUpContent->GetParent()) {
if (nsAccUtils::HasListener(walkUpContent, NS_LITERAL_STRING("click"))) {
mActionContent = walkUpContent;
mIsOnclick = PR_TRUE;
break;
}
PRBool isOnclick = nsAccUtils::HasListener(walkUpContent, NS_LITERAL_STRING("click"));
nsIAtom *tag = walkUpContent->Tag();
if ((tag == nsAccessibilityAtoms::a || tag == nsAccessibilityAtoms::area) &&
walkUpContent->IsNodeOfType(nsINode::eHTML)) {
@ -252,9 +248,15 @@ void nsLinkableAccessible::CacheActionContent()
// Don't let it keep walking up, otherwise we may report the wrong container
// as the action node
mActionContent = walkUpContent;
mIsOnclick = isOnclick;
break;
}
}
if (isOnclick) {
mActionContent = walkUpContent;
mIsOnclick = PR_TRUE;
break;
}
}
}