Closed
Bug 1620415
Opened 6 years ago
Closed 6 years ago
Remove GMPProvider.jsm/GMPInstallManager.jsm from Fenix (and all other Geckoview uses)
Remove GMPProvider.jsm/GMPInstallManager.jsm from Fenix (and all other Geckoview uses)
---
---
---
---
---
QA Whiteboard:
---
Has STR:
---
Change Request:
---
0
Bug Flags:
| Reporter | |
Description•6 years ago
|
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•6 years ago
|
Blocks: geckoview-startup
| Assignee | |
Updated•6 years ago
|
Assignee: nobody → snorp
Comment 1•6 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•6 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•6 years ago
|
|
| bugherder | |
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox75:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla75
Updated•4 years ago
|
Performance Impact: --- → ?
Whiteboard: [qf]
Updated•4 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.