WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
93312
[V8] Remove #include V8BindingMacros.h
https://bugs.webkit.org/show_bug.cgi?id=93312
Summary
[V8] Remove #include V8BindingMacros.h
Kentaro Hara
Reported
2012-08-06 17:39:47 PDT
We are factoring out V8Binding methods to separate files. After the refactoring, V8Binding.h will include binding related header files (e.g. V8BindingMacros.h, V8BindingHelper.h, etc), so that custom V8 binding files just need to include V8Binding.h only. We can remove '#include V8BindingMacros.h' from V8 binding files.
Attachments
Patch
(20.36 KB, patch)
2012-08-06 17:42 PDT
,
Kentaro Hara
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
Kentaro Hara
Comment 1
2012-08-06 17:42:00 PDT
Created
attachment 156811
[details]
Patch
Eric Seidel (no email)
Comment 2
2012-08-07 14:58:01 PDT
Comment on
attachment 156811
[details]
Patch OK.
WebKit Review Bot
Comment 3
2012-08-07 16:30:12 PDT
Comment on
attachment 156811
[details]
Patch Clearing flags on attachment: 156811 Committed
r124933
: <
http://trac.webkit.org/changeset/124933
>
WebKit Review Bot
Comment 4
2012-08-07 16:30:16 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.
Top of Page
Format For Printing
XML
Clone This Bug