Summary: | [WK2] --no-backing-store build fails because FloatPoint is not declared | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Ed Bartosh <bartosh> | ||||||||
Component: | WebKit2 | Assignee: | Nobody <webkit-unassigned> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | ap, benjamin, commit-queue, gyuyoung.kim, lucas.de.marchi, rakuco, simon.fraser, webkit.review.bot | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | PC | ||||||||||
OS: | Linux | ||||||||||
Bug Depends on: | |||||||||||
Bug Blocks: | 113153 | ||||||||||
Attachments: |
|
Description
Ed Bartosh
2013-03-29 16:05:53 PDT
Created attachment 195816 [details]
Patch
Reverting back to UNCONFIRMED after reading https://bugs.webkit.org/page.cgi?id=fields.html#status LGTM. You need to get WK2 owner's sign off. BTW, I wonder if this patch is only for EFL port. It looks this patch will be influence on other ports as well. Changing to WK2 as suggested. 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. 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.
Simon, can you please review this really small one? Comment on attachment 196522 [details]
Patch
Please just forward-declare to avoid extra includes.
Good point, I don't know what I was thinking when saying r+. Created attachment 196690 [details]
Patch
Updated according to reviewer's suggestion. Please review. Comment on attachment 196690 [details] Patch Clearing flags on attachment: 196690 Committed r147823: <http://trac.webkit.org/changeset/147823> All reviewed patches have been landed. Closing bug. |