Add disabled origin trial feature list

content/public/common/content_client.h

Index: content/public/common/content_client.h
diff --git a/content/public/common/content_client.h b/content/public/common/content_client.h
index 5510cb27f9145e769ea02cf7df17502e72cafe1a..a39f02239af621710ae3943f2b2590d4c5117657 100644
--- a/content/public/common/content_client.h
+++ b/content/public/common/content_client.h
@@ -177,6 +177,10 @@ class CONTENT_EXPORT ContentClient {
   // origin trials are not enabled in this context.
   virtual base::StringPiece GetOriginTrialPublicKey();
 
+  // Returns true if the named experimental feature should be disabled in origin
+  // trials, even if a valid token is present.
+  virtual bool IsOriginTrialFeatureDisabled(base::StringPiece feature);

[chasej, 2016/06/07 15:51:31]

With additional methods, would it make sense to collect them into a single class
(e.g. OriginTrialPolicy)? Then, could just expose the class (similar to
GetMediaClientAndroid below). The base class could provide the default
implementation (rather than making content_client.cc do so), and be overridden
as necessary.

I'm not sure that's the best thing to do, just asking the question.

[Sorin Jianu, 2016/06/07 17:20:24]

can the virtual function be declared const?

[iclelland, 2016/06/08 13:37:56]

Yes, and GetOriginTrialPublicKey above it.

[iclelland, 2016/06/08 15:38:52]

I've made that refactor in a follow-up CL; see the result here:
https://codereview.chromium.org/2049783002

+
 #if defined(OS_ANDROID)
   // Returns true for clients like Android WebView that uses synchronous
   // compositor. Note setting this to true will permit synchronous IPCs from