Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2024-10-23 02:05:42 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Backed out changeset 28bbd9862551 (bug 891107) for mochitest-other failure on a CLOSED TREE

Browse Source
This commit is contained in:
Tooru Fujisawa 2015-04-18 05:51:22 +09:00
parent c7e3f01856
commit 3ca77f8529
12 changed files with 71 additions and 243 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

159
js/src/ctypes/CTypes.cpp
View File

@ -1234,15 +1234,6 @@ ArgumentConvError(JSContext* cx, HandleValue actual, const char* funStr,
return false;
}
static bool
ArgumentLengthError(JSContext* cx, const char* fun, const char* count,
const char* s)
{
JS_ReportErrorNumber(cx, GetErrorMessage, nullptr,
CTYPESMSG_WRONG_ARG_LENGTH, fun, count, s);
return false;
}
static bool
ArrayLengthMismatch(JSContext* cx, unsigned expectedLength, HandleObject arrObj,
unsigned actualLength, HandleValue actual,
@ -3843,7 +3834,8 @@ CType::ConstructBasic(JSContext* cx,
const CallArgs& args)
{
if (args.length() > 1) {
return ArgumentLengthError(cx, "CType constructor", "at most one", "");
JS_ReportError(cx, "CType constructor takes zero or one argument");
return false;
}
// construct a CData object
@ -4365,7 +4357,8 @@ CType::CreateArray(JSContext* cx, unsigned argc, jsval* vp)
// Construct and return a new ArrayType object.
if (args.length() > 1) {
return ArgumentLengthError(cx, "CType.prototype.array", "at most one", "");
JS_ReportError(cx, "array takes zero or one argument");
return false;
}
// Convert the length argument to a size_t.
@ -4504,7 +4497,8 @@ ABI::ToSource(JSContext* cx, unsigned argc, jsval* vp)
{
CallArgs args = CallArgsFromVp(argc, vp);
if (args.length() != 0) {
return ArgumentLengthError(cx, "ABI.prototype.toSource", "no", "s");
JS_ReportError(cx, "toSource takes zero arguments");
return false;
}
JSObject* obj = JS_THIS_OBJECT(cx, vp);
@ -4548,7 +4542,8 @@ PointerType::Create(JSContext* cx, unsigned argc, jsval* vp)
CallArgs args = CallArgsFromVp(argc, vp);
// Construct and return a new PointerType object.
if (args.length() != 1) {
return ArgumentLengthError(cx, "PointerType", "one", "");
JS_ReportError(cx, "PointerType takes one argument");
return false;
}
jsval arg = args[0];
@ -4613,8 +4608,8 @@ PointerType::ConstructData(JSContext* cx,
}
if (args.length() > 3) {
return ArgumentLengthError(cx, "PointerType constructor", "0, 1, 2, or 3",
"s");
JS_ReportError(cx, "constructor takes 0, 1, 2, or 3 arguments");
return false;
}
RootedObject result(cx, CData::Create(cx, obj, NullPtr(), nullptr, true));
@ -4648,7 +4643,8 @@ PointerType::ConstructData(JSContext* cx,
//
if (!looksLikeClosure) {
if (args.length() != 1) {
return ArgumentLengthError(cx, "FunctionType constructor", "one", "");
JS_ReportError(cx, "first argument must be a function");
return false;
}
return ExplicitConvert(cx, args[0], obj, CData::GetData(result),
ConversionType::Construct);
@ -4849,7 +4845,8 @@ ArrayType::Create(JSContext* cx, unsigned argc, jsval* vp)
CallArgs args = CallArgsFromVp(argc, vp);
// Construct and return a new ArrayType object.
if (args.length() < 1 || args.length() > 2) {
return ArgumentLengthError(cx, "ArrayType", "one or two", "s");
JS_ReportError(cx, "ArrayType takes one or two arguments");
return false;
}
if (args[0].isPrimitive() ||
@ -4949,14 +4946,14 @@ ArrayType::ConstructData(JSContext* cx,
// with a length argument, or with an actual JS array.
if (CType::IsSizeDefined(obj)) {
if (args.length() > 1) {
return ArgumentLengthError(cx, "size defined ArrayType constructor",
"at most one", "");
JS_ReportError(cx, "constructor takes zero or one argument");
return false;
}
} else {
if (args.length() != 1) {
return ArgumentLengthError(cx, "size undefined ArrayType constructor",
"one", "");
JS_ReportError(cx, "constructor takes one argument");
return false;
}
RootedObject baseType(cx, GetBaseType(obj));
@ -5271,8 +5268,8 @@ ArrayType::AddressOfElement(JSContext* cx, unsigned argc, jsval* vp)
}
if (args.length() != 1) {
return ArgumentLengthError(cx, "ArrayType.prototype.addressOfElement",
"one", "");
JS_ReportError(cx, "addressOfElement takes one argument");
return false;
}
RootedObject baseType(cx, GetBaseType(typeObj));
@ -5392,7 +5389,8 @@ StructType::Create(JSContext* cx, unsigned argc, jsval* vp)
// Construct and return a new StructType object.
if (args.length() < 1 || args.length() > 2) {
return ArgumentLengthError(cx, "StructType", "one or two", "s");
JS_ReportError(cx, "StructType takes one or two arguments");
return false;
}
jsval name = args[0];
@ -5680,7 +5678,8 @@ StructType::Define(JSContext* cx, unsigned argc, jsval* vp)
}
if (args.length() != 1) {
return ArgumentLengthError(cx, "StructType.prototype.define", "one", "");
JS_ReportError(cx, "define takes one argument");
return false;
}
jsval arg = args[0];
@ -5767,14 +5766,9 @@ StructType::ConstructData(JSContext* cx,
return true;
}
size_t count = fields->count();
if (count >= 2) {
char fieldLengthStr[32];
JS_snprintf(fieldLengthStr, 32, "0, 1, or %u", count);
return ArgumentLengthError(cx, "StructType constructor", fieldLengthStr,
"s");
}
return ArgumentLengthError(cx, "StructType constructor", "at most one", "");
JS_ReportError(cx, "constructor takes 0, 1, or %u arguments",
fields->count());
return false;
}
const FieldInfoHash*
@ -5970,8 +5964,8 @@ StructType::AddressOfField(JSContext* cx, unsigned argc, jsval* vp)
}
if (args.length() != 1) {
return ArgumentLengthError(cx, "StructType.prototype.addressOfField",
"one", "");
JS_ReportError(cx, "addressOfField takes one argument");
return false;
}
if (!args[0].isString()) {
@ -6315,7 +6309,8 @@ FunctionType::Create(JSContext* cx, unsigned argc, jsval* vp)
// Construct and return a new FunctionType object.
CallArgs args = CallArgsFromVp(argc, vp);
if (args.length() < 2 || args.length() > 3) {
return ArgumentLengthError(cx, "FunctionType", "two or three", "s");
JS_ReportError(cx, "FunctionType takes two or three arguments");
return false;
}
AutoValueVector argTypes(cx);
@ -7160,7 +7155,8 @@ CData::Address(JSContext* cx, unsigned argc, jsval* vp)
{
CallArgs args = CallArgsFromVp(argc, vp);
if (args.length() != 0) {
return ArgumentLengthError(cx, "CData.prototype.address", "no", "s");
JS_ReportError(cx, "address takes zero arguments");
return false;
}
RootedObject obj(cx, JS_THIS_OBJECT(cx, vp));
@ -7194,7 +7190,8 @@ CData::Cast(JSContext* cx, unsigned argc, jsval* vp)
{
CallArgs args = CallArgsFromVp(argc, vp);
if (args.length() != 2) {
return ArgumentLengthError(cx, "ctypes.cast", "two", "s");
JS_ReportError(cx, "cast takes two arguments");
return false;
}
if (args[0].isPrimitive() || !CData::IsCData(&args[0].toObject())) {
@ -7234,7 +7231,8 @@ CData::GetRuntime(JSContext* cx, unsigned argc, jsval* vp)
{
CallArgs args = CallArgsFromVp(argc, vp);
if (args.length() != 1) {
return ArgumentLengthError(cx, "ctypes.getRuntime", "one", "");
JS_ReportError(cx, "getRuntime takes one argument");
return false;
}
if (args[0].isPrimitive() || !CType::IsCType(&args[0].toObject())) {
@ -7266,11 +7264,8 @@ ReadStringCommon(JSContext* cx, InflateUTF8Method inflateUTF8, unsigned argc, js
{
CallArgs args = CallArgsFromVp(argc, vp);
if (args.length() != 0) {
if (inflateUTF8 == JS::UTF8CharsToNewTwoByteCharsZ) {
return ArgumentLengthError(cx, "CData.prototype.readString", "no", "s");
}
return ArgumentLengthError(cx, "CData.prototype.readStringReplaceMalformed",
"no", "s");
JS_ReportError(cx, "readString takes zero arguments");
return false;
}
JSObject* obj = CDataFinalizer::GetCData(cx, JS_THIS_OBJECT(cx, vp));
@ -7385,7 +7380,8 @@ CData::ToSource(JSContext* cx, unsigned argc, jsval* vp)
{
CallArgs args = CallArgsFromVp(argc, vp);
if (args.length() != 0) {
return ArgumentLengthError(cx, "CData.prototype.toSource", "no", "s");
JS_ReportError(cx, "toSource takes zero arguments");
return false;
}
JSObject* obj = JS_THIS_OBJECT(cx, vp);
@ -7613,7 +7609,8 @@ CDataFinalizer::Construct(JSContext* cx, unsigned argc, jsval* vp)
}
if (args.length() != 2) {
return ArgumentLengthError(cx, "CDataFinalizer constructor", "two", "s");
JS_ReportError(cx, "CDataFinalizer takes 2 arguments");
return false;
}
JS::HandleValue valCodePtr = args[1];
@ -7827,8 +7824,8 @@ CDataFinalizer::Methods::Forget(JSContext* cx, unsigned argc, jsval* vp)
{
CallArgs args = CallArgsFromVp(argc, vp);
if (args.length() != 0) {
return ArgumentLengthError(cx, "CDataFinalizer.prototype.forget", "no",
"s");
JS_ReportError(cx, "CDataFinalizer.prototype.forget takes no arguments");
return false;
}
JS::Rooted<JSObject*> obj(cx, args.thisv().toObjectOrNull());
@ -7875,8 +7872,8 @@ CDataFinalizer::Methods::Dispose(JSContext* cx, unsigned argc, jsval* vp)
{
CallArgs args = CallArgsFromVp(argc, vp);
if (args.length() != 0) {
return ArgumentLengthError(cx, "CDataFinalizer.prototype.dispose", "no",
"s");
JS_ReportError(cx, "CDataFinalizer.prototype.dispose takes no arguments");
return false;
}
RootedObject obj(cx, JS_THIS_OBJECT(cx, vp));
@ -8049,12 +8046,8 @@ Int64Base::ToString(JSContext* cx,
bool isUnsigned)
{
if (args.length() > 1) {
if (isUnsigned) {
return ArgumentLengthError(cx, "UInt64.prototype.toString",
"at most one", "");
}
return ArgumentLengthError(cx, "Int64.prototype.toString",
"at most one", "");
JS_ReportError(cx, "toString takes zero or one argument");
return false;
}
int radix = 10;
@ -8090,10 +8083,8 @@ Int64Base::ToSource(JSContext* cx,
bool isUnsigned)
{
if (args.length() != 0) {
if (isUnsigned) {
return ArgumentLengthError(cx, "UInt64.prototype.toSource", "no", "s");
}
return ArgumentLengthError(cx, "Int64.prototype.toSource", "no", "s");
JS_ReportError(cx, "toSource takes zero arguments");
return false;
}
// Return a decimal string suitable for constructing the number.
@ -8124,7 +8115,8 @@ Int64::Construct(JSContext* cx,
// Construct and return a new Int64 object.
if (args.length() != 1) {
return ArgumentLengthError(cx, "Int64 constructor", "one", "");
JS_ReportError(cx, "Int64 takes one argument");
return false;
}
int64_t i = 0;
@ -8187,10 +8179,8 @@ bool
Int64::Compare(JSContext* cx, unsigned argc, jsval* vp)
{
CallArgs args = CallArgsFromVp(argc, vp);
if (args.length() != 2) {
return ArgumentLengthError(cx, "Int64.compare", "two", "s");
}
if (args[0].isPrimitive() ||
if (args.length() != 2 ||
args[0].isPrimitive() ||
args[1].isPrimitive() ||
!Int64::IsInt64(&args[0].toObject()) ||
!Int64::IsInt64(&args[1].toObject())) {
@ -8222,10 +8212,8 @@ bool
Int64::Lo(JSContext* cx, unsigned argc, jsval* vp)
{
CallArgs args = CallArgsFromVp(argc, vp);
if (args.length() != 1) {
return ArgumentLengthError(cx, "Int64.lo", "one", "");
}
if (args[0].isPrimitive() || !Int64::IsInt64(&args[0].toObject())) {
if (args.length() != 1 || args[0].isPrimitive() ||
!Int64::IsInt64(&args[0].toObject())) {
JS_ReportError(cx, "lo takes one Int64 argument");
return false;
}
@ -8242,10 +8230,8 @@ bool
Int64::Hi(JSContext* cx, unsigned argc, jsval* vp)
{
CallArgs args = CallArgsFromVp(argc, vp);
if (args.length() != 1) {
return ArgumentLengthError(cx, "Int64.hi", "one", "");
}
if (args[0].isPrimitive() || !Int64::IsInt64(&args[0].toObject())) {
if (args.length() != 1 || args[0].isPrimitive() ||
!Int64::IsInt64(&args[0].toObject())) {
JS_ReportError(cx, "hi takes one Int64 argument");
return false;
}
@ -8263,7 +8249,8 @@ Int64::Join(JSContext* cx, unsigned argc, jsval* vp)
{
CallArgs args = CallArgsFromVp(argc, vp);
if (args.length() != 2) {
return ArgumentLengthError(cx, "Int64.join", "two", "s");
JS_ReportError(cx, "join takes two arguments");
return false;
}
int32_t hi;
@ -8299,7 +8286,8 @@ UInt64::Construct(JSContext* cx,
// Construct and return a new UInt64 object.
if (args.length() != 1) {
return ArgumentLengthError(cx, "UInt64 constructor", "one", "");
JS_ReportError(cx, "UInt64 takes one argument");
return false;
}
uint64_t u = 0;
@ -8362,10 +8350,8 @@ bool
UInt64::Compare(JSContext* cx, unsigned argc, jsval* vp)
{
CallArgs args = CallArgsFromVp(argc, vp);
if (args.length() != 2) {
return ArgumentLengthError(cx, "UInt64.compare", "two", "s");
}
if (args[0].isPrimitive() ||
if (args.length() != 2 ||
args[0].isPrimitive() ||
args[1].isPrimitive() ||
!UInt64::IsUInt64(&args[0].toObject()) ||
!UInt64::IsUInt64(&args[1].toObject())) {
@ -8393,10 +8379,8 @@ bool
UInt64::Lo(JSContext* cx, unsigned argc, jsval* vp)
{
CallArgs args = CallArgsFromVp(argc, vp);
if (args.length() != 1) {
return ArgumentLengthError(cx, "UInt64.lo", "one", "");
}
if (args[0].isPrimitive() || !UInt64::IsUInt64(&args[0].toObject())) {
if (args.length() != 1 || args[0].isPrimitive() ||
!UInt64::IsUInt64(&args[0].toObject())) {
JS_ReportError(cx, "lo takes one UInt64 argument");
return false;
}
@ -8413,10 +8397,8 @@ bool
UInt64::Hi(JSContext* cx, unsigned argc, jsval* vp)
{
CallArgs args = CallArgsFromVp(argc, vp);
if (args.length() != 1) {
return ArgumentLengthError(cx, "UInt64.hi", "one", "");
}
if (args[0].isPrimitive() || !UInt64::IsUInt64(&args[0].toObject())) {
if (args.length() != 1 || args[0].isPrimitive() ||
!UInt64::IsUInt64(&args[0].toObject())) {
JS_ReportError(cx, "hi takes one UInt64 argument");
return false;
}
@ -8434,7 +8416,8 @@ UInt64::Join(JSContext* cx, unsigned argc, jsval* vp)
{
CallArgs args = CallArgsFromVp(argc, vp);
if (args.length() != 2) {
return ArgumentLengthError(cx, "UInt64.join", "two", "s");
JS_ReportError(cx, "join takes two arguments");
return false;
}
uint32_t hi;

3
js/src/ctypes/ctypes.msg
View File

@ -32,6 +32,3 @@ MSG_DEF(CTYPESMSG_PROP_NONSTRING,3, JSEXN_TYPEERR, "property name {0} of {1} is
/* data finalizer */
MSG_DEF(CTYPESMSG_EMPTY_FIN, 1, JSEXN_TYPEERR, "attempting to convert an empty CDataFinalizer{0}")
MSG_DEF(CTYPESMSG_FIN_SIZE_ERROR,2, JSEXN_TYPEERR, "expected an object with the same size as argument 1 of {0}, got {1}")
/* native function */
MSG_DEF(CTYPESMSG_WRONG_ARG_LENGTH,3, JSEXN_TYPEERR, "{0} takes {1} argument{2}")

9
js/src/jit-test/tests/ctypes/argument-length-abi.js
View File

@ -1,9 +0,0 @@
load(libdir + 'asserts.js');
function test() {
assertTypeErrorMessage(() => { ctypes.default_abi.toSource(1); },
"ABI.prototype.toSource takes no arguments");
}
if (typeof ctypes === "object")
test();

15
js/src/jit-test/tests/ctypes/argument-length-array.js
View File

@ -1,15 +0,0 @@
load(libdir + 'asserts.js');
function test() {
assertTypeErrorMessage(() => { ctypes.ArrayType(); },
"ArrayType takes one or two arguments");
assertTypeErrorMessage(() => { ctypes.int32_t.array(10)(1, 2); },
"size defined ArrayType constructor takes at most one argument");
assertTypeErrorMessage(() => { ctypes.int32_t.array()(1, 2); },
"size undefined ArrayType constructor takes one argument");
assertTypeErrorMessage(() => { ctypes.int32_t.array(10)().addressOfElement(); },
"ArrayType.prototype.addressOfElement takes one argument");
}
if (typeof ctypes === "object")
test();

15
js/src/jit-test/tests/ctypes/argument-length-cdata.js
View File

@ -1,15 +0,0 @@
load(libdir + 'asserts.js');
function test() {
assertTypeErrorMessage(() => { ctypes.int32_t(0).address(1); },
"CData.prototype.address takes no arguments");
assertTypeErrorMessage(() => { ctypes.char.array(10)().readString(1); },
"CData.prototype.readString takes no arguments");
assertTypeErrorMessage(() => { ctypes.char.array(10)().readStringReplaceMalformed(1); },
"CData.prototype.readStringReplaceMalformed takes no arguments");
assertTypeErrorMessage(() => { ctypes.int32_t(0).toSource(1); },
"CData.prototype.toSource takes no arguments");
}
if (typeof ctypes === "object")
test();

11
js/src/jit-test/tests/ctypes/argument-length-ctypes.js
View File

@ -1,11 +0,0 @@
load(libdir + 'asserts.js');
function test() {
assertTypeErrorMessage(() => { ctypes.cast(); },
"ctypes.cast takes two arguments");
assertTypeErrorMessage(() => { ctypes.getRuntime(); },
"ctypes.getRuntime takes one argument");
}
if (typeof ctypes === "object")
test();

16
js/src/jit-test/tests/ctypes/argument-length-finalizer.js
View File

@ -1,16 +0,0 @@
load(libdir + 'asserts.js');
function test() {
assertTypeErrorMessage(() => { ctypes.CDataFinalizer(1); },
"CDataFinalizer constructor takes two arguments");
let fin = ctypes.CDataFinalizer(ctypes.int32_t(0), ctypes.FunctionType(ctypes.default_abi, ctypes.int32_t, [ctypes.int32_t]).ptr(x => x));
assertTypeErrorMessage(() => { fin.forget(1); },
"CDataFinalizer.prototype.forget takes no arguments");
assertTypeErrorMessage(() => { fin.dispose(1); },
"CDataFinalizer.prototype.dispose takes no arguments");
fin.forget();
}
if (typeof ctypes === "object")
test();

11
js/src/jit-test/tests/ctypes/argument-length-function.js
View File

@ -1,11 +0,0 @@
load(libdir + 'asserts.js');
function test() {
assertTypeErrorMessage(() => { ctypes.FunctionType(); },
"FunctionType takes two or three arguments");
assertTypeErrorMessage(() => { ctypes.FunctionType(ctypes.default_abi, ctypes.void_t, []).ptr({}, 1); },
"FunctionType constructor takes one argument");
}
if (typeof ctypes === "object")
test();

36
js/src/jit-test/tests/ctypes/argument-length-int64.js
View File

@ -1,36 +0,0 @@
load(libdir + 'asserts.js');
function test() {
assertTypeErrorMessage(() => { ctypes.Int64(1).toString(1, 2); },
"Int64.prototype.toString takes at most one argument");
assertTypeErrorMessage(() => { ctypes.Int64(1).toSource(1); },
"Int64.prototype.toSource takes no arguments");
assertTypeErrorMessage(() => { ctypes.Int64(); },
"Int64 constructor takes one argument");
assertTypeErrorMessage(() => { ctypes.Int64.compare(); },
"Int64.compare takes two arguments");
assertTypeErrorMessage(() => { ctypes.Int64.lo(); },
"Int64.lo takes one argument");
assertTypeErrorMessage(() => { ctypes.Int64.hi(); },
"Int64.hi takes one argument");
assertTypeErrorMessage(() => { ctypes.Int64.join(); },
"Int64.join takes two arguments");
assertTypeErrorMessage(() => { ctypes.UInt64(1).toString(1, 2); },
"UInt64.prototype.toString takes at most one argument");
assertTypeErrorMessage(() => { ctypes.UInt64(1).toSource(1); },
"UInt64.prototype.toSource takes no arguments");
assertTypeErrorMessage(() => { ctypes.UInt64(); },
"UInt64 constructor takes one argument");
assertTypeErrorMessage(() => { ctypes.UInt64.compare(); },
"UInt64.compare takes two arguments");
assertTypeErrorMessage(() => { ctypes.UInt64.lo(); },
"UInt64.lo takes one argument");
assertTypeErrorMessage(() => { ctypes.UInt64.hi(); },
"UInt64.hi takes one argument");
assertTypeErrorMessage(() => { ctypes.UInt64.join(); },
"UInt64.join takes two arguments");
}
if (typeof ctypes === "object")
test();

11
js/src/jit-test/tests/ctypes/argument-length-pointer.js
View File

@ -1,11 +0,0 @@
load(libdir + 'asserts.js');
function test() {
assertTypeErrorMessage(() => { ctypes.PointerType(); },
"PointerType takes one argument");
assertTypeErrorMessage(() => { ctypes.int32_t.ptr(1, 2, 3, 4); },
"PointerType constructor takes 0, 1, 2, or 3 arguments");
}
if (typeof ctypes === "object")
test();

11
js/src/jit-test/tests/ctypes/argument-length-primitive.js
View File

@ -1,11 +0,0 @@
load(libdir + 'asserts.js');
function test() {
assertTypeErrorMessage(() => { ctypes.int32_t(1, 2, 3); },
"CType constructor takes at most one argument");
assertTypeErrorMessage(() => { ctypes.int32_t.array(1, 2); },
"CType.prototype.array takes at most one argument");
}
if (typeof ctypes === "object")
test();

17
js/src/jit-test/tests/ctypes/argument-length-struct.js
View File

@ -1,17 +0,0 @@
load(libdir + 'asserts.js');
function test() {
assertTypeErrorMessage(() => { ctypes.StructType(); },
"StructType takes one or two arguments");
assertTypeErrorMessage(() => { ctypes.StructType("a").define(); },
"StructType.prototype.define takes one argument");
assertTypeErrorMessage(() => { ctypes.StructType("a", [])(1, 2, 3); },
"StructType constructor takes at most one argument");
assertTypeErrorMessage(() => { ctypes.StructType("a", [ {"x": ctypes.int32_t }, {"y": ctypes.int32_t }, {"z": ctypes.int32_t }])(1, 2); },
"StructType constructor takes 0, 1, or 3 arguments");
assertTypeErrorMessage(() => { ctypes.StructType("a", [ {"x": ctypes.int32_t } ])().addressOfField(); },
"StructType.prototype.addressOfField takes one argument");
}
if (typeof ctypes === "object")
test();