Make ResourceLoadPriority calculation simpler

third_party/WebKit/Source/core/html/LinkResource.cpp

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 59 matching lines @@
     : m_owner(owner)
     , m_url(owner->getNonEmptyURLAttribute(hrefAttr))
 {
     m_charset = m_owner->getAttribute(charsetAttr);
     if (m_charset.isEmpty() && m_owner->document().frame())
         m_charset = m_owner->document().characterSet();
 }
 
 FetchRequest LinkRequestBuilder::build(bool lowPriority) const
 {
-    ResourceLoadPriority priority = lowPriority ? ResourceLoadPriorityVeryLow : ResourceLoadPriorityUnresolved;
-    return FetchRequest(ResourceRequest(m_owner->document().completeURL(m_url)), m_owner->localName(), m_charset, priority);
+    FetchRequest request(ResourceRequest(m_owner->document().completeURL(m_url)), m_owner->localName(), m_charset);
+    if (lowPriority)
+        request.setDefer(FetchRequest::LazyLoad);

[Yoav Weiss, 2016/06/25 08:37:59]

AFAICT, this mainly impacts non-matching styles & alternate links, marking them
as lazy, which will effectively result in them being of very low priority (and
therefore no functional change here). Is that a correct analysis?  

[Nate Chapin, 2016/06/27 19:46:16]

Yes, that's my understanding.

+    return request;
 }
 
 } // namespace blink