Summary: | [EFL][GTK] Use WTF_USE_SOUP instead BUILDING_SOUP__ | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Jesus Sanchez-Palencia <jesus> | ||||||
Component: | WebKit EFL | Assignee: | Jesus Sanchez-Palencia <jesus> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | cgarcia, gustavo, gyuyoung.kim, lucas.de.marchi, mrobinson, rakuco, tonikitoo, webkit.review.bot, xan.lopez | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Jesus Sanchez-Palencia
2012-08-22 13:22:23 PDT
Created attachment 160001 [details]
Patch
Comment on attachment 160001 [details] Patch Attachment 160001 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/13573004 Comment on attachment 160001 [details] Patch Attachment 160001 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/13559432 Is WebKit2/Shared/API/c/WKBase.h is a public header? If so we probably shouldn't use WTF macros there. Comment on attachment 160001 [details] Patch Attachment 160001 [details] did not pass efl-ews (efl): Output: http://queues.webkit.org/results/13569376 Comment on attachment 160001 [details] Patch Attachment 160001 [details] did not pass mac-ews (mac): Output: http://queues.webkit.org/results/13567394 Comment on attachment 160001 [details] Patch Attachment 160001 [details] did not pass gtk-ews (gtk): Output: http://queues.webkit.org/results/13565473 (In reply to comment #4) > Is WebKit2/Shared/API/c/WKBase.h is a public header? If so we probably shouldn't use WTF macros there. Yep, you are right, we can't use the macros but we can use #if defined(WTF_USE_SOUP) I guess. I've tested it locally and it worked with efl, I'm trying Qt and Gtk now. Created attachment 160169 [details]
Patch
Comment on attachment 160169 [details]
Patch
This looks no worse than what we already have, go ahead =)
Committed r126453: <http://trac.webkit.org/changeset/126453> |