Revert "Whitelist chromeos wm segfaults until we have a long-term fix"
Now that Dan Erat has updated the VM tests to wait for Chrome
to start up completely before shutting it down, chromeos-wm
should not segfault anymore. So we can remove it from the
whitelist.
See http://gerrit.chromium.org/gerrit/5797 for Dan's CL
to fix the VM tests.
This reverts commit 17faa72c1c329b7bdd5af90218f1e1288328e694
Change-Id: Ib5a92f2b79dd9364ac1c01e403e8c5c4066338aa
Reviewed-on: http://gerrit.chromium.org/gerrit/7242
Reviewed-by: Daniel Erat <derat@chromium.org>
Tested-by: David James <davidjames@chromium.org>
diff --git a/utils_py/generate_test_report.py b/utils_py/generate_test_report.py
index 2c521e7..36e2ebd 100755
--- a/utils_py/generate_test_report.py
+++ b/utils_py/generate_test_report.py
@@ -25,10 +25,9 @@
class CrashWaiver:
"""Represents a crash that we want to ignore for now."""
- def __init__(self, signals, deadline, url, ignored_person):
+ def __init__(self, signals, deadline, ignored_url, ignored_person):
self.signals = signals
self.deadline = datetime.datetime.strptime(deadline, '%Y-%b-%d')
- self.url = url
# List of crashes which are okay to ignore. This list should almost always be
# empty. If you add an entry, include the bug URL and your name, something like
@@ -36,9 +35,6 @@
# ['sig 11'], '2011-Aug-18', 'http://crosbug/123456', 'developer'),
_CRASH_WHITELIST = {
- 'chromeos-wm':CrashWaiver(
- ['sig 6', 'sig 11'], '2011-Sep-11', 'http://crosbug.com/19052',
- 'davidjames')
}
@@ -134,7 +130,8 @@
if match.group(1) in _CRASH_WHITELIST:
w = _CRASH_WHITELIST[match.group(1)]
if match.group(2) in w.signals and w.deadline > datetime.datetime.now():
- print 'Ignoring crash in %s for waiver %s' % (match.group(1), w.url)
+ print 'Ignoring crash in %s for waiver that expires %s' % (
+ match.group(1), w.deadline.strftime('%Y-%b-%d'))
continue
crashes.append('%s %s' % match.groups())