1211642 - Whitelist test plugin for async init

Status: RESOLVED FIXED

(Core Graveyard :: Plug-ins, defect)

Description

[(No longer employed by Mozilla) Aaron Klotz]

We need to whitelist the test plugin to ensure that async init will be tested adequately.

Comment 1

[(No longer employed by Mozilla) Aaron Klotz]

Attachment: Patch

Comment 2

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/f02deaaeb828

Comment 3

[Carsten Book [:Tomcat]]

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

Comment 4

[(No longer employed by Mozilla) Aaron Klotz]

Comment on attachment 8673315 [details] [diff] [review]
Patch

Approval Request Comment
[Feature/regressing bug #]: async plugin init
[User impact if declined]: async init will be disabled during testing
[Describe test coverage new/current, TreeHerder]: plugin test suite
[Risks and why]: Low, this really only affects testing but does affect xul.dll so I'm requesting approval.
[String/UUID change made/needed]: None

Comment 5

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 8673315 [details] [diff] [review]
Patch

Taking anything to help testing async plugin init.
Should be in 42 beta 8

Comment 6

[Carsten Book [:Tomcat]]

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

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-beta/rev/f585fae6c50a

Comment 8

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Various plugin tests have been failing since this landed on beta:
https://treeherder.mozilla.org/logviewer.html#?job_id=588233&repo=mozilla-beta
https://treeherder.mozilla.org/logviewer.html#?job_id=588229&repo=mozilla-beta
https://treeherder.mozilla.org/logviewer.html#?job_id=588224&repo=mozilla-beta


Backing out to fix them up:
https://hg.mozilla.org/releases/mozilla-beta/rev/bff8b3d98f72

Comment 9

[Sylvestre Ledru [:Sylvestre]]

Too late for 42 (even more as we disabled async plugin init)

Comment 10

[(No longer employed by Mozilla) Aaron Klotz]

(In reply to Wes Kocher (:KWierso) from comment #8)
> Various plugin tests have been failing since this landed on beta:
> https://treeherder.mozilla.org/logviewer.html#?job_id=588233&repo=mozilla-
> beta
> https://treeherder.mozilla.org/logviewer.html#?job_id=588229&repo=mozilla-
> beta
> https://treeherder.mozilla.org/logviewer.html#?job_id=588224&repo=mozilla-
> beta
> 
> 
> Backing out to fix them up:
> https://hg.mozilla.org/releases/mozilla-beta/rev/bff8b3d98f72

That's due to bug 1218473 et al. Will be sorted out with that.