Do not initialize gestureconfiguration in renderer process

android_webview/lib/main/aw_main_delegate.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 "android_webview/lib/main/aw_main_delegate.h"
 
 #include <memory>
 
 #include "android_webview/browser/aw_content_browser_client.h"
 #include "android_webview/browser/browser_view_renderer.h"
@@ skipping 48 matching lines @@
 
 AwMainDelegate::~AwMainDelegate() {
 }
 
 bool AwMainDelegate::BasicStartupComplete(int* exit_code) {
   content::SetContentClient(&content_client_);
 
   content::RegisterMediaUrlInterceptor(new AwMediaUrlInterceptor());
 
   BrowserViewRenderer::CalculateTileMemoryPolicy();
-
-  // WebView apps can override WebView#computeScroll to achieve custom
-  // scroll/fling. As a result, fling animations may not be ticked, potentially
-  // confusing the tap suppression controller. Simply disable it for WebView.
-  ui::GestureConfiguration::GetInstance()
-      ->set_fling_touchscreen_tap_suppression_enabled(false);
-
+  const base::CommandLine& command_line =
+      *base::CommandLine::ForCurrentProcess();

[boliu, 2016/08/25 02:45:52]

don't create another instance, re-use cl* pointer from below

actually, there's a browser process section near the end of this method already,
just move stuff to there instead

[sgurun-gerrit only, 2016/08/25 15:04:48]

Done.

+  std::string process_type =
+      command_line.GetSwitchValueASCII(switches::kProcessType);
+  if (process_type != switches::kRendererProcess) {

[boliu, 2016/08/25 02:45:52]

you can also exclude RegisterMediaUrlInterceptor and CalculateTileMemoryPolicy
above

(doing this in separate CL is fine)

[sgurun-gerrit only, 2016/08/25 15:04:48]

Done.

+    // WebView apps can override WebView#computeScroll to achieve custom
+    // scroll/fling. As a result, fling animations may not be ticked,
+    // potentially
+    // confusing the tap suppression controller. Simply disable it for WebView
+    ui::GestureConfiguration::GetInstance()
+        ->set_fling_touchscreen_tap_suppression_enabled(false);
+  }
   base::CommandLine* cl = base::CommandLine::ForCurrentProcess();
   cl->AppendSwitch(cc::switches::kEnableBeginFrameScheduling);
 
   // WebView uses the Android system's scrollbars and overscroll glow.
   cl->AppendSwitch(switches::kDisableOverscrollEdgeEffect);
 
   // Pull-to-refresh should never be a default WebView action.
   cl->AppendSwitch(switches::kDisablePullToRefreshEffect);
 
   // Not yet supported in single-process mode.
@@ skipping 173 matching lines @@
 #if defined(VIDEO_HOLE)
 content::ExternalVideoSurfaceContainer*
 AwMainDelegate::CreateExternalVideoSurfaceContainer(
     content::WebContents* web_contents) {
   return external_video_surface::ExternalVideoSurfaceContainerImpl::Create(
       web_contents);
 }
 #endif
 
 }  // namespace android_webview