Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(378)

Issue 14028012: Revert 193968 "Roll the DEPS for google-url." (Closed)

Created:
7 years, 8 months ago by michaeln
Modified:
7 years, 8 months ago
Reviewers:
tfarina, brettw, jam
CC:
chromium-reviews
Visibility:
Public.

Description

Revert 193968 "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 > > Review URL: https://chromiumcodereview.appspot.com/14089011 TBR=tfarina@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=193983

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+181 lines, -93 lines) Patch
M trunk/src/DEPS View 1 chunk +1 line, -1 line 0 comments Download
MM trunk/src/ash/ash.gyp View 4 chunks +5 lines, -5 lines 0 comments Download
M trunk/src/build/all.gyp View 9 chunks +10 lines, -10 lines 0 comments Download
M trunk/src/build/temp_gyp/googleurl.gyp View 2 chunks +92 lines, -4 lines 0 comments Download
M trunk/src/chrome/chrome_browser.gypi View 2 chunks +1 line, -1 line 0 comments Download
M trunk/src/chrome/chrome_browser_chromeos.gypi View 2 chunks +4 lines, -4 lines 0 comments Download
M trunk/src/chrome/chrome_browser_extensions.gypi View 2 chunks +1 line, -1 line 0 comments Download
M trunk/src/chrome/chrome_browser_ui.gypi View 2 chunks +2 lines, -2 lines 0 comments Download
M trunk/src/chrome/chrome_browser_ui_views.gyp View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/chrome/chrome_common.gypi View 2 chunks +1 line, -1 line 0 comments Download
M trunk/src/chrome/chrome_tests.gypi View 5 chunks +10 lines, -10 lines 0 comments Download
M trunk/src/chrome_frame/chrome_frame.gyp View 6 chunks +10 lines, -10 lines 0 comments Download
M trunk/src/cloud_print/service/service.gyp View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/components/autofill.gypi View 3 chunks +2 lines, -2 lines 0 comments Download
M trunk/src/components/visitedlink.gypi View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/content/content_common.gypi View 2 chunks +1 line, -1 line 0 comments Download
M trunk/src/content/content_shell.gypi View 2 chunks +1 line, -1 line 0 comments Download
M trunk/src/content/content_tests.gypi View 1 chunk +4 lines, -4 lines 0 comments Download
M trunk/src/jingle/jingle.gyp View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/media/media.gyp View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/net/net.gyp View 10 chunks +10 lines, -10 lines 0 comments Download
M trunk/src/net/tools/tld_cleanup/tld_cleanup.gyp View 1 chunk +1 line, -1 line 0 comments Download
MM trunk/src/ppapi/ppapi_internal.gyp View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/printing/printing.gyp View 1 chunk +1 line, -1 line 0 comments Download
MM trunk/src/rlz/rlz.gyp View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/sync/sync_core.gypi View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/sync/sync_internal_api.gypi View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/ui/message_center/message_center.gyp View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/ui/oak/oak.gyp View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/ui/ui.gyp View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/ui/ui_unittests.gypi View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/ui/views/controls/webview/webview.gyp View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/ui/views/views.gyp View 3 chunks +3 lines, -3 lines 0 comments Download
M trunk/src/url/url.gyp View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/webkit/base/webkit_base.gypi View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/webkit/glue/webkit_glue.gypi View 2 chunks +1 line, -1 line 0 comments Download
M trunk/src/webkit/storage/webkit_storage.gypi View 1 chunk +1 line, -1 line 0 comments Download
M trunk/src/webkit/tools/test_shell/test_shell.gypi View 2 chunks +2 lines, -2 lines 0 comments Download
M trunk/src/win8/metro_driver/metro_driver.gyp View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
michaeln
7 years, 8 months ago (2013-04-12 19:22:37 UTC) #1
michaeln
Committed patchset #1 manually as r193983.
7 years, 8 months ago (2013-04-12 19:23:08 UTC) #2
tfarina
Why didn't you include the revert reason in the commit message?
7 years, 8 months ago (2013-04-12 19:37:13 UTC) #3
jam
7 years, 8 months ago (2013-04-12 19:44:05 UTC) #4
btw oddly enough, this compiled and linked fine for me on my windows
machine. it passed on the trybots as well. perhaps all what was needed was
a clobber on the builder.


On Fri, Apr 12, 2013 at 12:37 PM, <tfarina@chromium.org> wrote:

> Why didn't you include the revert reason in the commit message?
>
>
>
>
https://codereview.chromium.**org/14028012/<https://codereview.chromium.org/1...
>

Powered by Google App Engine
This is Rietveld 408576698