Open Bug 634073 Opened 10 years ago Updated 6 months ago

"http-on-modify-request" is not fired for cached images

Categories

(Core :: ImageLib, defect)

x86
Windows XP
defect
Not set
normal

Tracking

()

People

(Reporter: Honza, Unassigned)

Details

(Whiteboard: [firebug-p1][netmonitor])

Attachments

(1 file)

STR:

You need to enable: browser.dom.window.dump.enabled true

1) Install the attached extension
2) Load http://getfirebug.com/tests/issues/3742/issue3742.html
3) Do a non-revalidating reload (press ENTER in address bar)
4) See the system console, it's displaying:
--- observe: http://localhost/firebug/tests/issues/3742/issue3742.html
"http-on-modify-request" are not fired for images.



If you refresh the page by pressing F5 you'll see:
--- observe: http://localhost/firebug/tests/issues/3742/issue3742.html
--- observe: http://localhost/firebug/tests/issues/3742/image1.png
--- observe: http://localhost/firebug/tests/issues/3742/image2.png


Source code of the attached extension is available here:
http://code.google.com/p/fbug/source/browse/#svn%2Ftests%2Fmozilla%2FnetObserver

Honza
Whiteboard: [firebug-p1]
Confirming this issue,

For my add-on update at AMO (https://addons.mozilla.org/en-US/firefox/addon/6371) I need "http-on-modify-request" working for cached content.

Please also consider to make setResponseHeader working for Content-Length which currently is READONLY and therefore throws NS_ERROR_ILLEGAL_VALUE on modification attempts. Reason: Quite a lot servers send wrongly calculated Content-Length (possibly caused by invalid calculations posted at http://php.net/manual/en/function.header.php and elsewhere) which prevents caching (see bug 588507) - but this request probably needs to be filed as another new bug(?).
Whiteboard: [firebug-p1] → [firebug-p1][netpanel]
Whiteboard: [firebug-p1][netpanel] → [firebug-p1][netmonitor]
I can confirm that this is still an issue in Nightly 44.0a1 (2015-09-23).
And of course this is not just affecting Firebug but also the built-in devtools' Network panel.

Sebastian
Component: Networking → ImageLib
You need to log in before you can comment on or make changes to this bug.