Split rtc_task_queue target. Add separate target for sequenced_task_checker and weak_ptr.

webrtc/base/BUILD.gn

Index: webrtc/base/BUILD.gn
diff --git a/webrtc/base/BUILD.gn b/webrtc/base/BUILD.gn
index 83cb750dd22039d88eaebdf21c03963adf6a4135..7056c5b275f4b964a38e89475f62348986bbb099 100644
--- a/webrtc/base/BUILD.gn
+++ b/webrtc/base/BUILD.gn
@@ -23,6 +23,8 @@ group("base") {
     ":rtc_base",
     ":rtc_base_approved",
     ":rtc_task_queue",
+    ":sequenced_task_checker",
+    ":weak_ptr",
   ]
   if (is_android) {
     public_deps += [ ":base_java" ]
@@ -295,21 +297,13 @@ rtc_static_library("rtc_task_queue") {
     ":rtc_base_approved",
   ]
 
-  sources = [
-    "sequenced_task_checker.h",
-    "sequenced_task_checker_impl.cc",
-    "sequenced_task_checker_impl.h",
-    "weak_ptr.cc",
-    "weak_ptr.h",
-  ]
-
   if (build_with_chromium) {
-    sources += [
+    sources = [
       "../../webrtc_overrides/webrtc/base/task_queue.cc",
       "../../webrtc_overrides/webrtc/base/task_queue.h",
     ]
   } else {
-    sources += [
+    sources = [
       "task_queue.h",
       "task_queue_posix.h",
     ]
@@ -339,6 +333,31 @@ rtc_static_library("rtc_task_queue") {
   }
 }
 
+rtc_static_library("sequenced_task_checker") {
+  public_deps = [
+    ":rtc_base_approved",

[kjellander_webrtc, 2017/06/12 10:56:38]

Please move to deps or explain why public_deps is needed?

[perkj_webrtc, 2017/06/12 11:04:03]

moved to weak_ptr which is actually where its needed. Thanks.

+  ]
+
+  sources = [
+    "sequenced_task_checker.h",
+    "sequenced_task_checker_impl.cc",
+    "sequenced_task_checker_impl.h",
+  ]
+  deps = [
+    ":rtc_task_queue",
+  ]
+}
+
+rtc_static_library("weak_ptr") {
+  sources = [
+    "weak_ptr.cc",
+    "weak_ptr.h",
+  ]
+  deps = [
+    ":sequenced_task_checker",
+  ]
+}
+
 rtc_static_library("rtc_numerics") {
   sources = [
     "numerics/exp_filter.cc",
@@ -861,6 +880,8 @@ if (rtc_include_tests) {
       ":rtc_base_tests_main",
       ":rtc_base_tests_utils",
       ":rtc_task_queue",
+      ":sequenced_task_checker",
+      ":weak_ptr",
       "../test:test_support",
     ]
     if (!build_with_chromium && is_clang) {