Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(7)

Unified Diff: trunk/src/build/android/pylib/utils/test_environment.py

Issue 93793002: Revert 237667 "[Telemetry] Refactor common Android test harness ..." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: trunk/src/build/android/pylib/utils/test_environment.py
===================================================================
--- trunk/src/build/android/pylib/utils/test_environment.py (revision 237673)
+++ trunk/src/build/android/pylib/utils/test_environment.py (working copy)
@@ -1,36 +0,0 @@
-# Copyright 2013 The Chromium Authors. All rights reserved.
-# Use of this source code is governed by a BSD-style license that can be
-# found in the LICENSE file.
-
-import logging
-import os
-import psutil
-
-from pylib import android_commands
-from pylib import ports
-
-def _KillWebServers():
- for retry in xrange(5):
- for server in ['lighttpd', 'web-page-replay']:
- pids = [p.pid for p in psutil.process_iter() if server in p.name]
- for pid in pids:
- try:
- logging.warning('Killing %s %s', server, pid)
- os.kill(pid, signal.SIGQUIT)
- except Exception as e:
- logging.warning('Failed killing %s %s %s', server, pid, e)
-
-
-def CleanupLeftoverProcesses():
- """Clean up the test environment, restarting fresh adb and HTTP daemons."""
- _KillWebServers()
- did_restart_host_adb = False
- for device in android_commands.GetAttachedDevices():
- adb = android_commands.AndroidCommands(device)
- # Make sure we restart the host adb server only once.
- if not did_restart_host_adb:
- adb.RestartAdbServer()
- did_restart_host_adb = True
- adb.RestartAdbdOnDevice()
- adb.EnableAdbRoot()
- adb.WaitForDevicePm()

Powered by Google App Engine
This is Rietveld 408576698