Add in product messaging for IT Admins on chrome:policy

Add in product messaging for IT Admins on chrome:policy.

BUG=652356
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[Georges Khalil, 2016-10-03 17:24:38]

Description was changed from

==========
Add link to chrome for work in about:policy.

BUG=
==========

to

==========
Add link to chrome for work in about:policy.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Georges Khalil, 2016-10-03 17:25:24]

Description was changed from

==========
Add link to chrome for work in about:policy.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Add link to chrome for work in about:policy.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Georges Khalil, 2016-10-03 17:27:24]

Patchset #1 (id:1) has been deleted

[Georges Khalil, 2016-10-03 19:23:48]

Description was changed from

==========
Add link to chrome for work in about:policy.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Add in product messaging for IT Admins on chrome:policy.

BUG=652356
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Georges Khalil, 2016-10-03 19:29:56]

georgesak@chromium.org changed reviewers:
+ caitkp@chromium.org, dbeam@chromium.org

[Georges Khalil, 2016-10-03 19:29:57]

caitkp@chromium.org: Please review changes in components/policy_strings.grdp

dbeam@chromium.org: Please review changes in chrome/browser/resources/*

[Dan Beam, 2016-10-03 21:56:56]

lgtm

https://codereview.chromium.org/2386243002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/policy.css (right):

https://codereview.chromium.org/2386243002/diff/20001/chrome/browser/resource...
chrome/browser/resources/policy.css:82: padding-left: 25px;
-webkit-padding-start: 25px;

so it magically flips in RTL

https://codereview.chromium.org/2386243002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/policy.html (right):

https://codereview.chromium.org/2386243002/diff/20001/chrome/browser/resource...
chrome/browser/resources/policy.html:40: <span
i18n-content="chromeWorkLink"></span></a>
indent off

https://codereview.chromium.org/2386243002/diff/20001/components/policy_strin...
File components/policy_strings.grdp (right):

https://codereview.chromium.org/2386243002/diff/20001/components/policy_strin...
components/policy_strings.grdp:237: <message name="IDS_POLICY_CHROME_WORK_LINK"
desc="Title of the link to chrome for work website.">
nit: CHROME_AT_WORK OR CHROME_FOR_WORK (and similarly in other message IDs)

[Cait (Slow), 2016-10-04 13:36:18]

lgtm

[Georges Khalil, 2016-10-04 13:36:53]

Thanks for the reviews. Sending to CQ.

https://codereview.chromium.org/2386243002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/policy.css (right):

https://codereview.chromium.org/2386243002/diff/20001/chrome/browser/resource...
chrome/browser/resources/policy.css:82: padding-left: 25px;
On 2016/10/03 21:56:55, Dan Beam wrote:
> -webkit-padding-start: 25px;
> 
> so it magically flips in RTL

Done.

https://codereview.chromium.org/2386243002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/policy.html (right):

https://codereview.chromium.org/2386243002/diff/20001/chrome/browser/resource...
chrome/browser/resources/policy.html:40: <span
i18n-content="chromeWorkLink"></span></a>
On 2016/10/03 21:56:55, Dan Beam wrote:
> indent off

Done.

[Georges Khalil, 2016-10-04 13:36:57]

The CQ bit was checked by georgesak@chromium.org

[Georges Khalil, 2016-10-04 13:36:58]

The patchset sent to the CQ was uploaded after l-g-t-m from dbeam@chromium.org
Link to the patchset: https://codereview.chromium.org/2386243002/#ps40001
(title: "dbeam@ comments.")

[commit-bot: I haz the power, 2016-10-04 13:37:16]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-04 14:41:28]

Description was changed from

==========
Add in product messaging for IT Admins on chrome:policy.

BUG=652356
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Add in product messaging for IT Admins on chrome:policy.

BUG=652356
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[commit-bot: I haz the power, 2016-10-04 14:41:29]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2016-10-04 14:44:39]

Description was changed from

==========
Add in product messaging for IT Admins on chrome:policy.

BUG=652356
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Add in product messaging for IT Admins on chrome:policy.

BUG=652356
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[commit-bot: I haz the power, 2016-10-04 14:44:40]

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