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

Unified Diff: headless/lib/browser/devtools_api/domain_h.template

Issue 2666503002: Make headless_shell target compile for Windows (Closed)
Patch Set: Fix initlogging placing, always enable logging in Windows Created 3 years, 10 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
« no previous file with comments | « headless/lib/browser/devtools_api/domain_cc.template ('k') | headless/lib/browser/headless_browser_impl.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: headless/lib/browser/devtools_api/domain_h.template
diff --git a/headless/lib/browser/devtools_api/domain_h.template b/headless/lib/browser/devtools_api/domain_h.template
index 73110412f43c8b9fb51fd324caabf989df3d6afd..50073c08a353a3c07864e93789644828dcd35cde 100644
--- a/headless/lib/browser/devtools_api/domain_h.template
+++ b/headless/lib/browser/devtools_api/domain_h.template
@@ -18,7 +18,7 @@
{# Macro for defining a member function for a given command. #}
{% macro command_decl(command) %}
- {% set method_name = command.name | to_title_case %}
+ {% set method_name = command.name | sanitize_literal | to_title_case %}
{% if command.description %}
// {{ command.description }}
{% endif %}
@@ -108,7 +108,7 @@ class HEADLESS_EXPORT Domain {
{# Generate response handlers for commands that need them. #}
{% for command in domain.commands %}
{% if not "returns" in command %}{% continue %}{% endif %}
- {% set method_name = command.name | to_title_case %}
+ {% set method_name = command.name | sanitize_literal | to_title_case %}
static void Handle{{method_name}}Response(base::Callback<void(std::unique_ptr<{{method_name}}Result>)> callback, const base::Value& response);
{% endfor %}
« no previous file with comments | « headless/lib/browser/devtools_api/domain_cc.template ('k') | headless/lib/browser/headless_browser_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698