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

Unified Diff: third_party/WebKit/Source/core/loader/LinkLoader.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
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 31d6081e55d3bebf3b97a73875fa9e5347cffe99..041de74cc5d8129db4cf2fcaf7b277ca55511529 100644
--- a/third_party/WebKit/Source/core/loader/LinkLoader.cpp
+++ b/third_party/WebKit/Source/core/loader/LinkLoader.cpp
@@ -135,8 +135,9 @@ static void dnsPrefetchIfNeeded(
if (caller == LinkCalledFromHeader)
UseCounter::count(document, UseCounter::LinkHeaderDnsPrefetch);
Settings* settings = document.settings();
- // 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>.
+ // 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() &&
!href.isEmpty()) {
if (settings->logDnsPrefetchAndPreconnect())
@@ -387,8 +388,11 @@ bool LinkLoader::loadLink(const LinkRelAttribute& relAttribute,
const NetworkHintsInterface& networkHintsInterface) {
// TODO(yoav): Do all links need to load only after they're in document???
- // TODO(yoav): Convert all uses of the CrossOriginAttribute to CrossOriginAttributeValue. crbug.com/486689
- // FIXME(crbug.com/463266): We're ignoring type here, for everything but preload. Maybe we shouldn't.
+ // TODO(yoav): Convert all uses of the CrossOriginAttribute to
+ // CrossOriginAttributeValue. crbug.com/486689
+
+ // FIXME(crbug.com/463266): We're ignoring type here, for everything but
+ // preload. Maybe we shouldn't.
dnsPrefetchIfNeeded(relAttribute, href, document, networkHintsInterface,
LinkCalledFromMarkup);
@@ -440,8 +444,8 @@ bool LinkLoader::loadLink(const LinkRelAttribute& relAttribute,
}
void LinkLoader::released() {
- // Only prerenders need treatment here; other links either use the Resource interface, or are notionally
- // atomic (dns prefetch).
+ // Only prerenders need treatment here; other links either use the Resource
+ // interface, or are notionally atomic (dns prefetch).
if (m_prerender) {
m_prerender->cancel();
m_prerender.clear();
« no previous file with comments | « third_party/WebKit/Source/core/loader/LinkLoader.h ('k') | third_party/WebKit/Source/core/loader/LinkLoaderTest.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698