Introduce a layering in the invalidation component as public and impl

components/invalidation/BUILD.gn

Index: components/invalidation/BUILD.gn
diff --git a/components/invalidation/BUILD.gn b/components/invalidation/BUILD.gn
index 7fa7b8777024203528a7210c48cedecd25cd0c55..0092e9e77705d1227d9135c0be985b78de4b27a9 100644
--- a/components/invalidation/BUILD.gn
+++ b/components/invalidation/BUILD.gn
@@ -5,46 +5,58 @@ if (is_android) {
   import("//build/config/android/rules.gni")
 }
 
+static_library("invalidation_base") {
+  sources = [
+    "base/ack_handle.cc",
+    "base/ack_handle.h",
+    "base/ack_handler.cc",
+    "base/ack_handler.h",
+    "base/invalidation.cc",
+    "base/invalidation.h",
+    "base/invalidation_handler.cc",
+    "base/invalidation_handler.h",

[pavely, 2015/06/24 23:45:17]

Could you add base/invalidation_export.h here too?

+    "base/invalidation_service.h",
+    "base/invalidation_util.cc",
+    "base/invalidation_util.h",
+    "base/invalidator_state.cc",
+    "base/invalidator_state.h",
+    "base/object_id_invalidation_map.cc",
+    "base/object_id_invalidation_map.h",
+    "base/single_object_invalidation_set.cc",
+    "base/single_object_invalidation_set.h",
+  ]
+  public_deps = [
+    "//third_party/cacheinvalidation",
+  ]
+  deps = [
+    "//base",
+    # "//third_party/cacheinvalidation/src/google/cacheinvalidation:cacheinvalidation_proto_cpp",
+  ]
+}
+
 static_library("invalidation") {
   sources = [
-    "ack_handle.cc",
-    "ack_handle.h",
-    "ack_handler.cc",
-    "ack_handler.h",
-    "invalidation.cc",
-    "invalidation.h",
-    "invalidation_handler.cc",
-    "invalidation_handler.h",
     "invalidation_logger.cc",
     "invalidation_logger.h",
     "invalidation_logger_observer.h",
     "invalidation_prefs.cc",
     "invalidation_prefs.h",
-    "invalidation_service.h",
     "invalidation_service_util.cc",
     "invalidation_service_util.h",
     "invalidation_state_tracker.cc",
     "invalidation_state_tracker.h",
     "invalidation_switches.cc",
     "invalidation_switches.h",
-    "invalidation_util.cc",
-    "invalidation_util.h",
     "invalidator.cc",
     "invalidator.h",
     "invalidator_registrar.cc",
     "invalidator_registrar.h",
-    "invalidator_state.cc",
-    "invalidator_state.h",
     "invalidator_storage.cc",
     "invalidator_storage.h",
     "mock_ack_handler.cc",
     "mock_ack_handler.h",
-    "object_id_invalidation_map.cc",
-    "object_id_invalidation_map.h",
     "profile_invalidation_provider.cc",
     "profile_invalidation_provider.h",
-    "single_object_invalidation_set.cc",
-    "single_object_invalidation_set.h",
     "unacked_invalidation_set.cc",
     "unacked_invalidation_set.h",
   ]
@@ -78,23 +90,23 @@ static_library("invalidation") {
       "ticl_settings_provider.h",
     ]
   }
-
   public_deps = [
-    "//third_party/cacheinvalidation",
+    ":invalidation_base",
   ]
   deps = [
     "//base",
+    "//base:i18n",
+    "//base:prefs",
     "//components/gcm_driver",
     "//components/keyed_service/core",
     "//components/pref_registry",
     "//components/signin/core/browser",
     "//google_apis",
     "//jingle:notifier",
+    "//net:net",
 
     # TODO(sync): Remove this (http://crbug.com/133352);
     "//third_party/protobuf:protobuf_lite",
-
-    # "//third_party/cacheinvalidation/src/google/cacheinvalidation:cacheinvalidation_proto_cpp",
   ]
 
   if (is_android) {