Fixing keyboard control of forms

layout/html/forms/src/nsButtonControlFrame.cpp

@@ -521,7 +521,7 @@ nsButtonControlFrame::HandleEvent(nsIPresContext& aPresContext,
     case NS_KEY_DOWN:
       if (NS_KEY_EVENT == aEvent->eventStructType) {
         nsKeyEvent* keyEvent = (nsKeyEvent*)aEvent;
-        if (NS_VK_SPACE == keyEvent->keyCode) {
+        if (NS_VK_SPACE == keyEvent->keyCode || NS_VK_RETURN == keyEvent->keyCode) {
           MouseClicked(&aPresContext);
         }
       }

layout/html/forms/src/nsCheckboxControlFrame.cpp

@@ -353,13 +353,26 @@ nsCheckboxControlFrame::Paint(nsIPresContext& aPresContext,
 }
 
 NS_METHOD nsCheckboxControlFrame::HandleEvent(nsIPresContext& aPresContext, 
-                                          nsGUIEvent* aEvent,
-                                          nsEventStatus& aEventStatus)
+                                              nsGUIEvent* aEvent,
+                                              nsEventStatus& aEventStatus)
 {
   if (nsEventStatus_eConsumeNoDefault == aEventStatus) {
     return NS_OK;
   }
 
+  switch(aEvent->message) {
+    case NS_KEY_PRESS:
+      if (NS_KEY_EVENT == aEvent->eventStructType) {
+        nsKeyEvent* keyEvent = (nsKeyEvent*)aEvent;
+        if (NS_VK_SPACE == keyEvent->keyCode || NS_VK_RETURN == keyEvent->keyCode) {
+          MouseClicked(&aPresContext);
+        }
+      }
+      break;
+    default:
+      break;
+  }
+
   if (nsnull == mWidget) {
       // Handle GFX rendered widget Mouse Down event
     PRInt32 type;

layout/html/forms/src/nsRadioControlFrame.cpp

@@ -393,6 +393,31 @@ nsRadioControlFrame::Paint(nsIPresContext& aPresContext,
   return NS_OK;
 }
 
+NS_METHOD 
+nsRadioControlFrame::HandleEvent(nsIPresContext& aPresContext, 
+                                 nsGUIEvent* aEvent,
+                                 nsEventStatus& aEventStatus)
+{
+  if (nsEventStatus_eConsumeNoDefault == aEventStatus) {
+    return NS_OK;
+  }
+
+  switch(aEvent->message) {
+    case NS_KEY_PRESS:
+      if (NS_KEY_EVENT == aEvent->eventStructType) {
+        nsKeyEvent* keyEvent = (nsKeyEvent*)aEvent;
+        if (NS_VK_SPACE == keyEvent->keyCode || NS_VK_RETURN == keyEvent->keyCode) {
+          MouseClicked(&aPresContext);
+        }
+      }
+      break;
+    default:
+      break;
+  }
+
+  return(nsFormControlFrame::HandleEvent(aPresContext, aEvent, aEventStatus));
+}
+
 void nsRadioControlFrame::GetRadioControlFrameState(nsString& aValue)
 {
   if (nsnull != mWidget) {

layout/html/forms/src/nsRadioControlFrame.h

@@ -73,11 +73,15 @@ public:
                    nsIRenderingContext& aRenderingContext,
                    const nsRect& aDirtyRect,
                    nsFramePaintLayer aWhichLayer);
- 
+
   virtual void PaintRadioButton(nsIPresContext& aPresContext,
                         nsIRenderingContext& aRenderingContext,
                         const nsRect& aDirtyRect);
 
+  NS_IMETHOD HandleEvent(nsIPresContext& aPresContext, 
+                         nsGUIEvent* aEvent,
+                         nsEventStatus& aEventStatus);
+
   ///XXX: End o the temporary methods
 
 protected: