Closed Bug 638596 Opened 15 years ago Closed 15 years ago

document.getElementById(...).value returns reversed string under certain circumstances

Categories

(Core :: DOM: Editor, defect)

Product:
Core
Component:
DOM: Editor
Version:
1.9.2 Branch
Platform:
x86
Windows XP
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla2.0
Tracking Flags:
Tracking Status
status1.9.2 --- wontfix

People

(Reporter: ytkvjehvsiep, Assigned: ehsan.akhgari)

Details

Attachments

(2 files, 1 obsolete file)

More minimized testcase
184 bytes, text/html
Details
Test case
2.17 KB, patch
roc
: review+
Details | Diff | Splinter Review
User-Agent: Opera/9.80 (Windows NT 5.1; U; en) Presto/2.7.62 Version/11.01 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-GB; rv:1.9.2.14) Gecko/20110218 Firefox/3.6.14 document.getElementById(...).value will return the reversed value from an input element with font-size 0. Reproducible: Always Steps to Reproduce: 1. Run the attached SSCCE. 2. Type in a string; the string will be echoed reversed. 3. Change the font-size at the bottom of the HTML to anything other than 0 4. Type in a string 5. String will be echoed correctly Actual Results: Echoed string is reversed when font-size is 0. Expected Results: String should have been echoed as-is. This SSCCE uses Google's CDN-cached jQuery, but it has been shown to not be a jQuery bug.
Works for me using Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b13pre) Gecko/20110303 Firefox/4.0b13pre ID:20110303030406 Can you try with the latest nightly please: http://nightly.mozilla.org/
Yep, works fine with the latest Minefield. Wonder what the problem was...
If you were curious as to what fixed it, you could always try finding the regression range: http://harthur.github.com/mozregression/ :-)
Component: General → DOM: Core & HTML
Product: Firefox → Core
QA Contact: general → general
Version: unspecified → 1.9.2 Branch
Whiteboard: [fixed in gecko2.0; 1.9.2 wontfix?]
Looks like this got fixed in http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=84ee6bc0484d&tochange=5588c9796f0b Maybe Ehsan's changes? Not sure. In any case, we're not going to mess with the 1.9.2 branch here.
Status: UNCONFIRMED → RESOLVED
Closed: 15 years ago
Component: DOM: Core & HTML → Editor
QA Contact: general → editor
Resolution: --- → WORKSFORME
I have no idea what has fixed this, but I'm going to add an automated test which makes sure that this won't fail again.
Assignee: nobody → ehsan
Attachment #516715 - Attachment is obsolete: true
status1.9.2: --- → wontfix
Flags: in-testsuite?
Resolution: WORKSFORME → FIXED
Whiteboard: [fixed in gecko2.0; 1.9.2 wontfix?]
Attached patch Test case — — Splinter Review
Attachment #516990 - Flags: review?(roc)
Depends on: post2.0
Whiteboard: [post-2.0][needs landing]
Test case: http://hg.mozilla.org/mozilla-central/rev/5f7ab12daeb4
No longer depends on: post2.0
Flags: in-testsuite? → in-testsuite+
Whiteboard: [post-2.0][needs landing]
Target Milestone: --- → mozilla2.0
Blocks: 770422
No longer blocks: 770422
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: