[Webkit-unassigned] [Bug 91844] [WK2][GTK][EFL] Share WebKit2-GTK plugin process implementation with EFL port

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=91844





--- Comment #46 from Carlos Garcia Campos <cgarcia at igalia.com>  2012-09-20 05:11:04 PST ---
(In reply to comment #45)
> (In reply to comment #35)
> > (From update of attachment 164363 [details] [details])
> > I'm not sure about some of the files, the implementation is the same because it's currently unimplemented, not because the implementation is expected to be common to all ports in a unix platform. PluginProcess/unix/PluginProcessMainUnix.cpp should probably be X11 instead of Unix, but it requires a lot of #ifdefs so I'm not sure whether it would be better to keep separate files without #ifdefs. Note also that Qt port could also share code common to unix platform, like in Platform/CoreIPC/unix/ConnectionUnix.cpp, but it seems to me that most of the files shared here are not for platform specific code, but for port specific code.
> 
> PluginProcessMainUnix seems to be more "unix" than "x11" because X11 support adds only error handling.
> Sometimes it's hard to point the best solution: port, backend, operating system.
> What's your suggestion?

My suggestion is to keep separate files instead of a "common" file full of #ifdefs

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.