From f90b6ed5f2e8bb7a6d942191dad7abe293a0c7cb Mon Sep 17 00:00:00 2001 From: "maruel@chromium.org" Date: Fri, 6 Jul 2012 19:04:09 +0000 Subject: Remove default argument to Results.File() to ease refactoring. I may need to add support for 'existing file tested for existence but never opened' so I'll need to add another flag to the constructor. This is because some tests rely on files being present but they are never read. *Sigh*. TBR=cmp@chromium.org NOTRY=true BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/10736002 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@145610 0039d316-1c4b-4281-b951-d872f2087c98 --- tools/isolate/trace_inputs.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'tools') diff --git a/tools/isolate/trace_inputs.py b/tools/isolate/trace_inputs.py index 868a77a..f6d62d5 100755 --- a/tools/isolate/trace_inputs.py +++ b/tools/isolate/trace_inputs.py @@ -559,7 +559,7 @@ class Results(object): If tainted is true, it means it is not a real path anymore as a variable replacement occured. """ - def __init__(self, root, path, tainted=False): + def __init__(self, root, path, tainted): """Represents a file accessed. May not be present anymore.""" logging.debug('%s(%s, %s)' % (self.__class__.__name__, root, path)) self.root = root @@ -679,7 +679,7 @@ class Results(object): logging.debug('Process(%s, %d, ...)' % (pid, len(files))) self.pid = pid self.files = sorted( - (Results.File(None, f) for f in files), key=lambda x: x.path) + (Results.File(None, f, False) for f in files), key=lambda x: x.path) self.children = children self.executable = executable self.command = command -- cgit v1.1