Use .rc strings in fewer places for installer util.

Use .rc strings in fewer places for installer util.

The resource strings are needed only for the setup app and unit tests. Previously they were also linked into Chrome.

This clarifies the usage and renames the targets to force you to pick (previously the naming encouraged you to link to the strings, which is probably not correct for most cases.

Committed: https://crrev.com/e26e6d0a79391734160024bba2955353760c5020
Cr-Commit-Position: refs/heads/master@{#367057}

[brettw, 2015-12-22 22:28:57]

Description was changed from

==========
Use .rc strings in fewer places for installer util.

The resource strings are needed only for the setup app and unit tests.
Previuosly they were also linked into Chrome.

This clarifies the usage and renames the targets to force you to pick
(previously the naming encouraged you to link to the strings, which is probably
no correct for most cases.
==========

to

==========
Use .rc strings in fewer places for installer util.

The resource strings are needed only for the setup app and unit tests.
Previously they were also linked into Chrome.

This clarifies the usage and renames the targets to force you to pick
(previously the naming encouraged you to link to the strings, which is probably
no correct for most cases.
==========

[brettw, 2015-12-22 22:28:57]

brettw@chromium.org changed reviewers:
+ dpranke@chromium.org

[brettw, 2015-12-22 22:30:52]

Follow up from https://codereview.chromium.org/1534903002/

[brettw, 2015-12-22 22:30:56]

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

[commit-bot: I haz the power, 2015-12-22 22:32:50]

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

[commit-bot: I haz the power, 2015-12-22 23:07:02]

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

[commit-bot: I haz the power, 2015-12-22 23:07:02]

Dry run: This issue passed the CQ dry run.

[Dirk Pranke, 2015-12-23 03:32:26]

lgtm. I like this better than the previous version; good to know it was a
transitional state.

[brettw, 2015-12-29 06:12:16]

Description was changed from

==========
Use .rc strings in fewer places for installer util.

The resource strings are needed only for the setup app and unit tests.
Previously they were also linked into Chrome.

This clarifies the usage and renames the targets to force you to pick
(previously the naming encouraged you to link to the strings, which is probably
no correct for most cases.
==========

to

==========
Use .rc strings in fewer places for installer util.

The resource strings are needed only for the setup app and unit tests.
Previously they were also linked into Chrome.

This clarifies the usage and renames the targets to force you to pick
(previously the naming encouraged you to link to the strings, which is probably
not correct for most cases.
==========

[brettw, 2015-12-29 06:12:21]

The CQ bit was checked by brettw@chromium.org

[commit-bot: I haz the power, 2015-12-29 06:12:35]

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

[commit-bot: I haz the power, 2015-12-29 06:51:57]

Description was changed from

==========
Use .rc strings in fewer places for installer util.

The resource strings are needed only for the setup app and unit tests.
Previously they were also linked into Chrome.

This clarifies the usage and renames the targets to force you to pick
(previously the naming encouraged you to link to the strings, which is probably
not correct for most cases.
==========

to

==========
Use .rc strings in fewer places for installer util.

The resource strings are needed only for the setup app and unit tests.
Previously they were also linked into Chrome.

This clarifies the usage and renames the targets to force you to pick
(previously the naming encouraged you to link to the strings, which is probably
not correct for most cases.
==========

[commit-bot: I haz the power, 2015-12-29 06:51:57]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-29 06:52:37]

Description was changed from

==========
Use .rc strings in fewer places for installer util.

The resource strings are needed only for the setup app and unit tests.
Previously they were also linked into Chrome.

This clarifies the usage and renames the targets to force you to pick
(previously the naming encouraged you to link to the strings, which is probably
not correct for most cases.
==========

to

==========
Use .rc strings in fewer places for installer util.

The resource strings are needed only for the setup app and unit tests.
Previously they were also linked into Chrome.

This clarifies the usage and renames the targets to force you to pick
(previously the naming encouraged you to link to the strings, which is probably
not correct for most cases.

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

[commit-bot: I haz the power, 2015-12-29 06:52:38]

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