[Webkit-unassigned] [Bug 116492] New: Remove DriverProxy._running_drivers from webkitpy.layout_tests.port
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Mon May 20 19:10:18 PDT 2013
https://bugs.webkit.org/show_bug.cgi?id=116492
Summary: Remove DriverProxy._running_drivers from
webkitpy.layout_tests.port
Product: WebKit
Version: 528+ (Nightly build)
Platform: Unspecified
OS/Version: Unspecified
Status: NEW
Severity: Normal
Priority: P2
Component: Tools / Tests
AssignedTo: webkit-unassigned at lists.webkit.org
ReportedBy: rniwa at webkit.org
CC: simon.fraser at apple.com, ojan at chromium.org,
dpranke at chromium.org, benjamin at webkit.org
Merge https://chromium.googlesource.com/chromium/blink/+/1af2f0be5a1746784ff114952f29f7b8fa9895f3
The DriverProxy class had the concept of keeping a pool of started
DRTs around (indexed by command line) so that we didn't necessarily
need to restart DRT when the command line for a test changed (e.g., when
starting to run tests in a virtual suite). However, the pool was per-worker
(so not shared across the machine) and we didn't cap the number of processes
started, so we would end up with a whole bunch of DRTs (3x-4x the number of
workers started). If anything, this seemed to make the tests slightly slower
and probably added to memory pressure on the machine.
This change removes the pool code; now we'll simply restart DRT when the
command line changes. With this change, we can actually fold the restarting
code into Driver itself (and get rid of DriverProxy), but I'll leave that
for a subsequent patch just to make things easier to follow.
--
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
More information about the webkit-unassigned
mailing list