|Summary:||[WK2] --no-backing-store build fails because FloatPoint is not declared|
|Product:||WebKit||Reporter:||Ed Bartosh <bartosh>|
|Severity:||Normal||CC:||ap, benjamin, commit-queue, gyuyoung.kim, lucas.de.marchi, rakuco, simon.fraser, webkit.review.bot|
|Version:||528+ (Nightly build)|
|Bug Depends on:|
Description Ed Bartosh 2013-03-29 16:05:53 PDT
Here is the error: In file included from /home/ed/git/wk/WebKit/Source/WebKit2/PluginProcess/PluginControllerProxy.h:32:0, from /home/ed/git/wk/WebKit/Source/WebKit2/PluginProcess/PluginControllerProxy.cpp:27: /home/ed/git/wk/WebKit/Source/WebKit2/WebProcess/Plugins/Plugin.h:270:58: error: `FloatPoint' in namespace `WebCore' does not name a type /home/ed/git/wk/WebKit/Source/WebKit2/WebProcess/Plugins/Plugin.h:270:77: error: ISO C++ forbids declaration of `parameter' with no type [-fpermissive] make: *** [Source/WebKit2/CMakeFiles/ewebkit2.dir/PluginProcess/PluginControllerProxy.cpp.o] Error 1 make: *** [Source/WebKit2/CMakeFiles/ewebkit2.dir/all] Error 2 make: *** [all] Error 2
Comment 2 Ed Bartosh 2013-04-01 01:05:28 PDT
Reverting back to UNCONFIRMED after reading https://bugs.webkit.org/page.cgi?id=fields.html#status
Comment 3 Gyuyoung Kim 2013-04-01 03:35:26 PDT
LGTM. You need to get WK2 owner's sign off.
Comment 4 Gyuyoung Kim 2013-04-01 03:37:11 PDT
BTW, I wonder if this patch is only for EFL port. It looks this patch will be influence on other ports as well.
Comment 5 Ed Bartosh 2013-04-01 03:59:52 PDT
Changing to WK2 as suggested.
Comment 6 Alexey Proskuryakov 2013-04-02 14:31:29 PDT
Comment on attachment 195816 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=195816&action=review > Source/WebKit2/ChangeLog:3 > + [EFL] --no-tiled-backing-store build fails because FloatPoint is not declared Please fix the prefix here to match the bug title.
Comment 7 Ed Bartosh 2013-04-04 13:55:27 PDT
Created attachment 196522 [details] Patch Alexey, thank you for review. I'm sorry for the delay. I've missed your comment somehow. Updated the patch according to your suggestion.
Comment 8 Ed Bartosh 2013-04-05 13:53:27 PDT
Simon, can you please review this really small one?
Comment 9 Simon Fraser (smfr) 2013-04-05 14:07:18 PDT
Comment on attachment 196522 [details] Patch Please just forward-declare to avoid extra includes.
Comment 10 Alexey Proskuryakov 2013-04-05 15:13:39 PDT
Good point, I don't know what I was thinking when saying r+.
Comment 12 Ed Bartosh 2013-04-05 15:28:01 PDT
Updated according to reviewer's suggestion. Please review.
Comment 13 WebKit Commit Bot 2013-04-05 17:47:49 PDT
Comment on attachment 196690 [details] Patch Clearing flags on attachment: 196690 Committed r147823: <http://trac.webkit.org/changeset/147823>
Comment 14 WebKit Commit Bot 2013-04-05 17:47:53 PDT
All reviewed patches have been landed. Closing bug.