webkit: Kill support/ directory.

webkit: Kill support/ directory.

- Rename webkit_support.gyp to webkit_glue.gyp
- Move it into glue/ directory.
- Remove DEPS and OWNERS file.

BUG=265753
R=darin@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=224864

[tfarina, 2013-09-20 02:52:27]

Darin, I think you can cover everything since this is mostly mechanical.

Jochen -> chrome fyi
James -> webkit fyi
John -> content fyi
Joi -> components fyi

[tfarina, 2013-09-20 02:57:33]

tkent -> webkit/support fyi

[jamesr, 2013-09-20 03:45:57]

webkit/support is not linked into production targets but webkit/glue is.
Are you sure you aren't pulling test code into the shipping binary?

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[tfarina, 2013-09-20 12:49:07]

On 2013/09/20 03:45:57, jamesr wrote:
> webkit/support is not linked into production targets but webkit/glue is.
I'm not sure what you mean by this. There is no code left in webkit/support/

> Are you sure you aren't pulling test code into the shipping binary?
> 
As far as I can see, no. We call it as webkit_glue.gyp:glue or
webkit_glue.gyp:glue_child. The only place where we do webkit_glue.gyp:* is
all.gyp, but we were doing the same with webkit_support.gyp.

[jamesr, 2013-09-20 20:33:42]

Ah right, there's no code here :).

lgtm+++++

[tfarina, 2013-09-23 16:12:48]

More focused owners for approval.

Lei -> chrome
Joi -> components

Thanks!

[Jói, 2013-09-23 16:34:43]

//components LGTM.

[Lei Zhang, 2013-09-23 18:31:34]

chrome/ lgtm

[commit-bot: I haz the power, 2013-09-23 19:56:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tfarina@chromium.org/23583055/1

[commit-bot: I haz the power, 2013-09-23 20:12:21]

Retried try job too often on chromium_presubmit for step(s) presubmit
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

[tfarina, 2013-09-23 23:06:38]

On 2013/09/23 20:12:21, I haz the power (commit-bot) wrote:
> Retried try job too often on chromium_presubmit for step(s) presubmit
>
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=chromium_p...

Ahh, this is soooO annoying!!! :(

Avi, could you rubber stamp this for content/?

Thanks!

[darin (slow to review), 2013-09-23 23:19:29]

LGTM

[commit-bot: I haz the power, 2013-09-23 23:30:21]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/tfarina@chromium.org/23583055/1

[commit-bot: I haz the power, 2013-09-24 02:54:56]

Change committed as 224864