[Android WebView] Ensure we have user consent before uploading minidumps

android_webview/java/src/org/chromium/android_webview/crash/SynchronizedWebViewCommandLine.java

 // Copyright 2016 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.
 
 package org.chromium.android_webview.crash;
 
+import org.chromium.android_webview.command_line.CommandLineUtil;
 import org.chromium.base.CommandLine;
 import org.chromium.base.annotations.SuppressFBWarnings;
 
 /**
  * Class for fetching command line switches for WebView in a thread-safe way.
  */
 class SynchronizedWebViewCommandLine {
     private static final Object sLock = new Object();
     private static InitState sInitialized = InitState.NOT_STARTED;
-    // TODO(gsennton): this value is used in WebViewChromiumFactoryProvider as well - set it
-    // somewhere where it can be read from both classes.
-    private static final String WEBVIEW_COMMAND_LINE_FILE = "/data/local/tmp/webview-command-line";
 
     private enum InitState { NOT_STARTED, STARTED, DONE }
 
+    private SynchronizedWebViewCommandLine() {}
+
+    public static SynchronizedWebViewCommandLine getInstance() {
+        return new SynchronizedWebViewCommandLine();

[sgurun-gerrit only, 2017/01/24 08:58:49]

why is it necessary to be able to create different instances of
SynchronizedWebViewCommandLine?

[gsennton, 2017/01/24 10:41:48]

To be able to mock the behaviour of SynchronizedWebViewCommandLine (see
CrashReceiverServiceUnitTest.java).
It's not strictly necessary to create different instances - all I want is the
initOnSeparateThread, and hasSwitch methods to be mockable, and before this
change they weren't since they were static.

+    }
+
     /**
      * Initialize the global CommandLine using the WebView command line file.
      * This method includes IO operations - it shouldn't be performed on the main thread.
      */
-    public static void initOnSeparateThread() {
+    public void initOnSeparateThread() {
         synchronized (sLock) {
             if (sInitialized != InitState.NOT_STARTED) return;
             sInitialized = InitState.STARTED;
         }
         new Thread(new Runnable() {
             @SuppressFBWarnings("DMI_HARDCODED_ABSOLUTE_FILENAME")
             @Override
             public void run() {
-                CommandLine.initFromFile(WEBVIEW_COMMAND_LINE_FILE);
+                // Only read command line flags if the device is debuggable.
+                if (CommandLineUtil.isBuildDebuggable()) {
+                    CommandLine.initFromFile(CommandLineUtil.WEBVIEW_COMMAND_LINE_FILE);
+                } else {
+                    CommandLine.init(null);
+                }
                 synchronized (sLock) {
                     sInitialized = InitState.DONE;
                     sLock.notifyAll();
                 }
             }
         }, "WebView-command-line-init-thread").start();
     }
 
     /**
      * Returns true if this command line contains the given switch.
      */
-    public static boolean hasSwitch(String switchString) {
+    public boolean hasSwitch(String switchString) {
         synchronized (sLock) {
             while (sInitialized != InitState.DONE) {
                 try {
                     sLock.wait();
                 } catch (InterruptedException e) {
                     throw new RuntimeException(e);
                 }
             }
             return CommandLine.getInstance().hasSwitch(switchString);
         }
     }
 }