Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1)

Side by Side Diff: test/cctest/test-decls.cc

Issue 392243002: Reimplement SetProperty using the LookupIterator (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « src/v8natives.js ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2007-2008 the V8 project authors. All rights reserved. 1 // Copyright 2007-2008 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 405 matching lines...) Expand 10 before | Expand all | Expand 10 after
416 } 416 }
417 }; 417 };
418 418
419 419
420 TEST(ExistsInPrototype) { 420 TEST(ExistsInPrototype) {
421 HandleScope scope(CcTest::isolate()); 421 HandleScope scope(CcTest::isolate());
422 422
423 // Sanity check to make sure that the holder of the interceptor 423 // Sanity check to make sure that the holder of the interceptor
424 // really is the prototype object. 424 // really is the prototype object.
425 { ExistsInPrototypeContext context; 425 { ExistsInPrototypeContext context;
426 context.Check("this.x = 87; this.x", 426 context.Check("this.x = 87; this.x", 0, 0, 1, EXPECT_RESULT,
427 0, 427 Number::New(CcTest::isolate(), 87));
428 0,
429 0,
430 EXPECT_RESULT, Number::New(CcTest::isolate(), 87));
431 } 428 }
432 429
433 { ExistsInPrototypeContext context; 430 { ExistsInPrototypeContext context;
434 context.Check("var x; x", 431 context.Check("var x; x",
435 0, 432 0,
436 0, 433 0,
437 0, 434 0,
438 EXPECT_RESULT, Undefined(CcTest::isolate())); 435 EXPECT_RESULT, Undefined(CcTest::isolate()));
439 } 436 }
440 437
(...skipping 268 matching lines...) Expand 10 before | Expand all | Expand 10 after
709 Number::New(CcTest::isolate(), 1)); 706 Number::New(CcTest::isolate(), 1));
710 // TODO(rossberg): All tests should actually be errors in Harmony, 707 // TODO(rossberg): All tests should actually be errors in Harmony,
711 // but we currently do not detect the cases where the first declaration 708 // but we currently do not detect the cases where the first declaration
712 // is not lexical. 709 // is not lexical.
713 context.Check(seconds[j], 710 context.Check(seconds[j],
714 i < 2 ? EXPECT_RESULT : EXPECT_ERROR, 711 i < 2 ? EXPECT_RESULT : EXPECT_ERROR,
715 Number::New(CcTest::isolate(), 2)); 712 Number::New(CcTest::isolate(), 2));
716 } 713 }
717 } 714 }
718 } 715 }
OLDNEW
« no previous file with comments | « src/v8natives.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698