Pier Angelo Vendrame pushed to branch tor-browser-115.5.0esr-13.5-1 at The Tor Project / Applications / Tor Browser

Commits:

• 0c6e4916
  by Pier Angelo Vendrame at 2023-12-13T12:37:06+01:00

  Bug 1860020 - Remove the assertion on the value of toolkit.telemetry.enabled. r=KrisWright,chutten
  
  Bug 1444275 introduced an assertion on the parent process to check that
  the value of toolkit.telemetry.enabled is the expected one.
  However, this expected value could be different from the one set and
  locked e.g. in some forks. Therefore, the assertion prevented debug
  builds from working in these cases.
  
  Differential Revision: https://phabricator.services.mozilla.com/D195080

1 changed file:

• modules/libpref/Preferences.cpp

Changes:

   Preferences::Lock(kTelemetryPref);

 }

-static void CheckTelemetryPref() {

-  MOZ_ASSERT(!XRE_IsParentProcess());

-

-  // Make sure the children got passed the right telemetry pref details.

-  DebugOnly<bool> value;

-  MOZ_ASSERT(NS_SUCCEEDED(Preferences::GetBool(kTelemetryPref, &value)) &&

-             value == TelemetryPrefValue());

-  MOZ_ASSERT(Preferences::IsLocked(kTelemetryPref));

-}

-

 #endif  // MOZ_WIDGET_ANDROID

 /* static */

       Preferences::SetPreference(gChangedDomPrefs->ElementAt(i));

     }

     gChangedDomPrefs = nullptr;

-

-#ifndef MOZ_WIDGET_ANDROID

-    CheckTelemetryPref();

-#endif

-

   } else {

     // Check if there is a deployment configuration file. If so, set up the

     // pref config machinery, which will actually read the file.