Bug 343375 Crash when pressing the Enter key in the URL bar [@

g_type_check_value_holds - g_value_set_double -
nsDocAccessibleWrap::FireToolkitEvent]
patch by nian.liu at sun.com r=aaronleventhal sr=roc

accessible/src/atk/nsDocAccessibleWrap.cpp

@@ -154,8 +154,9 @@ NS_IMETHODIMP nsDocAccessibleWrap::FireToolkitEvent(PRUint32 aEvent,
          * Need handle them separately.
          */
     case nsIAccessibleEvent::EVENT_ATK_PROPERTY_CHANGE :
+      {
         AtkPropertyChange *pAtkPropChange;
-        AtkPropertyValues values;
+        AtkPropertyValues values = { NULL };
 
         MAI_LOG_DEBUG(("\n\nReceived: EVENT_ATK_PROPERTY_CHANGE\n"));
         if (!aEventData)
@@ -203,11 +204,15 @@ NS_IMETHODIMP nsDocAccessibleWrap::FireToolkitEvent(PRUint32 aEvent,
         case PROP_VALUE:
             {
               // Old value not used for anything other than state change events
-              nsCOMPtr<nsIAccessibleValue> accValue(do_QueryInterface(aAccessible));
-              NS_ENSURE_TRUE(accValue, NS_ERROR_FAILURE);
+              nsCOMPtr<nsIAccessibleValue> accValue(do_QueryInterface(aAccessible, &rv));
+              if (!NS_SUCCEEDED(rv)) {
+                break;
+              }
               double newValue;
               rv = accValue->GetCurrentValue(&newValue);
-              NS_ENSURE_SUCCESS(rv, rv);
+              if (!NS_SUCCEEDED(rv)) {
+                break;
+              }
               g_value_init(&values.new_value, G_TYPE_DOUBLE);
               g_value_set_double(&values.new_value, newValue);
             }
@@ -228,7 +233,7 @@ NS_IMETHODIMP nsDocAccessibleWrap::FireToolkitEvent(PRUint32 aEvent,
                                   &values, NULL);
             g_free (signal_name);
         }
-
+      }
         break;
 
     case nsIAccessibleEvent::EVENT_ATK_SELECTION_CHANGE: