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

Side by Side Diff: components/metrics/net/net_metrics_log_uploader.cc

Issue 291153013: Make MetricsService upload logs through an interface. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Casts for win64 compile Created 6 years, 6 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 | Annotate | Revision Log
OLDNEW
(Empty)
1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #include "components/metrics/net/net_metrics_log_uploader.h"
6
7 #include "base/metrics/histogram.h"
8 #include "components/metrics/net/compression_utils.h"
9 #include "net/base/load_flags.h"
10 #include "net/url_request/url_fetcher.h"
11 #include "url/gurl.h"
12
13 namespace metrics {
14
15 NetMetricsLogUploader::NetMetricsLogUploader(
16 net::URLRequestContextGetter* request_context_getter,
17 const std::string& server_url,
18 const std::string& mime_type,
19 const base::Callback<void(int)>& on_upload_complete)
20 : MetricsLogUploader(server_url, mime_type, on_upload_complete),
21 request_context_getter_(request_context_getter) {
22 }
23
24 NetMetricsLogUploader::~NetMetricsLogUploader() {
25 }
26
27 bool NetMetricsLogUploader::UploadLog(const std::string& log_data,
28 const std::string& log_hash) {
29 std::string compressed_log_data;
30 if (!GzipCompress(log_data, &compressed_log_data)) {
31 NOTREACHED();
32 return false;
33 }
34
35 UMA_HISTOGRAM_PERCENTAGE(
36 "UMA.ProtoCompressionRatio",
37 static_cast<int>(100 * compressed_log_data.size() / log_data.size()));
38 UMA_HISTOGRAM_CUSTOM_COUNTS(
39 "UMA.ProtoGzippedKBSaved",
40 static_cast<int>((log_data.size() - compressed_log_data.size()) / 1024),
41 1, 2000, 50);
42
43 current_fetch_.reset(
44 net::URLFetcher::Create(GURL(server_url_), net::URLFetcher::POST, this));
45 current_fetch_->SetRequestContext(request_context_getter_);
46 current_fetch_->SetUploadData(mime_type_, compressed_log_data);
47
48 // Tell the server that we're uploading gzipped protobufs.
49 current_fetch_->SetExtraRequestHeaders("content-encoding: gzip");
50
51 DCHECK(!log_hash.empty());
52 current_fetch_->AddExtraRequestHeader("X-Chrome-UMA-Log-SHA1: " + log_hash);
53
54 // We already drop cookies server-side, but we might as well strip them out
55 // client-side as well.
56 current_fetch_->SetLoadFlags(net::LOAD_DO_NOT_SAVE_COOKIES |
57 net::LOAD_DO_NOT_SEND_COOKIES);
58 return true;
59 }
60
61 void NetMetricsLogUploader::OnURLFetchComplete(const net::URLFetcher* source) {
62 // We're not allowed to re-use the existing |URLFetcher|s, so free them here.
63 // Note however that |source| is aliased to the fetcher, so we should be
64 // careful not to delete it too early.
65 DCHECK_EQ(current_fetch_.get(), source);
66
67 int response_code = source->GetResponseCode();
68 if (response_code == net::URLFetcher::RESPONSE_CODE_INVALID)
69 response_code = -1;
70 on_upload_complete_.Run(response_code);
71 current_fetch_.reset();
72 }
73
74 } // namespace metrics
OLDNEW
« no previous file with comments | « components/metrics/net/net_metrics_log_uploader.h ('k') | components/metrics/test_metrics_service_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698