[Extensions Bindings] Add a bindings/ subdirectory under renderer

[Extensions Bindings] Add a bindings/ subdirectory under renderer

The core bindings system has grown (55 files and 13,000 LOC). Move it to
a subdirectory at extensions/renderer/bindings. This also separates out
the core bindings system (which is actually extensions-agnostic) from
the native extensions binding implementation. Add a DEPS file to enforce
this distinction.

BUG=653596
Review-Url: https://codereview.chromium.org/2947463002
Cr-Commit-Position: refs/heads/master@{#482024}
Committed: https://chromium.googlesource.com/chromium/src/+/d4bcb1e2cf549030d85e0503c0d313f1c70be9cd

[Devlin, 2017-06-16 20:28:53]

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

[commit-bot: I haz the power, 2017-06-16 20:29:20]

Dry run: CQ is trying da patch.

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

[Devlin, 2017-06-16 22:23:33]

Description was changed from

==========
moveitall

BUG=
==========

to

==========
[Extensions Bindings] Add a bindings/ subdirectory under renderer

The core bindings system has grown (55 files and 13,000 LOC). Move it to
a subdirectory at extensions/renderer/bindings. This also separates out
the core bindings system (which is actually extensions-agnostic) from
the native extensions binding implementation. Add a DEPS file to enforce
this distinction.

BUG=653596
==========

[Devlin, 2017-06-16 22:25:11]

rdevlin.cronin@chromium.org changed reviewers:
+ jbroman@chromium.org, lazyboy@chromium.org

[Devlin, 2017-06-16 22:25:13]

Hey folks, mind taking a look?

This is purely moving files with the exception of the new DEPS and OWNERS files.

In a separate CL, I'll see about updating the documentation to mention the
distinction between the extensions-agnostic core bindings system and the
extension-y implementation.

https://codereview.chromium.org/2947463002/diff/20001/extensions/renderer/bin...
File extensions/renderer/bindings/OWNERS (right):

https://codereview.chromium.org/2947463002/diff/20001/extensions/renderer/bin...
extensions/renderer/bindings/OWNERS:3: jbroman@chromium.org
jbroman@, do you want to be an OWNER here?

[commit-bot: I haz the power, 2017-06-16 22:28:57]

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

[commit-bot: I haz the power, 2017-06-16 22:28:58]

Dry run: This issue passed the CQ dry run.

[jbroman, 2017-06-20 19:55:41]

lgtm

https://codereview.chromium.org/2947463002/diff/20001/extensions/renderer/bin...
File extensions/renderer/bindings/OWNERS (right):

https://codereview.chromium.org/2947463002/diff/20001/extensions/renderer/bin...
extensions/renderer/bindings/OWNERS:3: jbroman@chromium.org
On 2017/06/16 at 22:25:13, Devlin wrote:
> jbroman@, do you want to be an OWNER here?

My main concern with that is that I don't work on extensions enough to have much
background of how the rest of the system works, so we'd probably want to have
you or lazyboy as a reviewer regardless. But I don't feel strongly.

[lazyboy, 2017-06-23 07:33:28]

Thanks, lgtm.

[Devlin, 2017-06-23 17:55:55]

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

[commit-bot: I haz the power, 2017-06-23 17:56:09]

Dry run: 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, 2017-06-23 18:22:16]

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

[commit-bot: I haz the power, 2017-06-23 18:22:18]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Devlin, 2017-06-23 19:28:01]

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

[commit-bot: I haz the power, 2017-06-23 19:28:18]

Dry run: CQ is trying da patch.

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

[Devlin, 2017-06-23 20:52:54]

https://codereview.chromium.org/2947463002/diff/20001/extensions/renderer/bin...
File extensions/renderer/bindings/OWNERS (right):

https://codereview.chromium.org/2947463002/diff/20001/extensions/renderer/bin...
extensions/renderer/bindings/OWNERS:3: jbroman@chromium.org
On 2017/06/20 19:55:41, jbroman wrote:
> On 2017/06/16 at 22:25:13, Devlin wrote:
> > jbroman@, do you want to be an OWNER here?
> 
> My main concern with that is that I don't work on extensions enough to have
much
> background of how the rest of the system works, so we'd probably want to have
> you or lazyboy as a reviewer regardless. But I don't feel strongly.

Okay, I'm fine with it just being lazyboy@ and myself for now.  If you ever
change your mind... :)

[Devlin, 2017-06-23 20:53:02]

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

[Devlin, 2017-06-23 20:53:08]

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

[Devlin, 2017-06-23 20:53:09]

The patchset sent to the CQ was uploaded after l-g-t-m from
lazyboy@chromium.org, jbroman@chromium.org
Link to the patchset: https://codereview.chromium.org/2947463002/#ps80001
(title: ".")

[commit-bot: I haz the power, 2017-06-23 20:53:35]

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, 2017-06-23 21:01:06]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1498251188993540,
"parent_rev": "55e4d9a48a1965626872aabca38484814f0d063a", "commit_rev":
"d4bcb1e2cf549030d85e0503c0d313f1c70be9cd"}

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

Description was changed from

==========
[Extensions Bindings] Add a bindings/ subdirectory under renderer

The core bindings system has grown (55 files and 13,000 LOC). Move it to
a subdirectory at extensions/renderer/bindings. This also separates out
the core bindings system (which is actually extensions-agnostic) from
the native extensions binding implementation. Add a DEPS file to enforce
this distinction.

BUG=653596
==========

to

==========
[Extensions Bindings] Add a bindings/ subdirectory under renderer

The core bindings system has grown (55 files and 13,000 LOC). Move it to
a subdirectory at extensions/renderer/bindings. This also separates out
the core bindings system (which is actually extensions-agnostic) from
the native extensions binding implementation. Add a DEPS file to enforce
this distinction.

BUG=653596

Review-Url: https://codereview.chromium.org/2947463002
Cr-Commit-Position: refs/heads/master@{#482024}
Committed:
https://chromium.googlesource.com/chromium/src/+/d4bcb1e2cf549030d85e0503c0d3...
==========

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

Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/d4bcb1e2cf549030d85e0503c0d3...