RESOLVED FIXED 113518
use XMLHttpRequestResponseType enumeration in XMLHttpRequest.idl 
https://bugs.webkit.org/show_bug.cgi?id=113518
Summary use XMLHttpRequestResponseType enumeration in XMLHttpRequest.idl
arno.
Reported 2013-03-28 10:45:13 PDT
Hi, since idl enumerations are now supported, we can now use XMLHttpRequestResponseType for XMLHttpRequest responseType.
Attachments
patch proposal (4.31 KB, patch)
2013-03-28 11:00 PDT, arno. 		no flags
DetailsFormatted DiffDiff
updated patch: comment json in idl + tests all valid types in xmlhttprequest-responsetype-set-invalidtype (5.43 KB, patch)
2013-03-28 14:24 PDT, arno. 		no flags
DetailsFormatted DiffDiff
updated patch: rename xmlhttprequest-responsetype-set-invalidtype to xmlhttprequest-set-responsetype (8.08 KB, patch)
2013-03-28 14:35 PDT, arno. 		no flags
DetailsFormatted DiffDiff
updated patch: udpate xmlhttprequest-set-responsetype-expected; hopefully this one is good (8.26 KB, patch)
2013-03-28 14:45 PDT, arno. 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
arno.
Comment 1 2013-03-28 11:00:15 PDT
Created attachment 195600 [details] patch proposal
Kentaro Hara
Comment 2 2013-03-28 11:10:24 PDT
Comment on attachment 195600 [details] patch proposal View in context: https://bugs.webkit.org/attachment.cgi?id=195600&action=review > Source/WebCore/ChangeLog:16 > + Already covered by > + xmlhttprequest-responsetype-set-invalidtype-expected.txt Is there any test for valid types? > Source/WebCore/xml/XMLHttpRequest.idl:34 > + "json", I thought "json" is not supported: https://bugs.webkit.org/show_bug.cgi?id=73648 Maybe it would be better to comment out "json" with a FIXME.
arno.
Comment 3 2013-03-28 14:24:24 PDT
Created attachment 195641 [details] updated patch: comment json in idl + tests all valid types in xmlhttprequest-responsetype-set-invalidtype
Kentaro Hara
Comment 4 2013-03-28 14:30:31 PDT
Comment on attachment 195641 [details] updated patch: comment json in idl + tests all valid types in xmlhttprequest-responsetype-set-invalidtype View in context: https://bugs.webkit.org/attachment.cgi?id=195641&action=review Looks OK with nits. > LayoutTests/ChangeLog:12 > + * fast/xmlhttprequest/xmlhttprequest-responsetype-set-invalidtype-expected.txt: Let's rename the test to xmlhttprequest-set-responsetype.html, as we're now testing valid types as well.
arno.
Comment 5 2013-03-28 14:35:42 PDT
Created attachment 195645 [details] updated patch: rename xmlhttprequest-responsetype-set-invalidtype to xmlhttprequest-set-responsetype
Kentaro Hara
Comment 6 2013-03-28 14:39:00 PDT
Comment on attachment 195645 [details] updated patch: rename xmlhttprequest-responsetype-set-invalidtype to xmlhttprequest-set-responsetype LGTM
Kentaro Hara
Comment 7 2013-03-28 14:40:00 PDT
Comment on attachment 195645 [details] updated patch: rename xmlhttprequest-responsetype-set-invalidtype to xmlhttprequest-set-responsetype Oops, it looks like you forgot to update xmlhttprequest-set-responsetype-expected.txt.
arno.
Comment 8 2013-03-28 14:45:54 PDT
Created attachment 195648 [details] updated patch: udpate xmlhttprequest-set-responsetype-expected; hopefully this one is good
Kentaro Hara
Comment 9 2013-03-28 15:12:09 PDT
Comment on attachment 195648 [details] updated patch: udpate xmlhttprequest-set-responsetype-expected; hopefully this one is good Looks good.
WebKit Review Bot
Comment 10 2013-03-28 15:38:03 PDT
Comment on attachment 195648 [details] updated patch: udpate xmlhttprequest-set-responsetype-expected; hopefully this one is good Clearing flags on attachment: 195648 Committed r147172: <http://trac.webkit.org/changeset/147172>
WebKit Review Bot
Comment 11 2013-03-28 15:38:07 PDT
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.