Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(853)

Side by Side Diff: content/browser/histogram_internals_url_loader.cc

Issue 2875143002: Reduce boilerplate when creating simple mojom::URLLoaders. (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 1 // Copyright 2017 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/histogram_internals_url_loader.h" 5 #include "content/browser/histogram_internals_url_loader.h"
6 6
7 #include "base/metrics/histogram_macros.h" 7 #include "base/metrics/histogram_macros.h"
8 #include "base/metrics/statistics_recorder.h" 8 #include "base/metrics/statistics_recorder.h"
9 #include "content/browser/histogram_internals_request_job.h" 9 #include "content/browser/histogram_internals_request_job.h"
10 #include "mojo/common/data_pipe_utils.h" 10 #include "mojo/common/data_pipe_utils.h"
11 11
12 namespace content { 12 namespace content {
13 13
14 void StartHistogramInternalsURLLoader(const ResourceRequest& request, 14 void StartHistogramInternalsURLLoader(const ResourceRequest& request,
15 mojom::URLLoaderClientPtr client) { 15 mojom::URLLoaderClientPtr client) {
16 scoped_refptr<net::HttpResponseHeaders> headers( 16 scoped_refptr<net::HttpResponseHeaders> headers(
17 new net::HttpResponseHeaders("HTTP/1.1 200 OK")); 17 new net::HttpResponseHeaders("HTTP/1.1 200 OK"));
18 ResourceResponseHead resource_response; 18 ResourceResponseHead resource_response;
19 resource_response.headers = headers; 19 resource_response.headers = headers;
20 resource_response.mime_type = "text/html"; 20 resource_response.mime_type = "text/html";
21 client->OnReceiveResponse(resource_response, base::nullopt, nullptr); 21 client->OnReceiveResponse(resource_response, base::nullopt, nullptr);
22 22
23 base::StatisticsRecorder::ImportProvidedHistograms(); 23 base::StatisticsRecorder::ImportProvidedHistograms();
24 std::string data = HistogramInternalsRequestJob::GenerateHTML(request.url); 24 std::string data = HistogramInternalsRequestJob::GenerateHTML(request.url);
25 25 mojo::DataPipe data_pipe(data.size());
26 MojoCreateDataPipeOptions options;
27 options.struct_size = sizeof(MojoCreateDataPipeOptions);
28 options.flags = MOJO_CREATE_DATA_PIPE_OPTIONS_FLAG_NONE;
29 options.element_num_bytes = 1;
30 options.capacity_num_bytes = data.size();
31 mojo::DataPipe data_pipe(options);
32
33 DCHECK(data_pipe.producer_handle.is_valid());
34 DCHECK(data_pipe.consumer_handle.is_valid());
35
36 CHECK(mojo::common::BlockingCopyFromString(data, data_pipe.producer_handle)); 26 CHECK(mojo::common::BlockingCopyFromString(data, data_pipe.producer_handle));
37 27
38 client->OnStartLoadingResponseBody(std::move(data_pipe.consumer_handle)); 28 client->OnStartLoadingResponseBody(std::move(data_pipe.consumer_handle));
39 29 client->OnComplete(ResourceRequestCompletionStatus(data.size()));
40 ResourceRequestCompletionStatus request_complete_data;
41 request_complete_data.error_code = net::OK;
42 request_complete_data.exists_in_cache = false;
43 request_complete_data.completion_time = base::TimeTicks::Now();
44 request_complete_data.encoded_data_length = data.size();
45 request_complete_data.encoded_body_length = data.size();
46 client->OnComplete(request_complete_data);
47 } 30 }
48 31
49 } // namespace content 32 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/blob_storage/blob_internals_url_loader.cc ('k') | content/browser/webui/web_ui_url_loader_factory.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698