Do not build device/bluetooth strings when extensions are disabled.

Do not build device/bluetooth strings when extensions are disabled.

BUG=334407
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=288953

[rpaquay, 2014-08-11 23:25:21]

On 2014/08/11 22:41:36, Lei Zhang wrote:

I have very little context to review this change as I have not been involved
much with string resource generation code before. Maybe keybuk@ would have more
insight.

[Lei Zhang, 2014-08-11 23:32:48]

+miket, who reviewed both r183953 and r203652

[miket_OOO, 2014-08-11 23:35:32]

lgtm

https://codereview.chromium.org/461773002/diff/1/chrome/tools/build/repack_lo...
File chrome/tools/build/repack_locales.py (right):

https://codereview.chromium.org/461773002/diff/1/chrome/tools/build/repack_lo...
chrome/tools/build/repack_locales.py:105: #e.g.
'<(SHARED_INTERMEDIATE_DIR)/ui/strings/app_locale_settings_da.pak',
I'm not sure why this moved, but it seems to be a no-op change, which is
probably fine.

[Lei Zhang, 2014-08-11 23:37:55]

https://codereview.chromium.org/461773002/diff/1/chrome/tools/build/repack_lo...
File chrome/tools/build/repack_locales.py (right):

https://codereview.chromium.org/461773002/diff/1/chrome/tools/build/repack_lo...
chrome/tools/build/repack_locales.py:105: #e.g.
'<(SHARED_INTERMEDIATE_DIR)/ui/strings/app_locale_settings_da.pak',
On 2014/08/11 23:35:32, miket wrote:
> I'm not sure why this moved, but it seems to be a no-op change, which is
> probably fine.

It's just the way the diff came out when I moved bluetooth and extensions down.

[Lei Zhang, 2014-08-11 23:37:58]

The CQ bit was checked by thestig@chromium.org

[commit-bot: I haz the power, 2014-08-11 23:40:02]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/thestig@chromium.org/461773002/1

[commit-bot: I haz the power, 2014-08-12 14:29:20]

Change committed as 288953