300419 - Documentation for "http-on-examine-merged-response" should mention whether it's always broadcast

Status: NEW

(Core :: Networking, defect, P3)

Description

[Christian :Biesinger (don't email me, ping me on IRC)]

152 /**
153  * The observer of this topic is notified after the received HTTP response
154  * is merged with data from the browser cache.  This means that, among other
155  * things, the Content-Type header will be set correctly.
156  */
157 #define NS_HTTP_ON_EXAMINE_MERGED_RESPONSE_TOPIC
"http-on-examine-merged-response"

It looks to me like this topic is only broadcast when there was indeed some
cached data (actually, only if that cached data was used _and_ there was a
server hit). It would be nice if the comment explicitly said when it was broadcast.

Comment 1

[Darin Fisher]

-> default owner

Comment 2

[Xavier]

It seems to me that this topic is broadcasted when you hit the reload button, but **not** when you hit the Enter key on an URL, except for the favicon request...

When I hit the Enter key, the page is fetched from the cache, and not even parsed to check its images and such other "included" files.
Only http-on-modify-request is broadcasted, making it difficult to know when the request completes

When I hit the reload button, the page is fetched from the cache, and both http-on-examine-response and http-on-examine-merged-response are broadcasted...

Comment 3

[Amit Kr. Chandra [:wizard_A]]

I would like to work on this bug... need some info as to how to reproduce the bug and the documentation that needs to be changed.

Comment 4

[Firefox Bug Husbandry Bot]

Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258

Comment 5

[Firefox Bug Husbandry Bot]

Bulk change to priority: https://bugzilla.mozilla.org/show_bug.cgi?id=1399258

Comment 6

[Somu]

Hi, I would like to work on this. Can this issue please be assigned to me? Thankyou.