Simplify OriginTrialContext and the way it validates tokens.

third_party/WebKit/Source/core/origin_trials/testing/InternalsFrobulate.cpp

Index: third_party/WebKit/Source/core/origin_trials/testing/InternalsFrobulate.cpp
diff --git a/third_party/WebKit/Source/core/origin_trials/testing/InternalsFrobulate.cpp b/third_party/WebKit/Source/core/origin_trials/testing/InternalsFrobulate.cpp
index 033c67834fef1206b01fe6a60b5c5a4df7aecbbc..61a958aa52b7114e6668a6502c1af1c64f93a7ec 100644
--- a/third_party/WebKit/Source/core/origin_trials/testing/InternalsFrobulate.cpp
+++ b/third_party/WebKit/Source/core/origin_trials/testing/InternalsFrobulate.cpp
@@ -14,8 +14,8 @@ namespace blink {
 bool InternalsFrobulate::frobulate(ScriptState* scriptState, Internals& internals, ExceptionState& exceptionState)
 {
     String errorMessage;
-    if (!OriginTrials::originTrialsSampleAPIEnabled(scriptState->getExecutionContext(), errorMessage)) {
-        exceptionState.throwDOMException(NotSupportedError, errorMessage);
+    if (!OriginTrials::originTrialsSampleAPIEnabled(scriptState->getExecutionContext())) {
+        exceptionState.throwDOMException(NotSupportedError, "API is not enabled");

[iclelland, 2016/07/13 14:30:09]

Minor nit -- as a test-only interface, we can be as explicit as we need to be
about what went wrong here. Would it be helpful for future engineers debugging
tests to say something like "The Origin Trials Sample API has not been enabled
in this context"?

[Marijn Kruisselbrink, 2016/07/13 19:16:08]

Done

         return false;
     }
     return frobulateNoEnabledCheck(internals);