"SmoothWheel" add-on does not work with e10s

RESOLVED WONTFIX

Status

()

Firefox
Extension Compatibility
RESOLVED WONTFIX
4 years ago
2 years ago

People

(Reporter: Daniel Kosiński, Unassigned)

Tracking

({addon-compat})

Trunk
x86_64
Linux
addon-compat
Points:
---

Firefox Tracking Flags

(e10s+)

Details

(Whiteboard: [bugday-20140818])

Attachments

(1 attachment)

(Reporter)

Description

4 years ago
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

4 years ago
Related to Bug 905436

Updated

4 years ago
Blocks: 905436
Whiteboard: [bugday-20140818]

Updated

4 years ago
No longer blocks: 905436
Status: UNCONFIRMED → NEW
tracking-e10s: --- → ?
Component: Untriaged → Extension Compatibility
Ever confirmed: true
Keywords: addon-compat

Updated

4 years ago
Blocks: 905436
tracking-e10s: ? → +
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

4 years ago
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

4 years ago
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

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

Comment 6

3 years ago
This addon is listed as "Untested" on http://arewee10syet.com/, no bug is assigned to it.
(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

3 years ago
Created attachment 8578376 [details]
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

3 years ago
Current AMO version still broken. Left a message on AMO.

Updated

2 years ago
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.