Detect HDR capability

gpu/ipc/service/direct_composition_surface_win.cc

 // Copyright 2017 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.
 
 #include "gpu/ipc/service/direct_composition_surface_win.h"
 
 #include <d3d11_1.h>
 #include <dcomptypes.h>
 
 #include <deque>
@@ skipping 17 matching lines @@
 #include "ui/gfx/transform.h"
 #include "ui/gl/dc_renderer_layer_params.h"
 #include "ui/gl/egl_util.h"
 #include "ui/gl/gl_angle_util_win.h"
 #include "ui/gl/gl_context.h"
 #include "ui/gl/gl_image_dxgi.h"
 #include "ui/gl/gl_image_memory.h"
 #include "ui/gl/gl_surface_egl.h"
 #include "ui/gl/scoped_make_current.h"
 
+#if defined(NTDDI_WIN10_RS2)
+#define ENABLE_HDR_DETECTION
+#include <dxgi1_6.h>
+#endif
+
 #ifndef EGL_ANGLE_flexible_surface_compatibility
 #define EGL_ANGLE_flexible_surface_compatibility 1
 #define EGL_FLEXIBLE_SURFACE_COMPATIBILITY_SUPPORTED_ANGLE 0x33A6
 #endif /* EGL_ANGLE_flexible_surface_compatibility */
 
 namespace gpu {
 namespace {
 
 // Some drivers fail to correctly handle BT.709 video in overlays. This flag
 // converts them to BT.601 in the video processor.
@@ skipping 937 matching lines @@
 }
 
 // static
 bool DirectCompositionSurfaceWin::AreOverlaysSupported() {
   if (!HardwareSupportsOverlays())
     return false;
 
   return base::FeatureList::IsEnabled(switches::kDirectCompositionOverlays);
 }
 
+// static
+bool DirectCompositionSurfaceWin::IsHDRSupported() {
+#if defined(ENABLE_HDR_DETECTION)
+  base::win::ScopedComPtr<ID3D11Device> d3d11_device =
+      gl::QueryD3D11DeviceObjectFromANGLE();
+  if (!d3d11_device) {
+    DLOG(ERROR) << "Failing to detect HDR, couldn't retrieve D3D11 "
+                << "device from ANGLE.";
+    return false;
+  }
+  base::win::ScopedComPtr<IDXGIDevice> dxgi_device;
+  d3d11_device.CopyTo(dxgi_device.GetAddressOf());
+  base::win::ScopedComPtr<IDXGIAdapter> dxgi_adapter;
+  dxgi_device->GetAdapter(dxgi_adapter.GetAddressOf());
+
+  unsigned int i = 0;

[palmer, 2017/06/26 19:16:41]

The style guide says not to use unsigned as a way to ensure integers aren't
negative. If this is a regular counter, I guess it should be a signed int? If
it's a size or array index, it should be a size_t.

But, if |EnumOutputs| wants an unsigned int, it's OK for now. I know that
changing those interfaces can sometimes turn into a huge CL. :)

[hubbe, 2017/06/26 19:20:42]

EnumOutputs is a microsoft API, so changing it seems unlikely.

+  while (true) {
+    base::win::ScopedComPtr<IDXGIOutput> output;
+    if (FAILED(dxgi_adapter->EnumOutputs(i++, output.GetAddressOf())))
+      break;
+    base::win::ScopedComPtr<IDXGIOutput6> output6;
+    if (FAILED(output.CopyTo(output6.GetAddressOf())))
+      continue;
+
+    DXGI_OUTPUT_DESC1 desc;
+    if (FAILED(output6->GetDesc1(&desc)))
+      continue;
+
+    UMA_HISTOGRAM_SPARSE_SLOWLY("GPU.Output.ColorSpace", desc.ColorSpace);
+    UMA_HISTOGRAM_SPARSE_SLOWLY("GPU.Output.MaxLuminance", desc.MaxLuminance);
+
+    if (desc.ColorSpace == DXGI_COLOR_SPACE_RGB_FULL_G2084_NONE_P2020) {
+      UMA_HISTOGRAM_BOOLEAN("GPU.Output.HDR", true);
+      return true;
+    }
+  }
+  UMA_HISTOGRAM_BOOLEAN("GPU.Output.HDR", false);
+#endif
+  return false;
+}
+
 bool DirectCompositionSurfaceWin::InitializeNativeWindow() {
   if (window_)
     return true;
 
   bool result = child_window_.Initialize();
   window_ = child_window_.window();
   return result;
 }
 
 bool DirectCompositionSurfaceWin::Initialize(gl::GLSurfaceFormat format) {
@@ skipping 164 matching lines @@
 DirectCompositionSurfaceWin::GetWindowTaskRunnerForTesting() {
   return child_window_.GetTaskRunnerForTesting();
 }
 
 base::win::ScopedComPtr<IDXGISwapChain1>
 DirectCompositionSurfaceWin::GetLayerSwapChainForTesting(size_t index) const {
   return layer_tree_->GetLayerSwapChainForTesting(index);
 }
 
 }  // namespace gpu