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

Unified Diff: content/common/font_warmup_win.cc

Issue 1557513002: Add logic to switch to DirectWrite font proxy via field trial. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add a comment and remove a blank line Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: content/common/font_warmup_win.cc
diff --git a/content/common/font_warmup_win.cc b/content/common/font_warmup_win.cc
index f9825fef0c734f36243cd7be4259db2e596b8a1f..6090a81c06a479c050ee8d2234a6c3c8653bdccc 100644
--- a/content/common/font_warmup_win.cc
+++ b/content/common/font_warmup_win.cc
@@ -23,6 +23,7 @@
#include "base/win/iat_patch_function.h"
#include "base/win/windows_version.h"
#include "content/public/common/dwrite_font_platform_win.h"
+#include "ppapi/shared_impl/proxy_lock.h"
#include "skia/ext/fontmgr_default_win.h"
#include "skia/ext/refptr.h"
#include "third_party/WebKit/public/web/win/WebFontRendering.h"
@@ -35,6 +36,7 @@ namespace content {
namespace {
+// The Skia font manager, used for the life of the process (leaked at the end).
SkFontMgr* g_warmup_fontmgr = nullptr;
base::win::IATPatchFunction g_iat_patch_open_sc_manager;
@@ -255,6 +257,7 @@ skia::RefPtr<SkTypeface> GetTypefaceFromLOGFONT(const LOGFONTW* log_font) {
: SkFontStyle::kUpright_Slant);
std::string family_name = base::WideToUTF8(log_font->lfFaceName);
+ ppapi::ProxyAutoLock lock; // Needed for DirectWrite font proxy.
return skia::AdoptRef(
g_warmup_fontmgr->matchFamilyStyle(family_name.c_str(), style));
}
@@ -486,7 +489,10 @@ SkFontMgr* GetPreSandboxWarmupFontMgr() {
}
GdiFontPatchData* PatchGdiFontEnumeration(const base::FilePath& path) {
- // We assume the fontmgr is already warmed up before calling this.
+ if (content::ShouldUseDirectWriteFontProxyFieldTrial() && !g_warmup_fontmgr)
jam 2016/01/11 05:05:45 nit: no content::
Ilya Kulshin 2016/01/11 20:56:52 Done.
+ g_warmup_fontmgr = SkFontMgr_New_DirectWrite();
+ // If not using the font proxy, we assume |g_warmup_fontmgr| is already
+ // initialized before this function is called.
DCHECK(g_warmup_fontmgr);
return new GdiFontPatchDataImpl(path);
}

Powered by Google App Engine
This is Rietveld 408576698