Closed Bug 1620415 Opened 5 years ago Closed 5 years ago

Remove GMPProvider.jsm/GMPInstallManager.jsm from Fenix (and all other Geckoview uses)

Categories

(Core Graveyard :: Plug-ins, enhancement)

Unspecified
Android
enhancement
Not set
normal

Tracking

(Performance Impact:?, firefox75 fixed)

RESOLVED FIXED
mozilla75
Performance Impact ?
Tracking Status
firefox75 --- fixed

People

(Reporter: jesup, Assigned: snorp)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

GMPProvider and GMPInstallManager are useless now on Android, since we can't use GMP if we're in the Play Store. GMPProvider takes considerable time at startup to initialize

Assignee: nobody → snorp

Profile: https://perfht.ml/3aq7Ds6

Note: The GMPProvider.jsm import takes 42ms in this profile, but a lot of that is because it's the first module that requires OS.File. Once we no longer use GMPProvider, the OS.File cost is probably going to hit a different import rather than go away.

Pushed by jwillcox@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2b1a361482a5 Don't include GMP in GeckoView r=geckoview-reviewers,agi
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla75
Performance Impact: --- → ?
Whiteboard: [qf]
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: