webRequest: Set frameId on beacon/ping requests.

third_party/WebKit/Source/core/loader/PingLoader.cpp

Index: third_party/WebKit/Source/core/loader/PingLoader.cpp
diff --git a/third_party/WebKit/Source/core/loader/PingLoader.cpp b/third_party/WebKit/Source/core/loader/PingLoader.cpp
index e87262768e3104daf14c384223c9f08c52d4ee9d..7ebf643ea14c828dcf9cd4ad447712097ce84587 100644
--- a/third_party/WebKit/Source/core/loader/PingLoader.cpp
+++ b/third_party/WebKit/Source/core/loader/PingLoader.cpp
@@ -137,10 +137,12 @@ PingLoader::PingLoader(LocalFrame* frame, ResourceRequest& request, const FetchI
     , m_identifier(createUniqueIdentifier())
     , m_keepAlive(this)
 {
+    ResourceResponse redirectResponse;
     frame->loader().client()->didDispatchPingLoader(request.url());
+    frame->loader().client()->dispatchWillSendRequest(frame->loader().documentLoader(), m_identifier, request, redirectResponse);

[Nate Chapin, 2016/05/23 21:28:04]

Could we call frame->document()->fetcher()->context().dispatchWillSendRequest()
instead of doing this and the 2 lines below? This code appears to be
unnecessarily (and until your change, incorrectly!) duplicated.

[robwu, 2016/05/23 22:28:31]

Done.

 
     TRACE_EVENT_INSTANT1("devtools.timeline", "ResourceSendRequest", TRACE_EVENT_SCOPE_THREAD, "data", InspectorSendRequestEvent::data(m_identifier, frame, request));
-    InspectorInstrumentation::willSendRequest(frame, m_identifier, frame->loader().documentLoader(), request, ResourceResponse(), initiatorInfo);
+    InspectorInstrumentation::willSendRequest(frame, m_identifier, frame->loader().documentLoader(), request, redirectResponse, initiatorInfo);
 
     m_loader = adoptPtr(Platform::current()->createURLLoader());
     ASSERT(m_loader);