corysanin/tubestation
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bug 1446711 part 5. Get rid of nsIDOMMouseEvent::GetButton. r=qdot

Browse Source 
MozReview-Commit-ID: AZWzObh01uI
This commit is contained in:
Boris Zbarsky
2018-03-20 00:16:06 -04:00
parent 038b06c23d
commit 3352976d14
10 changed files with 26 additions and 58 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
toolkit/components/satchel/nsFormFillController.cpp
View File

@@ -14,6 +14,7 @@
#include "mozilla/dom/HTMLInputElement.h"
#include "mozilla/dom/KeyboardEvent.h"
#include "mozilla/dom/KeyboardEventBinding.h"
#include "mozilla/dom/MouseEvent.h"
#include "mozilla/dom/PageTransitionEvent.h"
#include "mozilla/Logging.h"
#include "nsIFormAutoComplete.h"
@@ -34,7 +35,6 @@
#include "nsIPresShell.h"
#include "nsRect.h"
#include "nsILoginManager.h"
#include "nsIDOMMouseEvent.h"
#include "mozilla/ModuleUtils.h"
#include "nsToolkitCompsCID.h"
#include "nsEmbedCID.h"
@@ -1179,7 +1179,7 @@ nsFormFillController::KeyPress(nsIDOMEvent* aEvent)
nsresult
nsFormFillController::MouseDown(nsIDOMEvent* aEvent)
{
nsCOMPtr<nsIDOMMouseEvent> mouseEvent(do_QueryInterface(aEvent));
MouseEvent* mouseEvent = aEvent->InternalDOMEvent()->AsMouseEvent();
if (!mouseEvent) {
return NS_ERROR_FAILURE;
}
@@ -1190,8 +1190,7 @@ nsFormFillController::MouseDown(nsIDOMEvent* aEvent)
return NS_OK;
}
int16_t button;
mouseEvent->GetButton(&button);
int16_t button = mouseEvent->Button();
// In case of a right click we set a timestamp that
// will be checked in Focus() to avoid showing