Remove the requirement of D3D_FEATURE_LEVEL_11_0, but export the D3D_FEATURE_LEVEL through APIs

webrtc/modules/desktop_capture/win/dxgi_duplicator_controller.cc

 /*
  *  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.
  */
 
@@ skipping 26 matching lines @@
 DxgiDuplicatorController::~DxgiDuplicatorController() {
   rtc::CritScope lock(&lock_);
   Deinitialize();
 }
 
 bool DxgiDuplicatorController::IsSupported() {
   rtc::CritScope lock(&lock_);
   return Initialize();
 }
 
+bool DxgiDuplicatorController::SupportedFeatureLevels(
+    D3D_FEATURE_LEVEL* min_feature_level,
+    D3D_FEATURE_LEVEL* max_feature_level) {
+  rtc::CritScope lock(&lock_);
+  if (!Initialize()) {
+    return false;
+  }
+  if (min_feature_level != nullptr) {

[Sergey Ulanov, 2016/11/07 19:37:16]

nit: remove "!=nullptr" part.

[Hzj_jie, 2016/11/08 01:25:40]

Done.

+    *min_feature_level = min_feature_level_;
+  }
+  if (max_feature_level != nullptr) {
+    *max_feature_level = max_feature_level_;
+  }
+  return true;
+}
+
 DesktopVector DxgiDuplicatorController::dpi() {
   rtc::CritScope lock(&lock_);
   if (Initialize()) {
     return dpi_;
   }
   return DesktopVector();
 }
 
 DesktopRect DxgiDuplicatorController::desktop_rect() {
   rtc::CritScope lock(&lock_);
@@ skipping 57 matching lines @@
     return true;
   }
   Deinitialize();
   return false;
 }
 
 bool DxgiDuplicatorController::DoInitialize() {
   RTC_DCHECK(desktop_rect_.is_empty());
   RTC_DCHECK(duplicators_.empty());
 
+  // The maximum and minimum D3D_FEATURE_LEVEL supported by D3D11CreateDevice by
+  // default.
+  min_feature_level_ = D3D_FEATURE_LEVEL_11_0;
+  max_feature_level_ = D3D_FEATURE_LEVEL_9_1;

[Sergey Ulanov, 2016/11/07 19:37:16]

Maybe set these to -1 for initialization. Otherwise you have min > max, which
just doesn't look right.
Also this code won't work correctly if any of the devices has feature level
below 9.1 or above 11.

[Hzj_jie, 2016/11/08 01:25:39]

In D3dDevice::EnumDevices(), since we do not provide a D3D_FEATURE_LEVEL*, the
default feature levels is 9.1 to 11.0 (https://goo.gl/lcMhss). So here 11.0 and
9.1 is the largest and smallest possibilities.

But I do not think -1 is a safe value, considering Microsoft uses top 20 bits
for major version, middle 4 bits for minor version, low 8 bits as preserved
(https://goo.gl/tU4DoL). -1 means version 1048575.255.*(?). It looks like 0 is
not really possible, since there should be no 0.0.0(?).

+
   std::vector<D3dDevice> devices = D3dDevice::EnumDevices();
   if (devices.empty()) {
     return false;
   }
 
   for (size_t i = 0; i < devices.size(); i++) {
     duplicators_.emplace_back(devices[i]);
     if (!duplicators_.back().Initialize()) {
       return false;
     }
+
+    D3D_FEATURE_LEVEL feature_level =
+        devices[i].d3d_device()->GetFeatureLevel();
+    if (feature_level > max_feature_level_) {
+      max_feature_level_ = feature_level;
+    }
+    if (feature_level < min_feature_level_) {
+      min_feature_level_ = feature_level;
+    }
+
     if (desktop_rect_.is_empty()) {
       desktop_rect_ = duplicators_.back().desktop_rect();
     } else {
       const DesktopRect& left = desktop_rect_;
       const DesktopRect& right = duplicators_.back().desktop_rect();
       desktop_rect_ =
           DesktopRect::MakeLTRB(std::min(left.left(), right.left()),
                                 std::min(left.top(), right.top()),
                                 std::max(left.right(), right.right()),
                                 std::max(left.bottom(), right.bottom()));
@@ skipping 83 matching lines @@
       Deinitialize();
       return false;
     }
   }
   // id >= ScreenCount(). This is a user error, so we do not need to
   // deinitialize.
   return false;
 }
 
 }  // namespace webrtc