Bug 32854 - Remove COM code generation related files
Summary: Remove COM code generation related files
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit API (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows XP
: P2 Normal
Assignee: Kent Tamura
URL:
Keywords:
Depends on: 33190 33193
Blocks:
  Show dependency treegraph
 
Reported: 2009-12-22 00:06 PST by Kent Tamura
Modified: 2010-01-07 05:47 PST (History)
7 users (show)

See Also:


Attachments
Proposed patch (66.67 KB, patch)
2009-12-22 00:08 PST, Kent Tamura
no flags Details | Formatted Diff | Diff
Proposed patch (rev.2) (85.95 KB, patch)
2009-12-22 01:16 PST, Kent Tamura
sam: review-
Details | Formatted Diff | Diff
Proposed patch (rev.3) (146.77 KB, patch)
2010-01-04 13:27 PST, Kent Tamura
mjs: review+
tkent: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kent Tamura 2009-12-22 00:06:43 PST
We won't support COM code generation anymore. There are some files related to it in WebKit repository yet.
Comment 1 Kent Tamura 2009-12-22 00:08:30 PST
Created attachment 45368 [details]
Proposed patch
Comment 2 WebKit Review Bot 2009-12-22 00:11:16 PST
style-queue ran check-webkit-style on attachment 45368 [details] without any errors.
Comment 3 Kent Tamura 2009-12-22 01:16:09 PST
Created attachment 45373 [details]
Proposed patch (rev.2)

Remove !LANGUAGE_COM in *.idl too.
Comment 4 WebKit Review Bot 2009-12-22 01:17:50 PST
style-queue ran check-webkit-style on attachment 45373 [details] without any errors.
Comment 5 Eric Seidel (no email) 2009-12-22 12:59:14 PST
This is definitely one for the Apple-Win folks to review.
Comment 6 Maciej Stachowiak 2009-12-28 17:54:00 PST
Perhaps Adam, Steve or Sam want to comment on whether it's ok to strip this code out entirely or if we want to keep it around for some reason.
Comment 7 Sam Weinig 2009-12-30 15:19:23 PST
Comment on attachment 45373 [details]
Proposed patch (rev.2)

It is fine to rip this out, we can always bring it back through the magic of svn.  That said, you are missing some COM specific things, namely the InterfaceUUID and ImplementationUUID extended attributes.  Those should be removed as well.
Comment 8 Kent Tamura 2010-01-04 13:27:09 PST
Created attachment 45825 [details]
Proposed patch (rev.3)

- Remove UUID attributes in *.idl too.
Comment 9 WebKit Review Bot 2010-01-04 13:29:49 PST
Attachment 45825 [details] did not build on chromium:
Build output: http://webkit-commit-queue.appspot.com/results/162587
Comment 10 WebKit Review Bot 2010-01-04 13:30:39 PST
style-queue ran check-webkit-style on attachment 45825 [details] without any errors.
Comment 11 Eric Seidel (no email) 2010-01-05 13:54:44 PST
Comment on attachment 45825 [details]
Proposed patch (rev.3)

Is WebKit/win/WebKit.vcproj/build-generated-files.sh only used by the COM bindings?

The rest all looks fine.
Comment 12 Maciej Stachowiak 2010-01-05 14:11:45 PST
Comment on attachment 45825 [details]
Proposed patch (rev.3)

Looks like this addresses Sam's comments. I believe the answer to Eric's question is "yes". r=me
Comment 13 Eric Seidel (no email) 2010-01-06 09:30:09 PST
Attachment 45825 [details] was posted by a committer and has review+, assigning to Kent Tamura for commit.
Comment 14 Kent Tamura 2010-01-07 05:47:08 PST
Landed as r52921 <http://trac.webkit.org/changeset/52921>