Send UMA recording state to TranslateRanker.

components/translate/core/browser/translate_ranker_impl_unittest.cc

Index: components/translate/core/browser/translate_ranker_impl_unittest.cc
diff --git a/components/translate/core/browser/translate_ranker_impl_unittest.cc b/components/translate/core/browser/translate_ranker_impl_unittest.cc
index b61b4b7bbfc5534808a8684d30668216657b0be9..bdc3306be474fc1cf7df751932eeab79e746b7c7 100644
--- a/components/translate/core/browser/translate_ranker_impl_unittest.cc
+++ b/components/translate/core/browser/translate_ranker_impl_unittest.cc
@@ -350,10 +350,11 @@ TEST_F(TranslateRankerImplTest, LoggingDisabledViaOverride) {
   ranker->FlushTranslateEvents(&flushed_events);
   EXPECT_EQ(3U, flushed_events.size());
 
-  // Override the feature setting to disable logging.
+  ranker->RecordTranslateEvent(0, GURL(), &translate_event1_);
+  // Override the feature setting to disable logging. The event cache will be
+  // cleared.
   ranker->EnableLogging(false);
 

[Roger McFarlane (Chromium), 2017/06/06 19:28:27]

before and after EnableLogging(), can we assert the cache size?

[hamelphi, 2017/06/06 20:28:06]

Not easily, since the cache is private. However, RecordTranslateEvent and
FlushTranslateEvents are tested a few lines above.

I don't think it warrants exposing the cache just for testing.

-  ranker->RecordTranslateEvent(0, GURL(), &translate_event1_);
   ranker->RecordTranslateEvent(1, GURL(), &translate_event2_);
   ranker->RecordTranslateEvent(2, GURL(), &translate_event3_);