92453 - Remove lowsrc support

Status: RESOLVED FIXED

(Core :: Layout, defect, P2)

Description

[Chris Petersen]

Build: 2001072418
Platform: All
Expected Results: LOWSRC image should be replaced by higher resolution image
What I got: LOWSRC image is displayed only

Steps to reproduce: 

1) Go to url. Look at the center image (Free Preview CD- Filemaker 5.5). This
image should be the high resolution version.

2) Now , press the reload button.

3) LOWSRC image replaces the high rez image

4) Multiple reloads doesn't seem to correct the problem. User must place focus
in URL field and press return to reflow the document.

Comment 1

[Chris Petersen]

You might see the problem when upon visiting the site. If not , reloading the
page should cause the problem.

Comment 2

[Chris Petersen]

Attachment: simple test case that reproduces

Comment 3

[Chris Petersen]

Reassigning to [e-mail]

Comment 4

[Marc Attinasi]

LOWSRC is totally broken. We need some tests to really assess the extent of the
problems, but for now I think we are better off disabling it until we get it
working right. Currently, the lowsrc image either fails totally or gets stuck
and prevents the proper image from displaying.

Question: do we even want to implement lowsrc? At the very least we need to
restrict it to Quirks mode since it is not standard.

Comment 5

[Christopher Aillon (sabbatical, not receiving bugmail)]

Off the top of my head, I don't recall MSIE handling lowsrc.  I always thought
it was just NS 4. Can someone verify this?  If this is the case, then I think we
shouldn't support it either.  It is outdated and not standard, and although it
gives a "cool" effect on some sites if the user is on a dialup, high speed
connections are gaining popularity and this is less of a concern.  I always
thought it more of a nuisance while visiting gimp.org on a high speed
connection.  It creates a weird effect that looks like the browser is having
issues or something.  If a webmaster really wants this feature, he can do it
with DOM to gain more control of the invocation and extent of it.  But that's
just my opinion...

Adding 4xp keyword.

Comment 6

[Christopher Hoess (gone)]

Bleah. "lowsrc" isn't in HTML4, deprecated or otherwise; it's a disgusting hack
outside the proper realm of HTML that should be laid to rest in the same grave
as <layer> (all the more so because it has no impact on the final rendering of
the page). Changing summary.

Comment 7

[Stuart Parmenter]

I'd vote for removing support for it all together.  Currently do we the load for
lowsrc properly, but we never display it.. so we waste time downloading the
image if one is present.

Comment 8

[Stuart Parmenter]

Attachment: Fix

Comment 9

[Stuart Parmenter]

reassigning to myself.

need r= and sr=

Comment 10

[Stuart Parmenter]

Comment on attachment 53322 [details] [diff] [review]
Fix

this patch was missing the removal of a html attribute

Comment 11

[Stuart Parmenter]

Attachment: Real removal of lowsrc

Comment 12

[jag (Peter Annema)]

Comment on attachment 53439 [details] [diff] [review]
Real removal of lowsrc

r=jag

(including the other bug fix that's in there).

Comment 13

[Marc Attinasi]

Comment on attachment 53439 [details] [diff] [review]
Real removal of lowsrc

sr=attinasi - Thanks Pavlov

Comment 14

[Stuart Parmenter]

fixed.  no more lowsrc.  we may want to note this somewhere so people know.

Comment 15

[Marc Attinasi]

layout newsgroup might be good. 

BTW: I checked Composer and it has no lowsrc stuff int eh image attributes lists
- good thing! :)

Comment 16

[Dwayne]

Agggh, ya bastards.  I spent so much time getting this fixed (bug 27388) just to
get scrapped.  I understand why we don't need it anymore, but damn...

BTW, these window tabs are sweet.  They make bug tracking a lot easier!  :) 
(Hmm, what bug was that?  Oh yeah, it says it at the top)

Comment 17

[[e-mail]]

According to a comment in 23066 there's a DOM level 1 property lowSrc - has this
been removed as well?

Comment 18

[Stuart Parmenter]

lowSrc in the dom1 spec was, as i understand it, an error.  it is going to be
removed from future revisions of the spec.

Comment 19

[Brian Wilson]

I was just doing some HTML support testing and noticed that you had dropped 
LOWSRC. It looks like the train has long-since left on this issue, but someone 
raised the hypothetical Q of whether IE supports this attribute so I thought I 
would address that.

It does. It has since IE4. (Opera does not, BTW.) 

Granted, this is a small feature that IE implemented as a "me too" to netscape, 
but now IE is the "me first" case. I have no idea if that will affect your 
decision here.

Comment 20

[Stuart Parmenter]

this bug is fixed.  if you want to file a new bug, go ahead, but it isn't going
to get fixed.

Comment 21

[Shawn Wright]

Opera 6.05 *does* support lowsrc, for the record. It's hard to explain to users
that something that worked in Netscape 4.79, does work in both IE and Opera, and
worked in earlier Mozilla releases, now has been removed from Mozilla and Netscape.

Comment 22

[Michael T. Babcock]

I'm actually upset that lowsrc doesn't exist anymore, but I guess I'll just
browse slow sites with IE instead, right?

Comment 23

[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).