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

Side by Side Diff: third_party/WebKit/Source/build/scripts/templates/InternalSettingsGenerated.cpp.tmpl

Issue 2589143003: Add 'get' prefix for Settings.in generated code. (Closed)
Patch Set: Capitalize the prefix. Created 3 years, 12 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 unified diff | Download patch
OLDNEW
1 {% from "macros.tmpl" import license %} 1 {% from "macros.tmpl" import license %}
2 {{ license() }} 2 {{ license() }}
3 3
4 #include "InternalSettingsGenerated.h" 4 #include "InternalSettingsGenerated.h"
5 5
6 #include "core/page/Page.h" 6 #include "core/page/Page.h"
7 #include "core/frame/Settings.h" 7 #include "core/frame/Settings.h"
8 8
9 namespace blink { 9 namespace blink {
10 10
11 InternalSettingsGenerated::InternalSettingsGenerated(Page* page) 11 InternalSettingsGenerated::InternalSettingsGenerated(Page* page)
12 : m_page(page) 12 : m_page(page)
13 {% for setting in settings if setting.type|to_idl_type %} 13 {% for setting in settings if setting.type|to_idl_type %}
14 , m_{{setting.name}}(page->settings().{{setting.name}}()) 14 , m_{{setting.name}}(page->settings().Get{{setting.name|upper_first}}())
Łukasz Anforowicz 2016/12/21 21:07:36 |upper_first| causes ...FEtch... issue. I wonder
nasko 2017/01/04 18:18:51 Done.
15 {% endfor %} 15 {% endfor %}
16 { 16 {
17 } 17 }
18 18
19 InternalSettingsGenerated::~InternalSettingsGenerated() {} 19 InternalSettingsGenerated::~InternalSettingsGenerated() {}
20 20
21 void InternalSettingsGenerated::resetToConsistentState() { 21 void InternalSettingsGenerated::resetToConsistentState() {
22 {% for setting in settings if setting.type|to_idl_type %} 22 {% for setting in settings if setting.type|to_idl_type %}
23 m_page->settings().set{{setting.name|upper_first}}(m_{{setting.name}}); 23 m_page->settings().set{{setting.name|upper_first}}(m_{{setting.name}});
24 {% endfor %} 24 {% endfor %}
25 } 25 }
26 {% for setting in settings if setting.type|to_idl_type %} 26 {% for setting in settings if setting.type|to_idl_type %}
27 27
28 void InternalSettingsGenerated::set{{setting.name|upper_first}}({{setting.type|t o_passing_type}} {{setting.name}}) { 28 void InternalSettingsGenerated::set{{setting.name|upper_first}}({{setting.type|t o_passing_type}} {{setting.name}}) {
29 m_page->settings().set{{setting.name|upper_first}}({{setting.name}}); 29 m_page->settings().set{{setting.name|upper_first}}({{setting.name}});
30 } 30 }
31 {% endfor %} 31 {% endfor %}
32 32
33 DEFINE_TRACE(InternalSettingsGenerated) { 33 DEFINE_TRACE(InternalSettingsGenerated) {
34 visitor->trace(m_page); 34 visitor->trace(m_page);
35 } 35 }
36 36
37 } // namespace blink 37 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698