12152 - setting image.src doesn't work first time

Status: VERIFIED FIXED

(Core :: DOM: Core & HTML, defect, P3)

Description

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

DESCRIPTION: Setting HTMLImageElement::src should change the image.  It doesn't
happen the first time, but does happen later times.  This makes me think it only
works when the image is in the memory cache.

STEPS TO REPRODUCE:  Load attachment (to be attached) in apprunner.

ACTUAL RESULTS: First time loaded, you see HTML image.  Later tries, you see CSS
image.

EXPECTED RESULTS:  You should see the CSS image, although the first time you
might see the HTML image first for a short time.

BUILDS TESTED:
 * Linux, apprunner, 1999-08-16-08-M9

ADDITIONAL INFORMATION:
This is a spinoff of bug 11518.

Comment 1

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

Attachment: testcase described in report

Comment 2

[leger]

Is this Linux specific?

Comment 3

[David Baron :dbaron: (⌚️UTC-5, no longer working on Mozilla)]

ALSO DOES NOT WORK CORRECTLY ON:
 * Windows, apprunner, 1999-08-24-09-M10

Marking All/All.

Comment 4

[vidur (gone)]

In an attempt to get my bug list in order again, marking all the bugs I have
currently as ASSIGNED.

Comment 5

[ekrock's old account (dead)]

Bulk moving [testcase] code to new testcase keyword. Sorry for the spam!

Comment 6

[vidur (gone)]

This one works for me now. It's rather old, so I'm not surprised.

Comment 7

[Prashant Desale]

Reopening this one because I could still see this happening. First time you can 
see only HTML image, when you reload, then you can see CSS image.
Tested with 2000-04-10-09 with win-95.

Comment 8

[ekrock's old account (dead)]

Nominating nsbeta2. We have to start drawing a line on DOM0 backward 
compatibility; these bugs are supposed to be a high priority for nsbeta2 per the 
beta2 criteria.

Comment 9

[ekrock's old account (dead)]

I still see this with 5/15 build on WinNT 4.0 SP4. In fact, I never see 
the CSS image no matter how many times I reload the page. Setting image SRC is 
one of most widely used JS commands (e.g. the ubiquitous image rollovers). 
Setting to [nsbeta2+].

Comment 10

[Johnny Stenback (:jst)]

Taking this bug off Vidurs list.

Comment 11

[Nisheeth Ranjan]

Taking this from jst as discussed in the XML/DOM bug distribution meeting.

Comment 12

[Nisheeth Ranjan]

When the page contains a strict doctype declaration, the image tag in the 
document never makes it through the parser to the content sink.  That is why 
you don't see any image.  I've filed bug 44028 on that problem and assigned it 
to harishd.

When I remove the strict doctype declaration, then the original behavior 
described in this bug shows itself - the HTML 4.0 image gets displayed but does 
not get replaced with the CSS image.  I am going to explore that problem 
further.

Comment 13

[Nisheeth Ranjan]

Attachment: testcase without strict dtd doctype declaration

Comment 14

[Nisheeth Ranjan]

Fix in hand.  Awaiting code review by vidur.

Comment 15

[Nisheeth Ranjan]

Attachment: Attempted Fix

Comment 16

[Nisheeth Ranjan]

Attachment: Updated fix

Comment 17

[Nisheeth Ranjan]

The fix is checked in.  <IMG> and <INPUT type="image"> both behave
correctly now.  Thanks to Eric and Vidur for helping review my code and provide
testing strategies.

Comment 18

[Prashant Desale]

Verified with 2000-07-05-09.