RESOLVED FIXED 35956
[Chromium] Have a grd file live upstream for resources that we pull from WebKit/WebCore 
https://bugs.webkit.org/show_bug.cgi?id=35956
Summary [Chromium] Have a grd file live upstream for resources that we pull from WebK...
Tony Chang
Reported 2010-03-10 00:13:51 PST
We should have a grd file in WebKit/chromium/ for resources that live upstream. This has a couple of advantages: - If a resource we depend on upstream moves, whoever moves the file can fix the chromium build by updating the grd file. - As part of the chromium DRT work, we need to be able to build webkit_resources.grd upstream. The references to third_party files won't resolve upstream, so this allows us to get rid of the third_party references. Patch coming with a small grd file. The Chromium side patch that will start using this file is here: http://codereview.chromium.org/792005
Attachments
Patch (1.46 KB, patch)
2010-03-10 00:16 PST, Tony Chang 		fishd: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Tony Chang
Comment 1 2010-03-10 00:16:13 PST
Created attachment 50375 [details] Patch
Tony Chang
Comment 2 2010-03-10 00:17:53 PST
(In reply to comment #1) > Created an attachment (id=50375) [details] > Patch I am open to suggestions on where to put the file (e.g., the WebKit/win port has a .rc file in WebKit/win/WebKit.vcproj/) and on what to name the grd file. Right now they are all dev tool resources, but you could imagine us wanting to include other resources here (like the missing image png).
Tony Chang
Comment 3 2010-03-10 00:28:28 PST
Comment on attachment 50375 [details] Patch Based on offline discussion, I'm going to give this a more general filename so we can add other resources (not devtool related) to it in the future.
Tony Chang
Comment 4 2010-03-10 00:41:37 PST
Committed r55769: <http://trac.webkit.org/changeset/55769>
Note You need to log in before you can comment on or make changes to this bug.