GN: Avoid failing unconditionally on iOS due to the lack of a target for icudata

BUILD.gn

Index: BUILD.gn
diff --git a/BUILD.gn b/BUILD.gn
index b80d1530f871a63d3ceb7cd1365c2db1428c5485..24fe92716c7dec50e0431a769956848c20e12c5d 100644
--- a/BUILD.gn
+++ b/BUILD.gn
@@ -503,12 +503,11 @@ component("icuuc") {
 # TODO(GYP) support use_system_icu.
 if (icu_use_data_file) {
   if (is_ios) {
-    # TODO(GYP): Support mac resource bundle shown below.
-    # 'link_settings': {
-    #   'mac_bundle_resources': [
-    #     'source/data/in/icudtl.dat',
-    #   ],
-    # }
+    group("icudata") {
+      # TODO(GYP): Provide an empty group so that this target does not
+      # unconditionally fail. The dat file needs to be bundled separately. In
+      # the longer term, need to figure out how to use system ICU

[sdefresne, 2015/06/16 15:10:10]

Will we have to move all bundle resources to "executable" type targets? This may
proves problematic as Chrome for iOS already defines 15 targets and this will
only increase as more unit tests are upstreamed. 

[Dirk Pranke, 2015/06/16 16:06:16]

I expect we'll support a bundle() target type, but I'm not sure if that answers
your question?

+    }
   } else {
     copy("icudata") {
       if (is_android) {