Use a precise target type for Beacon requests.

Source/core/loader/BeaconLoader.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "config.h"
 #include "core/loader/BeaconLoader.h"
 
 #include "core/FetchInitiatorTypeNames.h"
 #include "core/fetch/CrossOriginAccessControl.h"
 #include "core/fetch/FetchContext.h"
 #include "core/fileapi/File.h"
 #include "core/frame/LocalFrame.h"
 #include "core/html/DOMFormData.h"
 #include "platform/network/FormData.h"
 #include "platform/network/ParsedContentType.h"
 #include "platform/network/ResourceRequest.h"
 #include "public/platform/WebURLRequest.h"
 #include "wtf/ArrayBufferView.h"
 
 namespace blink {
 
 void BeaconLoader::prepareRequest(LocalFrame* frame, ResourceRequest& request)
 {
-    // NOTE: do not distinguish Beacon by target type.
-    request.setRequestContext(blink::WebURLRequest::RequestContextPing);

[Mike West, 2014/08/13 13:54:41]

Why was this note here? It's certainly less relevant now that we've removed the
TargetType enum, but was there a good reason for its being there in the first
place?

[sof, 2014/08/13 14:05:38]

Just a remark that Blink didn't expose the "Beacon-ness" of the request, as
there was no known need for doing so.

It would have been better if that went hand-in-hand with not defining
RequestContextBeacon, as it wasn't being used. But no longer an issue.

+    request.setRequestContext(WebURLRequest::RequestContextBeacon);
     request.setHTTPMethod("POST");
     request.setHTTPHeaderField("Cache-Control", "max-age=0");
     request.setAllowStoredCredentials(true);
     frame->loader().fetchContext().addAdditionalRequestHeaders(frame->document(), request, FetchSubresource);
     frame->loader().fetchContext().setFirstPartyForCookies(request);
 }
 
 void BeaconLoader::issueRequest(LocalFrame* frame, ResourceRequest& request)
 {
     FetchInitiatorInfo initiatorInfo;
@@ skipping 85 matching lines @@
     AtomicString contentType = AtomicString("multipart/form-data; boundary=", AtomicString::ConstructFromLiteral) + entityBody->boundary().data();
     request.setHTTPBody(entityBody.release());
     request.setHTTPContentType(contentType);
 
     issueRequest(frame, request);
     payloadLength = entitySize;
     return true;
 }
 
 } // namespace blink