Add ScopedTaskScheduler to RenderWidgetHostViewChildFrameTest

Add ScopedTaskScheduler to RenderWidgetHostViewChildFrameTest

RenderWidgetHostViewChildFrameTest.VerifyRequestContextTypeForFrameTree
was failing due to missing TaskScheduler used by RenderWidgetHostImpl
constructor. This CL adds a ScopedTaskScheduler to the test fixture
and re-enables the disabled test.

BUG=696919
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

[tzik, 2017-02-28 14:00:04]

Description was changed from

==========
Add ScopedTaskScheduler to RenderWidgetHostViewChildFrameTest

RenderWidgetHostViewChildFrameTest.VerifyRequestContextTypeForFrameTree
was failing due to missing TaskScheduler used by RenderWidgetHostImpl
constructor. This CL adds a ScopedTaskScheduler to the test fixture
and re-enables the disabled test.
==========

to

==========
Add ScopedTaskScheduler to RenderWidgetHostViewChildFrameTest

RenderWidgetHostViewChildFrameTest.VerifyRequestContextTypeForFrameTree
was failing due to missing TaskScheduler used by RenderWidgetHostImpl
constructor. This CL adds a ScopedTaskScheduler to the test fixture
and re-enables the disabled test.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[tzik, 2017-02-28 14:01:20]

Description was changed from

==========
Add ScopedTaskScheduler to RenderWidgetHostViewChildFrameTest

RenderWidgetHostViewChildFrameTest.VerifyRequestContextTypeForFrameTree
was failing due to missing TaskScheduler used by RenderWidgetHostImpl
constructor. This CL adds a ScopedTaskScheduler to the test fixture
and re-enables the disabled test.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Add ScopedTaskScheduler to RenderWidgetHostViewChildFrameTest

RenderWidgetHostViewChildFrameTest.VerifyRequestContextTypeForFrameTree
was failing due to missing TaskScheduler used by RenderWidgetHostImpl
constructor. This CL adds a ScopedTaskScheduler to the test fixture
and re-enables the disabled test.

BUG=696919
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[tzik, 2017-02-28 14:01:20]

tzik@chromium.org changed reviewers:
+ alexmos@chromium.org

[tzik, 2017-02-28 14:01:29]

The CQ bit was checked by tzik@chromium.org to run a CQ dry run

[tzik, 2017-02-28 14:01:40]

PTAL

[commit-bot: I haz the power, 2017-02-28 14:01:50]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-28 14:13:56]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-28 14:13:57]

Dry run: Try jobs failed on following builders:
  linux_site_isolation on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_site_isol...)

[tzik, 2017-02-28 16:53:55]

The CQ bit was checked by tzik@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-28 16:54:21]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-28 17:07:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-28 17:07:35]

Dry run: Try jobs failed on following builders:
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Charlie Reis, 2017-02-28 17:38:21]

Thanks for getting this started!  It looks like it's still failing on the try
bots, though?

[alexmos, 2017-02-28 18:00:37]

Thanks for fixing this!  This seems to be the same as issue 695375?

https://codereview.chromium.org/2720163003/diff/1/content/browser/frame_host/...
File content/browser/frame_host/render_widget_host_view_child_frame_unittest.cc
(right):

https://codereview.chromium.org/2720163003/diff/1/content/browser/frame_host/...
content/browser/frame_host/render_widget_host_view_child_frame_unittest.cc:74:
RenderWidgetHostViewChildFrameTest() {}
Not sure if that'll fix the trybot failures, but do you need to initialize
task_scheduler_ with the message_loop_, like you did in
https://codereview.chromium.org/2710053003/ ?