Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(323)

Unified Diff: third_party/WebKit/Source/build/scripts/templates/OriginTrials.h.tmpl

Issue 1635593004: Rename Experimental Framework classes and concepts (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rename RuntimeEnabledFeatures flag to experimental_feature_name Created 4 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/build/scripts/templates/OriginTrials.h.tmpl
diff --git a/third_party/WebKit/Source/build/scripts/templates/ExperimentalFeatures.h.tmpl b/third_party/WebKit/Source/build/scripts/templates/OriginTrials.h.tmpl
similarity index 77%
rename from third_party/WebKit/Source/build/scripts/templates/ExperimentalFeatures.h.tmpl
rename to third_party/WebKit/Source/build/scripts/templates/OriginTrials.h.tmpl
index c44d07b7d02995a36959428ae428c54182c20da3..ccbf5552e2ebeb594ea8ddae6dfc7e185c5ff42d 100644
--- a/third_party/WebKit/Source/build/scripts/templates/ExperimentalFeatures.h.tmpl
+++ b/third_party/WebKit/Source/build/scripts/templates/OriginTrials.h.tmpl
@@ -1,8 +1,8 @@
{% from 'macros.tmpl' import license %}
{{license()}}
-#ifndef ExperimentalFeatures_h
-#define ExperimentalFeatures_h
+#ifndef OriginTrials_h
+#define OriginTrials_h
#include "core/CoreExport.h"
#include "wtf/text/WTFString.h"
@@ -14,20 +14,20 @@ class ExecutionContext;
// A class that stores dynamic tests for experimental features which can be
// enabled through the experimental framework via API keys.
-class CORE_EXPORT ExperimentalFeatures {
+class CORE_EXPORT OriginTrials {
public:
{% for feature in features %}
- {% if feature.api_name %}
+ {% if feature.experimental_feature_name %}
chasej 2016/01/26 16:56:52 Subject to change, based on comment on naming for
iclelland 2016/01/26 19:05:48 Acknowledged.
iclelland 2016/01/27 18:56:46 Done.
static bool {{feature.first_lowered_name}}Enabled(ExecutionContext* executionContext, String& errorMessage);
static bool {{feature.first_lowered_name}}Enabled(ExecutionContext* executionContext);
{% endif %}
{% endfor %}
private:
- ExperimentalFeatures() { }
+ OriginTrials() { }
{% for feature in features %}
- {% if feature.api_name %}
+ {% if feature.experimental_feature_name %}
static bool {{feature.first_lowered_name}}EnabledImpl(ExecutionContext* executionContext, String* errorMessage);
{% endif %}
{% endfor %}
@@ -35,4 +35,4 @@ private:
} // namespace blink
-#endif // ExperimentalFeatures_h
+#endif // OriginTrials_h

Powered by Google App Engine
This is Rietveld 408576698