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

Side by Side Diff: src/isolate.cc

Issue 1979473003: Version 5.1.281.35 (cherry-pick) (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@5.1
Patch Set: Created 4 years, 7 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
« no previous file with comments | « src/builtins.cc ('k') | src/objects.h » ('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 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "src/isolate.h" 5 #include "src/isolate.h"
6 6
7 #include <stdlib.h> 7 #include <stdlib.h>
8 8
9 #include <fstream> // NOLINT(readability/streams) 9 #include <fstream> // NOLINT(readability/streams)
10 #include <sstream> 10 #include <sstream>
(...skipping 2535 matching lines...) Expand 10 before | Expand all | Expand 10 after
2546 2546
2547 return cell_reports_intact; 2547 return cell_reports_intact;
2548 } 2548 }
2549 2549
2550 bool Isolate::IsArraySpeciesLookupChainIntact() { 2550 bool Isolate::IsArraySpeciesLookupChainIntact() {
2551 if (!FLAG_harmony_species) return true; 2551 if (!FLAG_harmony_species) return true;
2552 // Note: It would be nice to have debug checks to make sure that the 2552 // Note: It would be nice to have debug checks to make sure that the
2553 // species protector is accurate, but this would be hard to do for most of 2553 // species protector is accurate, but this would be hard to do for most of
2554 // what the protector stands for: 2554 // what the protector stands for:
2555 // - You'd need to traverse the heap to check that no Array instance has 2555 // - You'd need to traverse the heap to check that no Array instance has
2556 // a constructor property or a modified __proto__ 2556 // a constructor property
2557 // - To check that Array[Symbol.species] == Array, JS code has to execute, 2557 // - To check that Array[Symbol.species] == Array, JS code has to execute,
2558 // but JS cannot be invoked in callstack overflow situations 2558 // but JS cannot be invoked in callstack overflow situations
2559 // All that could be checked reliably is that 2559 // All that could be checked reliably is that
2560 // Array.prototype.constructor == Array. Given that limitation, no check is 2560 // Array.prototype.constructor == Array. Given that limitation, no check is
2561 // done here. In place, there are mjsunit tests harmony/array-species* which 2561 // done here. In place, there are mjsunit tests harmony/array-species* which
2562 // ensure that behavior is correct in various invalid protector cases. 2562 // ensure that behavior is correct in various invalid protector cases.
2563 2563
2564 PropertyCell* species_cell = heap()->species_protector(); 2564 PropertyCell* species_cell = heap()->species_protector();
2565 return species_cell->value()->IsSmi() && 2565 return species_cell->value()->IsSmi() &&
2566 Smi::cast(species_cell->value())->value() == kArrayProtectorValid; 2566 Smi::cast(species_cell->value())->value() == kArrayProtectorValid;
(...skipping 424 matching lines...) Expand 10 before | Expand all | Expand 10 after
2991 // Then check whether this scope intercepts. 2991 // Then check whether this scope intercepts.
2992 if ((flag & intercept_mask_)) { 2992 if ((flag & intercept_mask_)) {
2993 intercepted_flags_ |= flag; 2993 intercepted_flags_ |= flag;
2994 return true; 2994 return true;
2995 } 2995 }
2996 return false; 2996 return false;
2997 } 2997 }
2998 2998
2999 } // namespace internal 2999 } // namespace internal
3000 } // namespace v8 3000 } // namespace v8
OLDNEW
« no previous file with comments | « src/builtins.cc ('k') | src/objects.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698