Commit 853deaf9 authored by vangelis's avatar vangelis Committed by Commit bot

Revert of Delay font cache building by few seconds to yield for startup...

Revert of Delay font cache building by few seconds to yield for startup resource contention. (patchset #3 id:40001 of https://codereview.chromium.org/992413003/)

Reason for revert:
Suspected to be causing failures on Intel Release GPU bot.

See crbug.com/467706

Original issue's description:
> Delay font cache building by few seconds to yield for startup resource contention.
>
> During first renderer start there are lot of things happening simultaneously some of them are:
> 1. Renderer is going through all font files on the system to create a font collection.
> 2. Renderer loading up startup URL, accessing HTML/JS File cache, net activity etc.
> 3. Extension initialization.
>
> We delay building of this cache mainly to avoid parallel font file loading along with Renderer. Some systems have significant number of font files which takes long time process.
>
> BUG=458751,436195
> R=cpu,scottmg
>
> Committed: https://crrev.com/787e29c61825173adaf3414ff8e3d3c0c26f529b
> Cr-Commit-Position: refs/heads/master@{#320226}

TBR=cpu@chromium.org,scottmg@chromium.org,gab@chromium.org,shrikant@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=458751,436195

Review URL: https://codereview.chromium.org/1009403004

Cr-Commit-Position: refs/heads/master@{#320811}
parent a3889aa7
...@@ -291,26 +291,9 @@ void ChromeBrowserMainPartsWin::PostProfileInit() { ...@@ -291,26 +291,9 @@ void ChromeBrowserMainPartsWin::PostProfileInit() {
// otherwise it will spawn utility process to build cache file, which will // otherwise it will spawn utility process to build cache file, which will
// be used during next browser start/postprofileinit. // be used during next browser start/postprofileinit.
if (!content::LoadFontCache(path)) { if (!content::LoadFontCache(path)) {
// We delay building of font cache until first startup page loads. content::BrowserThread::PostTask(
// Selected 5 seconds as delay based on 50th percentile value content::BrowserThread::IO, FROM_HERE,
// for Startup.FirstWebContents.NonEmptyPaint. base::Bind(ExecuteFontCacheBuildTask, path));
// During first renderer start there are lot of things happening
// simultaneously some of them are:
// - Renderer is going through all font files on the system to create
// a font collection.
// - Renderer loading up startup URL, accessing HTML/JS File cache,
// net activity etc.
// - Extension initialization.
// We delay building of cache mainly to avoid parallel font file
// loading along with Renderer. Some systems have significant number of
// font files which takes long time to process.
// Related information is at http://crbug.com/436195.
const int kBuildFontCacheDelaySec = 5;
content::BrowserThread::PostDelayedTask(
content::BrowserThread::IO,
FROM_HERE,
base::Bind(ExecuteFontCacheBuildTask, path),
base::TimeDelta::FromSeconds(kBuildFontCacheDelaySec));
} }
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment