Split NQE to multiple files

components/cronet/android/BUILD.gn

 # Copyright 2015 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.
 
 import("//build/buildflag_header.gni")
 import("//build/config/android/config.gni")
 import("//build/config/android/rules.gni")
 import("//build/util/version.gni")
 import("//chrome/version.gni")
 import("//testing/test.gni")
@@ skipping 23 matching lines @@
     "chromium_url_request.h",
   ]
 }
 
 java_cpp_enum("net_request_priority_java") {
   sources = [
     "//net/base/request_priority.h",
   ]
 }
 
-java_cpp_enum("network_quality_observations_java") {
+java_cpp_enum("network_quality_observation_source_java") {
   sources = [
-    "//net/nqe/network_quality_estimator.h",
+    "//net/nqe/network_quality_observation_source.h",
   ]
 }
 
 java_cpp_enum("url_request_error_java") {
   sources = [
     "url_request_error.h",
   ]
 }
 
 # This target is a jar file containing classes that Cronet's javadocs
@@ skipping 200 matching lines @@
 
   deps = [
     "//third_party/android_tools:android_support_annotations_javalib",
   ]
 
   srcjar_deps = [
     ":cronet_version_srcjar",
     ":http_cache_type_java",
     ":url_request_error_java",
     ":load_states_list",
-    ":network_quality_observations_java",
+    ":network_quality_observation_source_java",
   ]
 
   run_findbugs_override = true
 }
 
 android_library("cronet_java") {
   java_files = [
     "java/src/org/chromium/net/ChromiumUrlRequest.java",
     "java/src/org/chromium/net/ChromiumUrlRequestContext.java",
     "java/src/org/chromium/net/ChromiumUrlRequestFactory.java",
@@ skipping 11 matching lines @@
     "java/src/org/chromium/net/urlconnection/CronetOutputStream.java",
     "java/src/org/chromium/net/urlconnection/CronetURLStreamHandlerFactory.java",
     "java/src/org/chromium/net/urlconnection/MessageLoop.java",
   ]
 
   deps = [
     ":cronet_api",
     "//base:base_java",
     "//net/android:net_java",
     "//third_party/android_tools:android_support_annotations_javalib",
     "//third_party/jsr-305:jsr_305_javalib",

[mmenke, 2016/05/06 15:04:04]

Should network_quality_observation_source_java be a dependency here?  It is in
the gyp file, or is it not needed there?  Or is there a reason it's only needed
for gyp?

[tbansal1, 2016/05/06 17:32:55]

Very likely it was not necessary in gyp file (Similar to http_cache_type_java).
I am not 100% sure because I did not try building it locally after commenting
that out.

Since gyp has been deprecated for Android, I am not sure if it is worth fixing
it.
https://groups.google.com/a/google.com/forum/#!searchin/cronet-team/gyp|sort:...

   ]
 
   srcjar_deps = [
     ":chromium_url_request_java",
     ":net_request_priority_java",
   ]
 
   run_findbugs_override = true
 }
 
@@ skipping 559 matching lines @@
     ":cronet_package_copy_native_lib",
     ":cronet_package_copy_native_lib_unstripped",
     ":generate_javadoc",
     ":generate_licenses",
     ":jar_cronet_api_source",
     ":jar_cronet_other_source",
     ":jar_cronet_sample_source",
     ":repackage_extracted_jars",
   ]
 }