Bug 20704

Summary: Replace the KJS namespace
Product: WebKit Reporter: Cameron Zwarich (cpst) <zwarich>
Component: JavaScriptCoreAssignee: Cameron Zwarich (cpst) <zwarich>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Proposed patch mjs: review+

Cameron Zwarich (cpst)
Reported 2008-09-07 15:22:57 PDT
We should remove the namespace and replace it with something else. JSC seems to be the current winner.
Attachments
Proposed patch (468.01 KB, patch)
2008-09-07 18:06 PDT, Cameron Zwarich (cpst) 		mjs: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Cameron Zwarich (cpst)
Comment 1 2008-09-07 18:06:13 PDT
Created attachment 23234 [details] Proposed patch Here is a patch that changes KJS to JSC. I just recently built clean Debug and Release builds on the Mac, and the revisions since should not affect anything. I'll be around to fix any build failures on the bots.
Maciej Stachowiak
Comment 2 2008-09-07 18:17:53 PDT
Comment on attachment 23234 [details] Proposed patch r=me I did not read in great detail but on skimming I did not see problems.
Cameron Zwarich (cpst)
Comment 3 2008-09-07 19:25:34 PDT
Landed in r36263.
Note You need to log in before you can comment on or make changes to this bug.