269124 - Use GetInputState rather than GetQueueStatus(QS_INPUT)

Status: RESOLVED INVALID

(Core :: XUL, defect)

Description

[Brian Ryner (not reading)]

We use GetQueueStatus(QS_INPUT) in nsWindow::GetLastInputEventTime() to
determine if there are new input messages waiting to be processed. 
GetInputState() has the same effect but is quite a bit faster (we call this a
ton during parsing).

References:

http://msdn.microsoft.com/library/en-us/winui/winui/windowsuserinterface/windowing/messagesandmessagequeues/messagesandmessagequeuesreference/messagesandmessagequeuesfunctions/getqueuestatus.asp
http://msdn.microsoft.com/library/en-us/winui/winui/windowsuserinterface/windowing/messagesandmessagequeues/messagesandmessagequeuesreference/messagesandmessagequeuesfunctions/getinputstate.asp

Comment 1

[Brian Ryner (not reading)]

Attachment: patch

Comment 2

[Brian Ryner (not reading)]

Comment on attachment 165533 [details] [diff] [review]
patch

Oops... this doesn't look at mouse moves, which are a significant part of
user-input events.

Comment 3

[Brian Ryner (not reading)]

resolving invalid... sorry for the spam.