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

Unified Diff: dart_microlytics/lib/io_channels.dart

Issue 515993003: Minimalistic analytics library used by Dart Server and try.dartlang.org (Closed) Base URL: https://github.com/lukechurch/dart-mircolytics.git@master
Patch Set: Created 6 years, 3 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: dart_microlytics/lib/io_channels.dart
diff --git a/dart_microlytics/lib/io_channels.dart b/dart_microlytics/lib/io_channels.dart
new file mode 100644
index 0000000000000000000000000000000000000000..49186a43755ed827e0170a284488f98f8f3f7ca5
--- /dev/null
+++ b/dart_microlytics/lib/io_channels.dart
@@ -0,0 +1,24 @@
+// Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file
+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
ahe 2014/09/02 13:44:24 Add line between copyright and library declaration
lukechurch 2014/09/02 19:52:37 Done.
+library microlytics.io_channels;
+
+import 'dart:io';
+import 'channels.dart';
+
+final String _ANALYTICS_URL = "https://www.google-analytics.com/collect";
ahe 2014/09/02 13:44:24 Can you share these constants to avoid defining th
lukechurch 2014/09/02 19:52:37 Done. I wanted to do this, but it breaks the _priv
+
+class HttpClientChannel extends Channel {
+ void sendData(String data) {
+ HttpClient client = new HttpClient();
+ client.postUrl(Uri.parse(_ANALYTICS_URL)).then((HttpClientRequest req) {
+ req.write(data);
+ return req.close();
+ }).then((HttpClientResponse response) {
+ print(response.statusCode);
ahe 2014/09/02 13:44:24 Debug code?
lukechurch 2014/09/02 19:52:37 /red face. Done.
+ response.drain();
+ });
+
ahe 2014/09/02 13:44:24 Remove extra line.
lukechurch 2014/09/02 19:52:37 Done.
+ }
+}
+

Powered by Google App Engine
This is Rietveld 408576698