Bug 1305280 - ContextualIdentityService.jsm must use UNIQUE_CONTAINERS_OPENED, r=me
authorAndrea Marchesini <amarchesini@mozilla.com>
Sun, 25 Sep 2016 08:25:54 +0200
changeset 315144 2e55d746c223d468d9d0b0e52721dfeda0aee7bc
parent 315143 04cda7684b40ba85af735d4b37bf8d45bb88888f
child 315145 aef020a0ba34408c4d416c554ea3797aa167c5db
push id82096
push useramarchesini@mozilla.com
push dateSun, 25 Sep 2016 06:26:20 +0000
treeherdermozilla-inbound@2e55d746c223 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersme
bugs1305280
milestone52.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 1305280 - ContextualIdentityService.jsm must use UNIQUE_CONTAINERS_OPENED, r=me
toolkit/components/contextualidentity/ContextualIdentityService.jsm
--- a/toolkit/components/contextualidentity/ContextualIdentityService.jsm
+++ b/toolkit/components/contextualidentity/ContextualIdentityService.jsm
@@ -319,17 +319,17 @@ function _ContextualIdentityService(path
   telemetry(userContextId) {
     let identity = this.getIdentityFromId(userContextId);
 
     // Let's ignore unknown identities for now.
     if (!identity || !identity.public) {
       return;
     }
 
-    if (this._openedIdentities.has(userContextId)) {
+    if (!this._openedIdentities.has(userContextId)) {
       this._openedIdentities.add(userContextId);
       Services.telemetry.getHistogramById("UNIQUE_CONTAINERS_OPENED").add(1);
     }
 
     Services.telemetry.getHistogramById("TOTAL_CONTAINERS_OPENED").add(1);
 
     if (identity.telemetryId) {
       Services.telemetry.getHistogramById("CONTAINER_USED")