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

Side by Side Diff: components/metrics/proto/translate_event.proto

Issue 2400503002: [Translate] Integrate TranslateEventProto UMA logging into TranslateManager. (Closed)
Patch Set: use unique_ptr Created 4 years, 2 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
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 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 // Stores information about translate UI, translate Ranekr and user interaction. 5 // Stores information about translate UI, translate Ranekr and user interaction.
6 6
7 syntax = "proto2"; 7 syntax = "proto2";
8 8
9 option optimize_for = LITE_RUNTIME; 9 option optimize_for = LITE_RUNTIME;
10 option java_outer_classname = "TranslateEventProtos"; 10 option java_outer_classname = "TranslateEventProtos";
(...skipping 93 matching lines...) Expand 10 before | Expand all | Expand 10 after
104 // many times. 104 // many times.
105 LANGUAGE_DISABLED_BY_AUTO_BLACKLIST = 14; 105 LANGUAGE_DISABLED_BY_AUTO_BLACKLIST = 14;
106 // The translation was not offered because Ranker dismissed it. 106 // The translation was not offered because Ranker dismissed it.
107 DISABLED_BY_RANKER = 15; 107 DISABLED_BY_RANKER = 15;
108 // The translation was not offered because the source or target 108 // The translation was not offered because the source or target
109 // language is not supported. 109 // language is not supported.
110 UNSUPPORTED_LANGUAGE = 16; 110 UNSUPPORTED_LANGUAGE = 16;
111 // The translation was not offered because the URL is not 111 // The translation was not offered because the URL is not
112 // supported (e.g. New Tab Page). 112 // supported (e.g. New Tab Page).
113 UNSUPPORTED_URL = 17; 113 UNSUPPORTED_URL = 17;
114 // The previous page was in the same language, so the translate UI was
115 // suppressed.
116 MATCHES_PREVIOUS_LANGUAGE = 18;
117 // The web context for the translate is no longer active.
hamelphi 2016/10/17 16:14:40 Please describe explicitly what happened with Tran
Roger McFarlane (Chromium) 2016/10/24 19:01:52 Done.
118 WEBCONTENTS_NOT_ACTIVE = 19;
119 // The browser context for translate prompt is no longer active.
120 BROSWER_WINDOW_NOT_ACTIVE = 20;
121 // The browser window is minimized.
122 BROWSER_WINDOW_MINIMIZED = 21;
123 // The focus in on an editable field.
rkaplow 2016/10/17 14:48:20 not sure what this last enum is really trying to s
Roger McFarlane (Chromium) 2016/10/24 19:01:52 Done.
124 EDITABLE_FIELD_IN_FOCUS = 22;
114 } 125 }
115 126
116 // Event received from translate UI. 127 // Event received from translate UI.
117 optional EventType event_type = 10; 128 optional EventType event_type = 10;
118 129
119 // The timestamp for the event, in seconds. 130 // The timestamp for the event, in seconds.
120 // This value comes from Chromium's TimeTicks::Now(), which is an abstract 131 // This value comes from Chromium's TimeTicks::Now(), which is an abstract
121 // time value that is guaranteed to always be increasing (regardless of 132 // time value that is guaranteed to always be increasing (regardless of
122 // Daylight Saving Time or any other changes to the system clock). 133 // Daylight Saving Time or any other changes to the system clock).
123 // These numbers are only comparable within a session. To sequence events 134 // These numbers are only comparable within a session. To sequence events
124 // across sessions, order by the |session_id| from the 135 // across sessions, order by the |session_id| from the
125 // ChromeUserMetricsExtension message. 136 // ChromeUserMetricsExtension message.
126 optional int64 event_timestamp_sec = 11; 137 optional int64 event_timestamp_sec = 11;
127 138
128 // Modified source language, if the user changed it. If changed more than 139 // Modified source language, if the user changed it. If changed more than
129 // once, we only keep the last one. 140 // once, we only keep the last one.
130 optional string modified_source_language = 12; 141 optional string modified_source_language = 12;
131 // Modified target language, if the user changed it. If changed more than 142 // Modified target language, if the user changed it. If changed more than
132 // once, we only keep the last one. 143 // once, we only keep the last one.
133 optional string modified_target_language = 13; 144 optional string modified_target_language = 13;
134 } 145 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698