Bug 14658 – REGRESSION (r24381): right-click/control-click broken

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKit Misc.

[Philippe Wittenbergh]

@rev 24399, summary says it all.
rev 24285 worked fine.

PowerPC OS X 10.4.10

[Matt Lilek]

Right click works fine for me with r24403 - can you provide some more information such as any extensions you have installed, what/where you were trying to right click on, etc?

[Philippe Wittenbergh]

(In reply to comment #1)
> Right click works fine for me with r24403 - can you provide some more
> information such as any extensions you have installed, what/where you were
> trying to right click on, etc?

Simply right click on a page (pretty simple html) to actually access the webinspector.
No extensions, haxies or other animals installed.
Both using the mouse (Kensington-mouse-in-a-box, 5buttons) and trackpad/keyboard.


e.g this page
http://nightly.webkit.org/start/?current-revision=24399

[Alexey Proskuryakov]

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

[David Kilzer (:ddkilzer)]

What type of mouse are you using?  This works for me using Ctrl-Click on a PowerBook G4 with the built-in trackpad with a local debug build of WebKit r24439 on Safari 3.0 (522.12) with Mac OS X 10.4.10 (8R218).

[Philippe Wittenbergh]

(In reply to comment #4)
> What type of mouse are you using? 

See comment 2, where I mention a Kensington mouse and the trackpad (PowerBook g4 1.5Ghz).
Just tested with an Apple Mouse (the mighty one). Idem dito. No reaction on right click.
And no reaction on control click either.

All other buttons of my Kensington mouse work as programmed with/for WebKit.
And - of course- the rodent works correctly everywhere else (OS, Camino, terminal.app, etc, etc, etc).

Comment 1 mentions build r24403 and you (comment 4) mention build r24439, which are more recent than what I have (r24399, latest from nightly.webkit.org). The issue might be fixed in those, I have no way to know. I can't build Webkit myself at the moment.

[Philippe Wittenbergh]

Ok, nightly.webkit.org now has r24441 available.
But it still has the same issue.

[Geoffrey F. Green]

Existence of bug confirmed on r24441, Intel-based Mac, 10.4.10. (Using a Microsoft Optical Blue USB mouse); Right/Control-clicking continues to work elsewhere.

[Robert McGovern]

Can confirm this bug on Intel MacBook using two finger right clicking. Basically right clicking on anything, including link, hasn't worked for the last two nightly builds I have tried: 18th July & 19th July

[Alexey Proskuryakov]

Marking as a confirmed regression per the above comments.

[David Kilzer (:ddkilzer)]

I see this with WebKit Nightly r24441 as well on Mac OS X 10.4.10 (8R218) with Safari 3.0 (522.12).

Is this only an issue with release builds, or just with WebKit Nightly packaging?

[David Kilzer (:ddkilzer)]

<rdar://problem/5346830>

[Mark Rowe (bdash)]

I don't see how this could be specific to the nightly builds.

[Matt Lilek]

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

[Matt Lilek]

From Bug 14703:

Actual Results:
No contextual menu appears.  An error is logged, e.g., *** -[NSCFArray
objectAtIndex:]: index (1) beyond bounds (1).

[Matt Lilek]

Doing a bit of snooping - this definitely regressed with r24381.  100% reproducible for me with Safari 2 and Coda, it's barfing on WebContentMenuClient.mm:106 (at least according to the NSException backtrace Coda printed to the console for me).

[Matt Lilek]

(In reply to comment #15)
> WebContentMenuClient.mm:106
> 

s/Content/Context

[Adam Roben (:aroben)]

I think I know what's wrong. Patch coming.

[Jakob Peterhänsel]

Just a confirmation, it suddently started to happen here as well.
Was working fine yesterday in nightly 24399, I then upgraded to latest, and then paired a Apple Mighty Mouse BT.. = no Contextual Menu.

Works fine in normal Safari 2.0.4, but not in nightly.

[Adam Roben (:aroben)]

Created attachment 15615 [details]
Patch with ChangeLog

Comment on attachment 15615 [details]
Patch with ChangeLog

r=me

[Adam Roben (:aroben)]

Landed in r24510