Sending an async GET request for doc.written blocked scripts.

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

Index: third_party/WebKit/Source/core/fetch/ResourceFetcher.cpp
diff --git a/third_party/WebKit/Source/core/fetch/ResourceFetcher.cpp b/third_party/WebKit/Source/core/fetch/ResourceFetcher.cpp
index c4ab711b0903d38ea0d8121df4aa7cffac881516..fa8336c6135d28f61147a772ff1fe75db787b89c 100644
--- a/third_party/WebKit/Source/core/fetch/ResourceFetcher.cpp
+++ b/third_party/WebKit/Source/core/fetch/ResourceFetcher.cpp
@@ -163,6 +163,8 @@ ResourceLoadPriority ResourceFetcher::computeLoadPriority(Resource::Type type, c
         priority = type == Resource::Raw ? ResourceLoadPriorityLow : ResourceLoadPriorityVeryLow;
     }
 
+    priority = request.resourceRequest().computePriorityForInterventions(priority);

[Nate Chapin, 2016/08/22 19:47:03]

This seems like overkill at first glance. Is there a reason marking the request
as LazyLoad is insufficient?

[shivanisha, 2016/08/22 20:44:00]

LazyLoad results in priority ResourceLoadPriorityLow which maps to net::LOWEST
but we want a priority net::IDLE for this case.

[Nate Chapin, 2016/08/22 20:55:12]

Ah, I forgot about the script-specific LazyLoad priority.

Adding an Idle value to the ResourceFetcher::DeferOption enum seemed like it'd
be cleaner than plumbing this through ResourceRequest. Can we do that instead?

[shivanisha, 2016/08/23 01:04:46]

That's a good suggestion. Done.

+
     // A manually set priority acts as a floor. This is used to ensure that synchronous requests
     // are always given the highest possible priority, as well as to ensure that there isn't priority
     // churn if images move in and out of the viewport, or is displayed more than once, both in and out