Closed Bug 1116001 Opened 9 years ago Closed 9 years ago

Save image as... re-downloads the image

Categories

(Core Graveyard :: File Handling, defect)

34 Branch
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 120809

People

(Reporter: tdhutt, Unassigned)

Details

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:34.0) Gecko/20100101 Firefox/34.0
Build ID: 20141126041045

Steps to reproduce:

I have a website that generates an image in response to a POST request (because one of the parameters is too long for GET). If you browse to that site, the image is displayed correctly in firefox. (Only the image is sent - it isn't embedded in a web page.)

If you then right-click the image and select Save Image As... Firefox re-fetches the image but using a GET, and without sending any parameters, which means that it just saves the error message.


Actual results:

Instead of the image being saved, an error message from my site ("Parameter missing") is saved as output.png. That's really going to confuse people!


Expected results:

Firefox should clearly cache the image file and just copy it when a save is requested, or re-encode the image from memory. Re-downloading it is wasteful, unexpected and impossible in some cases (e.g. when the image comes from a POST - you can't refetch it because POST is often used as shorthand for "changes something on the server").
Chrome behaves correctly by the way.
>Firefox should clearly cache the image file
That depends if caching is allowed and i don't know if Gecko caches Post requests without additional headers.

>or re-encode the image from memory
You would not get the same file and that means that this is no option.
This almost certainly is a duplicate.
QA Whiteboard: [bugday-20141229][DUPEME?]
Status: UNCONFIRMED → RESOLVED
Closed: 9 years ago
QA Whiteboard: [bugday-20141229][DUPEME?] → [bugday-20141229]
Component: Untriaged → File Handling
OS: Mac OS X → All
Product: Firefox → Core
Hardware: x86 → All
Resolution: --- → DUPLICATE
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.