Simplify the way to initialize context

Source/bindings/v8/ScriptPromiseResolverTest.cpp

 /*
  * Copyright (C) 2013 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:
  *
  *     * 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
@@ skipping 37 matching lines @@
     ScriptPromiseResolverTest()
         : m_isolate(v8::Isolate::GetCurrent())
         , m_handleScope(m_isolate)
         , m_context(m_isolate, v8::Context::New(m_isolate))
         , m_contextScope(m_context.newLocal(m_isolate))
     {
     }
 
     void SetUp()
     {
-        v8::Handle<v8::Context> context(m_context.newLocal(m_isolate));
-        V8PerContextDataHolder::install(context, DOMWrapperWorld::current(m_isolate));
-        m_perContextData = V8PerContextData::create(context);
-        m_perContextData->init();
+        // FIXME: Create a new world and pass it to V8PerContextData.

[haraken, 2014/02/28 11:14:05]

I'll address this in a follow-up CL. Passing 0 doesn't change any existing test
behavior because DOMWrapperWorld::current() has been returning 0.

+        m_perContextData = V8PerContextData::create(m_context.newLocal(m_isolate), 0);
         m_promise = ScriptPromise::createPending();
         m_resolver = ScriptPromiseResolver::create(m_promise);
     }
 
     void TearDown()
     {
         m_resolver = nullptr;
         m_promise.clear();
         m_perContextData.clear();
     }
@@ skipping 95 matching lines @@
     m_resolver->reject(ScriptValue(v8::Integer::New(m_isolate, 4), m_isolate));
     EXPECT_FALSE(m_resolver->isPending());
     EXPECT_EQ(V8PromiseCustom::Fulfilled, state());
     ASSERT_TRUE(result()->IsNumber());
     EXPECT_EQ(3, result().As<v8::Integer>()->Value());
 }
 
 } // namespace
 
 } // namespace WebCore