DirectX based screen capturer logic

webrtc/modules/desktop_capture/desktop_frame_win.h

 /*
  *  Copyright (c) 2013 The WebRTC project authors. All Rights Reserved.
  *
  *  Use of this source code is governed by a BSD-style license
  *  that can be found in the LICENSE file in the root of the source
  *  tree. An additional intellectual property rights grant can be found
  *  in the file PATENTS.  All contributing project authors may
  *  be found in the AUTHORS file in the root of the source tree.
  */
 
 #ifndef WEBRTC_MODULES_DESKTOP_CAPTURE_DESKTOP_FRAME_WIN_H_
 #define WEBRTC_MODULES_DESKTOP_CAPTURE_DESKTOP_FRAME_WIN_H_
 
 #include <memory>
 
+#include <DXGI.h>
 #include <windows.h>
+#include <wrl/client.h>
 
 #include "webrtc/modules/desktop_capture/desktop_frame.h"
 #include "webrtc/typedefs.h"
 
 namespace webrtc {
 
 // DesktopFrame implementation used by screen and window captures on Windows.
 // Frame data is stored in a GDI bitmap.
 class DesktopFrameWin : public DesktopFrame {
  public:
@@ skipping 10 matching lines @@
                   uint8_t* data,
                   std::unique_ptr<SharedMemory> shared_memory,
                   HBITMAP bitmap);
 
   HBITMAP bitmap_;
   std::unique_ptr<SharedMemory> owned_shared_memory_;
 
   RTC_DISALLOW_COPY_AND_ASSIGN(DesktopFrameWin);
 };
 
+// DesktopFrame implementation used by screen captures on Windows.
+// Frame data is stored in a IDXGISurface instance.
+class DesktopFrameWinDXGI : public DesktopFrame {

[Sergey Ulanov, 2016/03/31 18:41:15]

Put this to a separate file (in win directory)? It doesn't need to be in the
same file with DesktopFrameWin.

[Sergey Ulanov, 2016/03/31 18:41:15]

Call this DesktopFrameWinDxgi. Style guide is not prescriptive about
abbreviations in names like this, but it gives UrlTable and not URLTable as an
example of a good class name.

[Hzj_jie, 2016/04/05 23:15:17]

Done.

[Hzj_jie, 2016/04/05 23:15:17]

Done.

+ public:
+  virtual ~DesktopFrameWinDXGI();
+  DesktopFrameWinDXGI(DesktopSize size,
+                      Microsoft::WRL::ComPtr<IDXGISurface> surface,
+                      DXGI_MAPPED_RECT rect);
+
+  // Create a valid but empty DesktopFrame, no updated regions.
+  DesktopFrameWinDXGI(DesktopSize size);

[Sergey Ulanov, 2016/03/31 18:41:15]

Do you need this constructor? You could also use BasicDesktopFrame when you need
to create an empty DesktopFrame, but that won't be the right thing to do. See my
comments in screen_capturer_win_directx.cc

[Hzj_jie, 2016/04/05 23:15:17]

Done.

+
+ private:
+  // Make sure IDXGISurface won't be recycled during the lifetime of this
+  // instance.
+  Microsoft::WRL::ComPtr<IDXGISurface> surface_;
+};

[Sergey Ulanov, 2016/03/31 18:41:15]

add RTC_DISALLOW_COPY_AND_ASSIGN()

[Hzj_jie, 2016/04/05 23:15:17]

Done.

+
 }  // namespace webrtc
 
 #endif  // WEBRTC_MODULES_DESKTOP_CAPTURE_DESKTOP_FRAME_WIN_H_