Summary: | [EFL][WK2] Add missing files to build webkit2/Efl. | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ryuan Choi <ryuan.choi> | ||||||||||
Component: | WebKit2 | Assignee: | Ryuan Choi <ryuan.choi> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | mariusz.g, rakuco, webkit.review.bot | ||||||||||
Priority: | P2 | ||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | Unspecified | ||||||||||||
OS: | Unspecified | ||||||||||||
Bug Depends on: | |||||||||||||
Bug Blocks: | 61838 | ||||||||||||
Attachments: |
|
Description
Ryuan Choi
2012-01-11 18:53:37 PST
Created attachment 122155 [details]
Patch
Comment on attachment 122155 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=122155&action=review > Source/WebKit2/UIProcess/Plugins/efl/PluginProcessProxyEfl.cpp:35 > + Missing #endif for ENABLE(PLUGIN_PROCESS) Comment on attachment 122155 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=122155&action=review >> Source/WebKit2/UIProcess/Plugins/efl/PluginProcessProxyEfl.cpp:35 >> + > > Missing #endif for ENABLE(PLUGIN_PROCESS) Hmm, below line wasn't displayed on my browser certainly. Now, there is no problems. LGTM. Created attachment 135226 [details]
Patch
Comment on attachment 135226 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=135226&action=review Are there plans to build WK2 by default from now on? > Source/WebKit2/UIProcess/Plugins/efl/PluginProcessProxyEfl.cpp:23 > +#if ENABLE(PLUGIN_PROCESS) Don't you need to add this option to OptionsEfl.cmake? (In reply to comment #5) > (From update of attachment 135226 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=135226&action=review > > Are there plans to build WK2 by default from now on? Not now, but we are planning to enable webkit2 as default. > > > Source/WebKit2/UIProcess/Plugins/efl/PluginProcessProxyEfl.cpp:23 > > +#if ENABLE(PLUGIN_PROCESS) > > Don't you need to add this option to OptionsEfl.cmake? Yes, it should be an option. After thinking while, PluginProcessProxyEfl is not needed until it is ready to upstream because I want to finish minimum webkit2/efl first. thanks. Created attachment 135247 [details]
Patch
Comment on attachment 135247 [details]
Patch
The patch looks OK to me.
Comment on attachment 135247 [details]
Patch
LGTM too.
Comment on attachment 135247 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=135247&action=review > Source/WebKit2/PlatformEfl.cmake:41 > - UIProcess/Plugins/efl/PluginInfoStoreEfl.cpp > - UIProcess/Plugins/efl/PluginProcessProxyEfl.cpp > + UIProcess/Plugins/unix/PluginInfoStoreUnix.cpp What is this change? Created attachment 137477 [details]
Patch
Comment on attachment 137477 [details]
Patch
Thank you very much.
Comment on attachment 137477 [details] Patch Clearing flags on attachment: 137477 Committed r114345: <http://trac.webkit.org/changeset/114345> All reviewed patches have been landed. Closing bug. There is missing Source/WebKit2/PlatformEfl.cmake change in latest patch. |