Bug 100432 - [V8] We can merge the wrapper maps for DOM objects and active DOM objects
Summary: [V8] We can merge the wrapper maps for DOM objects and active DOM objects
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Adam Barth
URL:
Keywords:
Depends on: 100430
Blocks:
  Show dependency tree / graph
 
Reported: 2012-10-25 16:08 PDT by Adam Barth
Modified: 2012-10-26 00:16 PDT (History)
4 users (show)

See Also:

Attachments
Patch (10.26 KB, patch)
2012-10-25 16:09 PDT, Adam Barth 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Adam Barth 2012-10-25 16:08:15 PDT 
[V8] We can merge the wrapper maps for DOM objects and active DOM objects
Comment 1 Adam Barth 2012-10-25 16:09:02 PDT 
Created attachment 170761 [details]
Patch
Comment 2 Eric Seidel (no email) 2012-10-25 16:09:49 PDT 
Comment on attachment 170761 [details]
Patch

LGTM.
Comment 3 WebKit Review Bot 2012-10-25 21:58:56 PDT 
Comment on attachment 170761 [details]
Patch

Clearing flags on attachment: 170761

Committed r132570: <http://trac.webkit.org/changeset/132570>
Comment 4 WebKit Review Bot 2012-10-25 21:58:59 PDT 
All reviewed patches have been landed.  Closing bug.
Comment 5 Csaba Osztrogonác 2012-10-26 00:06:47 PDT 
(In reply to comment #3)
> (From update of attachment 170761 [details])
> Clearing flags on attachment: 170761
> 
> Committed r132570: <http://trac.webkit.org/changeset/132570>

It broke the bindings tests. Could you fix it, please?
Comment 6 Adam Barth 2012-10-26 00:13:52 PDT 
Sure.
Comment 7 Adam Barth 2012-10-26 00:16:09 PDT 
http://trac.webkit.org/changeset/132576