Document deprecated plugin policies as such.

Document deprecated plugin policies as such.

EnabledPlugins, DisabledPlugins and DisabledPluginsExcpetions are
deprecated as part of the general plugins handling framework deprecation
effort.

BUG=654731
TEST=none - Documentation only change.

Committed: https://crrev.com/231648330ec3ea62f4a238024b6d2e727a16f75c
Cr-Commit-Position: refs/heads/master@{#425640}

[pastarmovj, 2016-10-14 12:24:50]

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

[pastarmovj, 2016-10-14 12:24:51]

Please review this cl at your convenience.
Thanks!

[Thiemo Nagel, 2016-10-14 12:28:19]

https://codereview.chromium.org/2419043002/diff/1/components/policy/resources...
File components/policy/resources/policy_templates.json (right):

https://codereview.chromium.org/2419043002/diff/1/components/policy/resources...
components/policy/resources/policy_templates.json:1347: 'desc': '''This policy
is deprecated. Please use the DefaultPluginsSetting to control the avalability
of the Flash plugin and AlwaysOpenPdfExternally to control whether the
integrated PDF viewer should be used for opening PDF files.
Please use <ph name ...> to mark policy names as untranslateable.

[pastarmovj, 2016-10-14 12:45:49]

https://codereview.chromium.org/2419043002/diff/1/components/policy/resources...
File components/policy/resources/policy_templates.json (right):

https://codereview.chromium.org/2419043002/diff/1/components/policy/resources...
components/policy/resources/policy_templates.json:1347: 'desc': '''This policy
is deprecated. Please use the DefaultPluginsSetting to control the avalability
of the Flash plugin and AlwaysOpenPdfExternally to control whether the
integrated PDF viewer should be used for opening PDF files.
On 2016/10/14 12:28:19, Thiemo Nagel (slow) wrote:
> Please use <ph name ...> to mark policy names as untranslateable.

Done, although this is almost the only place we are doing this.

I think we should do it globally but not as part of this CL :)

[Thiemo Nagel, 2016-10-14 14:22:53]

Lgtm

https://codereview.chromium.org/2419043002/diff/1/components/policy/resources...
File components/policy/resources/policy_templates.json (right):

https://codereview.chromium.org/2419043002/diff/1/components/policy/resources...
components/policy/resources/policy_templates.json:1347: 'desc': '''This policy
is deprecated. Please use the DefaultPluginsSetting to control the avalability
of the Flash plugin and AlwaysOpenPdfExternally to control whether the
integrated PDF viewer should be used for opening PDF files.
> Done, although this is almost the only place we are doing this.

Yes, I know. Translators have asked for it though...

> I think we should do it globally but not as part of this CL :)

Full ack.

Nit: PLUGIN_SETTING is missing s after plugin.  Also the other users of that
syntax seem to append _POLICY.

[pastarmovj, 2016-10-17 08:39:14]

Done. Thanks!

[pastarmovj, 2016-10-17 08:39:22]

The CQ bit was checked by pastarmovj@chromium.org

[pastarmovj, 2016-10-17 08:39:23]

The patchset sent to the CQ was uploaded after l-g-t-m from tnagel@chromium.org
Link to the patchset: https://codereview.chromium.org/2419043002/#ps40001
(title: "Rename ph-s.")

[commit-bot: I haz the power, 2016-10-17 08:39:40]

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 09:24:12]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-17 09:26:38]

Description was changed from

==========
Document deprecated plugin policies as such.

EnabledPlugins, DisabledPlugins and DisabledPluginsExcpetions are
deprecated as part of the general plugins handling framework deprecation
effort.

BUG=654731
TEST=none - Documentation only change.
==========

to

==========
Document deprecated plugin policies as such.

EnabledPlugins, DisabledPlugins and DisabledPluginsExcpetions are
deprecated as part of the general plugins handling framework deprecation
effort.

BUG=654731
TEST=none - Documentation only change.

Committed: https://crrev.com/231648330ec3ea62f4a238024b6d2e727a16f75c
Cr-Commit-Position: refs/heads/master@{#425640}
==========

[commit-bot: I haz the power, 2016-10-17 09:26:39]

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