[translate] Fix shutdown race for translate ranker model loader.

components/translate/core/browser/translate_script_unittest.cc

Index: components/translate/core/browser/translate_script_unittest.cc
diff --git a/components/translate/core/browser/translate_script_unittest.cc b/components/translate/core/browser/translate_script_unittest.cc
index e6547841f67f617526bee9e580d228880dfa462e..5cc47984f0908748a529b58485b2f45a2673320d 100644
--- a/components/translate/core/browser/translate_script_unittest.cc
+++ b/components/translate/core/browser/translate_script_unittest.cc
@@ -7,7 +7,10 @@
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/macros.h"
+#include "base/memory/ref_counted.h"

[groby-ooo-7-16, 2017/04/24 19:12:46]

Why ref_counted.h?

[Roger McFarlane (Chromium), 2017/04/24 20:31:34]

I had originally added a scoped_refptr<net::URLRequestContextGetter> member to
the test fixture, but, since no-other code was referencing it, I just pass the
raw pointer into the download manager... and forgot to remove the inlcude.

Thanks.

 #include "base/strings/stringprintf.h"
+#include "base/test/scoped_task_scheduler.h"
+#include "base/threading/thread_task_runner_handle.h"
 #include "build/build_config.h"
 #include "components/translate/core/browser/translate_download_manager.h"
 #include "components/translate/core/common/translate_switches.h"
@@ -15,6 +18,7 @@
 #include "net/base/url_util.h"
 #include "net/http/http_request_headers.h"
 #include "net/url_request/test_url_fetcher_factory.h"
+#include "net/url_request/url_request_test_util.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/gurl.h"
 
@@ -29,6 +33,9 @@ class TranslateScriptTest : public testing::Test {
     script_.reset(new TranslateScript);
     DCHECK(script_.get());
     TranslateDownloadManager::GetInstance()->set_application_locale("en");
+    TranslateDownloadManager::GetInstance()->set_request_context(
+        new net::TestURLRequestContextGetter(
+            base::ThreadTaskRunnerHandle::Get()));
   }
 
   void TearDown() override { script_.reset(); }
@@ -46,7 +53,13 @@ class TranslateScriptTest : public testing::Test {
   void OnComplete(bool success, const std::string& script) {
   }
 
+  // Sets up the task scheduling/task-runner environment for each test.
+  base::test::ScopedTaskScheduler scoped_task_scheduler_;
+
+  // The translate script.
   std::unique_ptr<TranslateScript> script_;
+
+  // Factory to create programmatic URL fetchers.
   net::TestURLFetcherFactory url_fetcher_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(TranslateScriptTest);