Fix bug 1433: clear the global thread table when an isolate is disposed.

test/cctest/test-lockers.cc

 // Copyright 2007-2011 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 589 matching lines @@
     v8::Locker locker_;
     v8::HandleScope handle_scope;
     context = v8::Context::New();
   }
   i::List<JoinableThread*> threads(kNThreads);
   for (int i = 0; i < kNThreads; i++) {
     threads.Add(new LockUnlockLockDefaultIsolateThread(context));
   }
   StartJoinAndDeleteThreads(threads);
 }
+
+
+TEST(Regress1433) {
+  for (int i = 0; i < 10; i++) {
+    v8::Isolate* isolate = v8::Isolate::New();
+    {
+      v8::Locker l(isolate);

[Mads Ager (chromium), 2011/06/07 18:28:18]

l -> lock?

[Vitaly Repeshko, 2011/06/07 18:32:53]

Done.

+      v8::Isolate::Scope iscope(isolate);

[Mads Ager (chromium), 2011/06/07 18:28:18]

iscope -> isolate_scope

to match the other naming?

[Vitaly Repeshko, 2011/06/07 18:32:53]

Done.

+      v8::HandleScope handle_scope;
+      v8::Persistent<Context> context = v8::Context::New();
+      v8::Context::Scope context_scope(context);
+      v8::Handle<String> source = v8::String::New("1+1");
+      v8::Handle<Script> script = v8::Script::Compile(source);
+      v8::Handle<Value> result = script->Run();
+      v8::String::AsciiValue ascii(result);
+      context.Dispose();
+    }
+    isolate->Dispose();
+  }
+}