Fix use-after-move in blink/renderer/core/loader/

Fix use-after-move (potential) bugs found by the
"bugprone-use-after-move" clang-tidy check.

Bug: 1122844
Change-Id: I5307cf82ed89c1afeec735418f406d303bbcc1c1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2401142Reviewed-by: Yoichi Osato <yoichio@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Maksim Ivanov <emaxx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#806048}

third_party/blink/renderer/core/loader/ping_loader.cc

@@ -209,7 +209,7 @@ bool SendBeaconCommon(const ScriptState& state,
   params.MutableOptions().initiator_info.name =
       fetch_initiator_type_names::kBeacon;
 
-  frame->Client()->DidDispatchPingLoader(request.Url());
+  frame->Client()->DidDispatchPingLoader(url);
   Resource* resource =
       RawResource::Fetch(params, frame->DomWindow()->Fetcher(), nullptr);
   return resource->GetStatus() != ResourceStatus::kLoadError;

third_party/blink/renderer/core/loader/threadable_loader.cc

@@ -1038,16 +1038,16 @@ void ThreadableLoader::LoadRequest(
     }
   }
 
+  const mojom::RequestContextType request_context = request.GetRequestContext();
   FetchParameters new_params(std::move(request), resource_loader_options);
   DCHECK(!GetResource());
 
   checker_.WillAddClient();
-  if (request.GetRequestContext() == mojom::RequestContextType::VIDEO ||
-      request.GetRequestContext() == mojom::RequestContextType::AUDIO) {
+  if (request_context == mojom::RequestContextType::VIDEO ||
+      request_context == mojom::RequestContextType::AUDIO) {
     DCHECK(async_);
     RawResource::FetchMedia(new_params, resource_fetcher_, this);
-  } else if (request.GetRequestContext() ==
-             mojom::RequestContextType::MANIFEST) {
+  } else if (request_context == mojom::RequestContextType::MANIFEST) {
     DCHECK(async_);
     RawResource::FetchManifest(new_params, resource_fetcher_, this);
   } else if (async_) {