Switching Clank to use direction-based selection granularity strategy and adding a chrome flag.

content/renderer/render_view_impl.cc

Index: content/renderer/render_view_impl.cc
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc
index 9e0c4207e80039bff552964d85310cd9972ffedf..7625c049005f7bb86f73b870406448cdac515d50 100644
--- a/content/renderer/render_view_impl.cc
+++ b/content/renderer/render_view_impl.cc
@@ -761,6 +761,25 @@ void RenderViewImpl::Initialize(const ViewMsg_New_Params& params,
   if (switches::IsTouchEditingEnabled())
     webview()->settings()->setTouchEditingEnabled(true);
 
+#if defined(OS_ANDROID)
+  WebSettings::SelectionStrategyType selection_strategy_default =
+      WebSettings::StrategyDirection;
+#else
+  WebSettings::SelectionStrategyType selection_strategy_default =
+      WebSettings::StrategyCharacter;
+#endif
+
+  WebSettings::SelectionStrategyType selection_strategy =
+      selection_strategy_default;
+  const std::string selection_strategy_str =
+      base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
+          switches::kTouchTextSelectionStrategy);
+  if (selection_strategy_str == "0")

[Charlie Reis, 2015/04/22 19:04:59]

From the earlier discussion, I think "character" or "direction" do seem like
better values than the strings "0" or "1".  That's more sensible to pass in from
the command line, and it makes this easier to read.

[mfomitchev, 2015/04/22 21:17:24]

Done.

+    selection_strategy = WebSettings::StrategyCharacter;
+  else if (selection_strategy_str == "1")
+    selection_strategy = WebSettings::StrategyDirection;
+  webview()->settings()->setSelectionStrategy(selection_strategy);
+
   if (!params.frame_name.empty())
     webview()->mainFrame()->setName(params.frame_name);