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

Side by Side Diff: components/metrics/proto/ukm/source.proto

Issue 2649303004: UKM: Added support for navigation sources (Closed)
Patch Set: Removed redundant test steps Created 3 years, 10 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
« no previous file with comments | « no previous file | components/ukm/BUILD.gn » ('j') | components/ukm/test_ukm_service.h » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2017 The Chromium Authors. All rights reserved. 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 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 syntax = "proto2"; 5 syntax = "proto2";
6 6
7 package ukm; 7 package ukm;
8 8
9 option optimize_for = LITE_RUNTIME; 9 option optimize_for = LITE_RUNTIME;
10 10
11 // Source contains data related to a top-level navigation. 11 // Source contains data related to a top-level navigation.
12 // Next tag: 5 12 // Next tag: 5
13 message Source { 13 message Source {
14 // A identifier for the source. This should be unique even across sessions. 14 // A identifier for the source. This should be unique even across sessions.
15 optional int32 id = 1; 15 optional int32 id = 1;
16 16
17 // The url of the source, as recorded in history. If this URL has not been 17 // The url of the source, as recorded in history. If this URL has not been
18 // discovered by Google's crawler, it must be stripped to the security origin 18 // discovered by Google's crawler, it must be stripped to the security origin
19 // before recording to logs. If the security origin has not been discovered 19 // before recording to logs. If the security origin has not been discovered
20 // by the crawler, the security origin must not be logged. 20 // by the crawler, the security origin must not be logged.
21 optional string url = 2; 21 optional string url = 2;
22 22
23 // Timestamp of navigation to this Source, as seen by the client. Time of 23 // Timestamp of navigation to this Source, as seen by the client. Time of
24 // events related to this Source will generally be relative to this timestamp. 24 // events related to this Source will generally be relative to this timestamp.
25 optional int64 navigation_time_msec = 3; 25 optional int64 navigation_time_msec = 3;
26 26
27 // Time between navigation and the first meaningful paint, in milliseconds. 27 // Time between navigation and the first contentful paint, in milliseconds.
28 optional int64 first_meaningful_paint_msec = 4; 28 optional int64 first_contentful_paint_msec = 4;
Alexei Svitkine (slow) 2017/01/25 15:35:04 Please land this change server-side first before l
oystein (OOO til 10th of July) 2017/01/25 18:38:04 Acknowledged.
29 } 29 }
OLDNEW
« no previous file with comments | « no previous file | components/ukm/BUILD.gn » ('j') | components/ukm/test_ukm_service.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698