[Cronet] Respect user agent string from config

components/cronet/android/java/src/org/chromium/net/ChromiumUrlRequestFactory.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.os.Build;
 
 import org.chromium.base.UsedByReflection;
 
 import java.nio.channels.WritableByteChannel;
 import java.util.Map;
 
 /**
  * Network request factory using the native http stack implementation.
  */
 @UsedByReflection("HttpUrlRequestFactory.java")
 public class ChromiumUrlRequestFactory extends HttpUrlRequestFactory {
     private ChromiumUrlRequestContext mRequestContext;
 
     @UsedByReflection("HttpUrlRequestFactory.java")
     public ChromiumUrlRequestFactory(
             Context context, UrlRequestContextConfig config) {
         if (isEnabled()) {
-            mRequestContext =
-                    new ChromiumUrlRequestContext(context, UserAgent.from(context), config);
+            String userAgent = config.userAgent();
+            if (!userAgent.isEmpty()) {

[mef, 2015/03/16 15:39:55]

Suggest changing to: 

if (userAgent.isEmpty())
  userAgent = UserAgent.from(context);

and removing else clause.

[xunjieli, 2015/03/16 15:45:56]

Done.

+                mRequestContext = new ChromiumUrlRequestContext(context,
+                        userAgent, config);
+            } else {
+                mRequestContext = new ChromiumUrlRequestContext(context,
+                        UserAgent.from(context), config);
+            }
         }
     }
 
     @Override
     public boolean isEnabled() {
         return Build.VERSION.SDK_INT >= 14;
     }
 
     @Override
     public String getName() {
@@ skipping 22 matching lines @@
 
     @Override
     public void stopNetLog() {
         mRequestContext.stopNetLog();
     }
 
     public ChromiumUrlRequestContext getRequestContext() {
         return mRequestContext;
     }
 }