Closed Bug 538655 Opened 16 years ago Closed 16 years ago

Content Sink preferences investigation

Categories

(Firefox for Android Graveyard :: General, defect)

ARM
Maemo
defect
Not set
normal

Tracking

(fennec1.0+)

RESOLVED FIXED
Tracking Status
fennec 1.0+ ---

People

(Reporter: dougt, Assigned: dougt)

Details

Attachments

(2 files)

Using a build from Jan 6, from the tip of mobile-browser and mozilla-1.9.2. * Disk cache is off * Plugins are off. The build has two points instrumented to collect data: * Page load timing I have added hooks in the onStateChange function in browser.js to record the total page load time. * Panning delta times This is a crude way of measuring panning performance similar to tpan. The time deltas are based on the mouse events the front end sees. Basically, if the application is busing rendering, these delta times will increase. Long delays can be observe visually (e.g., a delta of > 500ms can be seen as a pause while panning). Steps 1) Modified content sink preference 2) started fennec on the command line 3) Loaded target url via the awesome bar 4) Waited until page completed 5) Reloaded target url 5 times 6) Repeated steps 3-5 for each of the target urls 6) Reloaded techcrunch.com and while the page was loaded proceed to pan in random directions which included bring out the sidebars I varied the deflection count and parse time looking for values that would keep raw load time consistent with the control (deflection count = 10, parse time = 50000) and decreased max time between mouse moves
based on the data, i found interactive Mode - deflection count = 1000; parse time = 5000 to be probably the best we can do to both ensure page load time and page interactive while loading.
Attached patch patch v.1Splinter Review
** we need to watch page load time regressions after this lands. **
Assignee: nobody → mozbugz
Attachment #420797 - Flags: review?
Attachment #420797 - Flags: review? → review?(pavlov)
Attachment #420797 - Flags: review?(pavlov) → review+
tracking-fennec: --- → 1.0+
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
from data that mfinkle collected, it look like this improved tp4 by 30%!
Component: Linux/Maemo → General
QA Contact: maemo-linux → general
Hardware: All → ARM
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: