linux: Use subpixel positioning for forced scale factor.

ui/gfx/font_render_params_linux.cc

Index: ui/gfx/font_render_params_linux.cc
diff --git a/ui/gfx/font_render_params_linux.cc b/ui/gfx/font_render_params_linux.cc
index 5e30f4dd947acf73a05e435d2dfe8420bdfd0aeb..1120ac86fd6e93cf12fc9b36de90562862e507ba 100644
--- a/ui/gfx/font_render_params_linux.cc
+++ b/ui/gfx/font_render_params_linux.cc
@@ -6,6 +6,7 @@
 
 #include "base/command_line.h"
 #include "base/logging.h"
+#include "ui/gfx/display.h"
 #include "ui/gfx/switches.h"
 
 #if defined(TOOLKIT_GTK)
@@ -23,9 +24,13 @@ namespace gfx {
 namespace {
 
 bool SubpixelPositioningRequested(bool renderer) {
+  if (renderer) {
+    return CommandLine::ForCurrentProcess()->HasSwitch(

[Alexei Svitkine (slow), 2014/04/09 20:43:23]

Nit: Extract CommandLine::ForCurrentProcess() into a local var to make this part
more concise. It can be above the if since it's also used below.

[Daniel Erat, 2014/04/09 21:03:11]

Done.

+               switches::kEnableWebkitTextSubpixelPositioning) ||
+           (Display::HasForceDeviceScaleFactor() &&

[Alexei Svitkine (slow), 2014/04/09 20:43:23]

Nit: Could you add a comment explaining the motivation behind this logic here?

[Daniel Erat, 2014/04/09 21:03:11]

Done.

+            Display::GetForcedDeviceScaleFactor() != 1.0);
+  }
   return CommandLine::ForCurrentProcess()->HasSwitch(
-      renderer ?
-      switches::kEnableWebkitTextSubpixelPositioning :
       switches::kEnableBrowserTextSubpixelPositioning);
 }