454472 - event.ctrlKey, event.shiftKey and event.altKey are undefined with DOMMouseScroll event

Status: RESOLVED FIXED

(Core :: DOM: Events, defect)

Description

[Alice0775 White]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.0.3pre) Gecko/2008090906 GranParadiso/3.0.3pre
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.0.3pre) Gecko/2008090906 Firefox/3.0.1

Ctrl + Mouse Wheel does not define event.ctrlKey property.
Shift and Alt key also.

Reproducible: Always

Steps to Reproduce:
1.Open Firefox
2.Go any page.
3.Ctrl + Mouse Wheel on the content area.
Actual Results:  
event.ctrlKey property is undefined.

Expected Results:  
event.ctrlKey property should be set to true.

This issue causes the problem that mouse wheel zoom level will not be stored.

Works:
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.0.3pre) Gecko/2008090906 GranParadiso/3.0.3pre

Not Works
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b1pre) Gecko/20080909105611 Minefield/3.1b1pre

Comment 1

[Alice0775 White]

Attachment: Test case of userChrome.js script.

This is Test case for userChrome.js.

Comment 2

[Alice0775 White]

Regression window:

Works:
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1a2pre) Gecko/20080812045727 Minefield/3.1a2pre


Not Works:
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1a2pre) Gecko/20080813032636 Minefield/3.1a2pre

Comment 3

[Alice0775 White]

To lack information of the event.ctrlKey, the following problems occur.

If Browser.zoom.Sitespecific is true,
Zooming by scroll-wheel to a larger/smaller zoom, switching to another tab and then switching back to the zoomed tab will reset the zoom level to the previous one.

(Zooming by pressing "Ctrl" + "=" works fine. Setting "Browser.zoom.Sitespecific" to false and scrolling zoom levels in individual tabs works fine.)

Comment 4

[Alice0775 White]

Attachment: Test case

Comment 5

[Alice0775 White]

Attachment: Test case

Comment 6

[Olli Pettay [:smaug][[e-mail]]]

Attachment: add nsIDOMMouseEvent to classinfo

Comment 7

[Alice0775 White]

Bug 454460 seems to be related to this bug.

Comment 9

[Markus Stange [:mstange]]

Bug 350471 added tests for this bug.