Put Data Reduction Proxy support in Cronet behind a gyp flag

components/cronet/android/test/javatests/src/org/chromium/net/CronetUrlRequestContextTest.java

 // Copyright 2014 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.net;
 
 import android.content.Context;
 import android.content.ContextWrapper;
 import android.os.ConditionVariable;
 import android.os.Handler;
@@ skipping 95 matching lines @@
     public void testDataReductionProxyEnabled() throws Exception {
         UrlRequestContextConfig config = new UrlRequestContextConfig();
         config.setLibraryName("cronet_tests");
         String[] commandLineArgs = {
                 CronetTestActivity.CONFIG_KEY, config.toString()
         };
         mActivity = launchCronetTestAppWithUrlAndCommandLineArgs(null,
                 commandLineArgs);
         assertTrue(NativeTestServer.startNativeTestServer(
                 getInstrumentation().getTargetContext()));
-
+        if (!NativeTestServer.isDataReductionProxySupported()) {
+            return;
+        }
         String serverHostPort = NativeTestServer.getHostPort();
         // Shutdown the original context and create a new context that uses the
         // NativeTestServer as the data reduction proxy.
         mActivity.mUrlRequestContext.shutdown();

[xunjieli, 2015/05/14 14:24:10]

The first context is useless right?
Can we instead do sth like,
System.loadLibrary("cronet_tests");
if (!NativeTestServer.isDataReductionProxySupported()) {
    return;
}

mActivity = launchCronetTestAppWithUrlAndCommandLineArgs(null,commandLineArgs);
// with config2

to avoid creating the first context?

[bengr, 2015/05/15 00:54:55]

Fixed in a CL that landed recently.

         UrlRequestContextConfig config2 = new UrlRequestContextConfig();
         config2.enableDataReductionProxy("test-key");
         config2.setDataReductionProxyOptions(
                 serverHostPort, "unused.net:9999",
                 NativeTestServer.getFileURL("/secureproxychecksuccess.txt"));
         config2.setLibraryName("cronet_tests");
         mActivity.mUrlRequestContext =
                 mActivity.mUrlRequestContext.createContext(
                         getInstrumentation().getTargetContext(), config2);
         TestUrlRequestListener listener = new TestUrlRequestListener();
@@ skipping 495 matching lines @@
                 new CronetUrlRequestContext(mActivity, mActivity.getContextConfig());
         CronetUrlRequestContext secondContext = new CronetUrlRequestContext(
                 mActivity.getApplicationContext(), mActivity.getContextConfig());
         CronetUrlRequestContext thirdContext = new CronetUrlRequestContext(
                 new ContextWrapper(mActivity), mActivity.getContextConfig());
         firstContext.shutdown();
         secondContext.shutdown();
         thirdContext.shutdown();
     }
 }