Closed
Bug 1528665
Opened 7 years ago
Closed 7 years ago
Remove GetAndroidSystemInfo sync IPC
Categories
(GeckoView :: General, enhancement)
Tracking
(firefox67 fixed)
RESOLVED
FIXED
mozilla67
| Tracking | Status | |
|---|---|---|
| firefox67 | --- | fixed |
People
(Reporter: m_kato, Assigned: m_kato)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
Content process in GV can access Java, but nsSystemInfo uses sync IPC to get model information via Java.
|
Assignee | |
Comment 1•7 years ago
|
||
nsSystemInfo is initialzied at first page load. Actually, content process uses
sync IPC to get Android OS information. But now, we can use Java code even if
on content process, so we should use JNI directly instead of sync IPC.
Also, nsSystemInfo still has unused extern android_sdk_version that is for
HoneyComp's DNS hack. So let's remote it.
Pushed by m_kato@ga2.so-net.ne.jp:
https://hg.mozilla.org/integration/autoland/rev/9327a3eb3732
Remove GetAndroidSystemInfo sync IPC. r=geckoview-reviewers,mccr8,snorp
|
||
Comment 3•7 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox67:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
You need to log in
before you can comment on or make changes to this bug.
Description
•