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

Unified Diff: components/metrics/proto/ukm/source.proto

Issue 2567263003: Basic UkmService implementation (Closed)
Patch Set: Proto and client_id Created 4 years 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: components/metrics/proto/ukm/source.proto
diff --git a/components/metrics/proto/ukm/source.proto b/components/metrics/proto/ukm/source.proto
new file mode 100644
index 0000000000000000000000000000000000000000..617de5160c27cf231c9504d08f915c49d407044d
--- /dev/null
+++ b/components/metrics/proto/ukm/source.proto
@@ -0,0 +1,27 @@
+// 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.
+
+syntax = "proto2";
+
+package ukm;
+
+option optimize_for = LITE_RUNTIME;
+
+// Source contains data related to a top-level navigation.
+// Next tag: 5
+message Source {
+ // A identifier for the source. This should be unique even across sessions.
+ optional int32 id = 1;
+
+ // The url of the source, as recorded in history. If this URL does not
+ // appear in FirstTimes, it should be stripped before recording to logs.
rkaplow 2017/01/04 17:57:57 I would reword this comment to not mention interna
+ optional string url = 2;
+
+ // Timestamp of navigation to this Source, as seen by the client. Time of
+ // events related to this Source will generally be relative to this timestamp.
+ optional int64 navigation_time_msec = 3;
+
+ // Time between navigation and the first meaningful paint, in milliseconds.
+ optional int64 first_meaningful_paint_msec = 4;
+}

Powered by Google App Engine
This is Rietveld 408576698