[ObjC ARC] Converts ios/chrome/test:test_support to ARC.

ios/chrome/test/BUILD.gn

Index: ios/chrome/test/BUILD.gn
diff --git a/ios/chrome/test/BUILD.gn b/ios/chrome/test/BUILD.gn
index 44ac22f58b65a0857655a95b1f8b729e5dfdf1c6..ce9b77b8b0154072d04369905a86d72271b99130 100644
--- a/ios/chrome/test/BUILD.gn
+++ b/ios/chrome/test/BUILD.gn
@@ -17,10 +17,9 @@ group("all_tests") {
 }
 
 source_set("test_support") {
+  configs += [ "//build/config/compiler:enable_arc" ]
   testonly = true
   sources = [
-    "block_cleanup_test.h",
-    "block_cleanup_test.mm",
     "ios_chrome_scoped_testing_chrome_browser_provider.h",
     "ios_chrome_scoped_testing_chrome_browser_provider.mm",
     "ios_chrome_scoped_testing_chrome_browser_state_manager.cc",
@@ -34,6 +33,10 @@ source_set("test_support") {
     "testing_application_context.mm",
   ]
 
+  public_deps = [
+    ":block_cleanup_test",
+  ]
+
   deps = [
     "//base",
     "//base/test:test_support",
@@ -57,6 +60,20 @@ source_set("test_support") {
   ]
 }
 
+source_set("block_cleanup_test") {
+  # The files in this target depend on non-ARC memeory management.

[rohitrao (ping after 24h), 2017/06/14 12:30:05]

I wonder if we should add a disable_arc target, so that this can be explicit and
enforced, rather than communicated via comment.

[rohitrao (ping after 24h), 2017/06/14 12:30:05]

Typo: memory.

[marq (ping after 24h), 2017/06/14 13:31:23]

Good idea, created a bug for that.

[marq (ping after 24h), 2017/06/14 13:31:23]

Done.

+  testonly = true
+  sources = [
+    "block_cleanup_test.h",
+    "block_cleanup_test.mm",
+  ]
+
+  deps = [
+    "//base",
+    "//testing/gtest",
+  ]
+}
+
 source_set("run_all_unittests") {
   testonly = true
   sources = [