Remove net & url small, iOS ICU alternatives, unit tests.

components/data_reduction_proxy/core/browser/BUILD.gn

 # 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.
 
 browser_sources = [
   "data_reduction_proxy_bypass_protocol.cc",
   "data_reduction_proxy_bypass_protocol.h",
   "data_reduction_proxy_bypass_stats.cc",
   "data_reduction_proxy_bypass_stats.h",
   "data_reduction_proxy_compression_stats.cc",
@@ skipping 34 matching lines @@
   "db_data_owner.cc",
   "db_data_owner.h",
 ]
 
 if (is_android) {
   source_set("browser_small") {
     sources = browser_sources
 
     deps = [
       "//base",
-      "//components/data_reduction_proxy/core/common:common_small",
+      "//components/data_reduction_proxy/core/common:common",

[brettw, 2016/04/04 22:54:11]

As mef says below, don't duplicate the target name, just remove ":common"

[kapishnikov, 2016/04/05 15:22:04]

Done.

       "//components/data_reduction_proxy/proto:data_reduction_proxy_proto",
       "//components/pref_registry",
       "//components/prefs",
       "//components/variations",
       "//crypto",
-      "//google_apis:google_apis_small",
-      "//net:net_small",
-      "//url:url_lib_use_icu_alternatives_on_android",
+      "//google_apis:google_apis",

[mef, 2016/04/04 22:25:43]

it seems that usually we don't repeat target name if it is the same as project
name (e.g. "//google_apis").

[kapishnikov, 2016/04/05 15:22:04]

Done.

+      "//net:net",
+      "//url:url",
     ]
   }
 }
 
 source_set("browser") {
   sources = browser_sources
   sources += [

[mef, 2016/04/04 22:25:43]

bengr: Would it make sense to extract data_store_impl.* into separate target and
get rid of browser/browser_small duplication?

     "data_store_impl.cc",
     "data_store_impl.h",
   ]
 
   deps = [
     "//base",
     "//components/data_reduction_proxy/core/common",
     "//components/data_reduction_proxy/proto:data_reduction_proxy_proto",
     "//components/pref_registry",
     "//components/prefs",
@@ skipping 72 matching lines @@
     "//base/test:test_support",
     "//components/data_reduction_proxy/core/common:test_support",
     "//components/data_reduction_proxy/proto:data_reduction_proxy_proto",
     "//components/prefs:test_support",
     "//components/variations",
     "//net:test_support",
     "//testing/gmock",
     "//testing/gtest",
   ]
 }