Close the media picker window when DesktopMediaPickerController is destructed.

chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller.mm

Index: chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller.mm
diff --git a/chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller.mm b/chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller.mm
index bffc0259bbfd7701111f88b08c9c994495bd0573..dfbb4520f7ad52d75aee3b448338d431049ca8e9 100644
--- a/chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller.mm
+++ b/chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller.mm
@@ -81,6 +81,7 @@ const int kExcessButtonPadding = 6;
 }
 
 - (void)dealloc {
+  [[self window] close];

[Scott Hess - ex-Googler, 2014/06/16 17:13:51]

Since this is being called from a destructor, I would put this _after_ all of
the places where you are disconnecting targets and delegates and stuff.

   [shareButton_ setTarget:nil];
   [cancelButton_ setTarget:nil];
   [sourceBrowser_ setDelegate:nil];