From 8de3c01fe4cc85dc25db049949a4dfd312e0c492 Mon Sep 17 00:00:00 2001
From: Guro Bokum <jiojiajiu@gmail.com>
Date: Fri, 6 Mar 2015 00:17:37 +0700
Subject: [PATCH] RootCollection doesn't check if its SmallVec has spilled
 #5037

---
 components/script/dom/bindings/js.rs | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/components/script/dom/bindings/js.rs b/components/script/dom/bindings/js.rs
index 7ada9f45d14a..ae09a8bf5b82 100644
--- a/components/script/dom/bindings/js.rs
+++ b/components/script/dom/bindings/js.rs
@@ -57,7 +57,7 @@ use js::jsval::JSVal;
 use layout_interface::TrustedNodeAddress;
 use script_task::STACK_ROOTS;
 
-use util::smallvec::{SmallVec, SmallVec16};
+use util::smallvec::{SmallVec, SmallVec24};
 
 use core::nonzero::NonZero;
 use std::cell::{Cell, UnsafeCell};
@@ -604,7 +604,7 @@ impl<T: Assignable<U>, U: Reflectable> TemporaryPushable<T> for Vec<JS<U>> {
 /// See also [*Exact Stack Rooting - Storing a GCPointer on the CStack*]
 /// (https://developer.mozilla.org/en-US/docs/Mozilla/Projects/SpiderMonkey/Internals/GC/Exact_Stack_Rooting).
 pub struct RootCollection {
-    roots: UnsafeCell<SmallVec16<*mut JSObject>>,
+    roots: UnsafeCell<SmallVec24<*mut JSObject>>,
 }
 
 /// A pointer to a RootCollection, for use in global variables.
@@ -616,7 +616,7 @@ impl RootCollection {
     /// Create an empty collection of roots
     pub fn new() -> RootCollection {
         RootCollection {
-            roots: UnsafeCell::new(SmallVec16::new()),
+            roots: UnsafeCell::new(SmallVec24::new()),
         }
     }
 
@@ -626,6 +626,7 @@ impl RootCollection {
             let roots = self.roots.get();
             (*roots).push(untracked.js_ptr);
             debug!("  rooting {:?}", untracked.js_ptr);
+            assert!(!(*roots).spilled());
         }
     }