Bug 96556

Summary: [V8] V8DOMWrapper::constructorForType is unnecessary now that we can get V8PerContextData from the v8::Context
Product: WebKit Reporter: Adam Barth <abarth>
Component: New BugsAssignee: Adam Barth <abarth>
Status: RESOLVED FIXED    
Severity: Normal CC: eric, haraken, japhet, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch for landing
none
Patch for landing none

Adam Barth
Reported 2012-09-12 14:00:19 PDT
[V8] V8DOMWrapper::constructorForType is unnecessary now that we can get V8PerContextData from the v8::Context
Attachments
Patch (8.01 KB, patch)
2012-09-12 14:01 PDT, Adam Barth 		no flags
DetailsFormatted DiffDiff
Patch for landing (9.03 KB, patch)
2012-09-12 14:12 PDT, Adam Barth 		no flags
DetailsFormatted DiffDiff
Patch for landing (9.03 KB, patch)
2012-09-12 14:12 PDT, Adam Barth 		no flags
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Adam Barth
Comment 1 2012-09-12 14:01:59 PDT
Created attachment 163689 [details] Patch
Eric Seidel (no email)
Comment 2 2012-09-12 14:09:52 PDT
Comment on attachment 163689 [details] Patch So much red...
Adam Barth
Comment 3 2012-09-12 14:12:08 PDT
Created attachment 163691 [details] Patch for landing
Adam Barth
Comment 4 2012-09-12 14:12:47 PDT
Created attachment 163692 [details] Patch for landing
Adam Barth
Comment 5 2012-09-12 14:13:01 PDT
Now with run-bindings-tests results.
WebKit Review Bot
Comment 6 2012-09-12 14:36:27 PDT
Comment on attachment 163692 [details] Patch for landing Clearing flags on attachment: 163692 Committed r128361: <http://trac.webkit.org/changeset/128361>
WebKit Review Bot
Comment 7 2012-09-12 14:36:31 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.