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)
Tracking
(Performance Impact:?, firefox75 fixed)
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
Reporter | ||
Updated•5 years ago
|
Blocks: geckoview-startup
Assignee | ||
Updated•5 years ago
|
Assignee: nobody → snorp
Comment 1•5 years ago
|
||
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.
Assignee | ||
Comment 2•5 years ago
|
||
Pushed by jwillcox@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2b1a361482a5
Don't include GMP in GeckoView r=geckoview-reviewers,agi
Comment 4•5 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox75:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla75
Updated•3 years ago
|
Performance Impact: --- → ?
Whiteboard: [qf]
Updated•3 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•