Fix DesktopCaptureDeviceTest to shutdown worker pool

content/browser/media/capture/desktop_capture_device_unittest.cc

Index: content/browser/media/capture/desktop_capture_device_unittest.cc
diff --git a/content/browser/media/capture/desktop_capture_device_unittest.cc b/content/browser/media/capture/desktop_capture_device_unittest.cc
index 0c1a360e2923a54142d95e17a3823a83db1789b1..9d2da0f31a0765e59feceb2d544398d3bcf3ba80 100644
--- a/content/browser/media/capture/desktop_capture_device_unittest.cc
+++ b/content/browser/media/capture/desktop_capture_device_unittest.cc
@@ -141,6 +141,11 @@ class DesktopCaptureDeviceTest : public testing::Test {
     worker_pool_ = new base::SequencedWorkerPool(3, "TestCaptureThread");
   }
 
+  virtual void TearDown() OVERRIDE {
+    worker_pool_->Shutdown();
+    message_loop_.RunUntilIdle();
+  }
+
   void CreateScreenCaptureDevice(scoped_ptr<webrtc::DesktopCapturer> capturer) {
     capture_device_.reset(new DesktopCaptureDevice(
         worker_pool_->GetSequencedTaskRunner(worker_pool_->GetSequenceToken()),
@@ -150,6 +155,8 @@ class DesktopCaptureDeviceTest : public testing::Test {
   }
 
  protected:
+  base::MessageLoop message_loop_;

[Ryan Sleevi, 2014/07/21 18:43:32]

Out of curiousity, does this use any //content BrowserThread concepts? Does it
make more sense to have a TestBrowserThreadBundle to support this (
http://src.chromium.org/viewvc/chrome/trunk/src/content/public/test/test_brow...
)

[Sergey Ulanov, 2014/07/22 00:08:22]

DesktopCaptureDevice::Create() does use BrowserThread, but
DesktopCaptureDevice::Create() is not used in the test because the test needs to
pass FakeScreenCapturer to the constructor.
But using TestBrowserThreadBundle does make this test cleaner. Done.

+
   scoped_refptr<base::SequencedWorkerPool> worker_pool_;
   scoped_ptr<base::Thread> thread_;
   scoped_ptr<DesktopCaptureDevice> capture_device_;