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

Issue 1232973002: Crash tests when BrowserTestBase::SetUp is not called (Closed)

Created:
5 years, 5 months ago by pval...(no longer on Chromium)
Modified:
5 years, 4 months ago
Reviewers:
Paweł Hajdan Jr.
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Crash tests when BrowserTestBase::SetUp is not called This is being done to prevent false positive test results that occur when subclasses of BrowserTestBase override SetUp but fail to call the superclass version. This technique (keep track of whether SetUp was called and crash in the destructor if it wasn't) was already used in ContentBrowserTest. This CL moves usage of that pattern up to BrowserTestBase. BUG=510649 Committed: https://crrev.com/632247ab88b66a6fe424a0a38d22b7623775468c Cr-Commit-Position: refs/heads/master@{#341135}

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -9 lines) Patch
M content/public/test/browser_test_base.h View 1 chunk +5 lines, -0 lines 0 comments Download
M content/public/test/browser_test_base.cc View 2 chunks +9 lines, -1 line 0 comments Download
M content/public/test/content_browser_test.h View 1 chunk +0 lines, -2 lines 0 comments Download
M content/public/test/content_browser_test.cc View 3 chunks +1 line, -6 lines 0 comments Download

Messages

Total messages: 31 (14 generated)
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1232973002/1
5 years, 5 months ago (2015-07-13 22:37:32 UTC) #2
commit-bot: I haz the power
Dry run: Try jobs failed on following builders: linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/85372) (exceeded global ...
5 years, 5 months ago (2015-07-13 23:25:27 UTC) #4
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1232973002/20001
5 years, 5 months ago (2015-07-16 00:40:29 UTC) #6
commit-bot: I haz the power
Dry run: Try jobs failed on following builders: win8_chromium_ng on tryserver.chromium.win (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_ng/builds/17091)
5 years, 5 months ago (2015-07-16 00:46:54 UTC) #8
pval...(no longer on Chromium)
phajdan.jr@, I'm sending this CL out now to get feedback, but we're not quite ready ...
5 years, 5 months ago (2015-07-20 22:32:05 UTC) #12
Paweł Hajdan Jr.
Looks fine to me. Please let me know when it's ready for the final review, ...
5 years, 5 months ago (2015-07-20 23:52:10 UTC) #13
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1232973002/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1232973002/40001
5 years, 5 months ago (2015-07-23 22:34:02 UTC) #15
commit-bot: I haz the power
Dry run: Try jobs failed on following builders: win_chromium_x64_rel_ng on tryserver.chromium.win (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_rel_ng/builds/82071)
5 years, 5 months ago (2015-07-23 23:34:28 UTC) #17
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1232973002/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1232973002/40001
5 years, 4 months ago (2015-07-28 22:27:07 UTC) #19
commit-bot: I haz the power
Dry run: Try jobs failed on following builders: linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_chromeos_rel_ng/builds/83834)
5 years, 4 months ago (2015-07-28 23:40:14 UTC) #21
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1232973002/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1232973002/40001
5 years, 4 months ago (2015-07-29 18:55:49 UTC) #23
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
5 years, 4 months ago (2015-07-29 19:27:42 UTC) #25
pval...(no longer on Chromium)
Paweł, this is ready for the final review!
5 years, 4 months ago (2015-07-29 22:26:18 UTC) #26
Paweł Hajdan Jr.
LGTM, thanks!
5 years, 4 months ago (2015-07-30 12:16:12 UTC) #27
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1232973002/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1232973002/40001
5 years, 4 months ago (2015-07-30 16:11:43 UTC) #29
commit-bot: I haz the power
Committed patchset #1 (id:40001)
5 years, 4 months ago (2015-07-30 16:49:28 UTC) #30
commit-bot: I haz the power
5 years, 4 months ago (2015-07-30 16:50:18 UTC) #31
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/632247ab88b66a6fe424a0a38d22b7623775468c
Cr-Commit-Position: refs/heads/master@{#341135}

Powered by Google App Engine
This is Rietveld 408576698