drag attachment to 'other' app doesn't work.

RESOLVED FIXED

Status

()

defect
RESOLVED FIXED
10 years ago
9 years ago

People

(Reporter: philbaseless-firefox, Assigned: philbaseless-firefox)

Tracking

({regression})

Trunk
x86
Windows XP
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(status1.9.2 ?, status1.9.1 ?)

Details

Attachments

(1 attachment, 3 obsolete attachments)

Assignee

Description

10 years ago
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1b3pre) Gecko/20090312 Shredder/3.0b3pre

Nothing happens when dragging attachment file to app.  e.g. open message with a .cpp file attachment and drag to VS.

Should open up file from a temporary location.

Outlook Express will copy dragged file to 'temporary internet files' and let outside app open from there.
Assignee

Comment 1

10 years ago
works in Thunderbird 2
"drag to VS", what is VS?
Assignee

Comment 3

10 years ago
dragging a c++ file to visual studio is suppose to open it for editing. 

But to make my comment more generic, drag a *.txt file to a text editor like word, that can handle dropped files, and it should open the file from the temporary location.
Keywords: regression
Summary: drag file to 'other' app doesn't work. → drag attachement to 'other' app doesn't work.
Assignee

Updated

10 years ago
Assignee: nobody → philbaseless-firefox
Assignee

Comment 4

10 years ago
this stems from widget/drag and drop in windows.  It need temp file support. TB2 only used temp files but the newer version of windows widget incorporated the stream type transfer but did not pick up the temp file support needed for this bug
Component: Attachments → Widget: Win32
Product: MailNews Core → Core
QA Contact: attachments → win32
Assignee

Updated

10 years ago
Depends on: 494989
Assignee

Comment 5

10 years ago
This enables dragging an attachment or email to text editor similar to depending bug does for images to image editor
Attachment #382661 - Flags: review?(emaijala)
Assignee

Comment 6

10 years ago
most of this code is the original image code only moved and shifted around and bracketed by conditionals (tab-formatted) to catch inserted new code. The new code covering the x-moz-file-promise flavor.
Assignee

Updated

10 years ago
Blocks: 497797
Assignee

Updated

10 years ago
Summary: drag attachement to 'other' app doesn't work. → drag attachment to 'other' app doesn't work.
Assignee

Updated

10 years ago
Flags: blocking1.9.1?
Assignee

Updated

10 years ago
Flags: blocking1.9.1? → wanted1.9.1.x?

Comment 7

10 years ago
Comment on attachment 382661 [details] [diff] [review]
utilizes CF_HDROP expected by external apps

Sorry, I was too slow and the patch doesn't apply to trunk anymore.
Attachment #382661 - Flags: review?(emaijala)
Assignee

Comment 8

10 years ago
(In reply to comment #7)
> (From update of attachment 382661 [details] [diff] [review])
> Sorry, I was too slow and the patch doesn't apply to trunk anymore.

it depends on bug 494989.  You can test this with that patch installed.

However, although other apps call GetData creating one temp file, windows shell has a nasty habit of calling GetData asking for CF_HDROP many times (~6) even though it uses the stream version.  so many temp files get created needlessly and that needs a module scope copy of the temp file for the many calls to GetData for this format.
I'll get that incorporated here but this will still work as is on top of 494989 patch
Assignee

Updated

10 years ago
Status: NEW → ASSIGNED
status1.9.1: --- → ?
Flags: wanted1.9.1.x?
Assignee

Comment 9

10 years ago
this updates due to changes to bugs upstream that aren't landing... yet
Attachment #382661 - Attachment is obsolete: true
Assignee

Updated

10 years ago
Duplicate of this bug: 213170
Assignee

Comment 11

10 years ago
Posted patch patched to trunk (obsolete) — Splinter Review
Attachment #402998 - Attachment is obsolete: true
Assignee

Updated

10 years ago
Duplicate of this bug: 415632
Assignee

Updated

10 years ago
Duplicate of this bug: 234921
Assignee

Comment 14

10 years ago
In summary, kFilePromiseMime ("application/x-moz-file-promise") registers CF_HDROP with windows and when windows asks for CF_HDROP there is no data. We know this can cause a freeze as it did with firefox. This patch returns data in the form of a temp file.

According to nsITransferable.idl the kFilePromiseMime is suppose to supply data through nsIFlavorDataProvider, implementing creation of the temp file, but it doesn't.

The other option for front end wanting to create temp files for passing in data would be to create a new flavor called kFileTempMime "application/x-moz-tempfile". I can do that here pretty easy if wanted but it will need a change to the idl and associated docs.

This patch will give back Thunderbird's ability to drag attachments to an application and most likely will be needed for xulrunner apps.
Attachment #412358 - Attachment is obsolete: true
Attachment #418504 - Flags: review?(roc)
Assignee

Updated

10 years ago
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/2484421e2013
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Assignee

Updated

10 years ago
Duplicate of this bug: 536103
Assignee

Updated

10 years ago
Duplicate of this bug: 539429
Assignee

Comment 18

10 years ago
This is a regression for Thunderbird all the way to TB2.  Anyway we can get this into 1.9.2 for TB3.1. Users are asking me when they can get the feature back.
status1.9.2: --- → ?
You need to log in before you can comment on or make changes to this bug.