Removes the custom toString callback for the various DOM constructors.

third_party/WebKit/Source/bindings/core/v8/V8PerIsolateData.cpp

 /*
  * Copyright (C) 2009 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 147 matching lines @@
 #endif
     V8PerIsolateData* data = from(isolate);
 
     // Clear everything before exiting the Isolate.
     if (data->m_scriptRegexpScriptState)
         data->m_scriptRegexpScriptState->disposePerContextData();
     data->m_liveRoot.clear();
     data->m_hiddenValue.clear();
     data->m_stringCache->dispose();
     data->m_stringCache.clear();
-    data->m_toStringTemplate.clear();
     data->m_domTemplateMapForNonMainWorld.clear();
     data->m_domTemplateMapForMainWorld.clear();
     if (isMainThread())
         mainThreadPerIsolateData = 0;
 
     // FIXME: Remove once all v8::Isolate::GetCurrent() calls are gone.
     isolate->Exit();
     delete data;
 }
 
@@ skipping 73 matching lines @@
 {
     if (value.IsEmpty() || !value->IsObject())
         return v8::Local<v8::Object>();
     DOMTemplateMap::iterator result = domTemplateMap.find(info);
     if (result == domTemplateMap.end())
         return v8::Local<v8::Object>();
     v8::Local<v8::FunctionTemplate> templ = result->value.Get(isolate());
     return v8::Local<v8::Object>::Cast(value)->FindInstanceInPrototypeChain(templ);
 }
 
-static void constructorOfToString(const v8::FunctionCallbackInfo<v8::Value>& info)
-{
-    // The DOM constructors' toString functions grab the current toString
-    // for Functions by taking the toString function of itself and then
-    // calling it with the constructor as its receiver. This means that
-    // changes to the Function prototype chain or toString function are
-    // reflected when printing DOM constructors. The only wart is that
-    // changes to a DOM constructor's toString's toString will cause the
-    // toString of the DOM constructor itself to change. This is extremely
-    // obscure and unlikely to be a problem.
-    v8::Isolate* isolate = info.GetIsolate();
-    v8::Local<v8::Value> value;
-    if (!info.Callee()->Get(isolate->GetCurrentContext(), v8AtomicString(isolate, "toString")).ToLocal(&value) || !value->IsFunction()) {
-        v8SetReturnValue(info, v8::String::Empty(isolate));
-        return;
-    }
-    v8::Local<v8::Value> result;
-    if (V8ScriptRunner::callInternalFunction(v8::Local<v8::Function>::Cast(value), info.This(), 0, 0, isolate).ToLocal(&result))
-        v8SetReturnValue(info, result);
-}
-
-v8::Local<v8::FunctionTemplate> V8PerIsolateData::toStringTemplate()
-{
-    if (m_toStringTemplate.isEmpty())
-        m_toStringTemplate.set(isolate(), v8::FunctionTemplate::New(isolate(), constructorOfToString));
-    return m_toStringTemplate.newLocal(isolate());
-}
-
 void V8PerIsolateData::addEndOfScopeTask(PassOwnPtr<EndOfScopeTask> task)
 {
     m_endOfScopeTasks.append(task);
 }
 
 void V8PerIsolateData::runEndOfScopeTasks()
 {
     Vector<OwnPtr<EndOfScopeTask>> tasks;
     tasks.swap(m_endOfScopeTasks);
     for (const auto& task : tasks)
         task->run();
     ASSERT(m_endOfScopeTasks.isEmpty());
 }
 
 void V8PerIsolateData::clearEndOfScopeTasks()
 {
     m_endOfScopeTasks.clear();
 }
 
 void V8PerIsolateData::setScriptDebugger(PassOwnPtr<ScriptDebuggerBase> debugger)
 {
     ASSERT(!m_scriptDebugger);
     m_scriptDebugger = debugger;
 }
 
 } // namespace blink