Adding a chrome flag to control the touch selection granularity strategy used by Blink.

chrome/browser/about_flags.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 "chrome/browser/about_flags.h"
 
 #include <iterator>
 #include <map>
 #include <set>
 #include <utility>
@@ skipping 155 matching lines @@
   { IDS_GENERIC_EXPERIMENT_CHOICE_ENABLED, "", "" },
   { IDS_GENERIC_EXPERIMENT_CHOICE_DISABLED,
     switches::kOverscrollHistoryNavigation,
     "0" },
   { IDS_OVERSCROLL_HISTORY_NAVIGATION_SIMPLE_UI,
     switches::kOverscrollHistoryNavigation,
     "2" }
 };
 #endif
 
+const Experiment::Choice kTouchTextSelectionStrategyChoices[] = {
+  { IDS_GENERIC_EXPERIMENT_CHOICE_DEFAULT, "", "" },
+  { IDS_TOUCH_SELECTION_STRATEGY_DIRECTION,
+    switches::kTouchTextSelectionStrategy,

[jdduke (slow), 2015/04/17 15:20:29]

I'd be OK if we just called this |kEnableSmartTextSelection| or something like
that. Not sure we even need it in about:flags, but if we do add it here we might
restrict the OS to where it will be available (Android).

[mfomitchev, 2015/04/17 16:22:39]

You are a bit ahead of me. Check out
https://codereview.chromium.org/1096773002/.

+    "1" }
+};
+
 #if !defined(DISABLE_NACL)
 const Experiment::Choice kNaClDebugMaskChoices[] = {
   // Secure shell can be used on ChromeOS for forwarding the TCP port opened by
   // debug stub to a remote machine. Since secure shell uses NaCl, we usually
   // want to avoid debugging that. The PNaCl translator is also a NaCl module,
   // so by default we want to avoid debugging that.
   // NOTE: As the default value must be the empty string, the mask excluding
   // the PNaCl translator and secure shell is substituted elsewhere.
   { IDS_NACL_DEBUG_MASK_CHOICE_EXCLUDE_UTILS_PNACL, "", "" },
   { IDS_NACL_DEBUG_MASK_CHOICE_DEBUG_ALL, switches::kNaClDebugMask, "*://*" },
@@ skipping 910 matching lines @@
   },
   {
     "enable-touch-editing",
     IDS_FLAGS_ENABLE_TOUCH_EDITING_NAME,
     IDS_FLAGS_ENABLE_TOUCH_EDITING_DESCRIPTION,
     kOsCrOS | kOsWin | kOsLinux,
     ENABLE_DISABLE_VALUE_TYPE(switches::kEnableTouchEditing,
                               switches::kDisableTouchEditing)
   },
   {
+      "touch-selection-strategy",
+      IDS_FLAGS_TOUCH_SELECTION_STRATEGY_NAME,
+      IDS_FLAGS_TOUCH_SELECTION_STRATEGY_DESCRIPTION,
+      kOsAll,
+      MULTI_VALUE_TYPE(kTouchTextSelectionStrategyChoices)
+  },
+  {
     "enable-stale-while-revalidate",
     IDS_FLAGS_ENABLE_STALE_WHILE_REVALIDATE_NAME,
     IDS_FLAGS_ENABLE_STALE_WHILE_REVALIDATE_DESCRIPTION,
     kOsAll,
     SINGLE_VALUE_TYPE(switches::kEnableStaleWhileRevalidate)
   },
   {
     "enable-suggestions-service",
     IDS_FLAGS_ENABLE_SUGGESTIONS_SERVICE_NAME,
     IDS_FLAGS_ENABLE_SUGGESTIONS_SERVICE_DESCRIPTION,
@@ skipping 1665 matching lines @@
 }
 
 const Experiment* GetExperiments(size_t* count) {
   *count = num_experiments;
   return experiments;
 }
 
 }  // namespace testing
 
 }  // namespace about_flags