[Origin Trials] Install origin trial bindings on V8 context conditionally

third_party/WebKit/Source/core/testing/v8/WebCoreTestSupport.cpp

 /*
  * Copyright (C) 2011 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
  *     documentation and/or other materials provided with the distribution.
  *
  * THIS SOFTWARE IS PROVIDED BY APPLE AND ITS CONTRIBUTORS "AS IS" AND ANY
  * EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
  * WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
  * DISCLAIMED. IN NO EVENT SHALL APPLE OR ITS CONTRIBUTORS BE LIABLE FOR ANY
  * DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES
  * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES;
  * LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND
  * ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
  * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #include "core/testing/v8/WebCoreTestSupport.h"
 
+#include "bindings/core/v8/DOMWrapperWorld.h"
+#include "bindings/core/v8/V8Binding.h"
 #include "bindings/core/v8/V8Internals.h"
+#include "bindings/core/v8/V8WorkerInternals.h"
 #include "core/dom/Document.h"
 #include "core/dom/ExecutionContext.h"
 #include "core/frame/LocalFrame.h"
+#include "core/origin_trials/OriginTrialContext.h"
 #include "core/testing/InternalSettings.h"
 #include "core/testing/Internals.h"
 #include "core/testing/WorkerInternals.h"
 
 using namespace blink;
 
 namespace WebCoreTestSupport {
 
+namespace {
+blink::initializeOriginTrialsFunction s_originalInitializeOriginTrialsFunction = nullptr;
+}
+
 v8::Local<v8::Value> createInternalsObject(v8::Local<v8::Context> context)
 {
     ScriptState* scriptState = ScriptState::from(context);
     v8::Local<v8::Object> global = scriptState->context()->Global();
     ExecutionContext* executionContext = scriptState->getExecutionContext();
     if (executionContext->isDocument())
         return toV8(Internals::create(scriptState), global, scriptState->isolate());
     if (executionContext->isWorkerGlobalScope())
         return toV8(WorkerInternals::create(scriptState), global, scriptState->isolate());
     return v8::Local<v8::Value>();
 }
 
 void injectInternalsObject(v8::Local<v8::Context> context)
 {
+    if (!s_originalInitializeOriginTrialsFunction) {
+        s_originalInitializeOriginTrialsFunction = setInitializeOriginTrialsFunction(initializeOriginTrialsForTests);
+    }
+
     ScriptState* scriptState = ScriptState::from(context);
     ScriptState::Scope scope(scriptState);
     v8::Local<v8::Object> global = scriptState->context()->Global();
     v8::Local<v8::Value> internals = createInternalsObject(context);
     if (internals.IsEmpty())
         return;
+
     v8CallOrCrash(global->Set(scriptState->context(), v8AtomicString(scriptState->isolate(), Internals::internalsId), internals));
+
+    // If Origin Trials have been registered before the internals object was ready,
+    // then inject them into the context now
+    ExecutionContext* executionContext = toExecutionContext(context);
+    if (executionContext) {
+        OriginTrialContext* originTrialContext = OriginTrialContext::from(executionContext);
+        if (originTrialContext)
+            originTrialContext->initializePendingTrials();
+    }
+}
+
+void initializeOriginTrialsForTests(ScriptState* scriptState)
+{
+    (*s_originalInitializeOriginTrialsFunction)(scriptState);
+
+    v8::Local<v8::Context> context = scriptState->context();
+    ExecutionContext* executionContext = toExecutionContext(scriptState->context());
+    if (!executionContext->isDocument() && !executionContext->isWorkerGlobalScope())
+        return;
+    OriginTrialContext* originTrialContext = OriginTrialContext::from(executionContext, OriginTrialContext::DontCreateIfNotExists);
+    if (!originTrialContext)
+        return;
+
+    ScriptState::Scope scope(scriptState);
+    v8::Local<v8::Object> global = context->Global();
+    v8::Isolate* isolate = scriptState->isolate();
+    const DOMWrapperWorld& world = scriptState->world();
+    // TODO(iclelland): This is a placeholder for the real interface object,
+    // which we don't try to obtain at this time. (An assertion in the bindings
+    // generation code prevents it from actually being used.) It should be
+    // replaced with the actual interface if possible, in order to allow origin
+    // trials on static attributes. (crbug.com/614352)
+    v8::Local<v8::Function> interface = v8::Function::New(context, nullptr).ToLocalChecked();
+
+    v8::Local<v8::String> internalsName = v8::String::NewFromUtf8(isolate, "internals", v8::NewStringType::kNormal).ToLocalChecked();
+    v8::Local<v8::Value> v8Internals = global->Get(context, internalsName).ToLocalChecked();
+    if (v8Internals->IsObject()) {
+        v8::Local<v8::Object> internals = v8Internals->ToObject();
+        if (!originTrialContext->featureBindingsInstalled("Frobulate") && originTrialContext->isFeatureEnabled("Frobulate", nullptr)) {

[haraken, 2016/05/27 22:25:17]

It would be nicer if we generate the if check inside the install method.

Nit: This CL mixes "initialize" and "install". Shall we unify them into
"install"?

[iclelland, 2016/05/30 12:15:35]

We'd have to pass the OriginTrialContext into the install method, though, to do
that, which seems ugly. I can do it, though, if you think it makes more sense
that way.
Question: There are two parts to the `if` -- the policy check (isFeatureEnabled)
and the only-install-this-once check
(featureBindingsInstalled/setFeatureBindingsInstalled). Should both of them go
into the install method, or just the featureBindingsInstalled calls?
Yes, install is the better of the two words. I'll change that.

+            if (executionContext->isDocument())
+                V8Internals::installOriginTrialsSampleAPI(isolate, world, internals, interface);
+            else if (executionContext->isWorkerGlobalScope())
+                V8WorkerInternals::installOriginTrialsSampleAPI(isolate, world, internals, interface);
+            originTrialContext->setFeatureBindingsInstalled("Frobulate");
+        }
+    }
 }
 
 void resetInternalsObject(v8::Local<v8::Context> context)
 {
     // This can happen if JavaScript is disabled in the main frame.
     if (context.IsEmpty())
         return;
 
     ScriptState* scriptState = ScriptState::from(context);
     ScriptState::Scope scope(scriptState);
     Document* document = toDocument(scriptState->getExecutionContext());
     ASSERT(document);
     LocalFrame* frame = document->frame();
     // Should the document have been detached, the page is assumed being destroyed (=> no reset required.)
     if (!frame)
         return;
     Page* page = frame->page();
     ASSERT(page);
     Internals::resetToConsistentState(page);
     InternalSettings::from(*page)->resetToConsistentState();
 }
 
 } // namespace WebCoreTestSupport