Drag and Drop not implemented under BeOS

RESOLVED FIXED

Status

()

Core
Drag and Drop
P3
normal
RESOLVED FIXED
16 years ago
12 years ago

People

(Reporter: Paul, Assigned: tqh)

Tracking

(Depends on: 1 bug, {fixed1.8})

Trunk
x86
BeOS
fixed1.8
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments, 10 obsolete attachments)

2.24 KB, text/plain
Mike Schroepfer
: approval1.8rc2+
Details
696 bytes, patch
Sergei Dolgov
: review+
Scott MacGregor
: approval1.8rc2+
Details | Diff | Splinter Review
74.31 KB, patch
Sergei Dolgov
: review+
Details | Diff | Splinter Review
73.98 KB, patch
Scott MacGregor
: approval1.8rc2+
Details | Diff | Splinter Review
(Reporter)

Description

16 years ago
Drag and Drop was not implemented in the BeOS port.
(Reporter)

Comment 1

16 years ago
I have written some code to implement Drag and Drop.  Currently, this code does
not support native dnd by default, but most of the code to do so, is there, but
turned off.  BeOS uses BMessages to do dnd, and sends multiple messages between
the target and the source when something gets dropped.  The final message was
not getting through, and since I was unable to find a reason for this, I decided
to at least make the drag and drop work for just mozilla, and worry about native
support later.

Below is the zip file containing the patch, as well as some new files.
(Reporter)

Comment 2

16 years ago
Created attachment 53496 [details]
Patch for DND on BeOS.  This does not support native drags.

Comment 3

16 years ago
Nothing to spec, take thee to xpapps: D&D!
Assignee: mpt → blakeross
Component: User Interface Design → XP Apps: Drag and Drop
QA Contact: zach → tpreston

Comment 4

16 years ago
it's a good start
Assignee: blakeross → arougthopher
(Reporter)

Comment 5

16 years ago
DND for bookmarks from the URL bar, mostly works.  DND for mail does not work. 
It calls the GetData, and does get it, and puts it in the transerable object,
but the mail does not get moved.
(Reporter)

Updated

16 years ago
Status: NEW → ASSIGNED

Comment 6

15 years ago
qa contact -> pmac
QA Contact: tpreston → pmac
(Reporter)

Comment 7

15 years ago
updated patch (finally) to come
(Reporter)

Comment 8

15 years ago
Created attachment 107553 [details] [diff] [review]
updated patch (native dnd not implemented yet) needs work

I am posting this patch because I need another set of eyes to look at it.  I
cannot, for the life of me, see why this is not working.  If, I remove my
preference directory, the first drag to the toolbar works for me, but no other
drags will.  The "IsDataFlavorSupported" method seems to be working just fine
(in mail, folders become hightlighted on drag over events)  But, GetData is not
always called (in the mail tree) and, even if it is, and gets the data, nothing
is actually dragged.  

I think I've looked at this code too long, and need some help.	It is more than
likely, something stupid I have overlooked.

To see the log, set NSPR_LOG_MODULES=nsDragService:5
Attachment #53496 - Attachment is obsolete: true
(Reporter)

Comment 9

15 years ago
Created attachment 107554 [details]
nsIDragSessionBeOS.h - needed for above patch

Extra metods needed for BeOS implementation
(Reporter)

Comment 10

14 years ago
Setting the priority, so I can better organize the issues in bugzilla I need to
take care of
Priority: -- → P3

Updated

14 years ago
QA Contact: pmac → timeless
(Assignee)

Updated

13 years ago
Blocks: 225778

Updated

13 years ago
Blocks: 266252
(Assignee)

Comment 11

13 years ago
Trying to understand how to do this. I've added nsIDragSessionBeOS.h and the
makefiles (in case it's needed). I'm a bit unsure about nsDragService.cpp
(InvokeDragSession) in the patch and how to do that. If anyone has any pointers
please let me know. 
(Assignee)

Comment 12

13 years ago
Created attachment 176358 [details] [diff] [review]
Half-updated old patch

This patch compiles together with nsIDragSessionBeOs.h (in widget/public) but
is not in any way functional. I don't think the changes in nsWindow.cpp (none
in this patch) will be all that hard if I just can figure out nsDragService.cpp
properly. 

(I suspect the old patch would have worked a bit better nowadays since we've
fixed a couple of things since then.)
Assignee: beos → thesuckiestemail
(Assignee)

Comment 13

13 years ago
Created attachment 176377 [details] [diff] [review]
nsDragService updated (drag working)

Ok, think I've translated the old version of DragService. Not sure exactly how
everything is supposed to work, but at least it works for dragging. Only
nsWindow patches left, which I'm a bit more confident about.
Attachment #176358 - Attachment is obsolete: true
(Assignee)

Comment 14

13 years ago
Simon, as I've updated DragService.cpp you maybe could take a look and see if
it's  ok. It seems to initiate every drag I've tried so that part might be ok.
(Assignee)

Comment 15

13 years ago
Ok, managed to get some drops going. I can now drag links within the same window
and drop them on the address-bar. Some times the page doesn't load properly, but
as I don't do a lot of cleanup right now that is not my main concern. Dragging
between Firefox-windows still seems broken though (so no Customize) or dragging
link to another url-bar.

Comment 16

13 years ago
Good work.

It's going to be another busy week for me though, so if you're happy that what
you have so far works OK, I'll leave it until you've got the drops going before
I take a look.

If you run into dificulties let me know and I'll see if I can find some time to
look at it.
(Assignee)

Comment 17

12 years ago
Created attachment 185337 [details]
Slightly updated nsIDragSessionBeOS.h
(Assignee)

Updated

12 years ago
Attachment #107554 - Attachment is obsolete: true
(Assignee)

Updated

12 years ago
Attachment #185337 - Attachment description: Slightly updated → Slightly updated nsIDragSessionBeOS.h
(Assignee)

Comment 18

12 years ago
Created attachment 185339 [details] [diff] [review]
Working Internal DND, so not from/to other apps yet.

This one seems to work nicely, and if I didn't miss anything in the patch I
suggest we review and commit this work in progress (it's dragging to/from
native apps that needs work). It will fix one bug and users can finally use the
customize feature...
(Assignee)

Updated

12 years ago
Attachment #107553 - Attachment is obsolete: true
Attachment #176377 - Attachment is obsolete: true
(Assignee)

Comment 19

12 years ago
Image dragging depends on bug 294234.
Depends on: 294234
(Assignee)

Comment 20

12 years ago
Hmm, seems I missed something in nsWindow from my experimental version, or I
fixed something in it, as customize in Firefox seems to work poorly and maybe
other things as well.
(Assignee)

Comment 21

12 years ago
Well it seems it's ok.
(Assignee)

Comment 22

12 years ago
Comment on attachment 185337 [details]
Slightly updated nsIDragSessionBeOS.h

r?
Attachment #185337 - Flags: review?(sergei_d)
(Assignee)

Updated

12 years ago
Attachment #185339 - Flags: review?(sergei_d)
(Assignee)

Comment 23

12 years ago
Comment on attachment 185337 [details]
Slightly updated nsIDragSessionBeOS.h

Soon replaced (hopefully)
Attachment #185337 - Flags: review?(sergei_d)
(Assignee)

Updated

12 years ago
Attachment #185339 - Flags: review?(sergei_d)
(Assignee)

Comment 24

12 years ago
Created attachment 187134 [details] [diff] [review]
How DND nsWindow should be 

This is just so I can use the nice diff-function, but I think this is probably
how nsWindow will end up.

Updated

12 years ago
Depends on: 287759

Comment 25

12 years ago
which of the above patches must be applied to test?  all?  just the last? 
Please advise.
(Assignee)

Comment 26

12 years ago
With our recent nsWindow changes I'll have to update this, otherwise I think it
will be very complicated to patch if you don't know how it works.

Comment 27

12 years ago
I can't claim to know how it works, but was willing to see if I could revise the
patch, out of respect for your schedule, tqh.  I'll gladly wait for something to
test if you have time to devote to this project.  I truly appreciate all the
progress you've made in this area.
(Assignee)

Comment 28

12 years ago
Ok, I think it's time to close this one, or Niels will get angry with me.
(Assignee)

Comment 29

12 years ago
Created attachment 199590 [details] [diff] [review]
Functionally complete DND-handling in nsWindow

This is only nsWindow changes, I will fix the other files as well.
Attachment #187134 - Attachment is obsolete: true

Comment 30

12 years ago
nsWindow part looks ok
(Assignee)

Updated

12 years ago
Attachment #185339 - Attachment is obsolete: true
(Assignee)

Updated

12 years ago
Attachment #185337 - Attachment is obsolete: true
(Assignee)

Comment 31

12 years ago
Comment on attachment 199590 [details] [diff] [review]
Functionally complete DND-handling in nsWindow

Out with the old...
Attachment #199590 - Attachment is obsolete: true
(Assignee)

Comment 32

12 years ago
Created attachment 199665 [details]
New dragsession interface for BeOS (widget/public)

And in with the new. (I think this one is final.)
(Assignee)

Comment 33

12 years ago
Created attachment 199666 [details] [diff] [review]
Makefile.in adds Interface (widget/public)

Also final I think.
(Assignee)

Comment 34

12 years ago
Created attachment 199668 [details] [diff] [review]
DND changes in widget/src/beos

I think all but nsDragService.cpp is final. I suggest we check this in as it is
better than current behaviour and the widget/public changes are hardest to
commit. I will then work on improving nsDragService.cpp for DND to and from
other apps...
(Assignee)

Updated

12 years ago
Attachment #199665 - Flags: review?(sergei_d)
(Assignee)

Updated

12 years ago
Attachment #199666 - Flags: review?(sergei_d)
(Assignee)

Comment 35

12 years ago
Comment on attachment 199668 [details] [diff] [review]
DND changes in widget/src/beos

Note that nsDragService.cpp is not complete, but it's a start, and I'd like to
have this first version in CVS and improve it from there. Also removed some
weird line in Makefile.in.
Attachment #199668 - Flags: review?(sergei_d)

Comment 36

12 years ago
Comment on attachment 199665 [details]
New dragsession interface for BeOS (widget/public)

r=sergei_d
Attachment #199665 - Flags: review?(sergei_d) → review+

Comment 37

12 years ago
Comment on attachment 199666 [details] [diff] [review]
Makefile.in adds Interface (widget/public)

r=sergei_d

probably needs sr, but unsure if rules weren't changed. Local BeOS-only change
in common file
Attachment #199666 - Flags: review?(sergei_d) → review+

Comment 38

12 years ago
tqh, which version of widget/src/beos you used for last patch?
I'm getting compilation errors:
    -DMOZILLA_CLIENT -include ../../../mozilla-config.h
-Wp,-MD,.deps/nsDragService.pp nsDragService.cpp
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/widget/src/beos/nsDragService.cpp:57:
warning: multiline `//' comment
In file included from
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/widget/src/beos/nsDragService.cpp:72:
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/dist/include/layout/nsPresContext.h:53:
nsILanguageAtomService.h: No such file or directory
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/widget/src/beos/nsDragService.cpp:74:
nsIView.h: No such file or directory
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/widget/src/beos/nsDragService.cpp:244:
warning: `#ifdef' argument starts with a digit
In file included from
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/widget/src/beos/nsDragService.h:44,
                 from
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/widget/src/beos/nsDragService.cpp:42:
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/widget/src/xpwidgets/nsBaseWidget.h:100:
warning: `nsBaseWidget::SetCursor(imgIContainer *, unsigned int, unsigned int)'
was hidden
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/widget/src/beos/nsWindow.h:148:
warning:   by `nsWindow::SetCursor(nsCursor)'
In file included from
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/widget/src/beos/nsDragService.cpp:72:
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/dist/include/layout/nsPresContext.h:673:
`nsILanguageAtomService' was not declared in this scope
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/dist/include/layout/nsPresContext.h:673:
template argument 1 is invalid
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/dist/include/layout/nsPresContext.h:673:
ANSI C++ forbids declaration `mLangService' with no type
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/widget/src/beos/nsDragService.cpp:
In method `nsresult nsDragService::InvokeDragSession(nsIDOMNode *,
nsISupportsArray *, nsIScriptableRegion *, unsigned int)':
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/widget/src/beos/nsDragService.cpp:193:
invalid use of undefined type `class nsIView'
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/dist/include/gfx/nsIDeviceContext.h:48:
forward declaration of `class nsIView'
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/widget/src/beos/nsDragService.cpp:204:
invalid use of undefined type `class nsIView'
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/dist/include/gfx/nsIDeviceContext.h:48:
forward declaration of `class nsIView'
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/widget/src/beos/nsDragService.cpp:
In function `static const char * nsDragService::FlavorToBeMime(const char *)':
/mozbuild/home/mozbone/2005-10-08-05-trunk/mozilla/widget/src/beos/nsDragService.cpp:782:
warning: control reaches end of non-void function
`nsDragService::FlavorToBeMime(const char *)' 

Comment 39

12 years ago
it compiles here if i change in 
nsPresContext.h
line
#include "nsILanguageAtomService.h"
to
#include "locale/nsILanguageAtomService.h"
and in nsDragService.cpp line
#include "nsIView.h"
to
#include "view/nsIView.h"
so, something is wrong in configuration

Comment 40

12 years ago
tried to move messages in SeaMonkeyMail from one folder to another.
Mail stays at old place, but some debug info appears in Terminal, like that:
BMessage: what = DATA (0x44415441, or 1145132097)
    entry  be:originator, type='CSTR', c=1, size=8, data[0]: "BeZilla"
    entry   be:clip_name, type='CSTR', c=1, size=18, data[0]: "BeZilla Drag Item"
    entry     be:actions, type='LONG', c=2, size= 4, data[0]: 0x44444350
(1145324368, 'DDCP')
                                                     data[1]: 0x44444d56
(1145326934, 'DDMV')
    entry       be:types, type='CSTR', c=2, size=19, data[0]: "text/x-moz-message"
                                            size=25, data[1]:
"application/octet-stream"
    entry   be:filetypes, type='CSTR', c=1, size=19, data[0]: "text/x-moz-message"
    entry be:type_descriptions, type='CSTR', c=1, size=19, data[0]:
"text/x-moz-message"
mailCharsetLoadListener: windows-1251                                          
        
(Assignee)

Comment 41

12 years ago
And that works on other platforms?

Comment 42

12 years ago
Sure it works. This i way i'm sorting my mails under MS Windows Mozilla.
Though, bookmark dragging in SeaMonkey Bookmark manager also does nothing
besides those terminal messages.
Sure, when i try to drag something, no dragging icon or any other sign
appearing, but destination gets marked/selected for moment when i try to drop there.
But no any other effect.

So give me what is supposed to work at this stage and how to test it.

Comment 43

12 years ago
Additional notices. 
1)Draggin bokmarks works, but only half of that. It appears on new place, but
remains at old
2)I know that outside dragging shouldn't work yet, but i tried - dragged image
file from desktop to Mozilla. It resulted in downloading not image, which i
dragged, but last dragged bookmark.
Maybe it helps

Comment 44

12 years ago
Also found crash bug.
With following lines
	Window()->CurrentMessage()->FindInt32("buttons", (int32 *)&buttons);
	if(NULL == msg && !restoreMouseMask && buttons)
		return;
in nsViewBeOS::MouseMoved
application crashes at mDragMessage->FindString
in nsDragService::IsInternalDrag()
if you try to drop something from outside onto (freshly launched) app

Comment 45

12 years ago
Comment on attachment 199668 [details] [diff] [review]
DND changes in widget/src/beos

1)in
IsInternalDrag you don't test if mDragMessage != 0
2)in UpdateDragMessageIfNeeded you neither check mDragMessage !=0 neither 
assign aDragMessage to it.

This is reason of crash from previous notice.
And that was serious crahs - if user even don't try to drag anything at
Mozilla, but just moves some item over desktop for other purpose, and crosses
mozilla border occasionally - Mozilla wil crash. 
Unacceptable
Attachment #199668 - Flags: review?(sergei_d) → review+

Comment 46

12 years ago
tqh, please look also at comment
https://bugzilla.mozilla.org/show_bug.cgi?id=298171#c11
Maybe I'm missing some logic there?

Comment 47

12 years ago
btw, totally after removed statement
if(NULL == msg && !restoreMouseMask && buttons) return;
in MouseDown(), i got DnD to work at least partially - now i can select links
and drop those on tabs, so URL opens there

Comment 48

12 years ago
sorry, links DnD work also without change from previous comment.
So, at least something is working, but group
of{UpdateDragMessageIfNeeded;IsInternalDrag;MouseMoved} must be fixed anyway,
even with that preliminary stage.
Actually i dislike nested "break;s" in MouseMoved, looks like ancient BASIC
program with mad GOTO placement, but i'm planning to change all mouse-related
code anyway in future, after we sort DnD things out.

Comment 49

12 years ago
I'm very happy.
(Assignee)

Comment 50

12 years ago
*personal opinion ahead*
Ew, don't say that about breaks. They are very well defined compared to goto's
and their state is very easy to follow. When you call a break  it's very simple
to see that this is an exitpoint out of the block, you can't see that with
if/elses which means you have to keep the whole block in your head in order to
see when and how it exits. You know that the code will continue after the block
as well. 

Break and continue are nice and powerful, while keeping code very readable.
Compared to long if/else-blocks that almost always gets messy, nested and 'goto'
like. Most times you even have to rely on formatting to even be able  to read
if/else-statements. The if/else code isn't always the answer even though most
ppl seem to think so. 


(Assignee)

Comment 51

12 years ago
> 2)in UpdateDragMessageIfNeeded you neither check mDragMessage !=0 neither 
> assign aDragMessage to it.

Yes that is a part that should be 'not yet implemented'. It's only done on
native drags, and this version isn't complete in that regard.

Comment 52

12 years ago
Comment on attachment 199668 [details] [diff] [review]
DND changes in widget/src/beos

Ooops, it seems i mistakenly granted review.
Two things must be fixed before commit:
1)mentioned crash in UpdateMessafeIfNeeded - maybe by adding input parameter to
IsInternal and feeding it with aDragMessage/mDragMessage depending on context
2)Strange compilation problems - maybe it is my propblem and i need just fresh
tree and patched mozilla/widget before global configure/make? At least i wait
explanations or ideas on that
Attachment #199668 - Flags: review+ → review-

Comment 53

12 years ago
problem 2) is clear now.
mozilla/widget/src/beos/Makefile.in
needs adding to 
REQUIRES	= xpcom \ section:
 view \
 locale \
and maybe removing timer ? (as you remove timer  in other place in Makefile.in)

Attachment #199665 - Attachment mime type: text/x-source-code → text/plain
widget/public files checked in

Checking in Makefile.in;
/cvsroot/mozilla/widget/public/Makefile.in,v  <--  Makefile.in
new revision: 1.101; previous revision: 1.100
done
RCS file: /cvsroot/mozilla/widget/public/nsIDragSessionBeOS.h,v
done
Checking in nsIDragSessionBeOS.h;
/cvsroot/mozilla/widget/public/nsIDragSessionBeOS.h,v  <--  nsIDragSessionBeOS.h
initial revision: 1.1
done
(Assignee)

Comment 55

12 years ago
Created attachment 199734 [details] [diff] [review]
DND changes in widget/src/beos 2

Corrected IsInternalDrag (made it a static function that takes a BMessage) and
added missing Makefile.in changes. I must've been tired yesterday.
Attachment #199668 - Attachment is obsolete: true
(Assignee)

Updated

12 years ago
Attachment #199734 - Flags: review?(sergei_d)

Comment 56

12 years ago
Comment on attachment 199734 [details] [diff] [review]
DND changes in widget/src/beos 2

r=sergei_d
functionality is minimalistic at the moment, but let it stay in tree - it eases
development
Attachment #199734 - Flags: review?(sergei_d) → review+

Comment 57

12 years ago
https://bugzilla.mozilla.org/attachment.cgi?id=199734
landed in trunk
Checking in mozilla/widget/src/beos/Makefile.in;
/cvsroot/mozilla/widget/src/beos/Makefile.in,v  <--  Makefile.in
new revision: 1.48; previous revision: 1.47
done
Checking in mozilla/widget/src/beos/nsDragService.cpp;
/cvsroot/mozilla/widget/src/beos/nsDragService.cpp,v  <--  nsDragService.cpp
new revision: 1.14; previous revision: 1.13
done
Checking in mozilla/widget/src/beos/nsDragService.h;
/cvsroot/mozilla/widget/src/beos/nsDragService.h,v  <--  nsDragService.h
new revision: 1.5; previous revision: 1.4
done
Checking in mozilla/widget/src/beos/nsWindow.cpp;
/cvsroot/mozilla/widget/src/beos/nsWindow.cpp,v  <--  nsWindow.cpp
new revision: 1.99; previous revision: 1.98
done
Checking in mozilla/widget/src/beos/nsWindow.h;
/cvsroot/mozilla/widget/src/beos/nsWindow.h,v  <--  nsWindow.h
new revision: 1.38; previous revision: 1.37
done 
(Assignee)

Comment 58

12 years ago
Closing bug. Finally. All changes to DND will be done in new bugs.
nsDragService.cpp needs more polishing.
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED

Comment 59

12 years ago
Created attachment 201642 [details] [diff] [review]
Same patch for the 1.8 branch

This is a BeOS-only change. This is a backport of attachment 199734 [details] [diff] [review] . It will not affect the other platforms in any way.
Attachment #201642 - Flags: approval1.8rc2?

Comment 60

12 years ago
Comment on attachment 199666 [details] [diff] [review]
Makefile.in adds Interface (widget/public)

Requesting approval for the 1.8 branch. This is a BeOS only change in shared territory, but it has been in the MAIN branch for a while and hasn't caused any problems. Impact of other platforms is non-existant.
Attachment #199666 - Flags: approval1.8rc2?

Comment 61

12 years ago
Comment on attachment 199665 [details]
New dragsession interface for BeOS (widget/public)
Attachment #199665 - Flags: review?
Attachment #199665 - Flags: review+
Attachment #199665 - Flags: approval1.8rc2?

Updated

12 years ago
Attachment #199665 - Flags: approval1.8rc2? → approval1.8rc2+

Updated

12 years ago
Attachment #199666 - Flags: approval1.8rc2? → approval1.8rc2+

Updated

12 years ago
Attachment #201642 - Flags: approval1.8rc2? → approval1.8rc2+

Comment 62

12 years ago
Comment on attachment 199665 [details]
New dragsession interface for BeOS (widget/public)

Accidentaly touched the review. View bug history to confirm that sergei_d reviewed this file.
Attachment #199665 - Flags: review?

Comment 63

12 years ago
Checked in by timeless.
Keywords: fixed1.8

Comment 64

12 years ago
Tqh, I'm bit confused by content of nsIDragSessionBeOS.h

There are two methods declared:
    NS_IMETHOD SetDragMessage(BMessage *aDragMessage) = 0;
    NS_IMETHOD GetDoingDrag(PRBool *aDoingDrag) = 0;

But i don't see any sign of implementation, even let it be stubs.
(Assignee)

Comment 65

12 years ago
(In reply to comment #64)
> Tqh, I'm bit confused by content of nsIDragSessionBeOS.h
> 
> There are two methods declared:
>     NS_IMETHOD SetDragMessage(BMessage *aDragMessage) = 0;
>     NS_IMETHOD GetDoingDrag(PRBool *aDoingDrag) = 0;
> 
> But i don't see any sign of implementation, even let it be stubs.
> 

Are you sure?
http://lxr.mozilla.org/seamonkey/source/widget/public/nsIDragSessionBeOS.h

Comment 66

12 years ago
Comment 65:
There are only declaration there, in your link, as I mentioned already.
LXR cannot find any occurance besides that file.
For other platforms methods from nsIDragSessionPLATFORM.h are implemented in nsDragService files.

Comment 67

12 years ago
or do you mean

 = 0;

as stubs? (with meaning notimplemented)
(Assignee)

Comment 68

12 years ago
They arn't declared at all in the lxr I linked. It only inherits nsISupports. So either you mean another header or I'm totally confused.
You need to log in before you can comment on or make changes to this bug.