Summary: | [chromium] Make use_default_render_theme compile the right set of files | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Scott Violet <sky> | ||||||
Component: | WebKit Misc. | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | abarth, dglazkov, fishd, jamesr, tkent+wkapi, tony, webkit.review.bot | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Scott Violet
2012-11-21 09:29:23 PST
Created attachment 175522 [details]
Patch
WebKit DEPS was rolled, so this compiles for me outside of chrome. Please wait for approval from abarth@webkit.org, dglazkov@chromium.org, fishd@chromium.org, jamesr@chromium.org or tkent@chromium.org before submitting, as this patch contains changes to the Chromium public API. See also https://trac.webkit.org/wiki/ChromiumWebKitAPI. Comment on attachment 175522 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=175522&action=review LGTM > Source/WebKit/chromium/WebKit.gyp:853 > + }], > + ['use_default_render_theme==0', { Nit: I would make this an else clause (delete from ] to the second ' inclusive). Created attachment 175528 [details]
Patch
Newest patch converts to else syntax as outlined in comment 4. Comment on attachment 175528 [details]
Patch
ok
Comment on attachment 175528 [details] Patch Clearing flags on attachment: 175528 Committed r135718: <http://trac.webkit.org/changeset/135718> All reviewed patches have been landed. Closing bug. |