Moved files under Source/bindings/v8 to Source/bindings/core/v8.

Moved files under Source/bindings/v8 to Source/bindings/core/v8.

- IDBBindingUtilities.* are moved to Source/bindings/modules/v8.
- other files are moved to Source/bindings/core/v8.

Because IDBBindingUtilities.* are used only by indexeddb (i.e.
Source/modules/indexeddb).

BUG=358074
TEST=no new tests, because of just refactoring.
R=haraken@chromium.org, jochen@chromium.org

Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=177274

[tasak, 2014-06-27 11:02:17]

Would you review this CL?

I moved Source/bindings/v8 files to Source/bindings/core/v8. 

However, this affects too many files to upload the patch. So I used both
--bypass-hooks and --bypass-watchlist.

So is it better to split the patch into smaller pieces?

I've not tried yet, but I'm afraid that only moving ScriptWrappable.h or
V8Bindings.h causes the same problem.

[haraken, 2014-06-27 11:11:48]

> - IDBBindingUtilities.* are moved to Source/bindings/modules/v8.
> - other files are moved to Source/bindings/core/v8.

I haven't reviewed all files, but LGTM assuming that you're doing the above two
things in this CL.

> So is it better to split the patch into smaller pieces?
>
> I've not tried yet, but I'm afraid that only moving ScriptWrappable.h or
> V8Bindings.h causes the same problem.

Yeah, splitting this CL into smaller pieces will impose a substantial amount of
burden on you to land this change. I'm OK with landing this CL in one go once
you're confident that it won't break anything.

[tasak, 2014-06-30 08:48:20]

jochen, would you review this CL?

This patch is just for moving files, i.e. no semantic changes.

[jochen (gone - plz use gerrit), 2014-07-01 07:25:53]

lgtm

[tasak, 2014-07-01 07:34:01]

On 2014/07/01 07:25:53, jochen wrote:
> lgtm

Thank you.

[tasak, 2014-07-01 07:56:34]

Committed patchset #5 manually as r177274 (tree was closed).