Bug 48097

Summary: [chromium] roll chromium DEPS to include forwarding headers refactor
Product: WebKit Reporter: Tony Chang <tony>
Component: New BugsAssignee: Tony Chang <tony>
Status: RESOLVED FIXED    
Severity: Normal CC: dglazkov, fishd, tkent
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Other   
OS: OS X 10.5   
Attachments:
Description Flags
Patch none

Tony Chang
Reported 2010-10-21 15:44:02 PDT
[chromium] roll chromium DEPS to include forwarding headers refactor
Attachments
Patch (1.75 KB, patch)
2010-10-21 15:49 PDT, Tony Chang 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Tony Chang
Comment 1 2010-10-21 15:49:36 PDT
Created attachment 71506 [details] Patch
Tony Chang
Comment 2 2010-10-21 15:52:01 PDT
In a webkit only checkout, we used to copy the webkit API header files into WebKit/chromium/third_party/WebKit/WebKit/chromium/public. This pulls in code from chromium that instead of copying the header files, we create forwarding headers in the build output directory. Since we're moving the headers, someone will have to delete WebKit/chromium/third_party/WebKit on the bots (sorry, it was my mistake for generating copies outside the build output dir).
Dimitri Glazkov (Google)
Comment 3 2010-10-21 15:56:24 PDT
Comment on attachment 71506 [details] Patch ok :)
Tony Chang
Comment 4 2010-10-21 15:58:50 PDT
Committed r70273: <http://trac.webkit.org/changeset/70273>
Csaba Osztrogonác
Comment 5 2010-10-22 05:41:20 PDT
Comment on attachment 71506 [details] Patch remove r+, because it was landed
Note You need to log in before you can comment on or make changes to this bug.