Make ResourceLoadPriority calculation simpler

third_party/WebKit/Source/core/fetch/ResourceFetcher.h

Index: third_party/WebKit/Source/core/fetch/ResourceFetcher.h
diff --git a/third_party/WebKit/Source/core/fetch/ResourceFetcher.h b/third_party/WebKit/Source/core/fetch/ResourceFetcher.h
index 31a65ed9874196c2940fc58bd707415619de1668..843163b6549a66730029cdb4341db75c0cdfcab0 100644
--- a/third_party/WebKit/Source/core/fetch/ResourceFetcher.h
+++ b/third_party/WebKit/Source/core/fetch/ResourceFetcher.h
@@ -127,8 +127,6 @@ public:
 
     void acceptDataFromThreadedReceiver(unsigned long identifier, const char* data, int dataLength, int encodedDataLength);
 
-    ResourceLoadPriority loadPriority(Resource::Type, const FetchRequest&, ResourcePriority::VisibilityStatus = ResourcePriority::NotVisible);
-
     enum ResourceLoadStartType {
         ResourceLoadingFromNetwork,
         ResourceLoadingFromCache
@@ -156,6 +154,7 @@ private:
     void initializeRevalidation(ResourceRequest&, Resource*);
     Resource* createResourceForLoading(FetchRequest&, const String& charset, const ResourceFactory&);
     void storeResourceTimingInitiatorInformation(Resource*);
+    ResourceLoadPriority loadPriority(Resource::Type, const FetchRequest&, ResourcePriority::VisibilityStatus);

[kinuko, 2016/06/17 05:40:47]

nit: while you're here can we rename this to computeLoadPriority or something
that's less confusing? (Is 'load' verb here?)

[Nate Chapin, 2016/06/17 18:38:16]

load is a noun here, I think. The name reads like a simple accessor, but it's
neither simple nor an accessor :/

computeLoadPriority() sounds good.

 
     Resource* resourceForStaticData(const FetchRequest&, const ResourceFactory&, const SubstituteData&);