Bug 94794 - [V8] Remove V8Proxy.{h,cpp}
Summary: [V8] Remove V8Proxy.{h,cpp}
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore JavaScript (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Kentaro Hara
URL:
Keywords:
Depends on:
Blocks: 85330
  Show dependency treegraph
 
Reported: 2012-08-23 02:39 PDT by Kentaro Hara
Modified: 2012-08-23 16:59 PDT (History)
8 users (show)

See Also:


Attachments
Patch (33.33 KB, patch)
2012-08-23 02:43 PDT, Kentaro Hara
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kentaro Hara 2012-08-23 02:39:50 PDT
Now it's time to kill V8Proxy.
Comment 1 Kentaro Hara 2012-08-23 02:43:14 PDT
Created attachment 160119 [details]
Patch
Comment 2 Dimitri Glazkov (Google) 2012-08-23 07:05:17 PDT
Comment on attachment 160119 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=160119&action=review

What a glorious moment!

> Source/WebCore/bindings/v8/ScriptController.h:67
> +// FIXME: We want to eventually be holding window shells instead of the

It would be good to file a bug and reference it here, so that someone could always check the staleness of this fixme.
Comment 3 Kentaro Hara 2012-08-23 16:58:51 PDT
Committed r126506: <http://trac.webkit.org/changeset/126506>