Add trial for DirectWrite, and reorder some initialization so it can take effect

content/common/sandbox_win.cc

Index: content/common/sandbox_win.cc
diff --git a/content/common/sandbox_win.cc b/content/common/sandbox_win.cc
index 18b30e2963cd5afc568bc839868238037fd4146b..dd79cdcd12a02da4fb67b6c69cacf07bf31135d3 100644
--- a/content/common/sandbox_win.cc
+++ b/content/common/sandbox_win.cc
@@ -12,6 +12,7 @@
 #include "base/debug/trace_event.h"
 #include "base/file_util.h"
 #include "base/hash.h"
+#include "base/metrics/field_trial.h"
 #include "base/path_service.h"
 #include "base/process/launch.h"
 #include "base/strings/string_util.h"
@@ -28,6 +29,7 @@
 #include "sandbox/win/src/sandbox.h"
 #include "sandbox/win/src/sandbox_nt_util.h"
 #include "sandbox/win/src/win_utils.h"
+#include "ui/gfx/win/dpi.h"
 
 static sandbox::BrokerServices* g_broker_services = NULL;
 static sandbox::TargetServices* g_target_services = NULL;
@@ -566,9 +568,22 @@ bool ShouldUseDirectWrite() {
   // DirectWrite. Skia does not require the additions to DirectWrite in QFE
   // 2670838, so a Win7 check is sufficient. We do not currently attempt to
   // support Vista, where SP2 and the Platform Update are required.
+  if (base::win::GetVersion() < base::win::VERSION_WIN7)
+    return false;
+
+  // If forced off, don't use it.
   const CommandLine& command_line = *CommandLine::ForCurrentProcess();
-  return !command_line.HasSwitch(switches::kDisableDirectWrite) &&
-         base::win::GetVersion() >= base::win::VERSION_WIN7;
+  if (command_line.HasSwitch(switches::kDisableDirectWrite))

[Alexei Svitkine (slow), 2014/06/17 21:30:29]

Do you want to see data from users who've manually toggled their flags as
separate groups on the dashboard? If so, you can follow the pattern suggested at
go/finch-and-flags to get that effect. If you don't care, then feel free to
ignore as this will be fine otherwise.

But it might be a good signal to see if people are manually disabling the
experimental group.

(If you do follow that suggested pattern, you should probably put the DPI scale
check above this, just below the version check.)

[scottmg, 2014/06/17 22:07:00]

Thanks, I think we don't care too much as there's many reasons (aesthetic,
preference, etc.) that people might force it off.

+    return false;
+
+  // Can't use GDI on HiDPI.
+  if (gfx::GetDPIScale() > 1.0f)
+    return true;
+
+  // Otherwise, Finch.
+  const std::string group_name =
+      base::FieldTrialList::FindFullName("DirectWrite");
+  return group_name != "Disabled";

[Alexei Svitkine (slow), 2014/06/17 21:30:29]

Instead of checking for != "Disabled", check for "Enabled" instead. That way,
you can have a group called "Control" that exercises the default behavior.

[Alexei Svitkine (slow), 2014/06/17 22:02:15]

Sorry, I just read your experiment description and you want the default to be
enabled, so the above makes sense now. Please ignore this comment.

[scottmg, 2014/06/17 22:07:00]

Done.

[scottmg, 2014/06/17 22:07:00]

Yeah, ok thanks. Seems like it'd work either way, depending on the server-side.
I'm still working through that CL.

 }
 
 base::ProcessHandle StartSandboxedProcess(