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

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

Issue 1895293002: Test fragment scrolling and history restoration interaction w/ scroll anchoring. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove .cpp.tmpl and other review comments Created 4 years, 8 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/InternalRuntimeFlags.h.tmpl
diff --git a/third_party/WebKit/Source/build/scripts/templates/InternalRuntimeFlags.h.tmpl b/third_party/WebKit/Source/build/scripts/templates/InternalRuntimeFlags.h.tmpl
index df2f2f6d0bfe665487c85e3556dbc5ebf12c4c1c..dc486ac5645a973990c2909039b173b7d781f65d 100644
--- a/third_party/WebKit/Source/build/scripts/templates/InternalRuntimeFlags.h.tmpl
+++ b/third_party/WebKit/Source/build/scripts/templates/InternalRuntimeFlags.h.tmpl
@@ -21,12 +21,12 @@ public:
return new InternalRuntimeFlags;
}
-{#
- Setting after startup does not work for most runtime flags, but we
- could add an option to print setters for ones which do:
+ // These are reset between layout tests from Internals::resetToConsistentState
+ // using RuntimeEnabledFeatures::Backup.
+ {% for feature in standard_features if feature.settable_from_internals %}
void set{{feature.name}}Enabled(bool isEnabled) { RuntimeEnabledFeatures::set{{feature.name}}Enabled(isEnabled); }
- If we do that, we also need to respect Internals::resetToConsistentState.
-#}
+ {% endfor %}
+
{% for feature in standard_features %}
bool {{feature.first_lowered_name}}Enabled() { return RuntimeEnabledFeatures::{{feature.first_lowered_name}}Enabled(); }
{% endfor %}

Powered by Google App Engine
This is Rietveld 408576698