Collect UMA data for Origin Trials

content/renderer/origin_trials/web_trial_token_validator_impl.h

Index: content/renderer/origin_trials/web_trial_token_validator_impl.h
diff --git a/content/renderer/origin_trials/web_trial_token_validator_impl.h b/content/renderer/origin_trials/web_trial_token_validator_impl.h
index 7cb118774295496eeb36a4ee92d8fc3305990bd6..feec27e0b9c09b65a5f476b798f28ba06523ac54 100644
--- a/content/renderer/origin_trials/web_trial_token_validator_impl.h
+++ b/content/renderer/origin_trials/web_trial_token_validator_impl.h
@@ -13,8 +13,10 @@
 
 namespace content {
 
-// The TrialTokenValidator is called by the Experimental Framework code in Blink
-// to validate tokens to enable experimental features.
+enum class WebOriginTrialTokenStatus;

[iclelland, 2016/04/28 17:00:58]

wrap this in "namespace blink" instead, if you need to declare it here.

(The one you're using below is being declared by including
WebTrialTokenValidator.h, I think.)

[chasej, 2016/05/02 15:53:11]

Wasn't needed, so just removed the declaration.

+
+// The TrialTokenValidator is called by the Origin Trials Framework code in
+// Blink to validate tokens to enable experimental features.
 //
 // This class is thread-safe.
 class CONTENT_EXPORT WebTrialTokenValidatorImpl
@@ -24,9 +26,10 @@ class CONTENT_EXPORT WebTrialTokenValidatorImpl
   ~WebTrialTokenValidatorImpl() override;
 
   // blink::WebTrialTokenValidator implementation
-  bool validateToken(const blink::WebString& token,
-                     const blink::WebSecurityOrigin& origin,
-                     const blink::WebString& featureName) override;
+  blink::WebOriginTrialTokenStatus validateToken(
+      const blink::WebString& token,
+      const blink::WebSecurityOrigin& origin,
+      const blink::WebString& featureName) override;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(WebTrialTokenValidatorImpl);