https://chromiumcodereview.appspot.com/10855098/diff/6036/test/mjsunit/compiler/inline-accessors.js
File test/mjsunit/compiler/inline-accessors.js (right):

https://chromiumcodereview.appspot.com/10855098/diff/6036/test/mjsunit/compiler/inline-accessors.js#newcode74
test/mjsunit/compiler/inline-accessors.js:74: TestInlinedGetter(context,
obj, expected);
On 2012/08/17 10:03:51, Michael Starzinger wrote:
Can we add assertFalse(expectException) after the test?

Done.

https://chromiumcodereview.appspot.com/10855098/diff/6036/test/mjsunit/compiler/inline-accessors.js#newcode234
test/mjsunit/compiler/inline-accessors.js:234:
TestInlinedSetter(context, obj, value, expected);
On 2012/08/17 10:03:51, Michael Starzinger wrote:
Likewise.

Done.

https://chromiumcodereview.appspot.com/10855098/

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to