Icon Capture Utility For Linux, Windows and Mac

chrome/browser/media/window_icon_util.h

Index: chrome/browser/media/window_icon_util.h
diff --git a/chrome/browser/media/window_icon_util.h b/chrome/browser/media/window_icon_util.h
new file mode 100644
index 0000000000000000000000000000000000000000..a5ee965fea9d6e4295aa95162583d1f13e32ae00
--- /dev/null
+++ b/chrome/browser/media/window_icon_util.h
@@ -0,0 +1,20 @@
+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_MEDIA_WINDOW_ICON_UTIL_H_
+#define CHROME_BROWSER_MEDIA_WINDOW_ICON_UTIL_H_
+
+#include "content/public/browser/desktop_media_id.h"
+#include "third_party/webrtc/modules/desktop_capture/desktop_capture_options.h"
+#include "ui/gfx/image/image_skia.h"
+
+class WindowIconUtil {
+ public:
+  virtual gfx::ImageSkia GetWindowIcon(content::DesktopMediaID id) = 0;
+
+  static std::unique_ptr<WindowIconUtil> Create(
+      const webrtc::DesktopCaptureOptions& options);

[Sergey Ulanov, 2016/08/15 22:28:39]

I don't think there is any reason to depend on webrtc::DesktopCaptureOptions
here. You are currently use it to pass X11 connection to the X11 version of the
capturer. It would be cleaner to just reuse the X11 connection that chrome
already has, i.e. gfx::GetXDisplay(). Then you wouldn't even need this as an
interface, and instead you would have a simple function that returns a window
icon..

[qiangchen, 2016/08/16 22:29:52]

Done.

+};
+
+#endif  // CHROME_BROWSER_MEDIA_WINDOW_ICON_UTIL_H_