aw: Fix multiprocess flag usage

android_webview/browser/aw_content_browser_client.cc

Index: android_webview/browser/aw_content_browser_client.cc
diff --git a/android_webview/browser/aw_content_browser_client.cc b/android_webview/browser/aw_content_browser_client.cc
index 45ab2d0ce581af72e9fa06c486b40e243a07956e..749d724459f102e1bfce9489f97f4c998b971390 100644
--- a/android_webview/browser/aw_content_browser_client.cc
+++ b/android_webview/browser/aw_content_browser_client.cc
@@ -270,9 +270,7 @@ bool AwContentBrowserClient::IsHandledURL(const GURL& url) {
 void AwContentBrowserClient::AppendExtraCommandLineSwitches(
     base::CommandLine* command_line,
     int child_process_id) {
-  if (command_line->HasSwitch(switches::kSingleProcess)) {
-    NOTREACHED() << "Android WebView does not support multi-process yet";

[Torne, 2017/04/10 14:06:29]

This is pretty weird; I assume we never reached this line? The message seems
misleading. I'm okay with removing it since it doesn't seem to be accomplishing
anything, but if you figured out why this was here in the first place I'm
curious :)

[Tobias Sargeant, 2017/04/10 14:21:14]

Since kSingleProcess was never set, I presume we never reached this.

The message itself goes way back (2012) and was unconditional. Mikhail added the
condition. I don't quite understand the point, because it seems like this method
is only be called when we're going to start a child process.

-  } else {
+  if (command_line->HasSwitch(switches::kWebViewSandboxedRenderer)) {
     // The only kind of a child process WebView can have is renderer.
     DCHECK_EQ(switches::kRendererProcess,
               command_line->GetSwitchValueASCII(switches::kProcessType));