OLD | NEW |
1 // Copyright 2014 The Chromium Authors. All rights reserved. | 1 // Copyright 2014 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 #include "chrome/browser/translate/chrome_translate_client.h" | 5 #include "chrome/browser/translate/chrome_translate_client.h" |
6 | 6 |
7 #include <memory> | 7 #include <memory> |
8 #include <vector> | 8 #include <vector> |
9 | 9 |
10 #include "base/logging.h" | 10 #include "base/logging.h" |
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
79 | 79 |
80 syncer::UserEventService* const user_event_service = | 80 syncer::UserEventService* const user_event_service = |
81 browser_sync::UserEventServiceFactory::GetForProfile(profile); | 81 browser_sync::UserEventServiceFactory::GetForProfile(profile); |
82 | 82 |
83 const auto* const entry = | 83 const auto* const entry = |
84 web_contents->GetController().GetLastCommittedEntry(); | 84 web_contents->GetController().GetLastCommittedEntry(); |
85 | 85 |
86 // If entry is null, we don't record the page. | 86 // If entry is null, we don't record the page. |
87 // The navigation entry can be null in situations like download or initial | 87 // The navigation entry can be null in situations like download or initial |
88 // blank page. | 88 // blank page. |
89 if (entry != nullptr) { | 89 DCHECK(web_contents); |
| 90 if (entry != nullptr && |
| 91 TranslateService::IsTranslatableURL(entry->GetVirtualURL())) { |
90 user_event_service->RecordUserEvent( | 92 user_event_service->RecordUserEvent( |
91 translate::ConstructLanguageDetectionEvent(details)); | 93 translate::ConstructLanguageDetectionEvent( |
| 94 entry->GetTimestamp().ToInternalValue(), details)); |
92 } | 95 } |
93 } | 96 } |
94 | 97 |
95 } // namespace | 98 } // namespace |
96 | 99 |
97 DEFINE_WEB_CONTENTS_USER_DATA_KEY(ChromeTranslateClient); | 100 DEFINE_WEB_CONTENTS_USER_DATA_KEY(ChromeTranslateClient); |
98 | 101 |
99 ChromeTranslateClient::ChromeTranslateClient(content::WebContents* web_contents) | 102 ChromeTranslateClient::ChromeTranslateClient(content::WebContents* web_contents) |
100 : content::WebContentsObserver(web_contents), | 103 : content::WebContentsObserver(web_contents), |
101 translate_driver_(&web_contents->GetController()), | 104 translate_driver_(&web_contents->GetController()), |
(...skipping 290 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
392 return ShowTranslateBubbleResult::SUCCESS; | 395 return ShowTranslateBubbleResult::SUCCESS; |
393 } | 396 } |
394 | 397 |
395 return TranslateBubbleFactory::Show(browser->window(), web_contents(), step, | 398 return TranslateBubbleFactory::Show(browser->window(), web_contents(), step, |
396 error_type); | 399 error_type); |
397 #else | 400 #else |
398 NOTREACHED(); | 401 NOTREACHED(); |
399 return ShowTranslateBubbleResult::SUCCESS; | 402 return ShowTranslateBubbleResult::SUCCESS; |
400 #endif | 403 #endif |
401 } | 404 } |
OLD | NEW |