Bug 775852. Actually follow the WebIDL spec for creating JS representations of sequences. r=khuey

This commit is contained in:
Boris Zbarsky 2012-07-27 00:09:09 -04:00
Родитель a54523e5c9
Коммит cf68660a3b
3 изменённых файлов: 54 добавлений и 1 удалений

Просмотреть файл

@ -2340,7 +2340,8 @@ if (%s.IsNull()) {
type.inner, descriptorProvider,
{
'result' : "%s[i]" % result,
'successCode': ("if (!JS_SetElement(cx, returnArray, i, &tmp)) {\n"
'successCode': ("if (!JS_DefineElement(cx, returnArray, i, tmp,\n"
" NULL, NULL, JSPROP_ENUMERATE)) {\n"
" return false;\n"
"}"),
'jsvalRef': "tmp",

Просмотреть файл

@ -58,6 +58,7 @@ MOCHITEST_FILES = \
test_traceProtos.html \
test_forOf.html \
forOf_iframe.html \
test_sequence_wrapping.html \
$(NULL)
# Include rules.mk before any of our targets so our first target is coming from

Просмотреть файл

@ -0,0 +1,51 @@
<!DOCTYPE HTML>
<html>
<!--
https://bugzilla.mozilla.org/show_bug.cgi?id=775852
-->
<head>
<meta charset="utf-8">
<title>Test for Bug 775852</title>
<script type="application/javascript" src="/tests/SimpleTest/SimpleTest.js"></script>
<link rel="stylesheet" type="text/css" href="/tests/SimpleTest/test.css"/>
</head>
<body>
<a target="_blank" href="https://bugzilla.mozilla.org/show_bug.cgi?id=775852">Mozilla Bug 775852</a>
<p id="display"></p>
<div id="content" style="display: none">
<canvas width="1" height="1" id="c"></canvas>
</div>
<pre id="test">
<script type="application/javascript">
/** Test for Bug 775852 **/
var gl = $("c").getContext("experimental-webgl");
var setterCalled = false;
extLength = gl.getSupportedExtensions().length;
ok(extLength > 0,
"This test won't work right if we have no supported extensions");
Object.defineProperty(Array.prototype, "0",
{
set: function(val) {
setterCalled = true;
}
});
// Test that our property got defined correctly
var arr = []
arr[0] = 5;
is(setterCalled, true, "Setter should be called when setting prop on array");
setterCalled = false;
is(gl.getSupportedExtensions().length, extLength,
"We should still have the same number of extensions");
is(setterCalled, false,
"Setter should not be called when getting supported extensions");
</script>
</pre>
</body>
</html>