Closed
Bug 1047557
Opened 10 years ago
Closed 7 years ago
Warm up Gecko while search activity is running
Categories
(Firefox for Android Graveyard :: Search Activity, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: Margaret, Unassigned)
References
Details
I'm not sure how to go about doing this, but it would be pretty sweet if gecko was ready to go by the time a user launched a URL from the search activity.
The search activity is built as part of the Fennec app, so we should have access to anything we need to call.
blassey or rnewman, got any ideas about how we could do this?
Flags: needinfo?(rnewman)
Flags: needinfo?(blassey.bugs)
Comment 1•10 years ago
|
||
It might be enough for your purposes to simply do this:
ThreadUtils.setUiThread(Thread.currentThread(), new Handler());
if (GeckoThread.checkAndSetLaunchState(GeckoThread.LaunchState.Launching, GeckoThread.LaunchState.Launched)) {
GeckoThread.createAndStart();
}
but be prepared for various things to break, either immediately (calls back from Gecko to get surfaces etc.) or when Fennec launches (profile not created? various listeners not inited?).
Fixing those will essentially involve reimplementing GeckoView#init, but there are landmines there: e.g., GeckoAppShell#setContextGetter will hold a reference to your activity if you let it.
snorp will have more to say about all this, but no harm in getting started.
Flags: needinfo?(rnewman)
Summary: Warm up gecko while search activity is running → Warm up Gecko while search activity is running
Comment 2•10 years ago
|
||
I think you will want to just wait for bug 834033 to land, or at least the headless part of it. Maybe snorp can split that out as a separate landable bug
Flags: needinfo?(blassey.bugs) → needinfo?(snorp)
Updated•10 years ago
|
status-firefox34:
--- → affected
Updated•10 years ago
|
status-firefox34:
affected → ---
Comment 3•10 years ago
|
||
(In reply to Brad Lassey [:blassey] (use needinfo?) from comment #2)
> I think you will want to just wait for bug 834033 to land, or at least the
> headless part of it. Maybe snorp can split that out as a separate landable
> bug
Yeah, that would be the way forward here. I can try to land that stuff.
Flags: needinfo?(snorp)
Comment 4•7 years ago
|
||
This is an ex-Search Activity now (bug 1221344).
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Updated•7 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•