[iOS] Define bundle_data target for //sql:sql_unittests.

[iOS] Define bundle_data target for //sql:sql_unittests.

This ensure that the files required by sql_unittests application
will be copied into the application bundle on iOS.

BUG=297668
Committed: https://crrev.com/c374755600bbe9135a3c3db738ce3b0f3fdc8fdd
Cr-Commit-Position: refs/heads/master@{#381976}

[sdefresne, 2016-03-16 18:09:30]

sdefresne@chromium.org changed reviewers:
+ dpranke@chromium.org, gbillock@chromium.org

[sdefresne, 2016-03-16 18:09:31]

dpranke: Please take a look.
gbillock: Could you review as OWNERS?

[Dirk Pranke, 2016-03-16 22:03:42]

lgtm

https://codereview.chromium.org/1811633002/diff/1/sql/BUILD.gn
File sql/BUILD.gn (right):

https://codereview.chromium.org/1811633002/diff/1/sql/BUILD.gn#newcode74
sql/BUILD.gn:74: "test/data",
nit: please use a trailing slash, assuming that works. If it doesn't, we need to
make it work, for consistency with how we treat data = [...] vars.

[sdefresne, 2016-03-17 17:54:00]

gbillock: ping?

https://codereview.chromium.org/1811633002/diff/1/sql/BUILD.gn
File sql/BUILD.gn (right):

https://codereview.chromium.org/1811633002/diff/1/sql/BUILD.gn#newcode74
sql/BUILD.gn:74: "test/data",
On 2016/03/16 at 22:03:42, Dirk Pranke wrote:
> nit: please use a trailing slash, assuming that works. If it doesn't, we need
to make it work, for consistency with how we treat data = [...] vars.

Will be addressed once
https://bugs.chromium.org/p/chromium/issues/detail?id=595769 is fixed.

[sdefresne, 2016-03-18 10:16:47]

sdefresne@chromium.org changed reviewers:
+ shess@chromium.org
- gbillock@chromium.org

[sdefresne, 2016-03-18 10:17:13]

-gbillock, +shess: Can you give OWNERS review?

[Scott Hess - ex-Googler, 2016-03-18 14:30:36]

lgtm

[sdefresne, 2016-03-18 15:39:05]

The CQ bit was checked by sdefresne@chromium.org

[sdefresne, 2016-03-18 15:39:06]

The patchset sent to the CQ was uploaded after l-g-t-m from dpranke@chromium.org
Link to the patchset: https://codereview.chromium.org/1811633002/#ps40001
(title: "Rebase on origin/master")

[commit-bot: I haz the power, 2016-03-18 15:39:27]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1811633002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1811633002/40001

[commit-bot: I haz the power, 2016-03-18 15:44:08]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-03-18 15:46:11]

Description was changed from

==========
[iOS] Define bundle_data target for //sql:sql_unittests.

This ensure that the files required by sql_unittests application
will be copied into the application bundle on iOS.

BUG=297668
==========

to

==========
[iOS] Define bundle_data target for //sql:sql_unittests.

This ensure that the files required by sql_unittests application
will be copied into the application bundle on iOS.

BUG=297668

Committed: https://crrev.com/c374755600bbe9135a3c3db738ce3b0f3fdc8fdd
Cr-Commit-Position: refs/heads/master@{#381976}
==========

[commit-bot: I haz the power, 2016-03-18 15:46:12]

Patchset 3 (id:??) landed as
https://crrev.com/c374755600bbe9135a3c3db738ce3b0f3fdc8fdd
Cr-Commit-Position: refs/heads/master@{#381976}