From 0733b51c366eb4932c46e28c9cccae7cea70da13 Mon Sep 17 00:00:00 2001
From: "aaronl%netscape.com" <aaronl%netscape.com>
Date: Thu, 14 Mar 2002 06:14:33 +0000
Subject: [PATCH] Bug 127724. Active Accessibility: nsIAccessible's accDoAction
 does not work for Html Check Box Node. r=aaronl, sr=sfraser, a=asa

---
 .../src/html/nsHTMLFormControlAccessible.cpp     | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/accessible/src/html/nsHTMLFormControlAccessible.cpp b/accessible/src/html/nsHTMLFormControlAccessible.cpp
index cfb0fc50f89..7d4cd649a03 100644
--- a/accessible/src/html/nsHTMLFormControlAccessible.cpp
+++ b/accessible/src/html/nsHTMLFormControlAccessible.cpp
@@ -97,10 +97,8 @@ NS_IMETHODIMP nsHTMLCheckboxAccessible::AccDoAction(PRUint8 index)
 {
   if (index == 0) {   // 0 is the magic value for default action
     nsCOMPtr<nsIDOMHTMLInputElement> htmlCheckboxElement(do_QueryInterface(mDOMNode));
-    PRBool checked = PR_FALSE;
     if (htmlCheckboxElement) {
-      htmlCheckboxElement->GetChecked(&checked);
-      htmlCheckboxElement->SetChecked(!checked);
+      htmlCheckboxElement->Click();
       return NS_OK;
     }
     return NS_ERROR_FAILURE;
@@ -134,8 +132,10 @@ NS_IMETHODIMP nsHTMLRadioButtonAccessible::AccDoAction(PRUint8 index)
 {
   if (index == eAction_Click) {
     nsCOMPtr<nsIDOMHTMLInputElement> element(do_QueryInterface(mDOMNode));
-    element->Click();
-    return NS_OK;
+    if (element) {
+      element->Click();
+      return NS_OK;
+    }
   }
   return NS_ERROR_INVALID_ARG;
 }
@@ -181,8 +181,10 @@ NS_IMETHODIMP nsHTMLButtonAccessible::AccDoAction(PRUint8 index)
 {
   if (index == 0) {
     nsCOMPtr<nsIDOMHTMLInputElement> element(do_QueryInterface(mDOMNode));
-    element->Click();
-    return NS_OK;
+    if (element) {
+      element->Click();
+      return NS_OK;
+    }
   }
   return NS_ERROR_INVALID_ARG;
 }