• Wez's avatar
    Reland "Disallow QuitCurrent*Deprecated() on RunLoops with Quit*Closure()s." · 325eafc6
    Wez authored
    This is a partial reland of b035cfda
    
    Original change's description:
    > Disallow QuitCurrent*Deprecated() on RunLoops with Quit*Closure()s.
    >
    > Use of RunLoop::QuitCurrent*Deprecated() is deprecated, and risky,
    > potentially causing unexpected early-exits if run under a different
    > RunLoop to the one intended.
    >
    > If a caller has taken a Quit*Closure() from the RunLoop then we now
    > assume that they intend to use it to quit the loop, and therefore use
    > of QuitCurrent*Deprecated() with it must be unintentional.
    >
    > Bug: 844016
    > Change-Id: Ib181d0cb34cdba2af29f557646cbe631101bc6b0
    > Reviewed-on: https://chromium-review.googlesource.com/1082980
    > Commit-Queue: Nico Weber <thakis@chromium.org>
    > Reviewed-by: Nico Weber <thakis@chromium.org>
    > Reviewed-by: Gabriel Charette <gab@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#571499}
    
    Bug: 844016
    Change-Id: I0666631fa736244c18f8848b317b41762881c748
    Reviewed-on: https://chromium-review.googlesource.com/1121296Reviewed-by: default avatarGabriel Charette <gab@chromium.org>
    Commit-Queue: Wez <wez@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#575687}
    325eafc6
run_loop.h 12.5 KB