Add logic to switch to DirectWrite font proxy via field trial.

content/common/sandbox_win.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/common/sandbox_win.h"
 
 #include <stddef.h>
 
 #include <string>
 
@@ skipping 717 matching lines @@
 #if !defined(NACL_WIN64)
   if (type_str == switches::kRendererProcess ||
       type_str == switches::kPpapiPluginProcess) {
     if (gfx::win::ShouldUseDirectWrite()) {
       AddDirectory(base::DIR_WINDOWS_FONTS,
                   NULL,
                   true,
                   sandbox::TargetPolicy::FILES_ALLOW_READONLY,
                   policy);
 
-      // If DirectWrite is enabled for font rendering then open the font cache
-      // section which is created by the browser and pass the handle to the
-      // renderer process. This is needed because renderer processes on
-      // Windows 8+ may be running in an AppContainer sandbox and hence their
-      // kernel object namespace may be partitioned.
-      std::string name(content::kFontCacheSharedSectionName);
-      name.append(base::UintToString(base::GetCurrentProcId()));
+      if (content::ShouldUseDirectWriteFontProxyFieldTrial()) {

[fdoray, 2015/12/30 01:15:08]

if (!...)

[Ilya Kulshin, 2015/12/30 01:59:46]

Done.

+        // If DirectWrite is enabled for font rendering then open the font
+        // cache section which is created by the browser and pass the handle to
+        // the renderer process. This is needed because renderer processes on
+        // Windows 8+ may be running in an AppContainer sandbox and hence their
+        // kernel object namespace may be partitioned.
+        std::string name(content::kFontCacheSharedSectionName);
+        name.append(base::UintToString(base::GetCurrentProcId()));
 
-      base::SharedMemory direct_write_font_cache_section;
-      if (direct_write_font_cache_section.Open(name, true)) {
-        void* shared_handle = policy->AddHandleToShare(
-            direct_write_font_cache_section.handle().GetHandle());
-        cmd_line->AppendSwitchASCII(
-            switches::kFontCacheSharedHandle,
-            base::UintToString(base::win::HandleToUint32(shared_handle)));
+        base::SharedMemory direct_write_font_cache_section;
+        if (direct_write_font_cache_section.Open(name, true)) {
+          void* shared_handle = policy->AddHandleToShare(
+              direct_write_font_cache_section.handle().GetHandle());
+          cmd_line->AppendSwitchASCII(
+              switches::kFontCacheSharedHandle,
+              base::UintToString(base::win::HandleToUint32(shared_handle)));
+        }
       }
     }
   }
 #endif
 
   if (type_str != switches::kRendererProcess) {
     // Hack for Google Desktop crash. Trick GD into not injecting its DLL into
     // this subprocess. See
     // http://code.google.com/p/chromium/issues/detail?id=25580
     cmd_line->AppendSwitchASCII("ignored", " --type=renderer ");
@@ skipping 91 matching lines @@
   }
 
   return false;
 }
 
 bool BrokerAddTargetPeer(HANDLE peer_process) {
   return g_broker_services->AddTargetPeer(peer_process) == sandbox::SBOX_ALL_OK;
 }
 
 }  // namespace content