Commit b2032572 authored by Brian Sheedy's avatar Brian Sheedy Committed by Commit Bot

Re-enable CrOS minidump tests

Re-enables the BrowserMinidumpTests on ChromeOS since the CL that was
supposedly causing extra crash dumps to appear unexpectedly on Betty
devices has been reverted.

TBR=crouleau@chromium.org

Bug: 1014754
Change-Id: I93645b8fb8994ad9e7506d7e2a07e4ce520c03f2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1933596Reviewed-by: default avatarBrian Sheedy <bsheedy@chromium.org>
Reviewed-by: default avatarCaleb Rouleau <crouleau@chromium.org>
Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#718783}
parent 983b73b1
......@@ -20,8 +20,7 @@ class BrowserMinidumpTest(tab_test_case.TabTestCase):
# Android is currently hard coded to return None for minidump paths.
# Minidump symbolization doesn't work in ChromeOS local mode if the rootfs is
# still read-only, so skip the test in that case.
# Flakes on chromeos: crbug.com/1014754
@decorators.Disabled('android', 'chromeos', 'chromeos-local')
@decorators.Disabled('android', 'chromeos-local')
def testSymbolizeMinidump(self):
# Wait for the browser to restart fully before crashing
self._LoadPageThenWait('var sam = "car";', 'sam')
......@@ -102,8 +101,7 @@ class BrowserMinidumpTest(tab_test_case.TabTestCase):
@decorators.Isolated
# Minidump symbolization doesn't work in ChromeOS local mode if the rootfs is
# still read-only, so skip the test in that case.
# Flakes on chromeos: crbug.com/1014754
@decorators.Disabled('android', 'chromeos', 'chromeos-local')
@decorators.Disabled('android', 'chromeos-local')
def testMultipleCrashMinidumps(self):
# Wait for the browser to restart fully before crashing
self._LoadPageThenWait('var cat = "dog";', 'cat')
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment