Bug 27788 - Don't export WebPluginController.h as a private header
Summary: Don't export WebPluginController.h as a private header
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Other OS X 10.5
: P2 Normal
Assignee: David Kilzer (:ddkilzer)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-07-28 17:52 PDT by David Kilzer (:ddkilzer)
Modified: 2009-07-29 17:10 PDT (History)
2 users (show)

See Also:


Attachments
Patch v1 (3.32 KB, patch)
2009-07-28 17:52 PDT, David Kilzer (:ddkilzer)
no flags Details | Formatted Diff | Diff
Patch v2 (3.38 KB, patch)
2009-07-29 13:55 PDT, David Kilzer (:ddkilzer)
mrowe: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description David Kilzer (:ddkilzer) 2009-07-28 17:52:20 PDT
Created attachment 33684 [details]
Patch v1

Reviewed by NOBODY (OOPS!).

WebPluginContainerCheck.h must be a private header because it is
included from WebPluginController.h, which is a private header.

WebKit:

* WebKit.xcodeproj/project.pbxproj: Export
WebPluginContainerCheck.h as a private header.

WebKit/mac:

* Plugins/WebPluginController.h: Changed #import of
WebPluginContainerCheck.h to use a framework-style path.
---
 4 files changed, 26 insertions(+), 2 deletions(-)
Comment 1 Mark Rowe (bdash) 2009-07-28 18:26:55 PDT
Is WebPluginController.h even used outside of WebKit?  If not, a better fix would be to remove it from the private headers.  Since the include in WebPluginController.h has been like this since April I'd would have expected that we would have noticed that this header wasn't usable as-is.
Comment 2 David Kilzer (:ddkilzer) 2009-07-28 18:41:34 PDT
(In reply to comment #1)
> Is WebPluginController.h even used outside of WebKit?  If not, a better fix
> would be to remove it from the private headers.  Since the include in
> WebPluginController.h has been like this since April I'd would have expected
> that we would have noticed that this header wasn't usable as-is.

Yes, +[WebPluginController plugInViewWithArguments:fromPluginPackage:] is being used outside of WebKit on iPhone OS.
Comment 3 David Kilzer (:ddkilzer) 2009-07-28 19:05:18 PDT
(In reply to comment #1)
> Is WebPluginController.h even used outside of WebKit?  If not, a better fix
> would be to remove it from the private headers.  Since the include in
> WebPluginController.h has been like this since April I'd would have expected
> that we would have noticed that this header wasn't usable as-is.

WebPluginController.h has been a private header since r9857 (July 21, 2005).

http://trac.webkit.org/changeset/9857

However, I can't find any uses of it outside WebKit.
Comment 4 David Kilzer (:ddkilzer) 2009-07-28 19:09:15 PDT
(In reply to comment #3)
> However, I can't find any uses of it outside WebKit.

On Mac OS X.
Comment 5 David Kilzer (:ddkilzer) 2009-07-29 13:55:55 PDT
Created attachment 33740 [details]
Patch v2
Comment 6 David Kilzer (:ddkilzer) 2009-07-29 17:10:02 PDT
Fixed in r46565.  <http://trac.webkit.org/changset/46565>