Mustash: Move GURL ParamTraits to url/ipc

url/ipc/BUILD.gn

Index: url/ipc/BUILD.gn
diff --git a/url/ipc/BUILD.gn b/url/ipc/BUILD.gn
new file mode 100644
index 0000000000000000000000000000000000000000..6daa0a87f933cd1a74241cc3d2ca9c9691bda623
--- /dev/null
+++ b/url/ipc/BUILD.gn
@@ -0,0 +1,24 @@
+# Copyright (c) 2016 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("//testing/test.gni")
+
+source_set("url_ipc") {
+  sources = [
+    "url_param_traits.cc",
+    "url_param_traits.h",
+  ]

[brettw, 2016/02/22 22:07:15]

This should depend on "//url" and "//ipc",

[Mark Dittmer, 2016/02/23 15:13:54]

Done.

+}
+
+test("url_ipc_unittests") {
+  sources = [
+    "url_param_traits_unittest.cc",

[brettw, 2016/02/22 22:43:40]

Actually, I think this can be compiled in to url_unittests in the parent
directory. Adding another target here will just be a big headache (have to
configure the bots to run it) and there's no reason why a unit test file
couldn't just depend on IPC.

What I would do is keep this target as-is but change to a source_set, then add
this source_set to the deps of //url:url_unittests

[Mark Dittmer, 2016/02/23 15:13:54]

Done.

+  ]
+
+  deps = [
+    "//base",
+    "//testing/gtest",
+    "//url:url",

[brettw, 2016/02/22 22:07:15]

This can just be "//url",

[Mark Dittmer, 2016/02/23 15:13:54]

N/A with removal.

+  ]
+}