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

Unified Diff: third_party/WebKit/Source/core/loader/DocumentLoader.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/DocumentLoader.cpp
diff --git a/third_party/WebKit/Source/core/loader/DocumentLoader.cpp b/third_party/WebKit/Source/core/loader/DocumentLoader.cpp
index 7328e4995ea61c863ae934427b1c18dea2be2ccf..ee4a229de9d081921752a8539267f345f3eda216 100644
--- a/third_party/WebKit/Source/core/loader/DocumentLoader.cpp
+++ b/third_party/WebKit/Source/core/loader/DocumentLoader.cpp
@@ -177,7 +177,7 @@ Resource* DocumentLoader::startPreload(Resource::Type type,
switch (type) {
case Resource::Image:
if (m_frame && m_frame->settings() &&
- m_frame->settings()->fetchImagePlaceholders()) {
+ m_frame->settings()->getFetchImagePlaceholders()) {
request.setAllowImagePlaceholder();
}
resource = ImageResource::fetch(request, fetcher());
@@ -714,7 +714,7 @@ void DocumentLoader::startLoadingMainResource() {
// PlzNavigate:
// The fetch has already started in the browser. Don't mark it again.
- if (!m_frame->settings()->browserSideNavigationEnabled()) {
+ if (!m_frame->settings()->getBrowserSideNavigationEnabled()) {
DCHECK(!timing().fetchStart());
timing().markFetchStart();
}
@@ -732,8 +732,9 @@ void DocumentLoader::startLoadingMainResource() {
// The final access checks are still performed here, potentially rejecting
// the "provisional" load, but the browser side already expects the renderer
// to be able to unconditionally commit.
- if (!m_mainResource || (m_frame->settings()->browserSideNavigationEnabled() &&
- m_mainResource->errorOccurred())) {
+ if (!m_mainResource ||
+ (m_frame->settings()->getBrowserSideNavigationEnabled() &&
+ m_mainResource->errorOccurred())) {
m_request = ResourceRequest(blankURL());
maybeLoadEmpty();
return;

Powered by Google App Engine
This is Rietveld 408576698