Verified Commit cbe86b42 authored by Alex Catarineu's avatar Alex Catarineu Committed by Pier Angelo Vendrame
Browse files

Bug 28369: Stop shipping pingsender executable

parent 3a6fb0ac

browser/app/macbuild/Contents/MacOS-files.in

+0 −1
Original line number Diff line number Diff line
@@ -17,7 +17,6 @@ 
#if defined(MOZ_CRASHREPORTER)

/minidump-analyzer

#endif

/pingsender

/pk12util

/ssltunnel

/xpcshell

browser/installer/package-manifest.in

+0 −4
Original line number Diff line number Diff line
@@ -423,10 +423,6 @@ bin/libfreebl_64int_3.so 
@BINPATH@/minidump-analyzer@BIN_SUFFIX@

#endif



; [ Ping Sender ]

;

@BINPATH@/pingsender@BIN_SUFFIX@



; Shutdown Terminator

@RESPATH@/components/terminator.manifest

browser/installer/windows/nsis/shared.nsh

+0 −1
Original line number Diff line number Diff line
@@ -1489,7 +1489,6 @@ ${RemoveDefaultBrowserAgentShortcut} 
  Push "crashreporter.exe"

  Push "default-browser-agent.exe"

  Push "minidump-analyzer.exe"

  Push "pingsender.exe"

  Push "updater.exe"

  Push "mozwer.dll"

  Push "${FileMainEXE}"

python/mozbuild/mozbuild/artifacts.py

+0 −2
Original line number Diff line number Diff line
@@ -490,7 +490,6 @@ class LinuxArtifactJob(ArtifactJob): 
        "{product}/{product}",

        "{product}/{product}-bin",

        "{product}/minidump-analyzer",

        "{product}/pingsender",

        "{product}/plugin-container",

        "{product}/updater",

        "{product}/**/*.so",
@@ -585,7 +584,6 @@ class MacArtifactJob(ArtifactJob): 
            "{product}-bin",

            "*.dylib",

            "minidump-analyzer",

            "pingsender",

            "plugin-container.app/Contents/MacOS/plugin-container",

            "updater.app/Contents/MacOS/org.mozilla.updater",

            # 'xpcshell',

toolkit/components/telemetry/app/TelemetrySend.jsm

+1 −27
Original line number Diff line number Diff line
@@ -1668,32 +1668,6 @@ var TelemetrySendImpl = { 
  },



  runPingSender(pings, observer) {

    if (AppConstants.platform === "android") {

    throw Components.Exception("", Cr.NS_ERROR_NOT_IMPLEMENTED);

    }



    let suppressPingsender = Services.prefs.getBoolPref(

      "toolkit.telemetry.testing.suppressPingsender",

      false

    );

    if (suppressPingsender) {

      this._log.trace("Silently skipping pingsender call in automation");

      return;

    }



    const exeName =

      AppConstants.platform === "win" ? "pingsender.exe" : "pingsender";



    let exe = Services.dirsvc.get("GreBinD", Ci.nsIFile);

    exe.append(exeName);



    let params = pings.flatMap(ping => [ping.url, ping.path]);

    let process = Cc["@mozilla.org/process/util;1"].createInstance(

      Ci.nsIProcess

    );

    process.init(exe);

    process.startHidden = true;

    process.noShell = true;

    process.runAsync(params, params.length, observer);

  },

};