Add reference counter of DxgiDuplicatorController to unload DXGI components

webrtc/modules/desktop_capture/win/dxgi_duplicator_controller.h

 /*
  *  Copyright (c) 2016 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_WIN_DXGI_DUPLICATOR_CONTROLLER_H_
 #define WEBRTC_MODULES_DESKTOP_CAPTURE_WIN_DXGI_DUPLICATOR_CONTROLLER_H_
 
 #include <D3DCommon.h>
 
-#include <memory>
+#include <atomic>
 #include <vector>
 
 #include "webrtc/base/criticalsection.h"
+#include "webrtc/base/scoped_ref_ptr.h"
 #include "webrtc/modules/desktop_capture/desktop_geometry.h"
 #include "webrtc/modules/desktop_capture/resolution_change_detector.h"
 #include "webrtc/modules/desktop_capture/shared_desktop_frame.h"
 #include "webrtc/modules/desktop_capture/win/d3d_device.h"
 #include "webrtc/modules/desktop_capture/win/dxgi_adapter_duplicator.h"
 #include "webrtc/modules/desktop_capture/win/dxgi_context.h"
 #include "webrtc/modules/desktop_capture/win/dxgi_frame.h"
 
 namespace webrtc {
 
 // A controller for all the objects we need to call Windows DirectX capture APIs
 // It's a singleton because only one IDXGIOutputDuplication instance per monitor
 // is allowed per application.
 //
 // Consumers should create a DxgiDuplicatorController::Context and keep it
 // throughout their lifetime, and pass it when calling Duplicate(). Consumers
 // can also call IsSupported() to determine whether the system supports DXGI
 // duplicator or not. If a previous IsSupported() function call returns true,
 // but a later Duplicate() returns false, this usually means the display mode is
 // changing. Consumers should retry after a while. (Typically 50 milliseconds,
 // but according to hardware performance, this time may vary.)
 class DxgiDuplicatorController {
+ private:
+  struct ReferenceReleaser final {

[Sergey Ulanov, 2017/06/19 18:52:43]

I don't think you need this anymore

[Hzj_jie, 2017/06/19 20:19:46]

Done.

+    void operator()(DxgiDuplicatorController* instance) const;
+  };
+
  public:
   using Context = DxgiFrameContext;
+  using Reference = rtc::scoped_refptr<DxgiDuplicatorController>;

[Sergey Ulanov, 2017/06/19 18:52:43]

I'd prefer to use scoped_refptr<> directly. It won't be used in too many places,
so adding this shortcut doesn't make code more readable.

[Hzj_jie, 2017/06/19 20:19:47]

Done.

 
   // A collection of D3d information we are interested on, which may impact
   // capturer performance or reliability.
   struct D3dInfo {
     // Each video adapter has its own D3D_FEATURE_LEVEL, so this structure
     // contains the minimum and maximium D3D_FEATURE_LEVELs current system
     // supports.
     // Both fields can be 0, which is the default value to indicate no valid
     // D3D_FEATURE_LEVEL has been retrieved from underlying OS APIs.
     D3D_FEATURE_LEVEL min_feature_level;
     D3D_FEATURE_LEVEL max_feature_level;
 
     // TODO(zijiehe): Add more fields, such as manufacturer name, mode, driver
     // version.
   };
 
   enum class Result {
     SUCCEEDED,
     FRAME_PREPARE_FAILED,
     INITIALIZATION_FAILED,
     DUPLICATION_FAILED,
     INVALID_MONITOR_ID,
   };
 
+  // Deprecated: use "GetRefence()" instead.

[Sergey Ulanov, 2017/06/19 18:52:43]

Maybe just update this function to return scoped_refptr<>?
There is only a handful places where it's called, so all callers can be updated
in the same CL.

[Hzj_jie, 2017/06/19 20:19:46]

Done. The only concern is the name "Instance()" does not reflect that this
function returns a scoped_refptr instead of an instance.

[Do not use (sergeyu), 2017/06/19 20:54:44]

I think Instance() is still fine. We still have only once instance, but it's
ref-counted now.

   // Returns the singleton instance of DxgiDuplicatorController.
   static DxgiDuplicatorController* Instance();
 
-  // Destructs current instance. We need to make sure COM components and their
-  // containers are destructed in correct order. This function calls
-  // Deinitialize() to do the real work.
-  ~DxgiDuplicatorController();
-
+  static Reference GetReference();
   // All the following public functions implicitly call Initialize() function.
 
   // Detects whether the system supports DXGI based capturer.
   bool IsSupported();
 
   // Returns a copy of D3dInfo composed by last Initialize() function call.
   bool RetrieveD3dInfo(D3dInfo* info);
 
   // Captures current screen and writes into |frame|.
   // TODO(zijiehe): Windows cannot guarantee the frames returned by each
@@ skipping 14 matching lines @@
   // Returns the count of screens on the system. These screens can be retrieved
   // by an integer in the range of [0, ScreenCount()). If system does not
   // support DXGI based capturer, this function returns 0.
   int ScreenCount();
 
  private:
   // DxgiFrameContext calls private Unregister(Context*) function during
   // destructing.
   friend DxgiFrameContext::~DxgiFrameContext();
 
+  // scoped_refptr<DxgiDuplicatorController> accesses private AddRef() and
+  // Release() functions.
+  friend class rtc::scoped_refptr<DxgiDuplicatorController>;
+
   // A private constructor to ensure consumers to use
-  // DxgiDuplicatorController::Instance().
+  // DxgiDuplicatorController::GetReference().
   DxgiDuplicatorController();
 
+  // Not implemented: The singleton DxgiDuplicatorController instance should not
+  // be deleted.
+  ~DxgiDuplicatorController();
+
+  // RefCountedInterface implementations.
+  void AddRef() const;
+  void Release();
+
   // Does the real duplication work. Setting |monitor_id| < 0 to capture entire
   // screen. This function calls Initialize(). And if the duplication failed,
   // this function calls Deinitialize() to ensure the Dxgi components can be
   // reinitialized next time.
   Result DoDuplicate(DxgiFrame* frame, int monitor_id);
 
+  // Unload all the DXGI components and releases the resources. This function
+  // wraps Deinitialize() with |lock_|.
+  void Unload();
+
   // Unregisters Context from this instance and all DxgiAdapterDuplicator(s)
   // it owns.
   void Unregister(const Context* const context);
 
   // All functions below should be called in |lock_| locked scope and should be
   // after a successful Initialize().
 
   // If current instance has not been initialized, executes DoInitialize()
   // function, and returns initialize result. Otherwise directly returns true.
   // This function may calls Deinitialize() if initialization failed.
@@ skipping 48 matching lines @@
   // According to http://crbug.com/682112, dxgi capturer returns a black frame
   // during first several capture attempts.
   bool EnsureFrameCaptured(Context* context, SharedDesktopFrame* target);
 
   // Moves |desktop_rect_| and all underlying |duplicators_|, putting top left
   // corner of the desktop at (0, 0). This is necessary because DXGI_OUTPUT_DESC
   // may return negative coordinates. Called from DoInitialize() after all
   // DxgiAdapterDuplicator and DxgiOutputDuplicator instances are initialized.
   void TranslateRect();
 
+  // The count of references which are now "living".
+  static std::atomic_int refcount_;
+
   // This lock must be locked whenever accessing any of the following objects.
   rtc::CriticalSection lock_;
 
   // A self-incremented integer to compare with the one in Context. It ensures
   // a Context instance is always initialized after DxgiDuplicatorController.
   int identity_ = 0;
   DesktopRect desktop_rect_;
   DesktopVector dpi_;
   std::vector<DxgiAdapterDuplicator> duplicators_;
   D3dInfo d3d_info_;
   ResolutionChangeDetector resolution_change_detector_;
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_MODULES_DESKTOP_CAPTURE_WIN_DXGI_DUPLICATOR_CONTROLLER_H_