From df8aa3ff05e2e3defe3258fa46b45606623426a0 Mon Sep 17 00:00:00 2001
From: "aaronl%netscape.com"
 <aaronl%netscape.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Fri, 14 Jun 2002 01:47:32 +0000
Subject: [PATCH] Bug 147976 - Active Accessibility: progress meter
 get_accValue() sometimes returning value with 2  Progress meter <label> not
 getting picked up in get_accName(). r=kyle, sr=hewitt

git-svn-id: svn://10.0.0.236/trunk@123281 18797224-902f-48f8-a5cc-f745e15eee43
---
 mozilla/accessible/src/base/nsAccessible.cpp          | 11 ++++++-----
 .../accessible/src/xul/nsXULFormControlAccessible.cpp |  6 +++---
 .../accessible/src/xul/nsXULFormControlAccessible.h   |  2 +-
 3 files changed, 10 insertions(+), 9 deletions(-)

diff --git a/mozilla/accessible/src/base/nsAccessible.cpp b/mozilla/accessible/src/base/nsAccessible.cpp
index 703444bc9d8..4fbd006e0bf 100644
--- a/mozilla/accessible/src/base/nsAccessible.cpp
+++ b/mozilla/accessible/src/base/nsAccessible.cpp
@@ -1489,20 +1489,21 @@ NS_IMETHODIMP nsAccessible::GetXULAccName(nsAString& _retval)
       //
       //   This should keep search times down and still get the relevant
       //   labels.
+      nsAutoString controlID;
+      domElement->GetAttribute(NS_LITERAL_STRING("id"), controlID);
       nsCOMPtr<nsIDOMNode> parent;
-      if (NS_SUCCEEDED(rv = mDOMNode->GetParentNode(getter_AddRefs(parent)))) {
+      if (!controlID.IsEmpty() && NS_SUCCEEDED(rv = mDOMNode->GetParentNode(getter_AddRefs(parent)))) {
         nsCOMPtr<nsIDOMXULElement> xulElement(do_QueryInterface(parent));
         NS_ASSERTION(xulElement, "No xulElement for parent DOM Node!");
         if (xulElement) {
-          nsAutoString controlID;
-          nsCOMPtr<nsIDOMNodeList>labelList;
+          nsCOMPtr<nsIDOMNodeList> labelList;
           if (NS_SUCCEEDED(rv = xulElement->GetElementsByAttribute(NS_LITERAL_STRING("control"), controlID, getter_AddRefs(labelList))))
           {
             PRUint32 length = 0;
             if (NS_SUCCEEDED(rv = labelList->GetLength(&length)) && length > 0) {
-              for (PRUint32 i = 0; i < length; ++i) {
+              for (PRUint32 index = 0; index < length; ++index) {
                 nsCOMPtr<nsIDOMNode> child;
-                if (NS_SUCCEEDED(rv = labelList->Item(i, getter_AddRefs(child) ))) {
+                if (NS_SUCCEEDED(rv = labelList->Item(index, getter_AddRefs(child) ))) {
                   rv = AppendLabelText(child, label);
                 }
               }
diff --git a/mozilla/accessible/src/xul/nsXULFormControlAccessible.cpp b/mozilla/accessible/src/xul/nsXULFormControlAccessible.cpp
index 250c8a52798..6d3806512a4 100644
--- a/mozilla/accessible/src/xul/nsXULFormControlAccessible.cpp
+++ b/mozilla/accessible/src/xul/nsXULFormControlAccessible.cpp
@@ -42,7 +42,6 @@
 #include "nsIDOMNodeList.h"
 #include "nsIDOMXULButtonElement.h"
 #include "nsIDOMXULCheckboxElement.h"
-#include "nsIDOMXULDescriptionElement.h"
 #include "nsIDOMXULDocument.h"
 #include "nsIDOMXULLabelElement.h"
 #include "nsIDOMXULMenuListElement.h"
@@ -437,7 +436,7 @@ NS_IMETHODIMP nsXULGroupboxAccessible::GetAccName(nsAString& _retval)
   */
 
 nsXULProgressMeterAccessible::nsXULProgressMeterAccessible(nsIDOMNode* aNode, nsIWeakReference* aShell):
-nsAccessible(aNode, aShell)
+nsFormControlAccessible(aNode, aShell)
 { 
 }
 
@@ -461,7 +460,8 @@ NS_IMETHODIMP nsXULProgressMeterAccessible::GetAccValue(nsAString& _retval)
   nsCOMPtr<nsIDOMElement> element(do_QueryInterface(mDOMNode));
   NS_ASSERTION(element, "No element for DOM node!");
   element->GetAttribute(NS_LITERAL_STRING("value"), _retval);
-  _retval.Append(NS_LITERAL_STRING("%"));
+  if (!_retval.IsEmpty() && _retval.Last() != '%')
+    _retval.Append(NS_LITERAL_STRING("%"));
   return NS_OK;
 }
 
diff --git a/mozilla/accessible/src/xul/nsXULFormControlAccessible.h b/mozilla/accessible/src/xul/nsXULFormControlAccessible.h
index 99e1a65c121..616c089a4ff 100644
--- a/mozilla/accessible/src/xul/nsXULFormControlAccessible.h
+++ b/mozilla/accessible/src/xul/nsXULFormControlAccessible.h
@@ -95,7 +95,7 @@ public:
   NS_IMETHOD GetAccName(nsAString& _retval);
 };
 
-class nsXULProgressMeterAccessible : public nsAccessible
+class nsXULProgressMeterAccessible : public nsFormControlAccessible
 {
 public:
   nsXULProgressMeterAccessible(nsIDOMNode* aNode, nsIWeakReference* aShell);