Bug 1273704 - cope with empty files in localisation r=kmag

MozReview-Commit-ID: 9HiwteAs48b

--HG--
extra : transplant_source : %01v%8E%2B%E2%D3%08%DB%1F%1B%29%86%3C%0A%90%B3E%EA%8E%60
This commit is contained in:
Andy McKay 2016-05-18 18:52:22 -07:00
parent 1c6bc4291a
commit 142be954b1
2 changed files with 12 additions and 1 deletions

View File

@ -123,3 +123,11 @@ add_task(function* testInvalidUUID() {
convService.asyncConvertData(FROM_TYPE, TO_TYPE, listener, uri);
}, expectInvalidContextException);
});
// Test that an empty stream does not throw an NS_ERROR_ILLEGAL_VALUE.
add_task(function* testEmptyStream() {
let stream = StringStream("");
let resultStream = convService.convert(stream, FROM_TYPE, TO_TYPE, URI);
equal(resultStream.data, "");
});

View File

@ -225,7 +225,10 @@ AddonLocalizationConverter.prototype = {
this.checkTypes(aFromType, aToType);
let addonId = this.getAddonId(aContext);
let string = NetUtil.readInputStreamToString(aStream, aStream.available());
let string = (
aStream.available() ?
NetUtil.readInputStreamToString(aStream, aStream.available()): ""
);
return this.convertToStream(addonId, string);
},