Bug 18868

Summary: SQUIRRELFISH: window.this shows up as a property, but it shouldn't
Product: WebKit Reporter: Maciej Stachowiak <mjs>
Component: JavaScriptCoreAssignee: Maciej Stachowiak <mjs>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Mac   
OS: OS X 10.5   
Bug Depends on:    
Bug Blocks: 18631    
Attachments:
Description Flags
don't keep "this" in the symbol table at runtime oliver: review+

Maciej Stachowiak
Reported 2008-05-03 01:35:49 PDT
"this" appears as a property of the window object to enumeration and lookup as "window.this", but it should not. This is likely a side effect of us treating it as a special local/global var.
Attachments
don't keep "this" in the symbol table at runtime (3.31 KB, patch)
2008-05-06 20:14 PDT, Maciej Stachowiak 		oliver: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Geoffrey Garen
Comment 1 2008-05-05 20:52:33 PDT
I think we should remove "this" from the symbol table after codegen.
Maciej Stachowiak
Comment 2 2008-05-06 20:14:37 PDT
Created attachment 20995 [details] don't keep "this" in the symbol table at runtime
Oliver Hunt
Comment 3 2008-05-06 20:23:05 PDT
Comment on attachment 20995 [details] don't keep "this" in the symbol table at runtime r=me, with the aswsertion changes we discussed
Note You need to log in before you can comment on or make changes to this bug.