[iOS] Define bundle_data target for //ui unittests

ui/gfx/BUILD.gn

 # Copyright 2014 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("//build/config/ui.gni")
 import("//testing/test.gni")
 
 if (is_android) {
   import("//build/config/android/config.gni")
   import("//build/config/android/rules.gni")
@@ skipping 618 matching lines @@
     sources -= [
       "test/ui_cocoa_test_helper.h",
       "test/ui_cocoa_test_helper.mm",
     ]
   }
   if (is_linux) {
     deps += [ "//build/linux:fontconfig" ]
   }
 }
 
+bundle_data("gfx_unittests_bundle_data") {
+  sources = [
+    "test/data",

[Dirk Pranke, 2016/03/16 22:06:05]

nit: trailing slash

[sdefresne, 2016/03/17 09:37:19]

No, adding a trailing slash will cause an error as sources needs to be converted
to SourceFile (the gn object representing a path to a source file). This errors
out if the path ends with a trailing slash saying that you cannot reference a
directory.

So, here we list the directory as a single file. This means that if the content
of the directory change, then the bundle_data may be incorrect. This was
discussed when I created the design for bundles, and the decision was that for
tests we were okay with having dependencies a bit stale as generally adding a
new test data will go in sync with new source change, and cause the target to be
rebuilt (as the create_bundle target transitively depends on the same target as
the executable target).

+  ]
+  outputs = [
+    "{{bundle_resources_dir}}/" +
+        "{{source_root_relative_dir}}/{{source_file_part}}",
+  ]
+}
+
 test("gfx_unittests") {
   sources = [
     "font_render_params_linux_unittest.cc",
     "font_unittest.cc",
     "image/image_family_unittest.cc",
     "image/image_ios_unittest.mm",
     "image/image_skia_unittest.cc",
     "image/image_unittest.cc",
     "ios/NSString+CrStringDrawing_unittest.mm",
     "ios/uikit_util_unittest.mm",
@@ skipping 63 matching lines @@
 
   if (!is_android && !is_ios) {
     sources += [ "render_text_unittest.cc" ]
   }
 
   # TODO(jschuh): crbug.com/167187 fix size_t to int truncations.
   configs += [ "//build/config/compiler:no_size_t_to_int_warning" ]
 
   deps = [
     ":gfx",
+    ":gfx_unittests_bundle_data",
     ":test_support",
     "//base",
     "//base/test:test_support",
     "//skia",
     "//testing/gtest",
     "//third_party/icu:icuuc",
     "//third_party/libpng",
     "//third_party/zlib",
     "//ui/base",
     "//ui/gfx/geometry",
@@ skipping 58 matching lines @@
 if (is_android) {
   generate_jni("gfx_jni_headers") {
     sources = [
       "../android/java/src/org/chromium/ui/gfx/BitmapHelper.java",
       "../android/java/src/org/chromium/ui/gfx/DeviceDisplayInfo.java",
       "../android/java/src/org/chromium/ui/gfx/ViewConfigurationHelper.java",
     ]
     jni_package = "gfx"
   }
 }