Reviewers: Michael Starzinger,

Description:
Collect garbage before running mjsunit/fast-prototype

When not using a snapshot, after turning on i18n, there's just enough
garbage after creating a context to trigger gc at the wrong moment.
Since the test uses natives syntax to access information that would
otherwise be hidden from javascript, this makes the test fail

BUG=none
[email protected]
TEST=mjsunit/fast-prototype passes on ia32.release with no snapshot

Please review this at https://codereview.chromium.org/23452047/

SVN Base: https://v8.googlecode.com/svn/branches/bleeding_edge

Affected files (+3, -1 lines):
  M test/mjsunit/fast-prototype.js


Index: test/mjsunit/fast-prototype.js
diff --git a/test/mjsunit/fast-prototype.js b/test/mjsunit/fast-prototype.js
index 83bcffe44fa5c15945691ba6f23236a98c443b6d..d700c3c3cce45e4245a477b364b4cf1a98f009d9 100644
--- a/test/mjsunit/fast-prototype.js
+++ b/test/mjsunit/fast-prototype.js
@@ -25,7 +25,7 @@
 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.

-// Flags: --allow-natives-syntax
+// Flags: --allow-natives-syntax --expose-gc

 // TODO(mstarzinger): This test does not succeed when GCs happen in
 // between prototype transitions, we disable GC stress for now.
@@ -84,6 +84,8 @@ function test(use_new, add_first, set__proto__, same_map_as) {
   return proto;
 }

+// TODO(mstarzinger): This test fails easily if gc happens at the wrong time.
+gc();

 for (var i = 0; i < 4; i++) {
   var set__proto__ = ((i & 1) != 0);


--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
--- You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to