headless: Ensure all commands have parameters and return types

headless: Ensure all commands have parameters and return types

When generating C++ bindings, ensure all commands have a method which
takes a full parameters object and returns a return value object. This
ensures the API for experimental commands is retained even if that
command is made stable.

BUG=640919
Committed: https://crrev.com/6b9268d993ae9fb5bcb2ec21082ac5fe4ec82dc0
Cr-Commit-Position: refs/heads/master@{#414431}

[Sami, 2016-08-25 13:24:25]

Description was changed from

==========
headless: Ensure all commands have parameters and return types

When generating C++ bindings, ensure all commands have a method which
takes a full parameters object and returns a return value object. This
ensures the API for experimental commands is retained even if that
command is made stable.

BUG=640919
==========

to

==========
headless: Ensure all commands have parameters and return types

When generating C++ bindings, ensure all commands have a method which
takes a full parameters object and returns a return value object. This
ensures the API for experimental commands is retained even if that
command is made stable.

BUG=640919
==========

[Sami, 2016-08-25 13:24:25]

skyostil@chromium.org changed reviewers:
+ altimin@chromium.org

[altimin, 2016-08-25 13:36:25]

LGTM, great!

Do you want to fix the problem with experimental params for a stable command
being available without going through GetExperimental()?

[Sami, 2016-08-25 13:39:57]

On 2016/08/25 13:36:25, altimin wrote:
> LGTM, great!
> 
> Do you want to fix the problem with experimental params for a stable command
> being available without going through GetExperimental()?

Yeah, that would be great to fix (separately). I'm not completely sure how to do
that yet -- especially if that param is later made stable. Let's figure out a
way.

[Sami, 2016-08-25 13:40:00]

The CQ bit was checked by skyostil@chromium.org

[commit-bot: I haz the power, 2016-08-25 13:40:18]

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-08-25 14:31:02]

Description was changed from

==========
headless: Ensure all commands have parameters and return types

When generating C++ bindings, ensure all commands have a method which
takes a full parameters object and returns a return value object. This
ensures the API for experimental commands is retained even if that
command is made stable.

BUG=640919
==========

to

==========
headless: Ensure all commands have parameters and return types

When generating C++ bindings, ensure all commands have a method which
takes a full parameters object and returns a return value object. This
ensures the API for experimental commands is retained even if that
command is made stable.

BUG=640919
==========

[commit-bot: I haz the power, 2016-08-25 14:31:04]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-25 14:32:38]

Description was changed from

==========
headless: Ensure all commands have parameters and return types

When generating C++ bindings, ensure all commands have a method which
takes a full parameters object and returns a return value object. This
ensures the API for experimental commands is retained even if that
command is made stable.

BUG=640919
==========

to

==========
headless: Ensure all commands have parameters and return types

When generating C++ bindings, ensure all commands have a method which
takes a full parameters object and returns a return value object. This
ensures the API for experimental commands is retained even if that
command is made stable.

BUG=640919

Committed: https://crrev.com/6b9268d993ae9fb5bcb2ec21082ac5fe4ec82dc0
Cr-Commit-Position: refs/heads/master@{#414431}
==========

[commit-bot: I haz the power, 2016-08-25 14:32:39]

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