1060088 - Don't ship PluginProvider.jsm and OpenH264Provider.jsm with Fennec

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[(inactive) Jim Chen [:jchen] [:darchons]]

They take up space and take up around 300ms in my startup profile.

Comment 1

[Richard Newman [:rnewman]]

Jim, are you planning to work on this? If not, can you mentor it?

Comment 2

[(inactive) Jim Chen [:jchen] [:darchons]]

I'll take it since I already have a patch for it.

Comment 3

[(inactive) Jim Chen [:jchen] [:darchons]]

Attachment: Don't ship unused addon providers in Fennec (v1)

We're trying to decrease Fennec's download size and startup time, so we want to not ship providers that are not used in Fennec.

Comment 4

[Blair McBride [:Unfocused] (UNAVAILABLE)]

Comment on attachment 8481504 [details] [diff] [review]
Don't ship unused addon providers in Fennec (v1)

Review of attachment 8481504 [details] [diff] [review]:
-----------------------------------------------------------------

You'll need to disable some tests too, and change some annotations for already disabled tests. Should be just xpcshell tests though.

Comment 5

[Blair McBride [:Unfocused] (UNAVAILABLE)]

Oh, and probably want to not ship GMPInstallManager.jsm either.

Comment 6

[(inactive) Jim Chen [:jchen] [:darchons]]

Attachment: Don't ship unused addon providers in Fennec (v2)

This patch excludes GMPInstallManager.jsm as well. I disabled the relevant tests that I saw, and try run was green, https://tbpl.mozilla.org/?tree=Try&rev=6a0c27209be8

Comment 7

[(inactive) Jim Chen [:jchen] [:darchons]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/70ddc2764516

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/70ddc2764516