Summary: | remove plugins/mac | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Alex Christensen <achristensen> | ||||
Component: | WebCore Misc. | Assignee: | Alex Christensen <achristensen> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | ap, bfulgham, commit-queue | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Alex Christensen
2014-08-12 15:53:35 PDT
Created attachment 236477 [details]
Patch
Are these files built when building 32-bit? They might be excluded from 64-bit build, because it doesn't support in-process plug-ins. I don't think so. Grepping for PluginPackageMac.cpp shows it's only referenced in CMake files (which I added by mistake) and ChangeLogs. If it were built, wouldn't it be in the Xcode files? Comment on attachment 236477 [details]
Patch
Fun, seems like these were only used by Qt on Mac. Looks like there are other unused files in WebCore/plugins (I found PluginDataNone.cpp).
Comment on attachment 236477 [details]
Patch
I'll look into this tomorrow.
Removed PluginDataNone.cpp, too, and mentioned in the committed change log. http://trac.webkit.org/changeset/172535 |