• Hayato Ito's avatar
    Revert "Check that TaskScheduler is not leaked between tests, or test-cases." · 78ea4f0f
    Hayato Ito authored
    This reverts commit a2a4e75d.
    
    Reason for revert: [sheriff] a culprit for cronet_tests failing on multiple builders
    See https://crbug.com/877355 for details.
    
    Original change's description:
    > Check that TaskScheduler is not leaked between tests, or test-cases.
    > 
    > - Move some APIs off of base::TestSuite, that we unnecessarily public.
    > - Rename the various internal TestEventListeners to express purpose.
    > - Add CheckForLeakedGlobals check, and DisableCheckForLeakedGlobals API.
    >   The latter is called by the content::ContentTestSuiteBase, since the
    >   various browser- and ui-tests running under that suite base run more
    >   like the browser itself, and so are expected to let global singletons
    >   leak.
    > 
    > Bug: 875486
    > Change-Id: If749dfe8530abc4a5c13a840bbf8258d48c377c5
    > Reviewed-on: https://chromium-review.googlesource.com/1182612
    > Reviewed-by: Scott Violet <sky@chromium.org>
    > Reviewed-by: Gabriel Charette <gab@chromium.org>
    > Commit-Queue: Wez <wez@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#585652}
    
    TBR=sky@chromium.org,wez@chromium.org,gab@chromium.org
    
    Change-Id: I906458f276100d9f924a0a78c7cca3a55bcb28b6
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 875486
    Reviewed-on: https://chromium-review.googlesource.com/1187982Reviewed-by: default avatarHayato Ito <hayato@chromium.org>
    Commit-Queue: Hayato Ito <hayato@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#585680}
    78ea4f0f
content_test_launcher.cc 4.6 KB