142477 - In <img008.html> JavaScript Testcase: 'Image.lowsrc' should return '../res/lowstar.gif' instead of 'undefined'.

Status: VERIFIED FIXED

(Core :: Graphics: ImageLib, defect)

Description

[pietro canevarolo]

img008.html
Description Pass Bug Number Actual Result Expected Result
Read Image.lowsrc ../res/lowstar.gif failed Bug report required undefined
../res/lowstar.gif

Comment 1

[Andreas Kunz]

Pietro, can you please explain, where to find this gif image?
Or send it here as an attachment? (maybe the html page, where it should be
loaded, too)

Comment 2

[Andreas Kunz]

Ok, the reporter explained to me, he ran the "Debug -> Verification ->
JavaScript" menu, clicked on "image" and then on "img008.html" and there it said
that this verification failed (actual result: "undefined", expected result: 
"../res/lowstar.gif").

Yes, I can reproduce it with me with 2002041711 on Win2k. (=> somebody may
change OS to "all")

You can easily test this yourself with:
http://mozilla.org/quality/ngdriver/suites/javascript/img008.html

Comment 3

[mmc]

I can reproduce it with build 2002042408 on linux.

Comment 4

[Matthias Versen [:Matti]]

This test fails because we don't support lowsrc images.(pavloc removed the
support a few months ago)

The testcase is out of date but who owns this ?

Comment 5

[Boris Zbarsky [:bzbarsky]]

desale owns the testcases, iirc.

Comment 6

[Phil Schwartau]

Setting QA accordingly -

Comment 7

[Ken]

>> This test fails because we don't support lowsrc images.(pavloc removed
>> the support a few months ago)

do you know the bug number where lowsrc was removed?

Comment 8

[Boris Zbarsky [:bzbarsky]]

A quick search on "remove lowsrc" in the summary shows bug 92453

Comment 9

[Olivier Cahagne]

*** Bug 144062 has been marked as a duplicate of this bug. ***

Comment 10

[Jo Hermans]

*** Bug 196849 has been marked as a duplicate of this bug. ***

Comment 11

[José Jeria]

*** Bug 203034 has been marked as a duplicate of this bug. ***

Comment 12

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Win95; en-US; rv:1.4a) Gecko/20030401]

Bug still there.


In the meantime, who can do the following to be more precise ?

In <http://mozilla.org/quality/ngdriver/suites/javascript/img008.html> source,
replace "Bug report required" by "Bug# 142477".

Comment 13

[Boris Zbarsky [:bzbarsky]]

The bug is that the test needs to be removed completely.  Updating it seems silly.

Comment 14

[Serge Gautherie (:sgautherie)]

Re comment 13:

Well, forgive me: I said "in the meantime".
I only thought that if there would be more delay for the bug fix, it could avoid
getting some more duplicates :-|

If there is no more "meantime", I kind of agree with you.

Comment 15

[Boris Zbarsky [:bzbarsky]]

My point is that anyone who can update the test can aslo remove it just as easily...

Comment 16

[Fabian Guisset]

lowSrc is still implemented in the DOM (for compat with DOM1 HTML), even though
it has no effect in imglib.
The testcase passes for me using Firebird0.7 on winxp. Marking WORKSFORME.
Please reopen if there's something I missed here, thanks!

Comment 17

[Andreas Kunz]

Moz 1.6a on Win2k also passes the test.
I guess this bug can be verified, however I don't do so in case someone wants to
(mis-/re-)use it to remove this testcase (it is still there, now under
Debug-Verification-DOM). But those are not built into the browser, but on the
moz.org website anyway, so...

Comment 18

[Serge Gautherie (:sgautherie)]

For the record:

Bug 158610 comment 6:
{
Created an attachment (id=124847)
Implement lowsrc attribute

This patch implements the setting and querying of the lowsrc attribute to allow
these sites to work.

It does NOT make lowsrc work, and we have no plans to do that.
}

LowSrc support:
First, this bug patch removed it completely;
Then, bug 158610 patch restored the get/set part (only).

Comment 19

[Serge Gautherie (:sgautherie)]

Re comment 18:

Read "First, bug _92453_ patch removed it completely;".


Re comment 13, comment 16:

Changing WorksForMe to Fixed, (see bug 158610).

Comment 20

[Fabian Guisset]

Thanks Serge, VERIFIED fixed.