1054737 - "SmoothWheel" add-on does not work with e10s

Status: RESOLVED WONTFIX

(Firefox :: Extension Compatibility, defect)

Description

[Daniel Kosiński]

User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:34.0) Gecko/20100101 Firefox/34.0 (Beta/Release)
Build ID: 20140804060850

Steps to reproduce:

Installed "SmoothWheel" https://addons.mozilla.org/en-US/firefox/addon/smoothwheel/
Opened an e10s tab in Nightly


Actual results:

Scrolling behaviour is the default one from firefox


Expected results:

Scrolling should be smooth (doh)

Comment 1

[Mikel]

Related to Bug 905436

Comment 2

[Chris Peterson [:cpeterson]]

Avi, if you have any questions about add-on support for multiprocess Firefox (e10s), just drop by the #e10s IRC channel on irc.mozilla.org. MDN also has a good introduction:

https://developer.mozilla.org/en-US/Add-ons/Working_with_multiprocess_Firefox

Comment 3

[Avi Halachmi (:avih)]

Yeah, I got a newer unreleased WIP version of SmoothWheel (restarless among others), thought admittedly I didn't test it yet with e10s. I should give it a go...

Thanks for the heads up.

Comment 4

[Avi Halachmi (:avih)]

So I made it mostly work with e10s.

Very ugly and somewhat dodgy code right now and with debug prints, but except for not unloading/loading on existing e10s tabs when disabling/enabling the addon (works fine for existing normal tabs and new e10s tabs/windows/restart), and except for not updating the stats (can't write prefs from frame scripts - should be fixable easily), it works, and even works well. By well I mean that I _think_ it's actually smoother with e10s, but it could totally be my imagination.

That being said, this is on top of an already WIP version mentioned on comment 3, which I've been touching occasionally but not very frequently over the past 3 months or so.

To release this e10s compatible version will require cleanups of both the WIP code and the e10s code, make sure it also works on Thunderbird/Seamonkey (the WIP version is already broken on those regardless of e10s - due to moving to bootstrapped and attaching to windows differently), and general testing and refinements.

So I wouldn't hold my breath for the next version, but technically it's possible that it'll get released at some stage .. ;)

Comment 5

[Avi Halachmi (:avih)]

Also, forgot to mention many thanks to the guys on #e10s on IRC, especially The_8472 and smaug. Cheers!

Comment 6

[tempmailkis]

This addon is listed as "Untested" on http://arewee10syet.com/, no bug is assigned to it.

Comment 7

[Chris Peterson [:cpeterson]]

(In reply to tempmailkis from comment #6)
> This addon is listed as "Untested" on http://arewee10syet.com/, no bug is
> assigned to it.

Thanks. I added a link from arewee10syet.com to this bug.

Comment 8

[Avi Halachmi (:avih)]

Attachment: smoothwheel-wip~2014-09-12-repackaged-2015-03-17.xpi

That's a version I hacked about 6 months ago. I should clean it up and tie loose ends and then post it officially. For now, remember that this is an unofficial version which is mostly a proof of concept.

Please don't distribute it, but you can report issues with e10s here. Thanks.

Changes from 0.45.8:
- Fixed text boxes scroll.
- Uses the W3C wheel events and should be more responsive on "heavy" pages (legacy events mode available at the advanced config panel).
- Can automatically disable itself when using a touchpad (uses heuristics).
- Hopefully reasonably e10s compatible.
- No statistics in e10s.
- Has some debug dumps and prints.
- Probably few others which I forgot.

Comment 9

[Jerod Lycett]

Current AMO version still broken. Left a message on AMO.