Fix for closing the desktop sharing notification bar when the shared window is closed

content/browser/renderer_host/media/media_stream_manager.h

Index: content/browser/renderer_host/media/media_stream_manager.h
diff --git a/content/browser/renderer_host/media/media_stream_manager.h b/content/browser/renderer_host/media/media_stream_manager.h
index 17bc6d231d1cd60e1aa56ca0d854975e42cf5eb8..8b718620e1a45700d9c860865f6ee708707ae206 100644
--- a/content/browser/renderer_host/media/media_stream_manager.h
+++ b/content/browser/renderer_host/media/media_stream_manager.h
@@ -156,6 +156,8 @@ class CONTENT_EXPORT MediaStreamManager
                       int capture_session_id) OVERRIDE;
   virtual void DevicesEnumerated(MediaStreamType stream_type,
                                  const StreamDeviceInfoArray& devices) OVERRIDE;
+  virtual void Aborted(MediaStreamType stream_type, int capture_session_id)
+      OVERRIDE;

[perkj_chrome, 2014/04/24 10:00:58]

nit: Don't break line just before OVERRIDE.

[mcasas, 2014/04/24 11:09:47]

I feel undecided if Aborted() is a good name for the window being closed. When
the window is closed, is this a normal circumstance? Then Aborted() sounds
harsh, but is not an Error(), right? What about stream is Finished(),
Stopped()...?

[jiayl, 2014/04/24 18:11:13]

Closing the window is not the expected way to stop the capturing. The expected
ways are (1) click the "stop sharing" button in the notification UI (2) call
stream.stop() from JS.
Plus, closing the window is only one scenario that triggers the OnError callback
of VideoCaptureController::Client. For example, it can be triggered when
capturing certain application window on Mac. 
So I think Aborted is better. It also clearly indicates that the capturing has
been aborted, making it more explicit than "Error" (e.g. the impl may choose to
ignore the error).

On 2014/04/24 11:09:47, mcasas wrote:

 
   // Implements base::SystemMonitor::DevicesChangedObserver.
   virtual void OnDevicesChanged(