Bug 1438872 - Document and test TelemetryEnvironment's e10sMultiProcesses. r=chutten
authorFelipe Gomes <felipc@gmail.com>
Tue, 16 Apr 2019 15:32:28 +0000
changeset 469687 3d5e46c5fd7c
parent 469686 d245ef38503c
child 469688 f93f4060ac7d
push id35879
push usernerli@mozilla.com
push dateTue, 16 Apr 2019 22:01:48 +0000
treeherdermozilla-central@12a60898fdc1 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewerschutten
bugs1438872
milestone68.0a1
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
Bug 1438872 - Document and test TelemetryEnvironment's e10sMultiProcesses. r=chutten Differential Revision: https://phabricator.services.mozilla.com/D27584
toolkit/components/telemetry/docs/data/environment.rst
toolkit/components/telemetry/tests/unit/test_TelemetryEnvironment.js
--- a/toolkit/components/telemetry/docs/data/environment.rst
+++ b/toolkit/components/telemetry/docs/data/environment.rst
@@ -42,16 +42,17 @@ Structure:
           name: <string>, // engine name, e.g. "Yahoo"; or "NONE" if no default
           loadPath: <string>, // where the engine line is located; missing if no default
           origin: <string>, // 'default', 'verified', 'unverified', or 'invalid'; based on the presence and validity of the engine's loadPath verification hash.
           submissionURL: <string> // set for default engines or well known search domains
         },
         searchCohort: <string>, // optional, contains an identifier for any active search A/B experiments
         launcherProcessState: <integer>, // optional, values correspond to values of mozilla::LauncherRegistryInfo::EnabledState enum
         e10sEnabled: <bool>, // whether e10s is on, i.e. browser tabs open by default in a different process
+        e10sMultiProcesses: <integer>, // Maximum number of processes that will be launched for regular web content
         telemetryEnabled: <bool>, // false on failure
         locale: <string>, // e.g. "it", null on failure
         intl: {
           requestedLocales: [ <string>, ... ], // The locales that are being requested.
           availableLocales: [ <string>, ... ], // The locales that are available for use.
           appLocales: [ <string>, ... ], // The negotiated locales that are being used.
           systemLocales: [ <string>, ... ], // The locales for the OS.
           regionalPrefsLocales: [ <string>, ... ], // The regional preferences for the OS.
--- a/toolkit/components/telemetry/tests/unit/test_TelemetryEnvironment.js
+++ b/toolkit/components/telemetry/tests/unit/test_TelemetryEnvironment.js
@@ -382,16 +382,17 @@ function checkBuildSection(data) {
   Assert.equal(data.build.updaterAvailable, AppConstants.MOZ_UPDATER,
                "build.updaterAvailable must equal AppConstants.MOZ_UPDATER");
 }
 
 function checkSettingsSection(data) {
   const EXPECTED_FIELDS_TYPES = {
     blocklistEnabled: "boolean",
     e10sEnabled: "boolean",
+    e10sMultiProcesses: "number",
     intl: "object",
     locale: "string",
     telemetryEnabled: "boolean",
     update: "object",
     userPrefs: "object",
   };
 
   Assert.ok("settings" in data, "There must be a settings section in Environment.");