Take ComputedStyleExtractor for a spin.
Created attachment 203206 [details] Patch
Comment on attachment 203206 [details] Patch Attachment 203206 [details] did not pass qt-ews (qt): Output: http://webkit-queues.appspot.com/results/742188
Comment on attachment 203206 [details] Patch Attachment 203206 [details] did not pass qt-wk2-ews (qt-wk2): Output: http://webkit-queues.appspot.com/results/750227
Created attachment 203208 [details] Patch
Comment on attachment 203208 [details] Patch r=me
Comment on attachment 203208 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=203208&action=review > Source/WebCore/editing/ApplyStyleCommand.cpp:459 > + int unicodeBidi = getIdentifierValue(ComputedStyleExtractor(n), CSSPropertyUnicodeBidi); Why does getIdentifierValue take ComputedStyleExtractor instead of a node? Seems unnecessarily abstract.
Committed r150945: <http://trac.webkit.org/changeset/150945>