getUserMeida: report device starting states

content/browser/media/capture/desktop_capture_device_aura_unittest.cc

Index: content/browser/media/capture/desktop_capture_device_aura_unittest.cc
diff --git a/content/browser/media/capture/desktop_capture_device_aura_unittest.cc b/content/browser/media/capture/desktop_capture_device_aura_unittest.cc
index 8f183081fd5583e1d8267d513767cbfcc1125569..25450396d9ae9f741250548daa86e1d4c9b2afd3 100644
--- a/content/browser/media/capture/desktop_capture_device_aura_unittest.cc
+++ b/content/browser/media/capture/desktop_capture_device_aura_unittest.cc
@@ -56,6 +56,7 @@ class MockDeviceClient : public media::VideoCaptureDevice::Client {
   MOCK_METHOD2(OnError,
                void(const tracked_objects::Location& from_here,
                     const std::string& reason));
+  MOCK_METHOD0(OnStarted, void(void));

[miu, 2017/02/09 22:21:49]

Where does DesktopCaptureDeviceAura call OnStarted()?

[miu, 2017/02/09 22:21:49]

One or more unit tests should probably EXPECT_CALL(OnStarted), right?

[braveyao, 2017/02/14 01:05:48]

It's called in ScreenCaptureDeviceCore::CaptureStarted,
https://codereview.chromium.org/2673373003/diff/20001/media/capture/content/s....

In the past we only forward the ERROR report. Now Success report will be
forwarded to Client too.

[braveyao, 2017/02/14 01:05:48]

Done.

Thanks for the suggestion!

[braveyao, 2017/02/15 21:47:15]

Oh, no. This test case is to stop immediately after start. So OnStarted won't be
guaranteed to be called. At it seems that the screen capture doesn't really
started in this unittest.
I tried not to call Stop and there is still no OnStart event.
I tried the method in desktop_capture_device_unittest.cc to wait for the event
and it will time out.
No ChromeOS at my hand to investigate more right now. So add the
EXCEPT_CAll(OnStarted) as a placeholder for now.

 
   // Trampoline methods to workaround GMOCK problems with std::unique_ptr<>.
   Buffer ReserveOutputBuffer(const gfx::Size& dimensions,