Commit 88e3114d authored by scottmg's avatar scottmg Committed by Commit bot

Raise kMaxFontThreshold

Looking at the UMA value of DirectWrite.Fonts.Loaded, 1000->1750 will
take us from ~96% to >99% of users having all their fonts loaded.

Hopefully this will alleviate most annoyance until the proxy code is
ready.

R=shrikant@chromium.org
BUG=413059

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

Cr-Commit-Position: refs/heads/master@{#294522}
parent 7faedca8
...@@ -350,7 +350,7 @@ IDWriteFontCollection* GetCustomFontCollection(IDWriteFactory* factory) { ...@@ -350,7 +350,7 @@ IDWriteFontCollection* GetCustomFontCollection(IDWriteFactory* factory) {
// We try here to put arbitrary limit on max number of fonts that could // We try here to put arbitrary limit on max number of fonts that could
// be loaded, otherwise we fallback to restricted set of fonts. // be loaded, otherwise we fallback to restricted set of fonts.
const UINT32 kMaxFontThreshold = 1000; const UINT32 kMaxFontThreshold = 1750;
HRESULT hr = E_FAIL; HRESULT hr = E_FAIL;
if (g_font_loader->GetFontMapSize() < kMaxFontThreshold) { if (g_font_loader->GetFontMapSize() < kMaxFontThreshold) {
hr = factory->CreateCustomFontCollection( hr = factory->CreateCustomFontCollection(
......
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