Index: content/browser/renderer_host/render_message_filter.cc |
diff --git a/content/browser/renderer_host/render_message_filter.cc b/content/browser/renderer_host/render_message_filter.cc |
index 6dbd4585a87fe0201dc89f97fe6817798f39a171..7e2d3bee7cabdebfa089fc8f61e74c8a1c550373 100644 |
--- a/content/browser/renderer_host/render_message_filter.cc |
+++ b/content/browser/renderer_host/render_message_filter.cc |
@@ -1171,9 +1171,11 @@ void RenderMessageFilter::OnDidLose3DContext( |
#if defined(OS_WIN) |
void RenderMessageFilter::OnPreCacheFontCharacters(const LOGFONT& font, |
const base::string16& str) { |
- // TODO(scottmg): Move this to FontCacheDispatcher, http://crbug.com/356346. |
- if (!ShouldUseDirectWrite()) |
- return; |
+ // TODO(scottmg): pdf/ppapi still require the renderer to be able to precache |
+ // GDI fonts (http://crbug.com/383227), even when using DirectWrite. |
+ // Eventually this shouldn't be added and should be moved to |
+ // FontCacheDispatcher too. http://crbug.com/356346. |
+ |
// First, comments from FontCacheDispatcher::OnPreCacheFont do apply here too. |
// Except that for True Type fonts, |
// GetTextMetrics will not load the font in memory. |