Add missing TaskScheduler on tests using RendereWidgetHostImpl

content/browser/renderer_host/render_widget_host_view_mac_editcommand_helper_unittest.mm

Index: content/browser/renderer_host/render_widget_host_view_mac_editcommand_helper_unittest.mm
diff --git a/content/browser/renderer_host/render_widget_host_view_mac_editcommand_helper_unittest.mm b/content/browser/renderer_host/render_widget_host_view_mac_editcommand_helper_unittest.mm
index 7c95711be0acb82a5f76300a4ea3a593b655ea0c..2fa64bc9b19a09ef933e6eb29e84477d7050e63b 100644
--- a/content/browser/renderer_host/render_widget_host_view_mac_editcommand_helper_unittest.mm
+++ b/content/browser/renderer_host/render_widget_host_view_mac_editcommand_helper_unittest.mm
@@ -10,6 +10,7 @@
 
 #include "base/mac/scoped_nsautorelease_pool.h"
 #include "base/message_loop/message_loop.h"
+#include "base/test/scoped_task_scheduler.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "content/browser/compositor/test/no_transport_image_transport_factory.h"
 #include "content/browser/gpu/compositor_util.h"
@@ -121,21 +122,24 @@ TEST_F(RenderWidgetHostViewMacEditCommandHelperTest,
        TestEditingCommandDelivery) {
   MockRenderWidgetHostDelegate delegate;
   TestBrowserContext browser_context;
-  MockRenderProcessHost process_host(&browser_context);
+  MockRenderProcessHostFactory process_host_factory;
+  RenderProcessHost* process_host =
+      process_host_factory.CreateRenderProcessHost(&browser_context, nullptr);
 
   // Populates |g_supported_scale_factors|.
   std::vector<ui::ScaleFactor> supported_factors;
   supported_factors.push_back(ui::SCALE_FACTOR_100P);
   ui::test::ScopedSetSupportedScaleFactors scoped_supported(supported_factors);
 
-  int32_t routing_id = process_host.GetNextRoutingID();
+  base::mac::ScopedNSAutoreleasePool pool;
+  base::MessageLoop message_loop;
+  base::test::ScopedTaskScheduler task_scheduler(&message_loop);

[gab, 2017/03/31 18:06:59]

I think you can merge the two lines above (ScopedTaskScheduler comes with an
implicit MessageLoop if not handed one and no callers below use |message_loop|
explicitly.

PS: Yeah, I know, this is kind of a mess... Aimed to be solved by
https://docs.google.com/document/d/1QabRo8c7D9LsYY3cEcaPQbOCLo8Tu-6VLykYXyl3P...

+
+  int32_t routing_id = process_host->GetNextRoutingID();
   RenderWidgetHostEditCommandCounter* render_widget =
-      new RenderWidgetHostEditCommandCounter(&delegate, &process_host,
+      new RenderWidgetHostEditCommandCounter(&delegate, process_host,
                                              routing_id);
 
-  base::mac::ScopedNSAutoreleasePool pool;
-
-  base::MessageLoop message_loop;
   ui::WindowResizeHelperMac::Get()->Init(base::ThreadTaskRunnerHandle::Get());
 
   // Owned by its |cocoa_view()|, i.e. |rwhv_cocoa|.