From 6cd54550a27e2f6ca0755a25328f769e41e524f4 Mon Sep 17 00:00:00 2001
From: Jeff Muizelaar <jmuizelaar@mozilla.com>
Date: Thu, 19 Mar 2020 02:37:21 +0000
Subject: [PATCH] Bug 1623492. Expose OS compositor to telemetry.

Differential Revision: https://phabricator.services.mozilla.com/D67415

--HG--
extra : moz-landing-system : lando
---
 gfx/thebes/gfxPlatform.cpp                |  4 ++++
 toolkit/components/telemetry/Scalars.yaml | 17 +++++++++++++++++
 2 files changed, 21 insertions(+)

diff --git a/gfx/thebes/gfxPlatform.cpp b/gfx/thebes/gfxPlatform.cpp
index d038b3d4ec4b..a0ca80ab6c92 100644
--- a/gfx/thebes/gfxPlatform.cpp
+++ b/gfx/thebes/gfxPlatform.cpp
@@ -3064,6 +3064,10 @@ void gfxPlatform::InitWebRenderConfig() {
     featureComp.UserEnable("Enabled");
   }
 
+  Telemetry::ScalarSet(
+      Telemetry::ScalarID::GFX_OS_COMPOSITOR,
+      gfx::gfxConfig::IsEnabled(gfx::Feature::WEBRENDER_COMPOSITOR));
+
   // Initialize WebRender partial present config.
   // Partial present is used only when WebRender compositor is not used.
   if (StaticPrefs::gfx_webrender_max_partial_present_rects_AtStartup() > 0) {
diff --git a/toolkit/components/telemetry/Scalars.yaml b/toolkit/components/telemetry/Scalars.yaml
index 615357804e30..424f504d971e 100644
--- a/toolkit/components/telemetry/Scalars.yaml
+++ b/toolkit/components/telemetry/Scalars.yaml
@@ -3066,6 +3066,23 @@ gfx:
     record_in_processes:
       - 'main'
     release_channel_collection: opt-out
+  os_compositor:
+    bug_numbers:
+      - 1623492
+    description: >
+      Boolean indicating whether the os compositor is being used by WebRender.
+      Only collected in the first subsession.
+    expires: never
+    kind: boolean
+    notification_emails:
+      - gfx-telemetry-alerts@mozilla.com
+      - jmuizelaar@mozilla.com
+    products:
+      - 'firefox'
+      - 'geckoview'
+    record_in_processes:
+      - 'main'
+    release_channel_collection: opt-out
 
 # The following section contains the form autofill related scalars.
 formautofill: