Backed out changeset 03a3ed5fbce1 (bug 1331915) for telemetry bustage. r=backout
authorSebastian Hengst <archaeopteryx@coole-files.de>
Thu, 27 Apr 2017 01:05:03 +0200
changeset 403353 b820712f307783b262fe59fdf5687386bc3d3a72
parent 403352 052a1bf45675e2f273b5f85afc755338ac61021b
child 403354 443cbeb485e5c48851818122c6644d8fb2920b20
push id7391
push usermtabara@mozilla.com
push dateMon, 12 Jun 2017 13:08:53 +0000
treeherdermozilla-beta@2191d7f87e2e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbackout
bugs1331915
milestone55.0a1
backs out03a3ed5fbce14c38e0b2bd452b6aed182dfcd957
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Backed out changeset 03a3ed5fbce1 (bug 1331915) for telemetry bustage. r=backout
gfx/thebes/gfxFont.cpp
toolkit/components/telemetry/Scalars.yaml
--- a/gfx/thebes/gfxFont.cpp
+++ b/gfx/thebes/gfxFont.cpp
@@ -2707,17 +2707,16 @@ gfxFont::ShapeText(DrawTarget      *aDra
     bool ok = false;
 
     // XXX Currently, we do all vertical shaping through harfbuzz.
     // Vertical graphite support may be wanted as a future enhancement.
     if (FontCanSupportGraphite() && !aVertical) {
         if (gfxPlatform::GetPlatform()->UseGraphiteShaping()) {
             if (!mGraphiteShaper) {
                 mGraphiteShaper = MakeUnique<gfxGraphiteShaper>(this);
-                Telemetry::ScalarAdd(Telemetry::ScalarID::BROWSER_USAGE_GRAPHITE, 1);
             }
             ok = mGraphiteShaper->ShapeText(aDrawTarget, aText, aOffset, aLength,
                                             aScript, aVertical, aRounding,
                                             aShapedText);
         }
     }
 
     if (!ok) {
--- a/toolkit/components/telemetry/Scalars.yaml
+++ b/toolkit/components/telemetry/Scalars.yaml
@@ -196,31 +196,16 @@ browser.engagement.navigation:
     kind: uint
     keyed: true
     notification_emails:
       - bcolloran@mozilla.com
     release_channel_collection: opt-out
     record_in_processes:
       - 'main'
 
-# The following section contains the browser usage scalars.
-browser.usage:
-  graphite:
-    bug_numbers:
-      - 1331915
-    description: >
-      The number of times a graphite2 font has been loaded.
-    expires: "60"
-    kind: uint
-    notification_emails:
-      - tom@mozilla.com
-    release_channel_collection: opt-in
-    record_in_processes:
-      -all
-
 # This section is for probes used to measure use of the Webextensions storage.sync API.
 storage.sync.api.usage:
   extensions_using:
     bug_numbers:
       - 1328974
     description: >
       The count of webextensions that have data stored in the chrome.storage.sync API.
       This includes extensions that have not used the storage.sync API this session.