Bug 113618 - [WK2] --no-backing-store build fails because FloatPoint is not declared
Summary: [WK2] --no-backing-store build fails because FloatPoint is not declared
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Linux
: P2 Normal
Assignee: Nobody
Depends on:
Blocks: 113153
  Show dependency treegraph
Reported: 2013-03-29 16:05 PDT by Ed Bartosh
Modified: 2013-04-05 17:47 PDT (History)
8 users (show)

See Also:

Patch (1.15 KB, patch)
2013-03-29 16:20 PDT, Ed Bartosh
ap: review+
Details | Formatted Diff | Diff
Patch (1.16 KB, patch)
2013-04-04 13:55 PDT, Ed Bartosh
no flags Details | Formatted Diff | Diff
Patch (1.14 KB, patch)
2013-04-05 15:26 PDT, Ed Bartosh
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
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[2]: *** [Source/WebKit2/CMakeFiles/ewebkit2.dir/PluginProcess/PluginControllerProxy.cpp.o] Error 1
make[1]: *** [Source/WebKit2/CMakeFiles/ewebkit2.dir/all] Error 2
make: *** [all] Error 2
Comment 1 Ed Bartosh 2013-03-29 16:20:04 PDT
Created attachment 195816 [details]
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]

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]

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]

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 11 Ed Bartosh 2013-04-05 15:26:54 PDT
Created attachment 196690 [details]
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]

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.