[Extensions] Clean and update runtime bindings

[Extensions] Clean and update runtime bindings

BUG=591164
BUG=609286
Committed: https://crrev.com/c7ce4fc66d9d3a2811306edbd9e69f2355efb62c
Cr-Commit-Position: refs/heads/master@{#392512}

[Devlin, 2016-05-08 16:33:29]

Patchset #1 (id:1) has been deleted

[Devlin, 2016-05-08 16:33:39]

Patchset #1 (id:20001) has been deleted

[Devlin, 2016-05-08 16:33:45]

Patchset #1 (id:40001) has been deleted

[Devlin, 2016-05-08 16:33:53]

Patchset #1 (id:60001) has been deleted

[Devlin, 2016-05-08 16:33:59]

Patchset #1 (id:80001) has been deleted

[Devlin, 2016-05-08 16:36:01]

Description was changed from

==========
Inprog
BUG=
==========

to

==========
[Extensions] Clean and update runtime bindings

BUG=591164
==========

[Devlin, 2016-05-08 17:13:34]

Description was changed from

==========
[Extensions] Clean and update runtime bindings

BUG=591164
==========

to

==========
[Extensions] Clean and update runtime bindings

BUG=591164
BUG=609286
==========

[Devlin, 2016-05-08 17:16:05]

rdevlin.cronin@chromium.org changed reviewers:
+ asargent@chromium.org, rob@robwu.nl

[robwu, 2016-05-08 18:06:50]

lgtm

https://codereview.chromium.org/1960903002/diff/100001/extensions/renderer/re...
File extensions/renderer/resources/runtime_custom_bindings.js (left):

https://codereview.chromium.org/1960903002/diff/100001/extensions/renderer/re...
extensions/renderer/resources/runtime_custom_bindings.js:152: });
Nice. Removing this unnecessary logic improves the error messages as a bonus.

"Invocation of form runtime.connect(....) doesn't match definition
runtime.connect(optional string extensionId, optional object connectInfo)"
instead of "Invalid arguments to connect"

[asargent_no_longer_on_chrome, 2016-05-09 23:20:21]

lgtm

[Devlin, 2016-05-10 00:15:34]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2016-05-10 00:16:11]

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

[commit-bot: I haz the power, 2016-05-10 01:05:18]

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

[commit-bot: I haz the power, 2016-05-10 01:05:19]

Try jobs failed on following builders:
  android_chromium_gn_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  cast_shell_linux on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_amd64-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)
  linux_chromium_chromeos_compile_dbg_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Devlin, 2016-05-10 01:16:12]

The CQ bit was checked by rdevlin.cronin@chromium.org

[commit-bot: I haz the power, 2016-05-10 01:16:32]

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

[commit-bot: I haz the power, 2016-05-10 01:41:48]

Description was changed from

==========
[Extensions] Clean and update runtime bindings

BUG=591164
BUG=609286
==========

to

==========
[Extensions] Clean and update runtime bindings

BUG=591164
BUG=609286
==========

[commit-bot: I haz the power, 2016-05-10 01:41:49]

Committed patchset #1 (id:100001)

[commit-bot: I haz the power, 2016-05-10 01:42:58]

Description was changed from

==========
[Extensions] Clean and update runtime bindings

BUG=591164
BUG=609286
==========

to

==========
[Extensions] Clean and update runtime bindings

BUG=591164
BUG=609286

Committed: https://crrev.com/c7ce4fc66d9d3a2811306edbd9e69f2355efb62c
Cr-Commit-Position: refs/heads/master@{#392512}
==========

[commit-bot: I haz the power, 2016-05-10 01:42:59]

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