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

Issue 2070001: Remove webkit_support_in_chromium.gyp and use webkit_support.gyp (Closed)

Created:
10 years, 7 months ago by tony
Modified:
9 years, 7 months ago
CC:
chromium-reviews
Base URL:
http://src.chromium.org/git/chromium.git
Visibility:
Public.

Description

Remove webkit_support_in_chromium.gyp and use webkit_support.gyp instead. features.gypi isn't needed by glue, appcache or database. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=47238

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+27 lines, -54 lines) Patch
M build/all.gyp View 1 chunk +1 line, -1 line 0 comments Download
M chrome/app/locales/locales.gyp View 1 chunk +1 line, -1 line 0 comments Download
M chrome/chrome.gyp View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/chrome_browser.gypi View 1 chunk +4 lines, -4 lines 0 comments Download
M chrome/chrome_common.gypi View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/chrome_dll.gypi View 1 chunk +1 line, -1 line 0 comments Download
M chrome/chrome_renderer.gypi View 1 chunk +2 lines, -2 lines 0 comments Download
M chrome/chrome_tests.gypi View 2 chunks +2 lines, -2 lines 0 comments Download
M chrome/nacl.gypi View 1 chunk +1 line, -1 line 0 comments Download
M chrome/test/interactive_ui/interactive_ui_tests.gypi View 1 chunk +1 line, -1 line 0 comments Download
M webkit/default_plugin/default_plugin.gyp View 1 chunk +2 lines, -2 lines 0 comments Download
M webkit/support/webkit_support.gyp View 1 chunk +0 lines, -5 lines 0 comments Download
D webkit/support/webkit_support_in_chromium.gyp View 1 chunk +0 lines, -22 lines 0 comments Download
M webkit/tools/test_shell/test_shell.gypi View 3 chunks +8 lines, -8 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
tony
This will have to wait until DEPS is rolled to pick up r59346.
10 years, 7 months ago (2010-05-13 06:24:23 UTC) #1
darin (slow to review)
10 years, 7 months ago (2010-05-13 06:29:12 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698