Status

RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: taras.mozilla, Assigned: taras.mozilla)

Tracking

(Blocks: 1 bug)

Trunk
x86
Linux

Details

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
Created attachment 363973 [details] [diff] [review]
nuke 1 line, saves 150ms of startup

it gets called twice since zoomtopage called on/during pageload updates page dimensions
Attachment #363973 - Flags: review?(gavin.sharp)
Hmm, I though there was some code that relied on having a sane viewport rect in widgetstack init, but perhaps that's no longer the case after my last resizing patch.
I tested this on desktop and device. The code seems to handle initialization fine - no visible issues. Using the Talos Ts test I see an avg Ts speedup of 150ms.
Attachment #363973 - Flags: review?(gavin.sharp) → review+
http://hg.mozilla.org/mobile-browser/rev/38cd1d8138fd
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Assignee: nobody → tglek
You need to log in before you can comment on or make changes to this bug.