reduce locationbar overhead

RESOLVED WORKSFORME

Status

Firefox for Android Graveyard
General
RESOLVED WORKSFORME
10 years ago
9 years ago

People

(Reporter: (dormant account), Unassigned)

Tracking

(Blocks: 1 bug, {perf})

Dependency tree / graph

Details

(Reporter)

Description

10 years ago
on startup 
this._editToolbar(false); is toggling location bar for no good reason. It should alread by in the view state.

Also should use the text field in readonly mode, instead of swapping elements.
I filed bug 468759 for the second part of that.
(Reporter)

Updated

10 years ago
Blocks: 459117
(In reply to comment #0)
> on startup 
> this._editToolbar(false); is toggling location bar for no good reason. It
> should alread by in the view state.

I'm pretty sure this happens because "about:blank" is loaded first and we go into edit mode when we see "about:blank" being loaded.

We should be able to fix this with bug 468880
Depends on: 468880
Taras - since bug 456880 landed and we removed the extra "about:blank", has the _editToolbar toggling stopped too?
(Reporter)

Comment 4

9 years ago
(In reply to comment #3)
> Taras - since bug 456880 landed and we removed the extra "about:blank", has the
> _editToolbar toggling stopped too?

We call _editToolbar 0 times on startup. As I said in the meeting, fennec js part of startup is starting to look pretty good :)
(Reporter)

Updated

9 years ago
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.