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

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

Issue 2757173002: Fix counting of ULP and DISABLED_BY_PREF. (Closed)
Patch Set: rebase to include lastest code changes. For real this time. Created 3 years, 9 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/translate/core/browser/translate_manager.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 113 matching lines...) Expand 10 before | Expand all | Expand 10 after
124 // The translate UI was not shown because the browser window for the 124 // The translate UI was not shown because the browser window for the
125 // translate prompt is no longer active. 125 // translate prompt is no longer active.
126 BROWSER_WINDOW_NOT_ACTIVE = 20; 126 BROWSER_WINDOW_NOT_ACTIVE = 20;
127 // The translate UI was not shown because the browser window is minimized. 127 // The translate UI was not shown because the browser window is minimized.
128 BROWSER_WINDOW_IS_MINIMIZED = 21; 128 BROWSER_WINDOW_IS_MINIMIZED = 21;
129 // The translate UI was not shown because the web context for the translate 129 // The translate UI was not shown because the web context for the translate
130 // prompt is no longer active. 130 // prompt is no longer active.
131 WEB_CONTENTS_NOT_ACTIVE = 22; 131 WEB_CONTENTS_NOT_ACTIVE = 22;
132 // The translate UI was not shown because the user is editing a form field. 132 // The translate UI was not shown because the user is editing a form field.
133 EDITABLE_FIELD_IS_ACTIVE = 23; 133 EDITABLE_FIELD_IS_ACTIVE = 23;
134 // The translate UI was not shown because the language is in the user's User
135 // Language Profile.
136 LANGUAGE_IN_ULP = 24;
134 } 137 }
135 138
136 // Event received from translate UI. 139 // Event received from translate UI.
137 optional EventType event_type = 10; 140 optional EventType event_type = 10;
138 141
139 // The timestamp for the event, in seconds. 142 // The timestamp for the event, in seconds.
140 // This value comes from Chromium's TimeTicks::Now(), which is an abstract 143 // This value comes from Chromium's TimeTicks::Now(), which is an abstract
141 // time value that is guaranteed to always be increasing (regardless of 144 // time value that is guaranteed to always be increasing (regardless of
142 // Daylight Saving Time or any other changes to the system clock). 145 // Daylight Saving Time or any other changes to the system clock).
143 // These numbers are only comparable within a session. To sequence events 146 // These numbers are only comparable within a session. To sequence events
144 // across sessions, order by the |session_id| from the 147 // across sessions, order by the |session_id| from the
145 // ChromeUserMetricsExtension message. 148 // ChromeUserMetricsExtension message.
146 optional int64 event_timestamp_sec = 11; 149 optional int64 event_timestamp_sec = 11;
147 150
148 // Modified source language, if the user changed it. If changed more than 151 // Modified source language, if the user changed it. If changed more than
149 // once, we only keep the last one. 152 // once, we only keep the last one.
150 optional string modified_source_language = 12; 153 optional string modified_source_language = 12;
151 // Modified target language, if the user changed it. If changed more than 154 // Modified target language, if the user changed it. If changed more than
152 // once, we only keep the last one. 155 // once, we only keep the last one.
153 optional string modified_target_language = 13; 156 optional string modified_target_language = 13;
154 } 157 }
OLDNEW
« no previous file with comments | « no previous file | components/translate/core/browser/translate_manager.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698