29219 - Wired stock charts do not display in M13

Status: VERIFIED FIXED

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

Description

[dan]

At wired news' stock charting page, the image containg the chart does not
display. Instead the text "dllchart' appears. I'm using mozilla M13 on WinNT
4.0(sp5).

To reproduce, simply go to the above URL and enter a ticker symbol, (eg SUNW)
then press GO.

The page works fine in Netscape 4.72 and IE5.0

Comment 1

[Gervase Markham [:gerv]]

WORKSFORME in M13 W95 (modulo loads of other page layout problems) and 
2000022608 W95.

We need an NT person to check this out...

Gerv

Comment 2

[Doron Rosenberg (IBM)]

seeing this on m15 nightlies on win32 and linux

moving to imagelib, not sure if it is the right component though...

the image link is:
http://chart.neural.com/scripts/chart/dllchart.dll?MfcISAPICommand=Chart&Sym1=SUNW&width=432&height=250&dRes=DAY&dCnt=12&dPeriod=MONTHS&PlotType=LINE&ind7=volume&csym1=000000&cSym2=FF0000&csym3=CC9966&csym4=99CC33&csym5=9999CC&cbckg=FFFFCC&cbcku=FFFFFF&cbckd=CCCCFF&cbckl=FFFFFF&ctxtu=000000&ctxtd=000000&ctxtl=000000&cind1a=red&cind2=blk&cind3=red&cind3a=lblu&cind4=7200AA&cind5=AA681A&cind6=mag&cavg1=0066FF&cavg2=FF00FF&Source=SP&Ignore=1220001147

Comment 3

[Doron Rosenberg (IBM)]

confirming bug. sorry for the spamm

Comment 4

[pnunn]

I'll dig into it.
-P

Comment 5

[Gervase Markham [:gerv]]

This is now working in every build I can lay my hands on, including the current 
one (20000428). Could it have been their problem?

Can [e-mail], [e-mail] and/or [e-mail] try this 
both in new and old builds, and say what they see?

Gerv

Comment 6

[pnunn]

Eli:
If this works for you, would you close the bug?
We don't have time to play with dead snakes...just make
sure its really dead.

thnx,
p

Comment 7

[Eli Goldberg]

Hmm...using today's build on NT, Linux & Mac and "MSFT" as the ticker symbol, I 
see no chart.

Comment 8

[Eli Goldberg]

...specifically, I see an empty image frame, which is sometimes (on Linux) 
replaced immediately by alt text (dllchart).

Clicking on the image and choosing 'View Image' from the context menu doesn't 
show any image, either.

Comment 9

[leger]

Putting on [nsbeta2+][6/01] radar.  This work must be done by 06/01 or we may 
pull this for PR2.

Comment 10

[pnunn]

Gagan:
I'm not sure who should own this.
Once the image is generated, the imglib can display it.
If I get the chart image from 4.x and save it, mozilla
displays it fine.

Looking at the data stream with traceplus, it looks like the
cgi is submitted with a GET, but nothing is ever returned.

Is there something funny with how we make the request? Or what
data stream we accept?

Reassigning to you in the hopes you will know where the
bug should go.
-P

Comment 11

[leger]

Changing from [6/01] to [6/15]

Comment 12

[Gagan]

->ruslan

Comment 13

[ruslan]

Yup. We've a serious problem it seems in nsSocketTransport. It entirely depends 
on the way the data is coming from the web server. It seems like we've a raise 
condition where we will stop reading after the pipe is full and not go back to 
PR_Read when the consumer read the data. I'm looking into it. It may explain 
some other partial loads and it may sometimes work or not work depending on the 
timing. UPgrading severity to critical.

Comment 14

[ruslan]

Ok. Everything is simpler then that. That particular site sends a 
bogus Content-Length header without a semicolon and we choke up on it. IE and 
NS4.x seem to tolerate this. I change the header parser to take care of this and 
will check this in when the tree opens up.

Comment 15

[ruslan]

Checked in

Comment 16

[Eli Goldberg]

This bug is verified fixed. Using today's builds, the chart displays correctly on 
Windows, Macintosh, and Linux.