bindings: Pass dictionaries to callback methods by const reference

bindings: Pass dictionaries to callback methods by const reference

Before this CL, strings and container types were passed to callback
method by const reference. Dictionary types should also be passed
by const ref to avoid unnecessary copying.

BUG=557590
Committed: https://crrev.com/02b9d4670fc8e06145343d19ea3e2ea28c7069f8
Cr-Commit-Position: refs/heads/master@{#360549}

[shalamov, 2015-11-18 11:15:45]

Description was changed from

==========
bindings: Pass dictionaries to callback methods by const reference

Before this CL, strings and container types were passed to callback
method by const reference. Dictionary types should also be passed
by const ref to avoid unnecessary copying.

BUG=557590
==========

to

==========
bindings: Pass dictionaries to callback methods by const reference

Before this CL, strings and container types were passed to callback
method by const reference. Dictionary types should also be passed
by const ref to avoid unnecessary copying.

BUG=557590
==========

[shalamov, 2015-11-18 11:15:45]

alexander.shalamov@intel.com changed reviewers:
+ haraken@chromium.org

[shalamov, 2015-11-18 11:21:44]

I'm implementing WebNFC API (crrev.com/1444303002) and I found out that V8
binding generator passes dictionaries to callback methods by values.

PTAL

[haraken, 2015-11-18 11:32:06]

On 2015/11/18 11:21:44, shalamov wrote:
> I'm implementing WebNFC API (crrev.com/1444303002) and I found out that V8
> binding generator passes dictionaries to callback methods by values.
> 
> PTAL

LGTM

[shalamov, 2015-11-18 11:38:35]

The CQ bit was checked by alexander.shalamov@intel.com

[commit-bot: I haz the power, 2015-11-18 11:39:07]

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

[commit-bot: I haz the power, 2015-11-18 12:26:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-11-18 12:26:48]

Try jobs failed on following builders:
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[shalamov, 2015-11-19 09:46:35]

The CQ bit was unchecked by alexander.shalamov@intel.com

[shalamov, 2015-11-19 09:46:39]

The CQ bit was checked by alexander.shalamov@intel.com

[commit-bot: I haz the power, 2015-11-19 09:47:32]

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

[commit-bot: I haz the power, 2015-11-19 10:37:54]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-11-19 10:39:16]

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