[Bindings] Expose CachedAccessorCallback

Expose CachedAccessorCallback in generated code

Makes these callbacks visible from other .cpp files.


BUG=617892
Review-Url: https://codereview.chromium.org/2617933002
Cr-Commit-Position: refs/heads/master@{#441888}
Committed: https://chromium.googlesource.com/chromium/src/+/1c0d372b93e0271311c106b02e140b551530d47b

[peria, 2017-01-06 04:59:48]

Description was changed from

==========
Expose CachedAccessorCallback

BUG=None
==========

to

==========
Expose CachedAccessorCallback in generated code

Makes these callbacks visible from other .cpp files.


BUG=None
==========

[peria, 2017-01-06 05:00:17]

peria@chromium.org changed reviewers:
+ bashi@chromium.org, haraken@chromium.org, yukishiino@chromium.org

[peria, 2017-01-06 05:00:18]

PTL

[peria, 2017-01-06 05:01:52]

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

[commit-bot: I haz the power, 2017-01-06 05:02:04]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[haraken, 2017-01-06 05:08:27]

LGTM

[peria, 2017-01-06 05:08:43]

The CQ bit was unchecked by peria@chromium.org

[peria, 2017-01-06 05:08:44]

The CQ bit was checked by peria@chromium.org

[commit-bot: I haz the power, 2017-01-06 05:08:50]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[bashi, 2017-01-06 05:17:08]

lgtm, but could you elaborate why do you want to expose it? If this is needed
for implementing something, adding an appropriate BUG= would be great :)

[peria, 2017-01-06 05:18:11]

The CQ bit was unchecked by peria@chromium.org

[peria, 2017-01-06 05:18:53]

Description was changed from

==========
Expose CachedAccessorCallback in generated code

Makes these callbacks visible from other .cpp files.


BUG=None
==========

to

==========
Expose CachedAccessorCallback in generated code

Makes these callbacks visible from other .cpp files.


BUG=617892
==========

[peria, 2017-01-06 05:20:49]

On 2017/01/06 05:17:08, bashi wrote:
> lgtm, but could you elaborate why do you want to expose it? If this is needed
> for implementing something, adding an appropriate BUG= would be great :)

Done.
Yes, I need to put a cached accessor callback pointer in the external reference
list
for the snapshotting project.  I update BUG= to figure it.

[peria, 2017-01-06 05:20:54]

The CQ bit was checked by peria@chromium.org

[commit-bot: I haz the power, 2017-01-06 05:21:09]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Yuki, 2017-01-06 05:56:45]

lgtm

[commit-bot: I haz the power, 2017-01-06 06:20:59]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1483680054351770, "parent_rev":
"82a51d454bb51140c5c32b14d30361bbf5069d2d", "commit_rev":
"1c0d372b93e0271311c106b02e140b551530d47b"}

[commit-bot: I haz the power, 2017-01-06 06:21:27]

Description was changed from

==========
Expose CachedAccessorCallback in generated code

Makes these callbacks visible from other .cpp files.


BUG=617892
==========

to

==========
Expose CachedAccessorCallback in generated code

Makes these callbacks visible from other .cpp files.


BUG=617892

Review-Url: https://codereview.chromium.org/2617933002
Cr-Commit-Position: refs/heads/master@{#441888}
Committed:
https://chromium.googlesource.com/chromium/src/+/1c0d372b93e0271311c106b02e14...
==========

[commit-bot: I haz the power, 2017-01-06 06:21:31]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/1c0d372b93e0271311c106b02e14...