Replace merge-ui-and-compositor-threads flag with no-merge-ui-and-compositor-threads flag

android_webview/test/shell/src/org/chromium/android_webview/shell/AwShellApplication.java

 // 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.
 
 package org.chromium.android_webview.shell;
 
 import android.app.Application;
 import android.content.Context;
 import android.os.Debug;
 import android.util.Log;
@@ skipping 10 matching lines @@
         "webviewchromium.pak", "en-US.pak"
     };
 
     @Override
     public void onCreate() {
         super.onCreate();
 
         AwShellResourceProvider.registerResources(this);
 
         CommandLine.initFromFile("/data/local/chrome-command-line");
+        // TODO(michaelbai) : Remove this once we fully move to UI/compositor thread merge mode.
+        if (!CommandLine.getInstance().hasSwitch("merge-ui-and-compositor-threads")) {
+            CommandLine.getInstance().appendSwitch("no-merge-ui-and-compositor-threads");
+        }

[joth, 2013/04/26 18:06:44]

The plan was not to add this here to get upstream running in the new mode...
although I'm OK we land this patch as it is and do a follow up to remove this
again.

I think we'll need to add a couple more "if (no-merge-thread)" around some other
places in the code (e.g. where we set the kEnableWebViewSynchronousAPI flag - we
shouldn't be doing that in merged thread mode).

[michaelbai, 2013/04/26 21:41:43]

We want to run downstream in no-merge mode and upstream in merge mode, there has
to be some difference between downstream and upstream. I were thinking the
difference is the setting in WebViewChromiumFactoryProvider.java, It seemed that
you also want to run these tests (android_webview_test) in upstream merge mode,
then where is the difference you think? The command line file?


On 2013/04/26 18:06:44, joth wrote:

[joth, 2013/04/26 21:52:35]

(Note this file is not used in downstream)
The difference is, down stream has these three lines (disable merge mode unless
the developer has specifically opted into it). Upstream won't (leaves code under
test in merge thread mode, unless developer has specifically opted out)

 
         if (CommandLine.getInstance().hasSwitch(CommandLine.WAIT_FOR_JAVA_DEBUGGER)) {
            Log.e(TAG, "Waiting for Java debugger to connect...");
            Debug.waitForDebugger();
            Log.e(TAG, "Java debugger connected. Resuming execution.");
         }
 
         ResourceExtractor.setMandatoryPaksToExtract(MANDATORY_PAKS);
         ResourceExtractor.setExtractImplicitLocaleForTesting(false);
         AwBrowserProcess.loadLibrary();
         AwBrowserProcess.start(this);
     }
 }