76728 - Can't drag a shortcut into browser

Status: VERIFIED FIXED

(Core :: DOM: Copy & Paste and Drag & Drop, defect, P3)

Description

[Mike Pinkerton (not reading bugmail)]

- go to www.mozilla.org
- drag proxy to desktop, proxy created
- go to another page
- drag shortcut from desktop into browser

expected:
- mozilla.org should load

actual:
- nada

something has regressed.

Comment 1

[Blake Ross]

<blake> pink: will you send me a mac for free, no questions asked?
<pinkerton> no

I don't have a Mac, so not much I can do with this one.  I've got bug 70302 
handling Windows issues.  --> pink, who asked to have this (at least in my mind)

Comment 2

[Mike Pinkerton (not reading bugmail)]

--> xpapps

Comment 3

[Viswanath Ramachandran]

accepting for mozilla0.9.2. P3. 

Comment 4

[Mike Pinkerton (not reading bugmail)]

if it's beta1+, it's catfood.

Comment 5

[Paul Chen]

nav triage team:

Umm, no, nsbeta1+ != nsCatFood+

Comment 6

[Mike Pinkerton (not reading bugmail)]

yes, this is catfood (because this is important functionality). if we're 
committed to fixing for beta1, why is it 0.9.2?

Comment 7

[Matthew T (active 1999-2002)]

This is also blocking the testing of several other bugs, described in bug 
44267.

Comment 8

[Mike Pinkerton (not reading bugmail)]

i have a patch. two lines, piece of cake.

Comment 9

[Mike Pinkerton (not reading bugmail)]

Index: mozilla/xpfe/communicator/resources/content/contentAreaDD.js
===================================================================
RCS file: /m/pub/mozilla/xpfe/communicator/resources/content/contentAreaDD.js,v
retrieving revision 1.18
diff -u -2 -r1.18 contentAreaDD.js
--- contentAreaDD.js	2001/05/12 20:07:12	1.18
+++ contentAreaDD.js	2001/06/04 23:22:17
@@ -191,6 +191,6 @@
     {
       var flavourSet = new FlavourSet();
-      flavourSet.appendFlavour("application/x-moz-file", "nsIFile");
       flavourSet.appendFlavour("text/x-moz-url");
+      flavourSet.appendFlavour("application/x-moz-file", "nsIFile");
       flavourSet.appendFlavour("text/unicode");
       return flavourSet;

Comment 10

[Blake Ross]

[s]r=blake

Comment 11

[Mike Pinkerton (not reading bugmail)]

Index: mozilla/xpfe/communicator/resources/content/contentAreaDD.js
===================================================================
RCS file: /m/pub/mozilla/xpfe/communicator/resources/content/contentAreaDD.js,v
retrieving revision 1.18
diff -u -2 -r1.18 contentAreaDD.js
--- contentAreaDD.js	2001/05/12 20:07:12	1.18
+++ contentAreaDD.js	2001/06/04 23:29:57
@@ -191,7 +191,7 @@
     {
       var flavourSet = new FlavourSet();
-      flavourSet.appendFlavour("application/x-moz-file", "nsIFile");
       flavourSet.appendFlavour("text/x-moz-url");
       flavourSet.appendFlavour("text/unicode");
+      flavourSet.appendFlavour("application/x-moz-file", "nsIFile");
       return flavourSet;
     },

making files last.

Comment 12

[Simon Fraser [no longer active]]

sr=sfraser.
See bug 84063 for a problem about dragging files with a / in the name.

Comment 13

[Asa Dotzler [:asa]]

a= [e-mail] for checkin to the trunk.
(on behalf of drivers)

Comment 14

[Mike Pinkerton (not reading bugmail)]

landed on trunk

Comment 15

[Mike Pinkerton (not reading bugmail)]

*** Bug 70302 has been marked as a duplicate of this bug. ***

Comment 16

[Terri Preston]

Verified on trunk build 2001060508, any chance of landing on branch?

Comment 17

[Christopher Blizzard (:blizzard)]

a=blizzard on behalf of drivers for 0.9.1

Comment 18

[Mike Pinkerton (not reading bugmail)]

landed on brach, 6/5/01, 3:30pm

Comment 19

[Terri Preston]

Verified branch 2001-06-06-03-0.9.1