1271462 - [e10s] Easy Screenshot add-on not functional when e10s is enabled

Status: RESOLVED FIXED

(Firefox :: Extension Compatibility, defect)

Description

[u554753]

The Easy Screenshot add-on is not functional when e10s is enabled. The add-on works as intended when e10s is disabled, and it does not matter whether Shim is enabled or not.

This is affecting the following user agents:
(Windows NT 6.1; rv:49.0) Gecko/20100101 Firefox/49.0
(Windows NT 10.0; WOW64; rv:49.0) Gecko/20100101 Firefox/49.0
(X11; Linux x86_64; rv:49.0) Gecko/20100101 Firefox/49.0

Steps to Reproduce:
Pre-Requisite: Start Firefox with a clean profile
1. Install the Easy Screenshot add-on https://addons.mozilla.org/addon/easyscreenshot
2. Restart the Browser
3. Access http://www.nytimes.com/
4. Click the add-on icon 
5. Select an option from the drop-down window

Expected Results:
The add-on should execute the functions as designed when the user selects from the drop down window.

Actual Results:
When e10s is enabled, there are two different errors. When shim is enabled, the functions from the drop-down window do not work as intended. You can drag the marker across the screen, but no option to take a screenshor appears. When shim is disabled, the addon button does not function entirely.

Comment 1

[:shell escalante]

Is there any chance you'd consider migrating this to webextensions?  We'd definitely check if something is missing....

Jorge is looking up for the person he spoke with in London from MozOnline

Comment 2

[Jorge Villalobos [:jorgev] (he/him)]

I think it was Hector I talked to. Hector, can you share your plans for e10s compatibility?

Comment 3

[Hector Zhao [:hectorz]]

(In reply to Jorge Villalobos [:jorgev] from comment #2)
> I think it was Hector I talked to. Hector, can you share your plans for e10s
> compatibility?

The e10s compatibilty works of our extensions started right after London. I'm currently working on bug fixes of {cehomepage,cpmanager}@mozillaonline.com, easyscreenshot will be the next (later this week or next week, I guess).

(In reply to :shell escalante from comment #1)
> Is there any chance you'd consider migrating this to webextensions?  We'd
> definitely check if something is missing....
> 

My original plan was just updating the current XUL overlay extension. Now that you've mentioned it, easyscreenshot is actually one of our extensions which doesn't involve too much XPCOM magics, it's a good chance for me to have a look at the WebExtensions APIs.

Comment 4

[Safwan Rahman (:safwan)]

Any update on this? as E10S is in the release channel, I am badly missing the add-on

Comment 5

[Hector Zhao [:hectorz]]

A new version was uploaded to and approved on AMO. It's a hybrid extension for the migration of editor prefs, and should be e10s compatible. I forgot to add mpc=true in its install.rdf, I'll do so with the next update, which depends on users' feedback on this version.

Comment 6

[Hector Zhao [:hectorz]]

A new version was uploaded and approved with mpc=true set, resolving this bug as fixed. Thanks for all your patience.