123099 - TRANSFER state not returned to nsIWebProgLstnr::OnStateChange() for several cases

Status: RESOLVED INCOMPLETE

(Core Graveyard :: Embedding: APIs, defect)

Description

[David Epstein]

Mozilla 0.9.7 gecko 20020122
1. Run testEmbed from bin folder (compile from /mozilla/embedding/qa/testEmbed
2. Add web progress listener from Tests menu.
3. Enter a url (I went to www.yahoo.com).
4. OK popup msgs.
5. Check TestOutput.txt logfile in C:/Temp
6. Track STATE_IS_TRANSFERRING entries.
Result: There is no transferring for image loads. Image lib isn't sending out
progress notifications. (e.g. for yahoo,
http://us.a1.yimg.com/us.yimg.com/a/1-/flash/compaq/ppc/redcompaq95x30.gif).
Note: This is happening for other protocols as well like ftp (which isn't
transfering for 'res:' entries).

Comment 1

[David Epstein]

Attachment: logfile of yahoo listener session

contains transition state entries for multiple requests at yahoo.com

Comment 2

[David Epstein]

changed qa contact to depstein

Comment 3

[David Epstein]

STATE_IS_NETWORK and ftp requests also aren't returning STATE_IS_TRANSFERRING. 

Comment 4

[David Epstein]

Rick landed a patch a couple of weeks ago which fixed some of these problems.
ftp sessions now include transfer states. Still broken are image loads and
STATE_IS_NETWORK.

Comment 5

[David Epstein]

Attachment: wplstnr session of local image load

shows that web prog lstnr is firing of STATE_IS_TRANSFERRING for DOCUMENT and
accompanied REQUEST, but not for other requests. I noticed that for the REQUEST
case where STATE_IS_TRANSFERRING appears, the string pathway is in the Win
directory form (i.e. C:\temp\file.txt) as opposed to the file url form (i.e.
file:///C:/temp/file.txt). STATE_START, REQUEST is in the file url form.

Comment 6

[David Epstein]

ccing kaie. Kai is there a security issue here with image loading? Web Progress
listener isn't firing off STATE_IS_TRANSFERRING notifications for image load
requests. The only transfer REQUEST it returns is the one which accompanies
DOCUMENT handling. But it doesn't notify for all other STATE_START (REQUEST)
instances.

Comment 7

[Kai Engert [:KaiE:]]

Yes. It is known that we do not receive notifications that would enable us to
track the security state of images.

Please see bug 135007 and bug 135011.

Comment 8

[David Epstein]

nominating topembed.

Comment 9

[Michael Buckland]

Rick-can you comment on this - does it affect any user?

Comment 10

[Judson Valeski]

.

Comment 11

[David Epstein]

this bug should be reassigned

Comment 12

[Benjamin Smedberg]

Marking a bunch of bugs in the "Embedding: APIs" component INCOMPLETE in preparation to archive that component. If I have done this incorrectly, please reopen the bugs and move them to a more correct component as we don't have "embedding" APIs any more.