Summary: | !HAVE(ACCESSIBILITY) should be able to build without linking in any code from WebCore/accessibility | ||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Adam Barth <abarth> | ||||||||||||
Component: | New Bugs | Assignee: | Adam Barth <abarth> | ||||||||||||
Status: | RESOLVED FIXED | ||||||||||||||
Severity: | Normal | CC: | cfleizach, eric, peter, webkit.review.bot | ||||||||||||
Priority: | P2 | ||||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||||
Hardware: | Unspecified | ||||||||||||||
OS: | Unspecified | ||||||||||||||
Bug Depends on: | |||||||||||||||
Bug Blocks: | 66687 | ||||||||||||||
Attachments: |
|
Description
Adam Barth
2012-05-21 11:36:24 PDT
Presumably this is a chromium-only issue? :) Created attachment 143067 [details]
Patch
Comment on attachment 143067 [details]
Patch
looks ok
Comment on attachment 143067 [details] Patch Attachment 143067 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/12731700 > Presumably this is a chromium-only issue? :)
It's likely to reduce the binary size on other ports that use !HAVE(ACCESSIBILITY) too.
Comment on attachment 143067 [details] Patch Attachment 143067 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/12736641 Created attachment 143110 [details]
Patch for landing
This patch should remove the duplicate symbols. Comment on attachment 143110 [details] Patch for landing Attachment 143110 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/12747180 Comment on attachment 143110 [details] Patch for landing Attachment 143110 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/12740569 Comment on attachment 143110 [details]
Patch for landing
Too tight. /me tries again.
Created attachment 143123 [details]
Patch
Comment on attachment 143123 [details] Patch Attachment 143123 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/12738599 Created attachment 143125 [details]
Patch
Comment on attachment 143125 [details] Patch Attachment 143125 [details] did not pass qt-wk2-ews (qt): Output: http://queues.webkit.org/results/12747194 Comment on attachment 143125 [details] Patch Attachment 143125 [details] did not pass qt-ews (qt): Output: http://queues.webkit.org/results/12742372 Created attachment 143126 [details]
Patch
Comment on attachment 143126 [details] Patch Clearing flags on attachment: 143126 Committed r117907: <http://trac.webkit.org/changeset/117907> All reviewed patches have been landed. Closing bug. |