removing "return NS_OK" because it keeps it from doing the call to

nsFrame which calls HandleRelease. Without HandleRelease the EventStateManager
mMouseGrabber never gets cleared and it keeps thinking events need to go to the view.
b=48792 r=pollmann

layout/forms/nsImageControlFrame.cpp

@@ -304,7 +304,6 @@ nsImageControlFrame::HandleEvent(nsIPresContext* aPresContext,
       mLastClickPoint.y = NSTwipsToIntPixels(aEvent->point.y, t2p);
 
       mGotFocus = PR_TRUE;
-      return NS_OK;
       break;
     }
   }

layout/html/forms/src/nsImageControlFrame.cpp

@@ -304,7 +304,6 @@ nsImageControlFrame::HandleEvent(nsIPresContext* aPresContext,
       mLastClickPoint.y = NSTwipsToIntPixels(aEvent->point.y, t2p);
 
       mGotFocus = PR_TRUE;
-      return NS_OK;
       break;
     }
   }