bluetooth: android: Strings included in Chrome build.

bluetooth: android: Strings included in Chrome build.

Upcoming patch https://codereview.chromium.org/1215303006/ produces
BluetoothDevices which may have empty names. Cross platform code
BluetoothDevice::GetAddressWithLocalizedDeviceTypeName() generates
names for devices that do not advertise a name and requires string
resources.

This patch packs bluetooth strings on all platforms except IOS
(previously this was guarded by "enable_extensions").

Bluetooth string pak files in GN build are also made consistent with the
GYP build (in a strings sub directory), as it was noticed that bluetooth
was the only module inconsistent when comparing with:

diff \
 <(find outandroid/Debug -name "*pak" | sort) \
 <(find outandroidgn/Debug/ -name "*pak" | sed s/gn// | sort)

BUG=488579
Committed: https://crrev.com/6ceefd2e4a8a04218995580534479a0dbd328404
Cr-Commit-Position: refs/heads/master@{#338595}

[scheib, 2015-07-10 23:59:32]

The CQ bit was checked by scheib@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-07-11 00:01:34]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1224273008/1

[commit-bot: I haz the power, 2015-07-11 00:07:17]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-07-11 00:07:18]

Dry run: Try jobs failed on following builders:
  linux_chromium_compile_dbg_32_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

(exceeded global retry quota)

[scheib, 2015-07-11 00:11:42]

scheib@chromium.org changed reviewers:
+ thakis@chromium.org

[scheib, 2015-07-11 00:11:43]

Splitting this change out from https://codereview.chromium.org/1215303006/

On 2015/07/09 20:47:55, Nico wrote:
> On 2015/07/09 20:35:18, scheib wrote:
> > thakis, OWNERS review please:
> >  chrome/chrome_resources.gyp
> >  chrome/tools/build/repack_locales.py
> 
> Do you need to change any chrome/ .gn files?

Done.

> How much binary size does this add?

ChromePublic.apk 
47557353 before
47563209 after
    5856 delta


> Since this used to be behind enable_extensions: How is this going to be
exposed?
> Regular web apis? ("bluetooth: android: Implement BluetoothDeviceAndroid for
Web
> Bluetooth's GATT's needs." is codeword-heavy enough that I don't understand
what
> it means

Web Bluetooth http://webbluetoothcg.github.io/web-bluetooth/ an open web api
currently behind a flag chrome://flags/#enable-web-bluetooth.

[scheib, 2015-07-11 00:13:51]

Patchset #1 (id:1) has been deleted

[scheib, 2015-07-11 00:23:42]

The CQ bit was checked by scheib@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-07-11 00:24:18]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1224273008/40001

[scheib, 2015-07-11 00:28:44]

scheib@chromium.org changed reviewers:
+ armansito@chromium.org

[scheib, 2015-07-11 00:28:45]

armansito, Review device/bluetooth/strings/OWNERS please.

[armansito, 2015-07-11 00:37:23]

On 2015/07/11 00:28:45, scheib wrote:
> armansito, Review device/bluetooth/strings/OWNERS please.

TBH I don't even know what uses these strings. Can't we just remove them
entirely?

[scheib, 2015-07-11 00:38:20]

On 2015/07/11 00:37:23, armansito wrote:
> On 2015/07/11 00:28:45, scheib wrote:
> > armansito, Review device/bluetooth/strings/OWNERS please.
> 
> TBH I don't even know what uses these strings. Can't we just remove them
> entirely?

BluetoothDevice::GetAddressWithLocalizedDeviceTypeName uses them.

[commit-bot: I haz the power, 2015-07-11 02:01:11]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-07-11 02:01:12]

Dry run: This issue passed the CQ dry run.

[Nico, 2015-07-11 03:36:12]

chrome lgtm

[armansito, 2015-07-13 22:56:13]

lgtm

[scheib, 2015-07-13 23:25:54]

The CQ bit was checked by scheib@chromium.org

[scheib, 2015-07-13 23:25:54]

The patchset sent to the CQ was uploaded after l-g-t-m from thakis@chromium.org
Link to the patchset: https://codereview.chromium.org/1224273008/#ps60001
(title: "Merge TOT")

[commit-bot: I haz the power, 2015-07-13 23:27:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1224273008/60001

[commit-bot: I haz the power, 2015-07-13 23:45:59]

Committed patchset #3 (id:60001)

[commit-bot: I haz the power, 2015-07-13 23:47:03]

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