Chromium Code Reviews| Index: chrome/browser/translate/chrome_translate_client.cc |
| diff --git a/chrome/browser/translate/chrome_translate_client.cc b/chrome/browser/translate/chrome_translate_client.cc |
| index c29d82fc79b9440dc06d7cac05cc956683739d89..b51735250d4906d7bbdfe80e977c77eca8d15996 100644 |
| --- a/chrome/browser/translate/chrome_translate_client.cc |
| +++ b/chrome/browser/translate/chrome_translate_client.cc |
| @@ -86,9 +86,11 @@ void LogLanguageDetectionEvent( |
| // If entry is null, we don't record the page. |
| // The navigation entry can be null in situations like download or initial |
| // blank page. |
| - if (entry != nullptr) { |
| + if (entry != nullptr && |
| + TranslateService::IsTranslatableURL(entry->GetVirtualURL())) { |
|
skym
2017/06/02 21:22:40
Would be nice to have a test case get filtered by
renjieliu1
2017/06/05 06:39:19
Done.
|
| user_event_service->RecordUserEvent( |
| - translate::ConstructLanguageDetectionEvent(details)); |
| + translate::ConstructLanguageDetectionEvent( |
| + entry->GetTimestamp().ToInternalValue(), details)); |
| } |
| } |