RESOLVED FIXED Bug 43699
Use getIdentifierValue to obtain direction and unicode-bidi properties in ApplyStyleCommand 
https://bugs.webkit.org/show_bug.cgi?id=43699
Summary Use getIdentifierValue to obtain direction and unicode-bidi properties in App...
Ryosuke Niwa
Reported 2010-08-08 18:02:17 PDT
Currently ApplyStyleCommand uses getPropertyCSSValue and static cast to CSSPrimitiveValue. We should simplify this by calling getIdentifierValue in all places.
Attachments
fixes the bug (14.60 KB, patch)
2010-08-08 18:27 PDT, Ryosuke Niwa 		no flags
DetailsFormatted DiffDiff
initialized highestAncestorUnicodeBidi (14.60 KB, patch)
2010-08-09 13:18 PDT, Ryosuke Niwa 		tony: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Ryosuke Niwa
Comment 1 2010-08-08 18:27:06 PDT
Created attachment 63853 [details] fixes the bug
WebKit Review Bot
Comment 2 2010-08-08 20:08:13 PDT
Attachment 63853 [details] did not build on win: Build output: http://queues.webkit.org/results/3635539
Eric Seidel (no email)
Comment 3 2010-08-09 09:46:36 PDT
Attachment 63853 [details] did not build on mac: Build output: http://queues.webkit.org/results/3637508
Ryosuke Niwa
Comment 4 2010-08-09 13:18:02 PDT
Created attachment 63922 [details] initialized highestAncestorUnicodeBidi
Tony Chang
Comment 5 2010-08-09 14:02:06 PDT
Comment on attachment 63922 [details] initialized highestAncestorUnicodeBidi OK
Ryosuke Niwa
Comment 6 2010-08-09 16:18:48 PDT
Committed r65013: <http://trac.webkit.org/changeset/65013>
Ryosuke Niwa
Comment 7 2010-08-09 17:06:43 PDT
(In reply to comment #5) > (From update of attachment 63922 [details]) > OK Thanks for the review, Tony.
Ryosuke Niwa
Comment 8 2011-05-25 22:39:15 PDT
I don't think the bug 60803 is anything to do with this one.
Note You need to log in before you can comment on or make changes to this bug.