129150 - NS_ERROR_UENC_NOMAPPING should not be used for nsIUnicodeDecoder

Status: VERIFIED FIXED

(Core :: Internationalization, defect)

Description

[nhottanscp]

NS_ERROR_UENC_NOMAPPING is an error code for nsIUnicodeEncoder but it is used as
a return value of ISO-2022-JP decoder,
http://lxr.mozilla.org/seamonkey/source/intl/uconv/ucvja/nsJapaneseToUnicode.cpp#970

The error for that case is invalid character code for the specified charset
instead of cannot map to Unicode. I think we can return
NS_ERROR_UDEC_ILLEGALINPUT instead.

Comment 1

[Rui Xu]

Should IQA be involved in this? if so, could you please provide the test cases?

Comment 2

[Andreas Becker]

Switching QA contact to ftang.  Frank, can you please handle this within 
development and re-assign further as you see is appropriate?  Feel free and 
re-assign back to IQA once you have a test case for IQA.  Thanks.

Comment 3

[Frank Tang]

this was fixed. 

Comment 4

[Roy Yokoyama]

Verifed.
-return NS_ERROR_UENC_NOMAPPING;                                              
+return NS_ERROR_UNEXPECTED;