Desktop chrome tab capture-chooseDesktopMedia()

chrome/browser/media/desktop_media_list.h

Index: chrome/browser/media/desktop_media_list.h
diff --git a/chrome/browser/media/desktop_media_list.h b/chrome/browser/media/desktop_media_list.h
index 86ac37c02f869e0c1f2813b84ac5bbbbac0c38f6..d310f502e6cad806fdeb9249ad2963c8a40789e0 100644
--- a/chrome/browser/media/desktop_media_list.h
+++ b/chrome/browser/media/desktop_media_list.h
@@ -5,9 +5,10 @@
 #ifndef CHROME_BROWSER_MEDIA_DESKTOP_MEDIA_LIST_H_
 #define CHROME_BROWSER_MEDIA_DESKTOP_MEDIA_LIST_H_
 
-#include "base/strings/string16.h"
 #include "base/time/time.h"
 #include "content/public/browser/desktop_media_id.h"
+#include "content/public/browser/web_contents_observer.h"
+#include "ui/gfx/image/image.h"
 #include "ui/gfx/image/image_skia.h"
 
 class DesktopMediaListObserver;
@@ -57,4 +58,19 @@ class DesktopMediaList {
   virtual const Source& GetSource(int index) const = 0;
 };
 
+// Create a image with an enlarged favicon on its center and black boundary.
+// If fails, an empty image will return.
+// The function is used to create a thumbnail for chrome tab preview.
+gfx::ImageSkia CreateEnlargedFaviconImage(gfx::Size size, gfx::Image& favicon);
+
+// Observer of WebContents to detect whether WebContents has been destroyed.
+class MediaListWebContentsObserver : public content::WebContentsObserver {

[Avi (use Gerrit), 2016/01/08 17:11:38]

This is not a useful class; it exists but overrides nothing. You use the
automatic nulling out of the pointer, but I've never seen that done before. Its
use doesn't make sense either; I'm commenting there.

You should remove it.

[GeorgeZ, 2016/01/08 19:39:02]

Done.

+ public:
+  explicit MediaListWebContentsObserver(content::WebContents* contents) {
+    Observe(contents);
+  }
+
+  DISALLOW_COPY_AND_ASSIGN(MediaListWebContentsObserver);
+};
+
 #endif  // CHROME_BROWSER_MEDIA_DESKTOP_MEDIA_LIST_H_