Quick fix for logging.

chrome/browser/translate/chrome_translate_client.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/translate/chrome_translate_client.h"
 
 #include <memory>
 #include <vector>
 
 #include "base/logging.h"
@@ skipping 68 matching lines @@
 
   syncer::UserEventService* const user_event_service =
       browser_sync::UserEventServiceFactory::GetForProfile(profile);
 
   const auto* const entry =
       web_contents->GetController().GetLastCommittedEntry();
 
   // 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 this.

[renjieliu1, 2017/06/05 06:39:19]

Done.

     user_event_service->RecordUserEvent(
-        translate::ConstructLanguageDetectionEvent(details));
+        translate::ConstructLanguageDetectionEvent(
+            entry->GetTimestamp().ToInternalValue(), details));
   }
 }
 
 }  // namespace
 
 DEFINE_WEB_CONTENTS_USER_DATA_KEY(ChromeTranslateClient);
 
 ChromeTranslateClient::ChromeTranslateClient(content::WebContents* web_contents)
     : content::WebContentsObserver(web_contents),
       translate_driver_(&web_contents->GetController()),
@@ skipping 290 matching lines @@
       return ShowTranslateBubbleResult::SUCCESS;
   }
 
   return TranslateBubbleFactory::Show(browser->window(), web_contents(), step,
                                       error_type);
 #else
   NOTREACHED();
   return ShowTranslateBubbleResult::SUCCESS;
 #endif
 }