RESOLVED FIXED 95966
Expose DOM interface for CSS Device Adaptation
https://bugs.webkit.org/show_bug.cgi?id=95966
Summary Expose DOM interface for CSS Device Adaptation
Thiago Marcos P. Santos
Reported 2012-09-06 03:12:04 PDT
Add it for both V8 and JSC.
Attachments
WIP (26.24 KB, patch)
2012-11-20 01:11 PST, Thiago Marcos P. Santos
no flags
Patch (24.02 KB, patch)
2012-11-20 08:18 PST, Thiago Marcos P. Santos
no flags
Patch (25.12 KB, patch)
2012-11-22 06:27 PST, Thiago Marcos P. Santos
no flags
Thiago Marcos P. Santos
Comment 1 2012-11-20 01:11:58 PST
Created attachment 175170 [details] WIP Build test (with the feature enabled by default on all the ports) to see if it the EWS explodes.
Adam Barth
Comment 2 2012-11-20 01:30:48 PST
Comment on attachment 175170 [details] WIP View in context: https://bugs.webkit.org/attachment.cgi?id=175170&action=review > Source/WebKit/chromium/features.gypi:45 > - 'ENABLE_CSS_DEVICE_ADAPTATION=0', > + 'ENABLE_CSS_DEVICE_ADAPTATION=1', I assume you don't plan to land this part of this patch.
Thiago Marcos P. Santos
Comment 3 2012-11-20 01:40:03 PST
Comment on attachment 175170 [details] WIP View in context: https://bugs.webkit.org/attachment.cgi?id=175170&action=review >> Source/WebKit/chromium/features.gypi:45 >> + 'ENABLE_CSS_DEVICE_ADAPTATION=1', > > I assume you don't plan to land this part of this patch. Nop, it is just a build test as I wrote on the comments. Just to make sure it will work on all the ports if they want to enable it one day.
Thiago Marcos P. Santos
Comment 4 2012-11-20 08:15:02 PST
The build test failed on GTK but I could build it locally in a clean environment. I'm guessing that GTK buildsystem might have a problem on tracking dependencies when a flag affects CSSValueKeywords.in.
Thiago Marcos P. Santos
Comment 5 2012-11-20 08:18:05 PST
Thiago Marcos P. Santos
Comment 6 2012-11-22 06:27:30 PST
WebKit Review Bot
Comment 7 2012-11-23 00:07:17 PST
Comment on attachment 175662 [details] Patch Clearing flags on attachment: 175662 Committed r135564: <http://trac.webkit.org/changeset/135564>
WebKit Review Bot
Comment 8 2012-11-23 00:07:24 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.