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

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

Issue 2589143003: Add 'get' prefix for Settings.in generated code. (Closed)
Patch Set: Only get prefix and no capitalization. Created 3 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/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 049d4851c03aed20bdc07de7a4b7998b458ac81c..40707b9ad19ed2a9a69d56429e8dfa4d30987ead 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() &&
!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())));
« no previous file with comments | « third_party/WebKit/Source/core/loader/ImageLoader.cpp ('k') | third_party/WebKit/Source/core/loader/MixedContentChecker.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698