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

Unified Diff: packages/usage/example/example.dart

Issue 2990843002: Removed fixed dependencies (Closed)
Patch Set: Created 3 years, 5 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
« no previous file with comments | « packages/usage/changelog.md ('k') | packages/usage/example/ga.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: packages/usage/example/example.dart
diff --git a/packages/usage/example/example.dart b/packages/usage/example/example.dart
index 06578c36812c9643246b1dcfbe6ad5929668c922..9d9342c2b41a156d9652449e47001d9500397b1f 100644
--- a/packages/usage/example/example.dart
+++ b/packages/usage/example/example.dart
@@ -2,7 +2,9 @@
// 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.
-/// A simple web app to hand-test the usage library.
+/**
+ * A simple web app to hand-test the usage library.
+ */
library usage_example;
import 'dart:html';
@@ -14,9 +16,9 @@ String _lastUa;
int _count = 0;
void main() {
- querySelector('#foo').onClick.listen((_) => _handleFoo());
- querySelector('#bar').onClick.listen((_) => _handleBar());
- querySelector('#page').onClick.listen((_) => _changePage());
+ querySelector('#foo').onClick.listen((_) => _handleFoo(getAnalytics()));
+ querySelector('#bar').onClick.listen((_) => _handleBar(getAnalytics()));
+ querySelector('#page').onClick.listen((_) => _changePage(getAnalytics()));
}
String _ua() => (querySelector('#ua') as InputElement).value.trim();
@@ -25,24 +27,22 @@ Analytics getAnalytics() {
if (_analytics == null || _lastUa != _ua()) {
_lastUa = _ua();
_analytics = new AnalyticsHtml(_lastUa, 'Test app', '1.0');
+ _analytics.optIn = true;
_analytics.sendScreenView(window.location.pathname);
}
return _analytics;
}
-void _handleFoo() {
- Analytics analytics = getAnalytics();
+void _handleFoo(Analytics analytics) {
analytics.sendEvent('main', 'foo');
}
-void _handleBar() {
- Analytics analytics = getAnalytics();
+void _handleBar(Analytics analytics) {
analytics.sendEvent('main', 'bar');
}
-void _changePage() {
- Analytics analytics = getAnalytics();
+void _changePage(Analytics analytics) {
window.history.pushState(null, 'new page', '${++_count}.html');
analytics.sendScreenView(window.location.pathname);
}
« no previous file with comments | « packages/usage/changelog.md ('k') | packages/usage/example/ga.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698