If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Avoid accessing AndroidBridge in child process

RESOLVED FIXED

Status

()

Core
Widget: Android
RESOLVED FIXED
7 years ago
7 years ago

People

(Reporter: mwu, Assigned: mwu)

Tracking

Trunk
All
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

7 years ago
Created attachment 455552 [details] [diff] [review]
Check for AndroidBridge

The child process doesn't have access to AndroidBridge because there's no JVM.
Attachment #455552 - Flags: review?(blassey.bugs)
(Assignee)

Updated

7 years ago
Attachment #455552 - Flags: review?(blassey.bugs) → review?(vladimir)
Comment on attachment 455552 [details] [diff] [review]
Check for AndroidBridge

Seems like a bandaid -- does the child actually need this info?  If so may want to file a followup bug to proxy the helper app service stuff..
Attachment #455552 - Flags: review?(vladimir) → review+
I'd really prefer if we assert if these methods are called from the child process and we're not proxying them.
(Assignee)

Comment 3

7 years ago
Filed bug 579388 for doing the helper app stuff properly.

http://hg.mozilla.org/mozilla-central/rev/525cbac0423c
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.