106692 - Rewrite mouse events to use CarbonEvents

Status: RESOLVED INVALID

(Core :: XUL, defect)

Description

[Mike Pinkerton (not reading bugmail)]

subject says it all

Comment 1

[Mike Pinkerton (not reading bugmail)]

-> 098

Comment 2

[Mike Pinkerton (not reading bugmail)]

not gonna happen in 098

Comment 3

[Mike Pinkerton (not reading bugmail)]

over to dagley, i probably won't have time for these by 099 with the embedding
work i'm getting pulled into.

Comment 4

[Steve Dagley]

-> mozilla1.0

Comment 5

[Steve Dagley]

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

Comment 6

[Steve Dagley]

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

Comment 7

[saari (gone)]

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

Comment 8

[saari (gone)]

marking topembed as it may be needed for embedding. needs investigation

Comment 9

[saari (gone)]

still needs investigation, but pushing to 1.1

Comment 10

[Greg K.]

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

Comment 11

[Steve Dagley]

Since I don't report into Internet Technologies anymore this bug needs a new
owner -> saari

Comment 12

[saari (gone)]

Can anyone remember why I said this might be needed for embedding? :-) I suspect
it wasn't?

Comment 13

[Frankie]

Saari, maybe you misinterpreted Pink's comment #3 where he said he was too busy
working on embedding, so he handed all the Carbon Event bugs off to sdagley.

Comment 14

[saari (gone)]

I was actually referring to my own comment, number 8

Comment 15

[Conrad Carlen (not reading bugmail)]

It's not really that mouse events in particular may need to be Carbon events for
the sake of embedding. But, all events in general that our event sinks take need
to be Carbon events. That's because the embedding app will likely not have a
classic WNE event pump and be using EventRecords to pass to our sink.

Or, in less words, this blocks bug 106689.

Comment 16

[saari (gone)]

gotcha

->bryner

Comment 17

[TGOS]

Isn't this bug obsolete, looking at bug 111230, which tries to replace Carbon
with Cocoa for widgets? If the widgets are Cocoa, why not using Cocoa event
handling?

Comment 18

[Josh Aas]

This bug should not be marked obsolete until Cocoa widgets are actually used in FF.

Comment 19

[Mark Mentovai]

We're part of the way there.  Mouse events are now handled as Carbon events outside of WNE, but are still converted to EventRecords and processed through the old dispatcher.  Reference bug 332579 (1.8 branch), bug 326273 (trunk), bug 338153 (consolidating 1.8 branch and trunk implementations).

Comment 20

[Ernst Sjostrand]

Close?

Comment 21

[Frankie]

RIP Carbon