diff options
author | Pale Moon <git-repo@palemoon.org> | 2015-02-02 04:49:27 +0100 |
---|---|---|
committer | Pale Moon <git-repo@palemoon.org> | 2015-02-02 04:50:36 +0100 |
commit | 1617d23a8b7ce01f641b9730feecaae7dc3ffb36 (patch) | |
tree | 4650799dc77197b278d156366defec5267b4b13a /build | |
parent | a06c52b959b396674629aad8cc7728f8ed552973 (diff) | |
download | palemoon-gre-1617d23a8b7ce01f641b9730feecaae7dc3ffb36.tar.gz |
Remove crashreporter from mobile
Diffstat (limited to 'build')
-rw-r--r-- | build/mobile/b2gautomation.py | 2 | ||||
-rw-r--r-- | build/mobile/remoteautomation.py | 34 |
2 files changed, 5 insertions, 31 deletions
diff --git a/build/mobile/b2gautomation.py b/build/mobile/b2gautomation.py index 06654f3e6..7c0e68a6c 100644 --- a/build/mobile/b2gautomation.py +++ b/build/mobile/b2gautomation.py @@ -76,7 +76,7 @@ class B2GRemoteAutomation(Automation): Automation.installExtension(self, extensionSource, profileDir, extensionID) # Set up what we need for the remote environment - def environment(self, env=None, xrePath=None, crashreporter=True): + def environment(self, env=None, xrePath=None): # Because we are running remote, we don't want to mimic the local env # so no copying of os.environ if env is None: diff --git a/build/mobile/remoteautomation.py b/build/mobile/remoteautomation.py index 2ebad37b7..6df61a99e 100644 --- a/build/mobile/remoteautomation.py +++ b/build/mobile/remoteautomation.py @@ -46,20 +46,17 @@ class RemoteAutomation(Automation): self._remoteLog = logfile # Set up what we need for the remote environment - def environment(self, env = None, xrePath = None, crashreporter = True): + def environment(self, env = None, xrePath = None): # Because we are running remote, we don't want to mimic the local env # so no copying of os.environ if env is None: env = {} # Except for the mochitest results table hiding option, which isn't - # passed to runtestsremote.py as an actual option, but through the - # MOZ_CRASHREPORTER_DISABLE environment variable. + # passed to runtestsremote.py as an actual option. if 'MOZ_HIDE_RESULTS_TABLE' in os.environ: env['MOZ_HIDE_RESULTS_TABLE'] = os.environ['MOZ_HIDE_RESULTS_TABLE'] - env['MOZ_CRASHREPORTER_DISABLE'] = '1' - return env def waitForFinish(self, proc, utilityPath, timeout, maxTime, startTime, debuggerInfo, symbolsPath): @@ -144,31 +141,8 @@ class RemoteAutomation(Automation): if javaException: return True - # If crash reporting is disabled (MOZ_CRASHREPORTER!=1), we can't say - # anything. - if not self.CRASHREPORTER: - return False - - try: - dumpDir = tempfile.mkdtemp() - remoteCrashDir = self._remoteProfile + '/minidumps/' - if not self._devicemanager.dirExists(remoteCrashDir): - # If crash reporting is enabled (MOZ_CRASHREPORTER=1), the - # minidumps directory is automatically created when Fennec - # (first) starts, so its lack of presence is a hint that - # something went wrong. - print "Automation Error: No crash directory (%s) found on remote device" % remoteCrashDir - # Whilst no crash was found, the run should still display as a failure - return True - self._devicemanager.getDirectory(remoteCrashDir, dumpDir) - crashed = Automation.checkForCrashes(self, dumpDir, symbolsPath) - - finally: - try: - shutil.rmtree(dumpDir) - except: - print "WARNING: unable to remove directory: %s" % dumpDir - return crashed + # No crash reporting + return False def buildCommandLine(self, app, debuggerInfo, profileDir, testURL, extraArgs): # If remote profile is specified, use that instead |