mojo: Fix Blink's shutdown sequence in AxProviderImplTest

This patch ensures the sequence used to tear down Blink in
AxProviderImplTest matches the real implementation in RenderThreadImpl.
Specifically, we need to shut down the scheduler before terminating
Blink, because otherwise the scheduler might keep pointers to tasks
that live on the Blink heap and access freed memory in its destructor.

We also add an assertion to the renderer scheduler that it is shut down
explicitly.

Note that Blink cannot shut down the scheduler internally because of
crbug.com/467369. We expect that this depedency can be cleaned up once
the message loop is better integrated with the scheduler
(crbug.com/465354) and when the Blink repository has been merged into
Chromium.

BUG=463143

Review URL: https://codereview.chromium.org/1145973011

Cr-Commit-Position: refs/heads/master@{#332221}

1 files changed, 8 insertions, 3 deletions

diff --git a/components/html_viewer/ax_provider_impl_unittest.cc b/components/html_viewer/ax_provider_impl_unittest.cc
index 7508f8a..bfbd9a7 100644
--- a/components/html_viewer/ax_provider_impl_unittest.cc
+++ b/components/html_viewer/ax_provider_impl_unittest.cc
@@ -47,7 +47,8 @@ class TestWebViewClient : public WebViewClient {
 class AxProviderImplTest : public testing::Test {
  public:
   AxProviderImplTest()
-      : renderer_scheduler_(scheduler::RendererScheduler::Create()) {
+      : message_loop_(new base::MessageLoopForUI()),
+        renderer_scheduler_(scheduler::RendererScheduler::Create()) {
 #if defined(V8_USE_EXTERNAL_STARTUP_DATA)
     gin::V8Initializer::LoadV8Snapshot();
 #endif
@@ -55,10 +56,14 @@ class AxProviderImplTest : public testing::Test {
         new html_viewer::BlinkPlatformImpl(nullptr, renderer_scheduler_.get()));
   }
 
-  ~AxProviderImplTest() override { blink::shutdown(); }
+  ~AxProviderImplTest() override {
+    renderer_scheduler_->Shutdown();
+    message_loop_.reset();
+    blink::shutdown();
+  }
 
  private:
-  base::MessageLoopForUI message_loop;
+  scoped_ptr<base::MessageLoopForUI> message_loop_;
   scoped_ptr<scheduler::RendererScheduler> renderer_scheduler_;
 };