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

Unified Diff: android_webview/native/aw_metrics_log_uploader.cc

Issue 2778203003: WebView: Create UMA uploader (Closed)
Patch Set: Created 3 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: android_webview/native/aw_metrics_log_uploader.cc
diff --git a/android_webview/native/aw_metrics_log_uploader.cc b/android_webview/native/aw_metrics_log_uploader.cc
new file mode 100644
index 0000000000000000000000000000000000000000..d9d470358412d4e716afc8d3a67f388d40677211
--- /dev/null
+++ b/android_webview/native/aw_metrics_log_uploader.cc
@@ -0,0 +1,32 @@
+// Copyright 2017 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "android_webview/native/aw_metrics_log_uploader.h"
+
+#include "android_webview/jni/AwMetricsLogUploader_jni.h"
+#include "base/android/jni_array.h"
+
+using base::android::ScopedJavaLocalRef;
+using base::android::ToJavaByteArray;
+
+namespace android_webview {
+
+AwMetricsLogUploader::AwMetricsLogUploader(
+ const base::Callback<void(int)>& on_upload_complete)
+ : on_upload_complete_(on_upload_complete) {}
+
+void AwMetricsLogUploader::UploadLog(const std::string& uncompressed_log_data) {
+ JNIEnv* env = base::android::AttachCurrentThread();
+ ScopedJavaLocalRef<jbyteArray> java_data = ToJavaByteArray(
+ env, reinterpret_cast<const uint8_t*>(uncompressed_log_data.data()),
+ uncompressed_log_data.size());
+ Java_AwMetricsLogUploader_uploadLog(env, java_data);
+
+ // The platform mechanism doesn't provide a response code or any way to handle
+ // failures, so we have nothing to pass to on_upload_complete. Just pass 200
+ // (HTTP OK) and pretend everything is peachy.
+ on_upload_complete_.Run(200);
+}
+
+} // namespace android_webview

Powered by Google App Engine
This is Rietveld 408576698