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

Unified Diff: third_party/WebKit/Source/core/loader/LinkLoader.cpp

Issue 2589143003: Add 'get' prefix for Settings.in generated code. (Closed)
Patch Set: Capitalize the prefix. Created 4 years 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/core/loader/LinkLoader.cpp
diff --git a/third_party/WebKit/Source/core/loader/LinkLoader.cpp b/third_party/WebKit/Source/core/loader/LinkLoader.cpp
index 5abb54d4f6fe7e33a76c0cf0a8e534bb4304abcd..462cb75f0e04e787be52cf97191b4d1c3231c0a2 100644
--- a/third_party/WebKit/Source/core/loader/LinkLoader.cpp
+++ b/third_party/WebKit/Source/core/loader/LinkLoader.cpp
@@ -140,9 +140,9 @@ static void dnsPrefetchIfNeeded(
// FIXME: The href attribute of the link element can be in "//hostname"
// form, and we shouldn't attempt to complete that as URL
// <https://bugs.webkit.org/show_bug.cgi?id=48857>.
- if (settings && settings->dnsPrefetchingEnabled() && href.isValid() &&
+ if (settings && settings->GetDNSPrefetchingEnabled() && href.isValid() &&
Łukasz Anforowicz 2016/12/21 21:07:36 Another interesting case for |upper_first| VS |upp
nasko 2017/01/04 18:18:52 Done.
!href.isEmpty()) {
- if (settings->logDnsPrefetchAndPreconnect()) {
+ if (settings->GetLogDnsPrefetchAndPreconnect()) {
document.addConsoleMessage(ConsoleMessage::create(
OtherMessageSource, DebugMessageLevel,
String("DNS prefetch triggered for " + href.host())));
@@ -165,7 +165,7 @@ static void preconnectIfNeeded(
if (caller == LinkCalledFromHeader)
UseCounter::count(document, UseCounter::LinkHeaderPreconnect);
Settings* settings = document.settings();
- if (settings && settings->logDnsPrefetchAndPreconnect()) {
+ if (settings && settings->GetLogDnsPrefetchAndPreconnect()) {
document.addConsoleMessage(ConsoleMessage::create(
OtherMessageSource, DebugMessageLevel,
String("Preconnect triggered for ") + href.getString()));
@@ -332,7 +332,7 @@ static Resource* preloadIfNeeded(const LinkRelAttribute& relAttribute,
crossOrigin);
}
Settings* settings = document.settings();
- if (settings && settings->logPreload()) {
+ if (settings && settings->GetLogPreload()) {
document.addConsoleMessage(ConsoleMessage::create(
OtherMessageSource, DebugMessageLevel,
String("Preload triggered for " + href.host() + href.path())));

Powered by Google App Engine
This is Rietveld 408576698