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

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

Issue 93120: Changed the ASSERT's in the cctest's to CHECK's (Closed) Base URL: http://v8.googlecode.com/svn/branches/bleeding_edge/
Patch Set: Created 11 years, 8 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 | « test/cctest/test-debug.cc ('k') | test/cctest/test-hashmap.cc » ('j') | 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 371 matching lines...) Expand 10 before | Expand all | Expand 10 after
382 // Force declaration by returning that the 382 // Force declaration by returning that the
383 // property is absent. 383 // property is absent.
384 state_ = INITIALIZE_IF_ASSIGN; 384 state_ = INITIALIZE_IF_ASSIGN;
385 return False(); 385 return False();
386 case INITIALIZE_IF_ASSIGN: 386 case INITIALIZE_IF_ASSIGN:
387 // Return that the property is present so we only get the 387 // Return that the property is present so we only get the
388 // setter called when initializing with a value. 388 // setter called when initializing with a value.
389 state_ = UNKNOWN; 389 state_ = UNKNOWN;
390 return True(); 390 return True();
391 default: 391 default:
392 ASSERT(state_ == UNKNOWN); 392 CHECK(state_ == UNKNOWN);
393 break; 393 break;
394 } 394 }
395 // Do the lookup in the object. 395 // Do the lookup in the object.
396 return v8::Local<Boolean>(); 396 return v8::Local<Boolean>();
397 } 397 }
398 398
399 private: 399 private:
400 State state_; 400 State state_;
401 }; 401 };
402 402
(...skipping 69 matching lines...) Expand 10 before | Expand all | Expand 10 after
472 state_ = INITIALIZE; 472 state_ = INITIALIZE;
473 return True(); 473 return True();
474 case INITIALIZE: 474 case INITIALIZE:
475 // Force an initialization by returning that 475 // Force an initialization by returning that
476 // the property is absent. This will make sure 476 // the property is absent. This will make sure
477 // that the setter is called and it will not 477 // that the setter is called and it will not
478 // lead to redeclaration conflicts (yet). 478 // lead to redeclaration conflicts (yet).
479 state_ = UNKNOWN; 479 state_ = UNKNOWN;
480 return False(); 480 return False();
481 default: 481 default:
482 ASSERT(state_ == UNKNOWN); 482 CHECK(state_ == UNKNOWN);
483 break; 483 break;
484 } 484 }
485 // Do the lookup in the object. 485 // Do the lookup in the object.
486 return v8::Local<Boolean>(); 486 return v8::Local<Boolean>();
487 } 487 }
488 488
489 private: 489 private:
490 State state_; 490 State state_;
491 }; 491 };
492 492
(...skipping 92 matching lines...) Expand 10 before | Expand all | Expand 10 after
585 HandleScope scope; 585 HandleScope scope;
586 586
587 { AbsentInPrototypeContext context; 587 { AbsentInPrototypeContext context;
588 context.Check("if (false) { var x = 0; }; x", 588 context.Check("if (false) { var x = 0; }; x",
589 0, 589 0,
590 0, 590 0,
591 1, // declaration 591 1, // declaration
592 EXPECT_RESULT, Undefined()); 592 EXPECT_RESULT, Undefined());
593 } 593 }
594 } 594 }
OLDNEW
« no previous file with comments | « test/cctest/test-debug.cc ('k') | test/cctest/test-hashmap.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698