API needed for hooks into dnd embedding

VERIFIED FIXED in mozilla0.9.9

Status

()

Core
Embedding: APIs
VERIFIED FIXED
16 years ago
15 years ago

People

(Reporter: Mike Pinkerton (not reading bugmail), Assigned: Mike Pinkerton (not reading bugmail))

Tracking

({topembed})

Trunk
mozilla0.9.9
topembed
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

16 years ago
an embedding app needs to be able to hook in at certain points to the dnd
process in order to do things like override what flavors go out on the drag, or
to cancel a drop outright.

we need to develop an api to do this.
(Assignee)

Updated

16 years ago
Status: NEW → ASSIGNED
Keywords: topembed
Target Milestone: --- → mozilla1.0
(Assignee)

Comment 1

16 years ago
Created attachment 71736 [details] [diff] [review]
new api and changes

new api and updated callers and impl

needs r/sr. i want to get this in for 099 so anyone on that branch can use it.
(Assignee)

Updated

16 years ago
Keywords: mozilla0.9.9
Target Milestone: mozilla1.0 → mozilla0.9.9
Comment on attachment 71736 [details] [diff] [review]
new api and changes

API looks nice. I'd call "disallowStart" "startIsAllowed" or "isStartAllowed"
Same for disallowDrop. These things are asking for permission, not saying that
it's been denied.
(Assignee)

Comment 3

16 years ago
Created attachment 71752 [details] [diff] [review]
update for ccarlen's changes

changed disallowXX to allowXX and reversed the logic in the c++
Attachment #71736 - Attachment is obsolete: true
Comment on attachment 71752 [details] [diff] [review]
update for ccarlen's changes

r=ccarlen
Attachment #71752 - Flags: review+

Comment 5

16 years ago
Comment on attachment 71752 [details] [diff] [review]
update for ccarlen's changes

sr=sfraser
Attachment #71752 - Flags: superreview+
Comment on attachment 71752 [details] [diff] [review]
update for ccarlen's changes

a=shaver for 0.9.9.
Attachment #71752 - Flags: approval+
(Assignee)

Comment 7

16 years ago
landed in 099
Status: ASSIGNED → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED

Comment 8

15 years ago
Over to depstein for verification
QA Contact: mdunn → depstein

Comment 9

15 years ago
Verified patch checkin against Mozilla 1.2b Mozilla/5.0 (Windows; U; WinNT4.0;
en-US; rv:1.2b) Gecko/20021026 build.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.