Remove experimental ExternalSymbolCallback feature. This is not needed...

test/cctest/test-api.cc

 // Copyright 2007-2008 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 5571 matching lines @@
       v8::Utils::OpenHandle(*global_template);
   CHECK(!internal_template->constructor()->IsUndefined());
   i::Handle<i::FunctionTemplateInfo> constructor(
       i::FunctionTemplateInfo::cast(internal_template->constructor()));
   CHECK(!constructor->access_check_info()->IsUndefined());
   v8::Persistent<Context> context0 = Context::New(NULL, global_template);
   CHECK(!constructor->access_check_info()->IsUndefined());
 }
 
 
-static String::ExternalStringResource* SymbolCallback(const char* chars,
-                                                      size_t length) {
-  uint16_t* buffer = i::NewArray<uint16_t>(length + 1);
-  for (size_t i = 0; i < length; i++) {
-    buffer[i] = chars[i];
-  }
-  buffer[length] = '\0';
-  return new TestResource(buffer);
-}
-
-
-static v8::Handle<Value> ExternalSymbolGetter(Local<String> name,
-                                              const AccessorInfo& info) {
-  ApiTestFuzzer::Fuzz();
-  CHECK(!name->Equals(v8_str("externalSymbol722")) || name->IsExternal());
-  return v8::True();
-}
-
-
-static void ExternalSymbolSetter(Local<String> name,
-                                 Local<Value> value,
-                                 const AccessorInfo&) {
-  ApiTestFuzzer::Fuzz();
-  CHECK(!name->Equals(v8_str("externalSymbol722")) || name->IsExternal());
-}
-
-
-THREADED_TEST(ExternalSymbols) {
-  TestResource::dispose_count = 0;
-  v8::V8::SetExternalSymbolCallback(SymbolCallback);
-  v8::HandleScope scope;
-  LocalContext context;
-  Local<ObjectTemplate> templ = ObjectTemplate::New();
-  // Use a bizare name so that the name does not clash with names used
-  // in natives files.  If running with snapshots enabled, variable
-  // names used in the native files will be normal symbols instead of
-  // external ones.  Also, make sure that the bizare name is used from
-  // JavaScript code before using it from C++ code.
-  Local<Value> value =
-      CompileRun("var o = { externalSymbol722: 42 }; o.externalSymbol722");
-  CHECK_EQ(42, value->Int32Value());
-  templ->SetAccessor(v8_str("externalSymbol722"),
-                     ExternalSymbolGetter,
-                     ExternalSymbolSetter);
-  context->Global()->Set(v8_str("obj"), templ->NewInstance());
-  value = CompileRun("obj.externalSymbol722");
-  CHECK_EQ(true, value->BooleanValue());
-  value = CompileRun("obj.externalSymbol722 = 42");
-  v8::V8::SetExternalSymbolCallback(NULL);
-}
-
-
 // This test verifies that pre-compilation (aka preparsing) can be called
 // without initializing the whole VM. Thus we cannot run this test in a
 // multi-threaded setup.
 TEST(PreCompile) {
   // TODO(155): This test would break without the initialization of V8. This is
   // a workaround for now to make this test not fail.
   v8::V8::Initialize();
   const char *script = "function foo(a) { return a+1; }";
   v8::ScriptData *sd = v8::ScriptData::PreCompile(script, strlen(script));
   CHECK_NE(sd->Length(), 0);
@@ skipping 49 matching lines @@
   context1->Global()->Set(v8_str("other"), context0->Global());
   Local<Value> value = CompileRun("var instance = new other.C(); instance.x");
   CHECK(value->IsInt32());
   CHECK_EQ(42, value->Int32Value());
   context1->Exit();
 
   // Dispose the contexts to allow them to be garbage collected.
   context0.Dispose();
   context1.Dispose();
 }