The default bug view has changed. See this FAQ.

CTRL+Enter to open link in a new tab broken since July 23, 2011 build

VERIFIED FIXED in Firefox 8

Status

()

Firefox
Keyboard Navigation
--
major
VERIFIED FIXED
6 years ago
6 years ago

People

(Reporter: MarcoZ, Assigned: khuey)

Tracking

({access, regression})

Trunk
Firefox 8
access, regression
Points:
---
Bug Flags:
in-testsuite ?

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
STR:
1. Open a page.
2. Tab to a link.
3. Press CTRL+Enter to open the link in a new tab.

Expected: Link should open in new tab.
Actual: Nothing happens.

This is a regression first seen in the July 23, 2011 build. Regression range is:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=19348341366b&tochange=887fad3ebc0b

Comment 1

6 years ago
Regresson window(build-system hourly):
Works:
http://hg.mozilla.org/projects/build-system/rev/8c2293dcca62
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:8.0a1) Gecko/20110719 Firefox/8.0a1 ID:20110719090011
Fails:
http://hg.mozilla.org/projects/build-system/rev/09b2569edec2
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:8.0a1) Gecko/20110719 Firefox/8.0a1 ID:20110719092300
Pushlog:
http://hg.mozilla.org/projects/build-system/pushloghtml?fromchange=8c2293dcca62&tochange=09b2569edec2
Triggered by;
09b2569edec2	Kyle Huey — Bug 646157: Prevent 2^n growth in event dispatch in a chain of nested labels. r=smaug Prior to this patch, Gecko allowed bubbling click events to trigger multiple <label>s, and for the synthetic click events generated by <label>s to also trigger additional <label>s. In contrast, Opera allows only the latter, and Webkit/IE allow neither. Gecko's behavior leads to 2^n events being dispatched in the case of markup like: <label for="thing1" /><input type="foo" name="thing1" /> <label for="thing2" /><input type="foo" name="thing2" /> ... After this patch a bubbling click event triggers at most one <label>, and the synthetic click event cannot trigger additional labels. Our behavior is still not identical to Webkit/IE though, because in Gecko a click event will still bubble up past a <label>, while in Webkit/IE the click event will stop at the <label>.
Blocks: 646157

Comment 2

6 years ago
This also broke
Shift+Enter to open link in new window.
Alt+Enter to save link.
Assignee: nobody → khuey
Created attachment 548203 [details] [diff] [review]
Patch

We're overwriting the trusted flag on the event, which causes much sadness.
Attachment #548203 - Flags: review?(Olli.Pettay)
Attachment #548203 - Flags: review?(Olli.Pettay) → review?(jonas)

Updated

6 years ago
Attachment #548203 - Flags: review?(jonas) → review+
http://hg.mozilla.org/mozilla-central/rev/58c04967ac5b

This needs a test ...
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Target Milestone: --- → Firefox 8

Updated

6 years ago
Duplicate of this bug: 674194
(Reporter)

Comment 6

6 years ago
Verified fixed in Mozilla/5.0 (Windows NT 6.1; WOW64; rv:8.0a1) Gecko/20110727 Firefox/8.0a1
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.