Add support for passing errors through PopulateArrayFromList.

Add support for passing errors through PopulateArrayFromList.

When a JSON schema includes an array of objects PopulateArrayFromList
calls a specialized PopulateItem instance for the generated object
type. When compiled with generate_error_messages=True the generated
type's Populate method expects to receive an error parameter. This
patch adds versions of the PopulateArrayFromList and PopulateItem
functions that pass through and provide that parameter.

This capability is required for a new manifest key that will include
an array of objects being added as part of issue 468955.

Committed: https://crrev.com/89a17f7839bfedeee84ae1dbc178fb13dfe03eba
Cr-Commit-Position: refs/heads/master@{#328471}

[Reilly Grant (use Gerrit), 2015-05-05 22:17:24]

reillyg@chromium.org changed reviewers:
+ rockot@chromium.org

[Reilly Grant (use Gerrit), 2015-05-05 22:17:24]

Please take a look.

[Ken Rockot(use gerrit already), 2015-05-05 22:24:31]

lgtm but here is a proposal:

could you make the error-string-aware implementations safely handle nullptrs,
and then change the non-error-string-aware implementations to just call their
error-string-aware counterparts with a nullptr?

[Reilly Grant (use Gerrit), 2015-05-05 22:46:32]

On 2015/05/05 22:24:31, Ken Rockot wrote:
> lgtm but here is a proposal:
> 
> could you make the error-string-aware implementations safely handle nullptrs,
> and then change the non-error-string-aware implementations to just call their
> error-string-aware counterparts with a nullptr?

Yes, but it's a pretty big code-generator change and this patch won't actually
increase code size because these new functions are unused until a schema that
needs them is added.

[Reilly Grant (use Gerrit), 2015-05-05 22:46:40]

The CQ bit was checked by reillyg@chromium.org

[commit-bot: I haz the power, 2015-05-05 22:49:38]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1128523003/1

[Reilly Grant (use Gerrit), 2015-05-05 23:37:39]

The CQ bit was unchecked by reillyg@chromium.org

[Reilly Grant (use Gerrit), 2015-05-06 01:28:16]

The CQ bit was checked by reillyg@chromium.org

[Reilly Grant (use Gerrit), 2015-05-06 01:28:16]

The patchset sent to the CQ was uploaded after l-g-t-m from rockot@chromium.org
Link to the patchset: https://codereview.chromium.org/1128523003/#ps20001
(title: "Updated JSON schema tests.")

[commit-bot: I haz the power, 2015-05-06 01:28:30]

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

[commit-bot: I haz the power, 2015-05-06 02:34:42]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-06 02:35:39]

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