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

Unified Diff: third_party/WebKit/Source/core/loader/appcache/ApplicationCacheHost.cpp

Issue 2383403002: Reflow comments in core/loader (Closed)
Patch Set: yhirano comments Created 4 years, 2 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 | « third_party/WebKit/Source/core/loader/ThreadableLoader.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/loader/appcache/ApplicationCacheHost.cpp
diff --git a/third_party/WebKit/Source/core/loader/appcache/ApplicationCacheHost.cpp b/third_party/WebKit/Source/core/loader/appcache/ApplicationCacheHost.cpp
index bf604afd074e4b15823771b01e992dbbd3d93c1d..e2f55c8b7830394223d6682416b3f9d96f6ad2e9 100644
--- a/third_party/WebKit/Source/core/loader/appcache/ApplicationCacheHost.cpp
+++ b/third_party/WebKit/Source/core/loader/appcache/ApplicationCacheHost.cpp
@@ -76,9 +76,9 @@ ApplicationCacheHost::~ApplicationCacheHost() {
void ApplicationCacheHost::willStartLoadingMainResource(
ResourceRequest& request) {
- // We defer creating the outer host object to avoid spurious creation/destruction
- // around creating empty documents. At this point, we're initiating a main resource
- // load for the document, so its for real.
+ // We defer creating the outer host object to avoid spurious
+ // creation/destruction around creating empty documents. At this point, we're
+ // initiating a main resource load for the document, so its for real.
if (!isApplicationCacheEnabled())
return;
@@ -105,10 +105,11 @@ void ApplicationCacheHost::willStartLoadingMainResource(
m_host->willStartMainResourceRequest(wrapped, spawningHost);
- // NOTE: The semantics of this method, and others in this interface, are subtly different
- // than the method names would suggest. For example, in this method never returns an appcached
- // response in the SubstituteData out argument, instead we return the appcached response thru
- // the usual resource loading pipeline.
+ // NOTE: The semantics of this method, and others in this interface, are
+ // subtly different than the method names would suggest. For example, in this
+ // method never returns an appcached response in the SubstituteData out
+ // argument, instead we return the appcached response thru the usual resource
+ // loading pipeline.
}
void ApplicationCacheHost::selectCacheWithoutManifest() {
@@ -136,11 +137,10 @@ void ApplicationCacheHost::selectCacheWithManifest(const KURL& manifestURL) {
ApplicationCacheManifestSelectInsecureHost);
}
if (m_host && !m_host->selectCacheWithManifest(manifestURL)) {
- // It's a foreign entry, restart the current navigation from the top
- // of the navigation algorithm. The navigation will not result in the
- // same resource being loaded, because "foreign" entries are never picked
- // during navigation.
- // see ApplicationCacheGroup::selectCache()
+ // It's a foreign entry, restart the current navigation from the top of the
+ // navigation algorithm. The navigation will not result in the same resource
+ // being loaded, because "foreign" entries are never picked during
+ // navigation. see ApplicationCacheGroup::selectCache()
frame->navigate(*document, document->url(), true, UserGestureStatus::None);
}
}
@@ -183,7 +183,8 @@ void ApplicationCacheHost::setApplicationCache(
}
void ApplicationCacheHost::detachFromDocumentLoader() {
- // Detach from the owning DocumentLoader and let go of WebApplicationCacheHost.
+ // Detach from the owning DocumentLoader and let go of
+ // WebApplicationCacheHost.
setApplicationCache(nullptr);
m_host.reset();
m_documentLoader = nullptr;
@@ -304,7 +305,8 @@ bool ApplicationCacheHost::isApplicationCacheEnabled() {
}
void ApplicationCacheHost::didChangeCacheAssociation() {
- // FIXME: Prod the inspector to update its notion of what cache the page is using.
+ // FIXME: Prod the inspector to update its notion of what cache the page is
+ // using.
}
void ApplicationCacheHost::notifyEventListener(
« no previous file with comments | « third_party/WebKit/Source/core/loader/ThreadableLoader.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698