Bug 104429

Summary: [CMake] Consolidate list of files to build for WebCore plugin support
Product: WebKit Reporter: Laszlo Gombos <laszlo.gombos>
Component: PlatformAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, gyuyoung.kim, mxie, paroga, rakuco, rwlbuis, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
proposed patch
none
rebase CMakeLists.txt
none
right patch this time
none
keep PluginDataNone.cpp for BlackBerry
buildbot: commit-queue-
rebase on top of r149876 none

Description Laszlo Gombos 2012-12-07 19:54:52 PST
It seems to me that we can consolidate the list of files to build in the CMake build system and move some some of the files from PlatformXXX.cmake to CMakeLists.txt.
Comment 1 Laszlo Gombos 2012-12-07 19:58:44 PST
Created attachment 178330 [details]
proposed patch

I only had the opportunity to test it on the EFL port.
Comment 2 Laszlo Gombos 2012-12-10 09:21:09 PST
Created attachment 178570 [details]
rebase CMakeLists.txt
Comment 3 Laszlo Gombos 2012-12-10 10:01:30 PST
Created attachment 178579 [details]
right patch this time
Comment 4 Laszlo Gombos 2012-12-26 20:14:38 PST
Rob, can you help review this patch ?
Comment 5 Rob Buis 2012-12-28 08:57:21 PST
Comment on attachment 178579 [details]
right patch this time

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

> Source/WebCore/PlatformBlackBerry.cmake:-255
> -        plugins/PluginDataNone.cpp

Is it correct this one was removed from the list? If it is correct, is it not in use anymore and can we remove the PluginDataNone.cpp?
Comment 6 Laszlo Gombos 2013-02-02 19:11:26 PST
Created attachment 186244 [details]
keep PluginDataNone.cpp for BlackBerry

Thanks Rob for the review. You're right in that PluginDataNone.cpp is still needed for the BlackBerry port. 

It seems to me that the file is only used by the BlackBerry port only and can be removed if the BlackBerry port starts to use USE(PLATFORM_STRATEGIES).
Comment 7 Build Bot 2013-02-02 21:14:31 PST
Comment on attachment 186244 [details]
keep PluginDataNone.cpp for BlackBerry

Attachment 186244 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/16357200
Comment 8 Build Bot 2013-02-02 22:14:40 PST
Comment on attachment 186244 [details]
keep PluginDataNone.cpp for BlackBerry

Attachment 186244 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/16353281
Comment 9 Rob Buis 2013-05-09 12:20:25 PDT
I think this may need another pass, it seems to not pass on win and gtk.
Comment 10 Patrick R. Gansterer 2013-05-09 12:22:31 PDT
(In reply to comment #9)
> I think this may need another pass, it seems to not pass on win and gtk.
Gtk & Win do not use CMake, so this failures seam unrelated.
Comment 11 Rob Buis 2013-05-09 12:38:36 PDT
(In reply to comment #10)
> (In reply to comment #9)
> > I think this may need another pass, it seems to not pass on win and gtk.
> Gtk & Win do not use CMake, so this failures seam unrelated.

You are correct. However, efl-wk2 also seems to fail. Laszlo, you want to check and maybe upload a new patch against ToT?
Comment 12 Laszlo Gombos 2013-05-10 09:00:57 PDT
Created attachment 201356 [details]
rebase on top of r149876
Comment 13 Rob Buis 2013-05-10 10:39:28 PDT
Comment on attachment 201356 [details]
rebase on top of r149876

Looks good.
Comment 14 WebKit Commit Bot 2013-05-10 12:09:19 PDT
Comment on attachment 201356 [details]
rebase on top of r149876

Clearing flags on attachment: 201356

Committed r149899: <http://trac.webkit.org/changeset/149899>
Comment 15 WebKit Commit Bot 2013-05-10 12:09:23 PDT
All reviewed patches have been landed.  Closing bug.