Added PPAPI win32k lockdown options to chrome://flags.

Added PPAPI win32k lockdown options to chrome://flags.
This patch adds the PPAPI win32k lockdown command line to the flags
page along with necessary changes to ensure tests work correctly.

BUG=523278
Committed: https://crrev.com/9745511be6490f393f829aa8acb85e66aba26cb9
Cr-Commit-Position: refs/heads/master@{#351542}

[forshaw, 2015-09-29 12:58:24]

forshaw@chromium.org changed reviewers:
+ isherman@chromium.org, jam@chromium.org

[forshaw, 2015-09-29 12:58:24]

isherman@chromium.org: Can you please review changes in the metrics component.
jam@chromium.org: Can you please review changes in chrome and content to add
win32k lockdown option to chrome://flags.

Thanks very much.

[jam, 2015-09-29 19:43:16]

On 2015/09/29 12:58:24, forshaw wrote:
> mailto:isherman@chromium.org: Can you please review changes in the metrics
component.
> mailto:jam@chromium.org: Can you please review changes in chrome and content
to add
> win32k lockdown option to chrome://flags.
> 
> Thanks very much.

rubberstamp lgtm

[forshaw, 2015-09-29 19:44:05]

On 2015/09/29 19:43:16, jam wrote:
> On 2015/09/29 12:58:24, forshaw wrote:
> > mailto:isherman@chromium.org: Can you please review changes in the metrics
> component.
> > mailto:jam@chromium.org: Can you please review changes in chrome and content
> to add
> > win32k lockdown option to chrome://flags.
> > 
> > Thanks very much.
> 
> rubberstamp lgtm

Thanks a lot jam@

[Ilya Sherman, 2015-09-30 04:13:37]

https://codereview.chromium.org/1373843004/diff/1/tools/metrics/actions/actio...
File tools/metrics/actions/actions.xml (right):

https://codereview.chromium.org/1373843004/diff/1/tools/metrics/actions/actio...
tools/metrics/actions/actions.xml:186: </action>
Hmm, I don't remember: Is this entry automatically added?  If so, LGTM.  If not,
do you really need this as both an action and in the histogram?

[forshaw, 2015-09-30 10:11:52]

On 2015/09/30 04:13:37, Ilya Sherman wrote:
>
https://codereview.chromium.org/1373843004/diff/1/tools/metrics/actions/actio...
> File tools/metrics/actions/actions.xml (right):
> 
>
https://codereview.chromium.org/1373843004/diff/1/tools/metrics/actions/actio...
> tools/metrics/actions/actions.xml:186: </action>
> Hmm, I don't remember: Is this entry automatically added?  If so, LGTM.  If
not,
> do you really need this as both an action and in the histogram?

I think you're right, it wasn't necessary in the end. So I've remove it from the
patch. Thanks very much for the review.

[forshaw, 2015-09-30 12:22:32]

The CQ bit was checked by forshaw@chromium.org

[forshaw, 2015-09-30 12:22:33]

The patchset sent to the CQ was uploaded after l-g-t-m from
isherman@chromium.org, jam@chromium.org
Link to the patchset: https://codereview.chromium.org/1373843004/#ps20001
(title: "Removed actions.xml patch.")

[commit-bot: I haz the power, 2015-09-30 12:22:57]

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

[commit-bot: I haz the power, 2015-09-30 13:29:07]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-09-30 13:29:57]

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