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

Side by Side Diff: content/browser/histogram_internals_request_job.h

Issue 2874623002: Implement chrome://histograms with network service. (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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #ifndef CONTENT_BROWSER_HISTOGRAM_INTERNALS_REQUEST_JOB_H_ 5 #ifndef CONTENT_BROWSER_HISTOGRAM_INTERNALS_REQUEST_JOB_H_
6 #define CONTENT_BROWSER_HISTOGRAM_INTERNALS_REQUEST_JOB_H_ 6 #define CONTENT_BROWSER_HISTOGRAM_INTERNALS_REQUEST_JOB_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/macros.h" 10 #include "base/macros.h"
11 #include "base/memory/weak_ptr.h" 11 #include "base/memory/weak_ptr.h"
12 #include "net/url_request/url_request_simple_job.h" 12 #include "net/url_request/url_request_simple_job.h"
13 #include "url/gurl.h"
13 14
14 namespace content { 15 namespace content {
15 16
16 class HistogramInternalsRequestJob : public net::URLRequestSimpleJob { 17 class HistogramInternalsRequestJob : public net::URLRequestSimpleJob {
17 public: 18 public:
18 HistogramInternalsRequestJob(net::URLRequest* request, 19 HistogramInternalsRequestJob(net::URLRequest* request,
19 net::NetworkDelegate* network_delegate); 20 net::NetworkDelegate* network_delegate);
20 21
21 // net::URLRequestSimpleJob: 22 // net::URLRequestSimpleJob:
22 void Start() override; 23 void Start() override;
23 int GetData(std::string* mime_type, 24 int GetData(std::string* mime_type,
24 std::string* charset, 25 std::string* charset,
25 std::string* data, 26 std::string* data,
26 const net::CompletionCallback& callback) const override; 27 const net::CompletionCallback& callback) const override;
27 28
29 // Generates the HTML for chrome://histograms.
30 // base::StatisticsRecorder::ImportProvidedHistograms must have been called
31 // on the UI thread first.
32 static std::string GenerateHTML(const GURL& url);
33
28 private: 34 private:
29 ~HistogramInternalsRequestJob() override; 35 ~HistogramInternalsRequestJob() override;
30 36
31 // Starts the real URL request. 37 // Starts the real URL request.
32 void StartUrlRequest(); 38 void StartUrlRequest();
33 39
34 // The string to select histograms which have |path_| as a substring. 40 // The string to select histograms which have |path_| as a substring.
yzshen1 2017/05/10 15:13:43 Please update the comment.
jam 2017/05/10 15:29:58 Done.
35 std::string path_; 41 GURL url_;
36 42
37 base::WeakPtrFactory<HistogramInternalsRequestJob> weak_factory_; 43 base::WeakPtrFactory<HistogramInternalsRequestJob> weak_factory_;
38 44
39 DISALLOW_IMPLICIT_CONSTRUCTORS(HistogramInternalsRequestJob); 45 DISALLOW_IMPLICIT_CONSTRUCTORS(HistogramInternalsRequestJob);
40 }; 46 };
41 47
42 } // namespace content 48 } // namespace content
43 49
44 #endif // CONTENT_BROWSER_HISTOGRAM_INTERNALS_REQUEST_JOB_H_ 50 #endif // CONTENT_BROWSER_HISTOGRAM_INTERNALS_REQUEST_JOB_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698