1254312 - [e10s] Fix browser_plugin_enabled_state_locked.js to run in e10s

Status: RESOLVED FIXED

(Toolkit :: Add-ons Manager, defect)

Description

[Jared Wein [:jaws] (please needinfo? me)]

Just need to replace Promise.defer() with `new Promise`...

Comment 1

[Jared Wein [:jaws] (please needinfo? me)]

https://treeherder.mozilla.org/#/jobs?repo=try&revision=fa84691cf082

Comment 2

[Jared Wein [:jaws] (please needinfo? me)]

Attachment: MozReview Request: Bug 1254312 - [e10s] Fix browser_plugin_enabled_state_locked.js to run in e10s. r?mconley

Review commit: https://reviewboard.mozilla.org/r/38545/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/38545/

Comment 3

[Mike Conley (:mconley) (:⚙️)]

Comment on attachment 8727616 [details]
MozReview Request: Bug 1254312 - [e10s] Fix browser_plugin_enabled_state_locked.js to run in e10s. r?mconley

https://reviewboard.mozilla.org/r/38545/#review35189

Go go go!

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/e89ea2cf1c20

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/e89ea2cf1c20

Comment 6

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8727616 [details]
MozReview Request: Bug 1254312 - [e10s] Fix browser_plugin_enabled_state_locked.js to run in e10s. r?mconley

Approval Request Comment
[Feature/regressing bug #]: e10s test
[User impact if declined]: none, will add expanded e10s test coverage
[Describe test coverage new/current, TreeHerder]: it is an automated test
[Risks and why]: none
[String/UUID change made/needed]: none

Comment 7

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Comment on attachment 8727616 [details]
MozReview Request: Bug 1254312 - [e10s] Fix browser_plugin_enabled_state_locked.js to run in e10s. r?mconley

test-only changes are auto-approved.

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/c63a2db7aafa