Summary: | Add feature defines for web-facing parts of CSS Regions and Exclusions | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Tim Horton <thorton> | ||||||||||||
Component: | CSS | Assignee: | Tim Horton <thorton> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | CC: | abarth, cmarcelo, gustavo, macpherson, menard, mitz, ojan, ossy, philn, webkit.review.bot, xan.lopez | ||||||||||||
Priority: | P2 | Keywords: | InRadar | ||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||||
Hardware: | Unspecified | ||||||||||||||
OS: | Unspecified | ||||||||||||||
Attachments: |
|
Description
Tim Horton
2012-05-24 17:03:38 PDT
Created attachment 143939 [details]
patch
Created attachment 143940 [details]
patch
Rebase time. Created attachment 143943 [details]
patch
Comment on attachment 143943 [details] patch Attachment 143943 [details] did not pass win-ews (win): Output: http://queues.webkit.org/results/12798350 Comment on attachment 143943 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=143943&action=review r+ but you need to figure out why this is breaking Windows > Source/WebCore/ChangeLog:9 > + No new tests. Either explain why there are no new tests, if you think it’s necessary, or just remove this line. If no tests are listed here, then there are no new tests! > Source/WebCore/bindings/objc/DOMCSS.mm:55 > +#if ENABLE(CSS_REGIONS) > #import "DOMWebKitCSSRegionRule.h" > +#endif We don’t stick #if directives into the main #import or #include block. You should create a separate section with just this #import. This comment applies to the rest of this patch. Created attachment 143954 [details]
patch for landing if windows builds
Created attachment 143957 [details]
gtk
Landed in http://trac.webkit.org/changeset/118484 (In reply to comment #9) > Landed in http://trac.webkit.org/changeset/118484 Enabled on Qt again: http://trac.webkit.org/changeset/118494 |