AboutURLLoader: check for data pipe creation.

The class mojo::DataPipe shouldn't be used. In its constructor, it
CHECKs the data pipe is created. It can fails when system resources are
insufficient.

This CL replaces the mojo::DataPipe and handles the insuficient resource
case.

Note: No crash reports has been received so far because this URLLoader
      is not used often enough.

Bug: 911652
Change-Id: I09e9ec7cf883a3682466e528b6010fe326916400
Reviewed-on: https://chromium-review.googlesource.com/c/1355124
Commit-Queue: Arthur Sonzogni <[email protected]>
Reviewed-by: Camille Lamy <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#613562}(cherry picked from commit 7d47a55)
Reviewed-on: https://chromium-review.googlesource.com/c/1361873
Reviewed-by: Arthur Sonzogni <[email protected]>
Cr-Commit-Position: refs/branch-heads/3626@{#68}
Cr-Branched-From: d897fb1-refs/heads/master@{#612437}

Author: ArthurSonzogni

content/browser/loader/navigation_url_loader_impl.cc

@@ -358,9 +358,18 @@ class AboutURLLoaderFactory : public network::mojom::URLLoaderFactory {
     network::ResourceResponseHead response_head;
     response_head.mime_type = "text/html";
     client->OnReceiveResponse(response_head);
-    // The response's body is empty, so this data pipe will not be filled.
-    mojo::DataPipe pipe;
-    client->OnStartLoadingResponseBody(std::move(pipe.consumer_handle));
+
+    // Create a data pipe for transmitting the empty response. The |producer|
+    // doesn't add any data.
+    mojo::ScopedDataPipeProducerHandle producer;
+    mojo::ScopedDataPipeConsumerHandle consumer;
+    if (CreateDataPipe(nullptr, &producer, &consumer) != MOJO_RESULT_OK) {
+      client->OnComplete(
+          network::URLLoaderCompletionStatus(net::ERR_INSUFFICIENT_RESOURCES));
+      return;
+    }
+
+    client->OnStartLoadingResponseBody(std::move(consumer));
     client->OnComplete(network::URLLoaderCompletionStatus(net::OK));
   }