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

Issue 8632023: Avoid references to untrusted.gypi in WebKit (Closed)

Created:
9 years, 1 month ago by noelallen1
Modified:
9 years ago
CC:
chromium-reviews
Visibility:
Public.

Description

Avoid references to untrusted.gypi in WebKit Remove include of untrusted.gypi in normal ppapi gyp files to prevent WebKit from depending on NaCl. Refactor PPAPI gyp files to pull out source lists which are shared between trusted and untrusted targets. Fix scons hack which pulls list of sources from GYP so that we can build with scons. This CL should allow us to revert the DEPS change: http://trac.webkit.org/changeset/101011 on the webkit side BUG= none TEST= Tools/Scripts/update-webkit --chromium Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=111606

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : '' #

Patch Set 4 : '' #

Patch Set 5 : '' #

Patch Set 6 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+497 lines, -588 lines) Patch
M DEPS View 1 2 3 4 5 1 chunk +1 line, -1 line 0 comments Download
M chrome/chrome_tests.gypi View 1 2 3 4 5 1 chunk +1 line, -1 line 0 comments Download
M ppapi/native_client/src/shared/ppapi/nacl.scons View 1 2 3 4 5 1 chunk +11 lines, -4 lines 0 comments Download
M ppapi/ppapi.gyp View 1 2 3 4 5 1 chunk +1 line, -1 line 0 comments Download
M ppapi/ppapi_cpp.gypi View 1 2 3 4 5 5 chunks +11 lines, -395 lines 0 comments Download
M ppapi/ppapi_internal.gyp View 1 2 3 4 5 1 chunk +1 line, -1 line 0 comments Download
A ppapi/ppapi_sources.gypi View 1 1 chunk +351 lines, -0 lines 0 comments Download
M ppapi/ppapi_tests.gypi View 1 2 3 4 5 3 chunks +1 line, -185 lines 0 comments Download
A ppapi/ppapi_untrusted.gyp View 1 2 1 chunk +119 lines, -0 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
noelallen1
9 years, 1 month ago (2011-11-22 20:00:06 UTC) #1
dmichael (off chromium)
It looks like this CL is missing ppapi_sources.gypi Does this replace http://codereview.chromium.org/8635015/ ?
9 years, 1 month ago (2011-11-22 20:05:34 UTC) #2
noelallen1
9 years, 1 month ago (2011-11-22 20:30:22 UTC) #3
dmichael (off chromium)
9 years, 1 month ago (2011-11-22 20:35:01 UTC) #4
lgtm

Powered by Google App Engine
This is Rietveld 408576698