Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(213)

Issue 2814019: Experimental pre-reading optimization to run on perf bots... (Closed)

Created:
10 years, 6 months ago by amit
Modified:
9 years ago
Reviewers:
slightlyoff, stoyan
CC:
chromium-reviews
Visibility:
Public.

Description

Experimental pre-reading optimization to run on perf bots Pre-reading chrome.dll prior to calling loadlibrary has shown significant improvement int he startup performance. Temporarily enabling this for headless mode so that we can try out the effect on the perf bots. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=50386

Patch Set 1 #

Total comments: 1

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+57 lines, -0 lines) Patch
M chrome/app/client_util.cc View 1 2 chunks +57 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
amit
10 years, 6 months ago (2010-06-19 00:05:29 UTC) #1
slightlyoff
http://codereview.chromium.org/2814019/diff/1/2 File chrome/app/client_util.cc (right): http://codereview.chromium.org/2814019/diff/1/2#newcode114 chrome/app/client_util.cc:114: if (IsRunningHeadless() || InstallUtil::IsChromeFrameProcess()) { We need to add ...
10 years, 6 months ago (2010-06-19 01:45:17 UTC) #2
slightlyoff
10 years, 6 months ago (2010-06-21 20:53:42 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698