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

Side by Side Diff: android_webview/native/aw_metrics_log_uploader.cc

Issue 2778203003: WebView: Create UMA uploader (Closed)
Patch Set: Created 3 years, 8 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
(Empty)
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
3 // found in the LICENSE file.
4
5 #include "android_webview/native/aw_metrics_log_uploader.h"
6
7 #include "android_webview/jni/AwMetricsLogUploader_jni.h"
8 #include "base/android/jni_array.h"
9
10 using base::android::ScopedJavaLocalRef;
11 using base::android::ToJavaByteArray;
12
13 namespace android_webview {
14
15 AwMetricsLogUploader::AwMetricsLogUploader(
16 const base::Callback<void(int)>& on_upload_complete)
17 : on_upload_complete_(on_upload_complete) {}
18
19 void AwMetricsLogUploader::UploadLog(const std::string& uncompressed_log_data) {
20 JNIEnv* env = base::android::AttachCurrentThread();
21 ScopedJavaLocalRef<jbyteArray> java_data = ToJavaByteArray(
22 env, reinterpret_cast<const uint8_t*>(uncompressed_log_data.data()),
23 uncompressed_log_data.size());
24 Java_AwMetricsLogUploader_uploadLog(env, java_data);
25
26 // The platform mechanism doesn't provide a response code or any way to handle
27 // failures, so we have nothing to pass to on_upload_complete. Just pass 200
28 // (HTTP OK) and pretend everything is peachy.
29 on_upload_complete_.Run(200);
30 }
31
32 } // namespace android_webview
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698