Firefox Startup Performance – Feb 11, 2010

Various minor updates this week:

  • Static build: Ted did the first pass of review. Getting close. Will land on the Places branch once binary tests build, to see perf impact.
  • Ben’s fastload cache replacement is still waiting on first-review from Ben Smedberg.
  • Neil took up bug 354048 to not rebuild toolbars at startup, should reduce the DOM activity some at least. The patch is in review by Dao and Mano.
  • Marco has nearly completed bug 542943, removing the bookmark redirect hash, which means less SQL at startup and less memory usage by the bookmarks service.
  • I put a patch up on bug 545516 that fixes some bugs and cleans up the layout of the Performance Snapshot. Will get it landed next week.
  • Started working with John O’Duinn and Mike Morgan to get a team together to work on the graph server.

2 Comments on “Firefox Startup Performance – Feb 11, 2010”

  1. Zack says:

    Yay for better graph server!

  2. Alfred Kayser says:

    It seems that a lot of performance work is stuck in waiting for reviews. Patches that remove deaad code, unused functions, decom, etc are for example:
    bug 405407 Merge nsDiskCacheStreamIO and nsDiskCacheStreamOutput
    bug 537223 Cleanup of gfxColor.h
    bug 331032 Decom nsICacheEntryInfo, replace by ExpirationTime attribute to nsICachingChannel
    bug 230675 ‘decom’ of nsICacheVisitor.idl: saves 10% / 150K from nkcache_s.lib
    bug 386937 ConvertPixel doesn’t work as nsColor and gfx_color are not compatible…

    To really address performance, start kicking the reviewers to focus on performance patches that are proven to work and proven to improve performance.

Leave a Reply

Fill in your details below or click an icon to log in: Logo

You are commenting using your account. Log Out /  Change )

Google+ photo

You are commenting using your Google+ account. Log Out /  Change )

Twitter picture

You are commenting using your Twitter account. Log Out /  Change )

Facebook photo

You are commenting using your Facebook account. Log Out /  Change )


Connecting to %s