RESOLVED FIXED 45547
Fix a windows build of plugins 
https://bugs.webkit.org/show_bug.cgi?id=45547
Summary Fix a windows build of plugins
Fridrich Strba
Reported 2010-09-10 09:20:08 PDT
Add a WebCore/plugins/win directory among the directories where to look for include files, since it contains some include files needed for the Windows build.
Attachments
Patch (1.01 KB, patch)
2010-09-10 09:21 PDT, Fridrich Strba 		no flags
DetailsFormatted DiffDiff
Patch (1002 bytes, patch)
2010-09-10 09:52 PDT, Fridrich Strba 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Fridrich Strba
Comment 1 2010-09-10 09:21:34 PDT
Created attachment 67193 [details] Patch
Andreas Kling
Comment 2 2010-09-10 09:28:05 PDT
Comment on attachment 67193 [details] Patch > + -I$(srcdir)/WebCore/plugins/win \ Wouldn't this fit better into the "TARGET_WIN32" section?
Fridrich Strba
Comment 3 2010-09-10 09:34:47 PDT
(In reply to comment #2) > (From update of attachment 67193 [details]) > > + -I$(srcdir)/WebCore/plugins/win \ > > Wouldn't this fit better into the "TARGET_WIN32" section? Just a matter of taste because the files there will be included only if the following condition is fulfilled: #elif defined(GDK_WINDOWING_WIN32) #include "PluginMessageThrottlerWin.h" #include <gdk/gdkwin32.h> #endif So, if you can accept it as it is, it is ok, if not I will redo the patch.
Fridrich Strba
Comment 4 2010-09-10 09:52:08 PDT
Created attachment 67198 [details] Patch
WebKit Commit Bot
Comment 5 2010-10-13 19:03:12 PDT
Comment on attachment 67198 [details] Patch Clearing flags on attachment: 67198 Committed r69726: <http://trac.webkit.org/changeset/69726>
WebKit Commit Bot
Comment 6 2010-10-13 19:03:17 PDT
All reviewed patches have been landed. Closing bug.
WebKit Review Bot
Comment 7 2010-10-13 21:32:38 PDT
http://trac.webkit.org/changeset/69726 might have broken GTK Linux 64-bit Debug The following tests are not passing: canvas/philip/tests/2d.gradient.radial.negative.html editing/selection/context-menu-on-text.html
Note You need to log in before you can comment on or make changes to this bug.