20188 - Throbber doesn't stop when image URL is invalid

Status: VERIFIED DUPLICATE of bug 26582

(Core :: Graphics: ImageLib, defect, P3)

Description

[Richard Zach]

The throbber and the download progress bar don't stop when displaying
directories over FTP.

Eg: go to ftp://ftp.mozilla.org/

Build 1999112808 Linux.

Comment 1

[Asa Dotzler [:asa]]

win32 on 98 and NT4 display same behavior in 12/08/08 build
Also the spinner doesn't stop spinning until you hit the stop button.
After you hit the stop button the console reports an error loading the ftp URL.

Comment 2

[Eric Vaughan]

xpapps bug

Comment 3

[Paul MacQuiddy]

Rick, searched through bug reports and saw someone else reported the same one I
had a couple days ago. And lo and behold it is happening again.

I tried it on 4.x and no problems. I don't know what to say. There are a few
other bugs I found, so it seems like this is an issue.

Comment 4

[Stephen Anderson]

I too have a throbber no stop problem.  But my problem is not related to ftp
and it shows up on Win95.  I first noticed this problem starting with M11 and
continues through build 1999121712.  The problem is seen on the page Debug |
Viewer Demos | #1 CSS Styles - aka test1.html.  The page loads fully and
completely, but the throbber and the status bar animation do not stop.

I had seen a previous bug dealing with animated gifs on the mac and test1.html
does include an animated gif, but the problem persists even with reference to
that image removed from the HTML.  You can find a fair amount of mention of
this specific page and the throbber on bug# 19114.

Comment 5

[Stephen Anderson]

Oh, I really should have mentioned that indeed, the throbber does not stop on
ftp links to directories as originally noted by zach for this bug.

Comment 6

[Stephen Anderson]

Okay, I've narrowed down exactly what is causing the problem with the viewer
demo page I mentioned.  It is this outside url in the style sheet
specification.  I'm attaching a stripped down copy of this page as a testcase.

Basically the testcase is:

<html>
<head>
<title>Example 1</title>
<style>
UL LI {list-style: square outside url(resource:/res/raptor.jpg);	//none;}
</style>
</head>
<body bgcolor="#FFFFFF" text="#000000">
<ul>
  <li>My style is the problem</li>
</ul>
</body>
</html>

Comment 7

[Stephen Anderson]

Attachment: Testcase

Comment 8

[Stephen Anderson]

Oops, I'm not real familliar with Style Sheet syntax.  I may have made a boo-
boo with that first testcase.

UL LI {list-style: square outside url(resource:/res/raptor.jpg);  //none; }

might need to be

UL LI {list-style: square outside url(resource:/res/raptor.jpg);  //none;
}

Just in case, I'm making a new attachment without that possible mistake.
Results are the same though.

Comment 9

[Stephen Anderson]

Attachment: Testcase without typo

Comment 10

[Stephen Anderson]

Added testcase to whiteboard because URL provided demonstrates FTP scenario,
and Debug | Viewer Demos | #1 CSS Styles (provided in Mozilla) demonstrates the
other mentioned scenario to reproduce this bug.

Comment 11

[Warren Harris]

Moving Rick's M14 bugs to M13 (since he won't be here for M14). He can triage
them to M15 as appropriate.

Comment 12

[Warren Harris]

Back to Jud.

Comment 13

[Judson Valeski]

I fixed this earlier this week.

Comment 14

[Richard Zach]

It's still broken (both for the testcase and ftp) on yesterday's Linux build
(2000-01-06-08). (For some reason I can't manage to download today's build).

Comment 15

[Paul MacQuiddy]

There have been major linux builds problems, please wait for builds from this
weekend - some changes are not getting into new builds apparently

Comment 16

[Richard Zach]

Works for the ftp urls, but the css testcase is still broken (Linux build
2000.01.08.08)

Comment 17

[Stephen Anderson]

My observations of build 2000010908 on Win95 also match the last comments.
Reopening bug because CSS Testcase still causes malfunction.

Comment 18

[leger]

Clearing FIXED resolution due to reopen.

Comment 19

[Judson Valeski]

reassigning to Pierre. please do not put multiple

Comment 20

[Judson Valeski]

sorry. "multiple... bugs in one bug report."

Comment 21

[Pierre Saslawsky]

Moved to M14

Comment 22

[ekrock's old account (dead)]

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

Comment 23

[Marc Attinasi]

Assigning to ekrock: 1/3 of Pierre's NEW bugs to help reduce his doomage factor

Comment 24

[Pierre Saslawsky]

Reassigned back to me these bugs that shouldn't have left my list.

Comment 25

[Marc Attinasi]

The problem is that the image URL in the CSS rule is invalid, so it keeps 
looking for it. If you put in a valid url then the throbber stops as it should. 
This appears to be intermittent as well, since most of the time it will stop 
throbbing and use a default image if the specified image cannot be loaded.

This problem is not restricted to URL's in style rules: if you specify an image 
with a bogus URL you get the same behavior (actually, it appears to be more 
consistent). Try something like:

<BODY>
<P>Bogus image to follow</P>
<IMG SRC="resource:/res/bogus-image.gif">
</BODY>

A problem with the image loader possibly?

Comment 26

[Marc Attinasi]

Attachment: Test case showing how valid URLs cease throbbing, and how IMG has same problem

Comment 27

[Pierre Saslawsky]

Reassigned to ImageLib. It could be a network problem too. I guess someone ignore 
an error code somewhere between these two.

Comment 28

[pnunn]

duplicate


*** This bug has been marked as a duplicate of 26582 ***

Comment 29

[Eli Goldberg]

Rubber-stamping verified as duplicate; left note in 26582 to verify this bug as
part of 26582's verification.