RESOLVED FIXED 129022
Add FeatureDefines for image controls 
https://bugs.webkit.org/show_bug.cgi?id=129022
Summary Add FeatureDefines for image controls
Brady Eidson
Reported 2014-02-18 20:37:43 PST
Add FeatureDefines for image controls Image controls will be platform-specific UI on top of images. To help with development that requires constant rolling-in and rolling-out of patches, it'll be a big help to land the feature defines separately.
Attachments
Patch v1 (29.03 KB, patch)
2014-02-18 20:45 PST, Brady Eidson 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Brady Eidson
Comment 1 2014-02-18 20:45:02 PST
Created attachment 224582 [details] Patch v1
WebKit Commit Bot
Comment 2 2014-02-19 09:30:34 PST
Comment on attachment 224582 [details] Patch v1 Clearing flags on attachment: 224582 Committed r164366: <http://trac.webkit.org/changeset/164366>
WebKit Commit Bot
Comment 3 2014-02-19 09:30:35 PST
All reviewed patches have been landed. Closing bug.
Joseph Pecoraro
Comment 4 2014-02-19 11:12:18 PST
Comment on attachment 224582 [details] Patch v1 View in context: https://bugs.webkit.org/attachment.cgi?id=224582&action=review > Source/JavaScriptCore/Configurations/FeatureDefines.xcconfig:96 > +ENABLE_IMAGE_CONTROLS_macosx_1080 = ; > +ENABLE_IMAGE_CONTROLS_macosx_1090 = ; > +ENABLE_IMAGE_CONTROLS_macosx_101000 = ENABLE_IMAGE_CONTROLS; I wonder why we include 1080 and 1090 if the feature is not enabled there. We could just leave them off right?
Note You need to log in before you can comment on or make changes to this bug.