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

Side by Side Diff: content/ppapi_plugin/ppapi_plugin_main.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: 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/base_paths.h" 5 #include "base/base_paths.h"
6 #include "base/command_line.h" 6 #include "base/command_line.h"
7 #include "base/debug/crash_logging.h" 7 #include "base/debug/crash_logging.h"
8 #include "base/debug/debugger.h" 8 #include "base/debug/debugger.h"
9 #include "base/files/file_path.h" 9 #include "base/files/file_path.h"
10 #include "base/i18n/rtl.h" 10 #include "base/i18n/rtl.h"
11 #include "base/message_loop/message_loop.h" 11 #include "base/message_loop/message_loop.h"
12 #include "base/path_service.h" 12 #include "base/path_service.h"
13 #include "base/threading/platform_thread.h" 13 #include "base/threading/platform_thread.h"
14 #include "build/build_config.h" 14 #include "build/build_config.h"
15 #include "content/child/child_process.h" 15 #include "content/child/child_process.h"
16 #include "content/common/content_constants_internal.h" 16 #include "content/common/content_constants_internal.h"
17 #include "content/common/sandbox_linux/sandbox_linux.h" 17 #include "content/common/sandbox_linux/sandbox_linux.h"
18 #include "content/ppapi_plugin/ppapi_thread.h" 18 #include "content/ppapi_plugin/ppapi_thread.h"
19 #include "content/public/common/content_client.h" 19 #include "content/public/common/content_client.h"
20 #include "content/public/common/content_switches.h" 20 #include "content/public/common/content_switches.h"
21 #include "content/public/common/main_function_params.h" 21 #include "content/public/common/main_function_params.h"
22 #include "content/public/plugin/content_plugin_client.h" 22 #include "content/public/plugin/content_plugin_client.h"
23 #include "ipc/ipc_sender.h"
23 #include "ppapi/proxy/plugin_globals.h" 24 #include "ppapi/proxy/plugin_globals.h"
24 #include "ppapi/proxy/proxy_module.h" 25 #include "ppapi/proxy/proxy_module.h"
25 #include "ui/base/ui_base_switches.h" 26 #include "ui/base/ui_base_switches.h"
26 27
27 #if defined(OS_WIN) 28 #if defined(OS_WIN)
28 #include "base/win/win_util.h" 29 #include "base/win/win_util.h"
29 #include "base/win/windows_version.h" 30 #include "base/win/windows_version.h"
31 #include "content/child/dwrite_font_proxy/dwrite_font_proxy_init_win.h"
30 #include "content/common/font_warmup_win.h" 32 #include "content/common/font_warmup_win.h"
33 #include "content/public/common/dwrite_font_platform_win.h"
31 #include "sandbox/win/src/sandbox.h" 34 #include "sandbox/win/src/sandbox.h"
32 #include "third_party/WebKit/public/web/win/WebFontRendering.h" 35 #include "third_party/WebKit/public/web/win/WebFontRendering.h"
33 #include "third_party/skia/include/ports/SkTypeface_win.h" 36 #include "third_party/skia/include/ports/SkTypeface_win.h"
34 #include "ui/gfx/win/direct_write.h" 37 #include "ui/gfx/win/direct_write.h"
35 #include "ui/gfx/win/dpi.h" 38 #include "ui/gfx/win/dpi.h"
36 #endif 39 #endif
37 40
38 #if defined(OS_CHROMEOS) 41 #if defined(OS_CHROMEOS)
39 #include "base/files/file_util.h" 42 #include "base/files/file_util.h"
40 #endif 43 #endif
(...skipping 15 matching lines...) Expand all
56 namespace content { 59 namespace content {
57 60
58 namespace { 61 namespace {
59 62
60 #if defined(OS_WIN) 63 #if defined(OS_WIN)
61 // Windows-only skia sandbox support 64 // Windows-only skia sandbox support
62 void SkiaPreCacheFont(const LOGFONT& logfont) { 65 void SkiaPreCacheFont(const LOGFONT& logfont) {
63 ppapi::proxy::PluginGlobals::Get()->PreCacheFontForFlash( 66 ppapi::proxy::PluginGlobals::Get()->PreCacheFontForFlash(
64 reinterpret_cast<const void*>(&logfont)); 67 reinterpret_cast<const void*>(&logfont));
65 } 68 }
69
70 // Helper function so that we can Bind() GetBrowserSender. We can't bind it
71 // as a member function because PluginGlobals is not ref counted.
fdoray 2015/12/30 01:15:08 You can use base::Bind with a non-refcounted class
Ilya Kulshin 2015/12/30 01:59:46 Done.
72 IPC::Sender* GetPluginGlobalsSender() {
73 return ppapi::proxy::PluginGlobals::Get()->GetBrowserSender();
74 }
75
66 #endif 76 #endif
67 77
68 } // namespace 78 } // namespace
69 79
70 // Main function for starting the PPAPI plugin process. 80 // Main function for starting the PPAPI plugin process.
71 int PpapiPluginMain(const MainFunctionParams& parameters) { 81 int PpapiPluginMain(const MainFunctionParams& parameters) {
72 const base::CommandLine& command_line = parameters.command_line; 82 const base::CommandLine& command_line = parameters.command_line;
73 83
74 #if defined(OS_WIN) 84 #if defined(OS_WIN)
75 g_target_services = parameters.sandbox_info->target_services; 85 g_target_services = parameters.sandbox_info->target_services;
(...skipping 56 matching lines...) Expand 10 before | Expand all | Expand 10 after
132 142
133 ChildProcess ppapi_process; 143 ChildProcess ppapi_process;
134 ppapi_process.set_main_thread( 144 ppapi_process.set_main_thread(
135 new PpapiThread(parameters.command_line, false)); // Not a broker. 145 new PpapiThread(parameters.command_line, false)); // Not a broker.
136 146
137 #if defined(OS_WIN) 147 #if defined(OS_WIN)
138 if (!base::win::IsUser32AndGdi32Available()) 148 if (!base::win::IsUser32AndGdi32Available())
139 gfx::win::MaybeInitializeDirectWrite(); 149 gfx::win::MaybeInitializeDirectWrite();
140 bool use_direct_write = gfx::win::IsDirectWriteEnabled(); 150 bool use_direct_write = gfx::win::IsDirectWriteEnabled();
141 if (use_direct_write) { 151 if (use_direct_write) {
142 WarmupDirectWrite(); 152 if (content::ShouldUseDirectWriteFontProxyFieldTrial())
153 InitializeDWriteFontProxy(base::Bind(&GetPluginGlobalsSender));
154 else
155 WarmupDirectWrite();
143 } else { 156 } else {
144 SkTypeface_SetEnsureLOGFONTAccessibleProc(SkiaPreCacheFont); 157 SkTypeface_SetEnsureLOGFONTAccessibleProc(SkiaPreCacheFont);
145 } 158 }
146 159
147 blink::WebFontRendering::setUseDirectWrite(use_direct_write); 160 blink::WebFontRendering::setUseDirectWrite(use_direct_write);
148 blink::WebFontRendering::setDeviceScaleFactor(gfx::GetDPIScale()); 161 blink::WebFontRendering::setDeviceScaleFactor(gfx::GetDPIScale());
149 #endif 162 #endif
150 163
151 main_message_loop.Run(); 164 main_message_loop.Run();
165
166 #if defined(OS_WIN)
167 if (content::ShouldUseDirectWriteFontProxyFieldTrial())
168 UninitializeDWriteFontProxy();
169 #endif
152 return 0; 170 return 0;
153 } 171 }
154 172
155 } // namespace content 173 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698