18724 - Progress notifications not hitting!

Status: VERIFIED FIXED

(Core :: Networking, defect, P3)

Description

[Warren Harris]

Hitting?

Comment 1

[leger]

Bulk move of all Necko (to be deleted component) bugs to new Networking

component.

Comment 2

[Tomi Leppikangas]

I made a hack that gets progress notifications from httpchannel
to progressmeter. Patch attached.

Comment 3

[Tomi Leppikangas]

Attachment: patch for progressmeter

Comment 4

[Gagan]

law: I am checking in this patch which seems to do the job from HTTP part. Could
you verify that you are indeed getting this information (I am not sure how/where
this would be displayed) Are we planning to display in on the status bar as in
4.x? 

Comment 5

[Tomi Leppikangas]

Attachment: more work with progress notifications

Comment 6

[Tomi Leppikangas]

Here is pach #2, this makes also notifications to statusbar:
3k of 10 loaded
Document loaded in 2.77 sec (6k 2.40 kb/sec)

There is some ugly static variables (should be attributes of 
nsBrowserAppCore),
also some stuff with frames, should check that if this is frame or not.

Gagan also says that this should live in webshell..

Comment 7

[Warren Harris]

Moving what's not done for M15 to M16.

Comment 8

[Gagan]

the progress notifications are in a state of big change right now and so I am
pushing this till M18...

Comment 9

[Warren Harris]

Moving beta2 bugs out of M18.

Comment 10

[Tomi Leppikangas]

Travis made nice job with progress notifications. I moved my progressbar patch
to JS, just like onStatus. I attach new patch, not so ugly hack anymore,
puts simple progressbar working.

Comment 11

[Tomi Leppikangas]

Attachment: patch #3 update progressbar with JS

Comment 12

[leger]

Per gagan..ok to mark Fixed.  Checked in.

Comment 13

[Tom Everingham]

verif:
NT 2000053108