Sync more DMServer proto comments to the client.

Sync more DMServer proto comments to the client.

Also replace mentionings of the deprecated platformKeysPrivate by
platformKeys.  Comments only, no functional changes.

BUG=645078
Committed: https://crrev.com/9a1580424d6de98575f9015f3ef3585a12c8d2dc
Cr-Commit-Position: refs/heads/master@{#426523}

[Thiemo Nagel, 2016-10-17 18:29:35]

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

[Thiemo Nagel, 2016-10-17 18:30:05]

tnagel@chromium.org changed reviewers:
+ peletskyi@chromium.org

[Thiemo Nagel, 2016-10-17 18:30:06]

Hi Sasha,

could you please take a look?

Thank you,
Thiemo

[commit-bot: I haz the power, 2016-10-17 18:30:07]

Dry run: 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-17 19:20:59]

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

[commit-bot: I haz the power, 2016-10-17 19:21:00]

Dry run: This issue passed the CQ dry run.

[Thiemo Nagel, 2016-10-18 15:20:59]

tnagel@chromium.org changed reviewers:
+ dkalin@chromium.org

[Thiemo Nagel, 2016-10-18 15:21:00]

Hi Dennis,

could you please take a look at the platformKeys changes to
policy_templates.json?

Thank you,
Thiemo

[Thiemo Nagel, 2016-10-18 15:22:34]

Description was changed from

==========
Sync more DMServer proto comments to the client.

Comments only, no functional changes.

BUG=645078
==========

to

==========
Sync more DMServer proto comments to the client.

Also replace mentionings of the deprecated platformKeysPrivate by platformKeys. 
Comments only, no functional changes.

BUG=645078
==========

[Thiemo Nagel, 2016-10-18 15:22:47]

Description was changed from

==========
Sync more DMServer proto comments to the client.

Also replace mentionings of the deprecated platformKeysPrivate by platformKeys. 
Comments only, no functional changes.

BUG=645078
==========

to

==========
Sync more DMServer proto comments to the client.

Also replace mentionings of the deprecated platformKeysPrivate by
platformKeys.  Comments only, no functional changes.

BUG=645078
==========

[peletskyi, 2016-10-18 17:08:21]

On 2016/10/18 15:21:00, Thiemo Nagel (slow) wrote:
> Hi Dennis,
> 
> could you please take a look at the platformKeys changes to
> policy_templates.json?
> 
> Thank you,
> Thiemo

*.proto LGTM (If Dennis confirms "app:" prefix)

[peletskyi, 2016-10-19 09:29:04]

https://codereview.chromium.org/2424003003/diff/20001/chrome/browser/chromeos...
File chrome/browser/chromeos/policy/proto/chrome_device_policy.proto (right):

https://codereview.chromium.org/2424003003/diff/20001/chrome/browser/chromeos...
chrome/browser/chromeos/policy/proto/chrome_device_policy.proto:296: // Example
of the package name: "com.android.camera". Do not include "app:"
Dennis, what about "app:" prefix? I saw there are discussions about it.

[dkalin, 2016-10-20 16:19:14]

On 2016/10/18 15:21:00, Thiemo Nagel (slow) wrote:
> Hi Dennis,
> 
> could you please take a look at the platformKeys changes to
> policy_templates.json?
> 
> Thank you,
> Thiemo

LG

[dkalin, 2016-10-20 16:20:59]

dkalin@google.com changed reviewers:
+ dkalin@google.com

[dkalin, 2016-10-20 16:20:59]

lgtm

https://codereview.chromium.org/2424003003/diff/20001/chrome/browser/chromeos...
File chrome/browser/chromeos/policy/proto/chrome_device_policy.proto (right):

https://codereview.chromium.org/2424003003/diff/20001/chrome/browser/chromeos...
chrome/browser/chromeos/policy/proto/chrome_device_policy.proto:296: // Example
of the package name: "com.android.camera". Do not include "app:"
On 2016/10/19 09:29:03, peletskyi wrote:
> Dennis, what about "app:" prefix? I saw there are discussions about it.

This CL LGTM. App: prefix should not be included. We were just discussing which
component strips out this prefix.

[Thiemo Nagel, 2016-10-20 16:38:02]

The CQ bit was checked by tnagel@chromium.org

[commit-bot: I haz the power, 2016-10-20 16:38:33]

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-20 17:46:25]

Description was changed from

==========
Sync more DMServer proto comments to the client.

Also replace mentionings of the deprecated platformKeysPrivate by
platformKeys.  Comments only, no functional changes.

BUG=645078
==========

to

==========
Sync more DMServer proto comments to the client.

Also replace mentionings of the deprecated platformKeysPrivate by
platformKeys.  Comments only, no functional changes.

BUG=645078
==========

[commit-bot: I haz the power, 2016-10-20 17:46:26]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-21 13:19:53]

Description was changed from

==========
Sync more DMServer proto comments to the client.

Also replace mentionings of the deprecated platformKeysPrivate by
platformKeys.  Comments only, no functional changes.

BUG=645078
==========

to

==========
Sync more DMServer proto comments to the client.

Also replace mentionings of the deprecated platformKeysPrivate by
platformKeys.  Comments only, no functional changes.

BUG=645078

Committed: https://crrev.com/9a1580424d6de98575f9015f3ef3585a12c8d2dc
Cr-Commit-Position: refs/heads/master@{#426523}
==========

[commit-bot: I haz the power, 2016-10-21 13:19:54]

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