Can't submit forms on Logitech Website

RESOLVED FIXED

Status

()

Core
DOM
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: Sylvain Pasche, Assigned: bz)

Tracking

({regression})

Trunk
regression
Points:
---
Bug Flags:
blocking1.9.2 +
in-testsuite +

Firefox Tracking Flags

(status1.9.2 beta4-fixed)

Details

Attachments

(3 attachments)

(Reporter)

Description

9 years ago
Created attachment 413327 [details]
testcase, needs reduction

This is a regression in Firefox 3.6 and 3.7. The Logitech Support website uses forms that can't be submitted with Firefox 3.6. The form works fine with Firefox 3.5 or other browsers.

Steps to reproduce:
1) Extract the attached archive and load the html file
2) Click on "Save Changes"
Expected:
 You should see an alert with the message: 'Preferred Language' requires a value.
Actual:
 An alert with: An email address must be unique across the Email Address, Email Alt 1, and Email Alt 2 fields.

On 3.6 and later, document.forms[1].p_email returns a NodeList with 2 identical nodes representing the email input.
On 3.5 it returns a single node with the email input.
(Reporter)

Updated

9 years ago
Flags: blocking1.9.2?
Assignee: nobody → bzbarsky
Keywords: regressionwindow-wanted, testcase-wanted
Flags: blocking1.9.2? → blocking1.9.2+

Comment 3

9 years ago
Comment on attachment 413408 [details] [diff] [review]
Fix

>       if(nsContentUtils::PositionIsBefore(list->GetNodeAt(list->Length() - 1), aChild)) {
>           list->AppendElement(aChild);
>           return NS_OK;
>       }
Um, this method seems to mix 2 and 4 space indentation.
Want to fix that while you're here? Also the 'while (last != first) {' a bit later in the
method.

In any case, r=me.
Attachment #413408 - Flags: review?(Olli.Pettay) → review+
> Um, this method seems to mix 2 and 4 space indentation.

Fixed, for both.
Pushed http://hg.mozilla.org/mozilla-central/rev/6deaa5b2b2d8
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.