Roll the DEPS for google-url.

Roll the DEPS for google-url.

And update all the references from build/temp_gyp/googleurl.gyp to url/url.gyp.

This also changes googleurl.gyp to reference the files under url/, so that we
don't break the Blink tree.

BUG=229660
R=brettw@chromium.org
TBR=brettw@chromium.org

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

[tfarina, 2013-04-12 00:07:27]

Let's see what the trybots has to say about this.

[tfarina, 2013-04-12 01:49:17]

Brett, looks like with patch set 5 we are good and read to go.

Please, could you review patch set 5?

Thanks,

[tfarina, 2013-04-12 01:49:34]

On 2013/04/12 01:49:17, tfarina wrote:
> Brett, looks like with patch set 5 we are good and read to go.
> 
s/read/ready

[brettw, 2013-04-12 04:37:43]

lgtm

https://codereview.chromium.org/14089011/diff/15001/chrome/chrome_tests.gypi
File chrome/chrome_tests.gypi (right):

https://codereview.chromium.org/14089011/diff/15001/chrome/chrome_tests.gypi#...
chrome/chrome_tests.gypi:3102: '../url/url.gyp:googleurl_unittests',
This one isn't in order.

https://codereview.chromium.org/14089011/diff/15001/url/url.gyp
File url/url.gyp (right):

https://codereview.chromium.org/14089011/diff/15001/url/url.gyp#newcode75
url/url.gyp:75: 'target_name': 'googleurl_unittests',
url_unittests made more sense here, is this change required?

[tfarina, 2013-04-12 14:07:12]

https://codereview.chromium.org/14089011/diff/15001/chrome/chrome_tests.gypi
File chrome/chrome_tests.gypi (right):

https://codereview.chromium.org/14089011/diff/15001/chrome/chrome_tests.gypi#...
chrome/chrome_tests.gypi:3102: '../url/url.gyp:googleurl_unittests',
On 2013/04/12 04:37:44, brettw wrote:
> This one isn't in order.

Done.

https://codereview.chromium.org/14089011/diff/15001/url/url.gyp
File url/url.gyp (right):

https://codereview.chromium.org/14089011/diff/15001/url/url.gyp#newcode75
url/url.gyp:75: 'target_name': 'googleurl_unittests',
On 2013/04/12 04:37:44, brettw wrote:
> url_unittests made more sense here, is this change required?
yes, the bots are building googleurl_unittests yet:
https://chromium.googlesource.com/chromium/tools/build/

[commit-bot: I haz the power, 2013-04-12 14:30:41]

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

[commit-bot: I haz the power, 2013-04-12 14:30:59]

Presubmit check for 14089011-25001 failed and returned exit status 1.

INFO:root:Found 39 file(s).

Running presubmit commit checks ...
Running /b/commit-queue/workdir/chromium/PRESUBMIT.py
Running /b/commit-queue/workdir/chromium/ash/PRESUBMIT.py
Running /b/commit-queue/workdir/chromium/chrome/PRESUBMIT.py
Running /b/commit-queue/workdir/chromium/jingle/PRESUBMIT.py
Running /b/commit-queue/workdir/chromium/net/PRESUBMIT.py
Running /b/commit-queue/workdir/chromium/ppapi/PRESUBMIT.py
Running /b/commit-queue/workdir/chromium/sync/PRESUBMIT.py

** Presubmit ERRORS **
Missing LGTM from an OWNER for these files:
    ui/views/controls/webview/webview.gyp
    media/media.gyp
    components/autofill.gypi
    ui/oak/oak.gyp
    cloud_print/service/service.gyp
    net/net.gyp
    chrome_frame/chrome_frame.gyp
    ui/ui_unittests.gypi
    ui/ui.gyp
    components/visitedlink.gypi
    rlz/rlz.gyp
    sync/sync_internal_api.gypi
    printing/printing.gyp
    jingle/jingle.gyp
    sync/sync_core.gypi
    ui/message_center/message_center.gyp
    ui/views/views.gyp
    win8/metro_driver/metro_driver.gyp
    net/tools/tld_cleanup/tld_cleanup.gyp
    ash/ash.gyp

Presubmit checks took 4.6s to calculate.

Was the presubmit check useful? Please send feedback & hate mail to
maruel@chromium.org!

[commit-bot: I haz the power, 2013-04-12 14:32:05]

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

[commit-bot: I haz the power, 2013-04-12 15:10:31]

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

[commit-bot: I haz the power, 2013-04-12 15:52:48]

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

[commit-bot: I haz the power, 2013-04-12 16:30:52]

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

[commit-bot: I haz the power, 2013-04-12 16:48:21]

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

[commit-bot: I haz the power, 2013-04-12 17:30:25]

Retried try job too often on ios_dbg_simulator for step(s) base_unittests,
compile, content_unittests, crypto_unittests, googleurl_unittests,
media_unittests, net_unittests, sql_unittests, ui_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=ios_dbg_si...

[commit-bot: I haz the power, 2013-04-12 17:31:49]

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

[commit-bot: I haz the power, 2013-04-12 18:10:07]

Retried try job too often on ios_dbg_simulator for step(s) base_unittests,
compile, content_unittests, crypto_unittests, googleurl_unittests,
media_unittests, net_unittests, sql_unittests, ui_unittests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=ios_dbg_si...

[commit-bot: I haz the power, 2013-04-12 18:19:19]

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

[commit-bot: I haz the power, 2013-04-12 18:29:56]

Change committed as 193968