Setup initial mock url request job tests for Cronet

components/cronet.gypi

 # 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.
 
 {
   'conditions': [
     ['OS=="android" and use_icu_alternatives_on_android==1', {
       # TODO(mef): Figure out what needs to be done for gn script.
       'targets': [
         {
@@ skipping 383 matching lines @@
             'resource_dir': 'cronet/android/sample/res',
             'is_test_apk': 1,
           },
           'includes': [ '../build/java_apk.gypi' ],
         },
         {
           'target_name': 'cronet_tests_jni_headers',
           'type': 'none',
           'sources': [
             'cronet/android/test/src/org/chromium/cronet_test_apk/CronetTestUtil.java',
+            'cronet/android/test/javatests/src/org/chromium/cronet_test_apk/MockUrlRequestJobTest.java',
           ],
           'variables': {
             'jni_gen_package': 'cronet_tests',
           },
           'includes': [ '../build/jni_generator.gypi' ],
         },
         {
           'target_name': 'libcronet_tests',
           'type': 'shared_library',
           'sources': [
             'cronet/android/test/cronet_tests_jni.cc',
+            'cronet/android/test/mock_url_request_job_test.cc',
+            'cronet/android/test/mock_url_request_job_test.h',
+            '../net/base/directory_lister.cc',
+            '../net/base/directory_lister.h',
+            '../net/url_request/url_request_file_job.cc',

[mef, 2014/09/23 15:24:57]

Is there particular reason to include these as source files instead of adding
dependencies on other projects?

[xunjieli, 2014/09/23 15:35:58]

I saw net.gyp
(https://code.google.com/p/chromium/codesearch#chromium/src/net/net.gyp&l=172)
excluded these source files if disable_file_support == 1. I don't see a target
that has these files included by default.

Matt, since you made the refactoring, any thoughts?

[mef, 2014/09/23 15:39:35]

Ah, that makes sense.

+            '../net/url_request/url_request_file_job.h',
           ],
           'dependencies': [
             'cronet_static',
             'cronet_tests_jni_headers',
             '../base/base.gyp:base',
             '../net/net.gyp:net',
+            '../net/net.gyp:net_test_support',
             '../net/net.gyp:quic_tools',
             '../url/url.gyp:url_lib',
+            '../base/base.gyp:base_i18n',
+            '../third_party/icu/icu.gyp:icui18n',
+            '../third_party/icu/icu.gyp:icuuc',
           ],
         },
         {
           'target_name': 'cronet_test_apk',
           'type': 'none',
           'dependencies': [
             'cronet',
           ],
           'variables': {
             'apk_name': 'CronetTest',
             'java_in_dir': 'cronet/android/test',
             'resource_dir': 'cronet/android/test/res',
+            'asset_location': 'cronet/android/test/assets',
             'native_lib_target': 'libcronet_tests',
           },
           'includes': [ '../build/java_apk.gypi' ],
         },
         {
           # cronet_test_apk creates a .jar as a side effect. Any java targets
           # that need that .jar in their classpath should depend on this target,
           # cronet_sample_apk_java. Dependents of cronet_test_apk receive its
           # jar path in the variable 'apk_output_jar_path'. This target should
           # only be used by targets which instrument cronet_test_apk.
@@ skipping 18 matching lines @@
             'java_in_dir': 'cronet/android/test/javatests',
             'resource_dir': 'cronet/android/test/res',
             'is_test_apk': 1,
           },
           'includes': [ '../build/java_apk.gypi' ],
         },
       ],
     }],  # OS=="android"
   ],
 }