use extractLibs code path in child if we use it in the parent

RESOLVED INCOMPLETE

Status

()

Core
Widget: Android
RESOLVED INCOMPLETE
8 years ago
2 years ago

People

(Reporter: blassey, Unassigned)

Tracking

Trunk
All
Android
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Created attachment 503044 [details] [diff] [review]
patch

this is a follow up from bug
Attachment #503044 - Flags: review?(tglek)

Comment 1

8 years ago
I don't like adding extra logic. Why not have the loader check for an extracted file before loading files from jar/cache/etc.

ie if(!open(/path/to/cached/lib)) load_it_usual_way() . I can post a patch tomorrow

Comment 2

8 years ago
Comment on attachment 503044 [details] [diff] [review]
patch

Nm previous comment, shouldn't review patches when tired. This looks good.
Attachment #503044 - Flags: review?(tglek) → review+

Comment 3

8 years ago
Comment on attachment 503044 [details] [diff] [review]
patch

I thought this patch got tested. I just tried it and it still exhibits slow child startup.
Attachment #503044 - Flags: review+ → review-
OS: Linux → Android
Hardware: x86_64 → All
This looks like it's from the XUL Fennec days and isn't relevant any more.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → INCOMPLETE
You need to log in before you can comment on or make changes to this bug.