Closed Bug 1097532 Opened 10 years ago Closed 9 years ago

[e10s] Right click context menu stops working with "All-in-One Gestures" if e10s is enabled

Categories

(Core :: DOM: Events, defect)

defect
Not set
normal

Tracking

()

RESOLVED WONTFIX
Tracking Status
e10s + ---

People

(Reporter: alice0775, Unassigned)

References

Details

(Keywords: regression)

Steps To reproduce:
1. Install "All-in-One Gestures 0.26" https://addons.mozilla.org/en-US/firefox/addon/all-in-one-gestures/?src=search
2. Open e10s window
3. Right click on contents area

Actual Results:
No context menu pop up

Expected Results:
Context menu should pop up

Regression window(m-i)
Good:
https://hg.mozilla.org/integration/mozilla-inbound/rev/24a710c83e90
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:35.0) Gecko/20100101 Firefox/35.0 ID:20140918085422
Bad:
https://hg.mozilla.org/integration/mozilla-inbound/rev/df6055033377
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:35.0) Gecko/20100101 Firefox/35.0 ID:20140918085721
Pushlog:
https://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=24a710c83e90&tochange=df6055033377

Suspect: Bug 1059207
Alice: did the "All-in-One Gestures" add-on work correctly with e10s before this regression?
Flags: needinfo?(wmccloskey)
Summary: Right click context menu stops working with "All-in-One Gestures" if e10s is enabled → [e10s] Right click context menu stops working with "All-in-One Gestures" if e10s is enabled
Bill: if this add-on was broken by your recent event handling changes, should we treat this regression as a higher priority than typical add-on compat bugs?
(In reply to Chris Peterson (:cpeterson) from comment #1)
> Alice: did the "All-in-One Gestures" add-on work correctly with e10s before
> this regression?

Mouse trail of the addon are not working. but this is a separate problem.
I think this should be the same priority as other add-on compat bugs.
Flags: needinfo?(wmccloskey)
The developer has been contacted through AMO.
Let me post some update regarding All-in-One-Gestures. The developer has not been responding since almost 2 years as can be seen in the comments of his extension. I also tried contacting him last year because I was planning to pick up his code and adapt it for SeaMonkey. Now my extension works in SeaMonkey and Firefox and is a continuation of All-in-One-Gestures: Mouse Gestures Suite at https://addons.mozilla.org/addon/mouse-gestures-suite/

I'm very familiar with the code of this extension and I can say that adapting it to work with e10s means doing a *major* rewrite of core mechanisms and other stuff as well. This is quite a complex extension and communication with content is happening all over the code, which means moving all relevant parts to frame scripts and the current shims are far from adequate in this case. I've started work on this already and e10s beta versions of Mouse Gestures Suite are available in the development channel.

But given the fact that All-in-One-Gestures has been abandoned for almost 2 years and that some of the code dates back to early 2000's means it's very unlikely it will ever be made e10s compatible. Even if the developer reappeared and was willing to do this he would need to become very active in the development to be able to adapt it.
Thank you for working on this!
(In reply to lemon_juice from comment #6)
> Let me post some update regarding All-in-One-Gestures. The developer has not
> been responding since almost 2 years as can be seen in the comments of his
> extension. I also tried contacting him last year because I was planning to
> pick up his code and adapt it for SeaMonkey. Now my extension works in
> SeaMonkey and Firefox and is a continuation of All-in-One-Gestures: Mouse
> Gestures Suite at https://addons.mozilla.org/addon/mouse-gestures-suite/
> 
> I'm very familiar with the code of this extension and I can say that
> adapting it to work with e10s means doing a *major* rewrite of core
> mechanisms and other stuff as well. This is quite a complex extension and
> communication with content is happening all over the code, which means
> moving all relevant parts to frame scripts and the current shims are far
> from adequate in this case. I've started work on this already and e10s beta
> versions of Mouse Gestures Suite are available in the development channel.
> 
> But given the fact that All-in-One-Gestures has been abandoned for almost 2
> years and that some of the code dates back to early 2000's means it's very
> unlikely it will ever be made e10s compatible. Even if the developer
> reappeared and was willing to do this he would need to become very active in
> the development to be able to adapt it.

Sorry to say this but your extension has nothing to do with ALl-in one sidebar : yours want to enable some shortcuts when doing some mouse gestures, but all-in one sidebar allow me to have a toolbar on the side of my screen which allow me to keep checking things like email, ads blocked, cookie destruction, tabs closed, bookmarks, ....
Yours could never replace it like this.
The extension the closest to all-in one sidebar I could find is Omni sidebar (https://addons.mozilla.org/fr/firefox/addon/omnisidebar/?src=search) and good news : it's e10s compatible.
But still, I would like to keep using all-in one sidebar.
Please read the title of this bug report again - it is about "All-in-One Gestures" not "All-in-One Sidebar", they are completely different extensions. Comments about All-in-One sidebar don't belong here.
Holy s***.
I'm really sorry, I made a wrong turn on : http://arewee10syet.com/ and selectec one line too far ...
Is there a way to delete my previous comment ?
Once again, really sorry.
No problem, the names are very similar, it happens:). I don't know if it's possible to delete a comment, maybe an admin can do it.
This is still not working.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.