33156 - [FEATURE] DocLoader needs to use the new web progress interfaces

Status: VERIFIED FIXED

(Core :: DOM: Navigation, defect, P1)

Description

[Scott MacGregor]

This is a tracking bug for Travis and I. I need to rewrite the doc loader to
send out notifications using his web progress interfaces.

Comment 1

[Scott MacGregor]

accepting.

Comment 2

[Scott MacGregor]

I checked in the first part of this which is to signal a status changed when we
start loading a document and when we are done loading a document. This causes
the network activity and window busy flags to be set and then cleared when the
document is done loading.

Comment 3

[Scott MacGregor]

triaging to M16 since m15 is tonight.

Comment 4

[Scott MacGregor]

Marking as one of my P1 bugs for M16.

Comment 5

[Scott MacGregor]

Here's a patch that properly implements OnStatusChange and OnChildStatusChange
for starting and stopping document loads. It includes properly firing the right
notifications when loading a document that contains a frameset.

You now get something like
OnStatusChange for the parent saying we are starting to load a document
    OnChildStatusChange for each frame starting to load
    OnChildStatusChange for each frame ending its load
OnStatusChange for the parent saying it's done loading.

I believe after this the only thing left is implementing progress on the doc loader.

Comment 6

[Christopher Blizzard (:blizzard)]

Hrm, I want this patch.  Are you going to actually attach it? :)

Comment 7

[Scott MacGregor]

Sorry I haven't been able to add attachments to bug reports in the last couple
days. We time out during the post.....still waiting for that to get fixed so I
can post the changes.

Comment 8

[Christopher Blizzard (:blizzard)]

Is that with Mozilla or Netscape 4?  I had problems posting large documents with
Mozilla but was able to do it with NS 4.  Can you mail me the patch otherwise? 
I'm working on something that depends on it.

Comment 9

[Scott MacGregor]

I only use mozilla so that means I can't post the patch. I emailed you the same
diffs I've been giving dougt.

Comment 10

[Scott MacGregor]

Over the weekend I checked in changes to now report progress load notification.
I belive the docloader work for this bug is now done.

I'm leaving this bug open 'cause I'm using it to check in navigator changes to
get the progress bar to use this new information.

Comment 11

[Scott MacGregor]

The work I needed to do for this bug is now done. Progress and start / stopping
of document load notifications all run through the new web progress interfaces now.

Comment 12

[Paul Wyskoczka]

Verified