622117 - Links with href broken if JavaScript is disabled and onclick attribute is present

Status: RESOLVED FIXED

(Core :: DOM: UI Events & Focus Handling, defect, P1)

Description

[d.a.]

User-Agent:       Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:2.0b9pre) Gecko/20101230 Firefox/4.0b9pre
Build Identifier: 

If JavaScript is disabled or blocked (via NoScript) and the onclick attribute is present the links will not work. Instead it will stay on the page where the link is, see test-case.

Pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=f5eb60e1bc6e&tochange=e928817fb4e9

Happens with JavaScript disabled through options and trace and method pref'd off.

Reproducible: Always

Comment 1

[d.a.]

Attachment: Testcase

Comment 2

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

I see this too in today's nightly. Confirming.

Comment 3

[Alice0775 White]

Regression window:
Works
http://hg.mozilla.org/mozilla-central/rev/699023bb3c9c
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b9pre) Gecko/20101222 Firefox/4.0b9pre ID:20101227122945
Fails:
http://hg.mozilla.org/mozilla-central/rev/b7acaa08a9c5
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b9pre) Gecko/20101223 Firefox/4.0b9pre ID:20101227124535
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=699023bb3c9c&tochange=b7acaa08a9c5

Comment 4

[Dave Garrett]

To be clear, this bug is for when the onclick attribute exists on an otherwise normal link with an href. The example case is:

<a href="http://www.google.com/" onclick="">Google</a>

(you'd want to mention that upfront, as otherwise disabling JavaScript would obviously be expected to break a link using only onclick)

Comment 5

[Boris Zbarsky [:bzbarsky]]

Attachment: Fix flag collision.

This is caused by a silent merge failure on hg's part when merging the patch in bug 331959 on top of the patch for bug 618196.  The result was the same value being used for both flags, so the exception from the onclick handler was treated as preventing anchor actions.

Comment 6

[Boris Zbarsky [:bzbarsky]]

Pushed http://hg.mozilla.org/mozilla-central/rev/9dead0bfa0f3

Comment 7

[LegNeato]

As per today's meeting, beta 9 will be a time-based release. Marking these all betaN+. Please move it back to beta9+ if  you believe it MUST be in the next beta (ie: trunk is in an unshippable state without this)