180592 - Scroll wheel does not work in PDF Browser Plugin

Status: RESOLVED FIXED

(Camino Graveyard :: Plug-ins, defect, P3)

Description

[Manfred Schubert]

User-Agent:       Mozilla/5.0 (Macintosh; U; PPC Mac OS X; en-US; rv:1.0.1) Gecko/20021104 Chimera/0.6
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X; en-US; rv:1.0.1) Gecko/20021104 Chimera/0.6

The scroll wheel of the mouse does not work in the PDF Browser Plugin
<http://www.schubert-it.com/plugin/>. It works in Internet Explorer and Mozilla.

Reproducible: Always

Steps to Reproduce:
1. Download and install the PDF Browser Plugin
2. Open a PDF
3. Use the scroll wheel

Actual Results:  
PDF does not scroll

Expected Results:  
PDF should scroll

This might well be a bug in the PDF Browser Plugin instead, since I could not
find any documentation on how to implement scroll wheel support for plugins in
Chimera. In that case it would be great if you could point me to the right
documentation.

Comment 1

[Greg K.]

Confirmed using Chimera/2002111204 on 10.1.5.

Comment 2

[Simon Fraser [no longer active]]

You'll have to register carbon event handlers to get scroll wheel events. There
is no way we can send you these via Event Records.

Comment 3

[Manfred Schubert]

I _do_ register a Carbon Event handler for the screll wheel. That's why it works
in Mozilla, Netscape, Internet Explorer etc.
The Carbon Event handler doesn't work in a Cocoa browser though.
You are already sending "fake" events via the EventRecord (key focus). Can't you
just send mouse scroll events too? I will then update the plugin to handle these.

Comment 4

[Peter Lubczynski]

bnesse gone, -->peterl

Comment 5

[Peter Lubczynski]

bnesse gone, -->peterl

Comment 6

[Simon Fraser [no longer active]]

Manfred: do you regiser your carbon event handler on the window?

Because we're a Cocoa app, no Carbon events get sent by default. We'd have to
synthesise events for you to allow your plugin to get them.

I'll take this.

Comment 7

[Manfred Schubert]

Yes, I install the scroll wheel handler on the window. I wouldn't have thought
it would make a difference whether I install it on the window or the
application, and the window seemed more logical to me.
Works fine in all Carbon browsers, but as far as I know Carbon Events just don't
work in a Cocoa application. So the plugin needs some other way to be informed
abou the Cocoa scroll events, which I think could be "fake" scroll events passed
via the EventRecord.

Comment 8

[Simon Fraser [no longer active]]

> which I think could be "fake" scroll events passed via the EventRecord.

But there are no standard events for this; we'd have to make up some new ones,
and add them to npapi.h. I think us faking up carbon events would be easier.

Comment 9

[Manfred Schubert]

Yes, if you can make Carbon Events work, even better.
It was just that I thought it's not possible for a Cocoa app, that's why I
suggested passing it through the classic event record. I have no preference
either way.

Comment 10

[Norbert Rittel]

I'd love to see this fixed as well. The Plug-in is a great extension of Chimera.
If you can make Carbon Events work that'll be great!

Comment 11

[Jonathan Brecher]

According to
http://developer.apple.com/techpubs/macosx/Essentials/CarbonCocoaDoc/cci_chap2/Carbon_and__mmunication.html,
 Cocoa apps *will* transmit Carbon Events under normal circumstances, starting
in Jaguar.  Or at least that's how I read that page.  I wonder if something in
Chimera's event handling is disabling that system functionality somehow.

Manfred's original issue here is more-or-less in the "polish" category -- his
plugin can function quite well without scroll wheel support.  We're in a worse
position. We create our own floating windows and dialogs, and we can't talk to
them at all without Carbon Events, even though we created them from our Carbon
plugin in the first place.  The plugin works with Mozilla, but after using
Chimera for a bit, I kinda like it, and I'd really like to support it if possible...

Comment 12

[Simon Fraser [no longer active]]

Jonathan: please point me to a a place where I can download your plugin, and see
see sample files.

Note that we have Carbon Event issues in other plugins too (see bug 163468 for
an example).

Comment 13

[Jonathan Brecher]

I've emailed a development version of our plugin to Simon.  Feel free to share it 
within the Netscape/Mozilla organization, but I'd appreciate if you didn't 
forward it outside without talking to me first.

Notwithstanding that url I cited in comment #11, I note that Apple's newly-
released Safari behaves exactly the same as Chimera in terms of (non)handling of 
Carbon Events.  I wouldn't be surprised if this issue had to be escalated within 
Apple.  I've started that process through my Apple contact, but additional 
pestering probably wouldn't hurt...

Comment 14

[Manfred Schubert]

Anything happening here?

Comment 15

[Derek Schrock]

Is bug 210157 a dup of this bug?

Comment 16

[Manfred Schubert]

I need Carbon Events in the Camino browser window.

Comment 17

[Simon Fraser [no longer active]]

Attachment: Patch to fire off scrollwheel carbon events

Comment 18

[Mike Pinkerton (not reading bugmail)]

won't this send the scroll event twice?

Comment 19

[Simon Fraser [no longer active]]

(In reply to comment #18)
> won't this send the scroll event twice?

Nah. The scroll events will only get picked up by someone who explicitly
registers a carbon event handler for them, and they'll never come back to Cocoa.

Comment 20

[Mike Pinkerton (not reading bugmail)]

Comment on attachment 190367 [details] [diff] [review]
Patch to fire off scrollwheel carbon events

r=pink

Comment 21

[Simon Fraser [no longer active]]

Fixed.

Comment 22

[L. H.]

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