Closed Bug 499515 Opened 15 years ago Closed 15 years ago

fix OS/2 Drag and Drop bugs

Categories

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

x86
OS/2
defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: dragtext, Assigned: dragtext)

Details

(Keywords: verified1.9.1)

Attachments

(1 file)

The attached patch fixes these issues:

- rearranging tabs or tearing one off to open in a new window can not be canceled by pressing Escape;  this bug was fixed months ago on other platforms but not OS/2.

- after rearranging tabs, dropping native objects (i.e. files & text) on the browser has no effect.

- after loading a page by dropping a native URL on the browser, native text can not be dropped into form fields on the page without pressing Ctrl.

The patch also eliminates a few compiler warning messages.
Attachment #384260 - Flags: review?(mozilla)
Comment on attachment 384260 [details] [diff] [review]
fixes nsDragService

I haven't tested the patch, but it does look OK to me. (I guess the bottom 3 chunks of the patch are not really part of the solution?)
Attachment #384260 - Flags: review?(mozilla) → review+
(In reply to comment #1)
> (From update of attachment 384260 [details] [diff] [review])
> I haven't tested the patch, but it does look OK to me. (I guess the
> bottom 3 chunks of the patch are not really part of the solution?)

They deal with warnings:  assigning a char constant to a non-const char ptr, and an "ambiguous" if statement.
OK, great. Pushed: http://hg.mozilla.org/mozilla-central/rev/311bae8dffe0

I take it that we also want this on the 1.9.1 branch for FF 3.5, so I'm going to wait a bit and push it to branch in a few hours.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Rich/Peter,

Would you be willing to verify the fix for this bug on latest branch 1.9.1 as it's on OS/2?

Thanks
I think this was verified some time ago.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: