Status

RESOLVED DUPLICATE of bug 459117
10 years ago
10 years ago

People

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

Tracking

(Depends on: 4 bugs, {meta, perf})

Trunk
x86
Linux
meta, perf
Dependency tree / graph

Details

(Assignee)

Description

10 years ago
I compared firefox to fennec on my c2d at 1Ghz and notice it had a constant overhead on loading pages(>0.2 seconds). From oprofile looked like nsXPCWrappedJSClass::CallMethod was an interesting function. I hacked in a map to collect cpu time of every function called and get stuff like

nsIAutoCompleteInput::onTextEntered 1 calls, 320.312012ms total
nsIDOMEventListener::handleEvent 21 calls, 1968.750977ms total
nsIFactory::createInstance 1 calls, 7.813000ms total
nsILoginManagerStorage::countLogins 1 calls, 46.875000ms total
nsILoginManagerStorage::init 1 calls, 15.625000ms total
nsIModule::getClassObject 1 calls, 0.000000ms total
nsIWebProgressListener::onLocationChange 1 calls, 70.313004ms total
nsIWebProgressListener::onProgressChange 4 calls, 0.000000ms total
nsIWebProgressListener::onSecurityChange 1 calls, 46.875000ms total
nsIWebProgressListener::onStateChange 58 calls, 4117.188965ms total
nsIWebProgressListener::onStatusChange 28 calls, 23.438000ms total


So with this info i cut stuff out of onStateChange http://mxr.mozilla.org/mobile-browser/source/chrome/content/browser.js#409

Taking out translatePhoneNumbers saves 2.6seconds(out 4.1 above)
Taking out updateBrowser saves another 0.3seconds

This using www.w3.org/ as a reference since it renders quickly.
Depends on: 463291
(Assignee)

Updated

10 years ago
Depends on: 463322
(Assignee)

Updated

10 years ago
Depends on: 463530
(Assignee)

Updated

10 years ago
Depends on: 463538
(Assignee)

Updated

10 years ago
Depends on: 463722
(Assignee)

Updated

10 years ago
Depends on: 463745
(Assignee)

Updated

10 years ago
Depends on: 464128
(Assignee)

Updated

10 years ago
Depends on: 464129
(Assignee)

Updated

10 years ago
Depends on: 465556
(Assignee)

Updated

10 years ago
Depends on: 465128
(Assignee)

Updated

10 years ago
Depends on: 464995
(Assignee)

Updated

10 years ago
Depends on: 466578

Updated

10 years ago
Assignee: nobody → tglek
Keywords: meta, perf
(Assignee)

Updated

10 years ago
Depends on: 466739
(Assignee)

Updated

10 years ago
Depends on: 466877
(Assignee)

Updated

10 years ago
Depends on: 468163
(Assignee)

Updated

10 years ago
Depends on: 468733
(Assignee)

Updated

10 years ago
Depends on: 468739
(Assignee)

Updated

10 years ago
Depends on: 468760
(Assignee)

Updated

10 years ago
Depends on: 468776
(Assignee)

Updated

10 years ago
Depends on: 468886
(Assignee)

Updated

10 years ago
Depends on: 469076
Taras:

Can you attach the patch for your printf-instrumented build?  Even if it's not pretty it might still be a useful reference for me when I start work on bug 467953.
(Assignee)

Comment 2

10 years ago
i'm gonna package it up nicely next week, but if you want it asap I can post it.
Actually next week is fine.  I'm not going to be able to get to it before then anyway.
(Assignee)

Updated

10 years ago
Depends on: 470116
(Assignee)

Updated

10 years ago
Depends on: 470507
(Assignee)

Updated

10 years ago
Depends on: 470810
(Assignee)

Updated

10 years ago
Depends on: 470859
(Assignee)

Updated

10 years ago
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 459117
You need to log in before you can comment on or make changes to this bug.