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

Issue 2842563002: Revert of Use TaskScheduler instead of blocking pool in automatic_reboot_manager.cc. (Closed)

Created:
3 years, 8 months ago by hcarmona
Modified:
3 years, 8 months ago
Reviewers:
stevenjb, fdoray
CC:
chromium-reviews, oshima+watch_chromium.org, davemoore+watch_chromium.org
Target Ref:
refs/heads/master
Project:
chromium
Visibility:
Public.

Description

Revert of Use TaskScheduler instead of blocking pool in automatic_reboot_manager.cc. (patchset #1 id:1 of https://codereview.chromium.org/2770783002/ ) Reason for revert: Reverting b/c of chromeos test failures: https://uberchromegw.corp.google.com/i/chromium.chromiumos/builders/Linux%20ChromiumOS%20Tests%20%281%29 Original issue's description: > Use TaskScheduler instead of blocking pool in automatic_reboot_manager.cc. > > The blocking pool is being deprecated in favor of TaskScheduler. > > BUG=667892 > R=stevenjb@chromium.org > > Review-Url: https://codereview.chromium.org/2770783002 > Cr-Commit-Position: refs/heads/master@{#466677} > Committed: https://chromium.googlesource.com/chromium/src/+/b67baf11ed95e0e7eb570d99b6de407e48d89476 TBR=stevenjb@chromium.org,fdoray@chromium.org # Skipping CQ checks because original CL landed less than 1 days ago. NOPRESUBMIT=true NOTREECHECKS=true NOTRY=true BUG=667892 Review-Url: https://codereview.chromium.org/2842563002 Cr-Commit-Position: refs/heads/master@{#466736} Committed: https://chromium.googlesource.com/chromium/src/+/764bea2483c240b36b16a1a9196fa07c573ae42a

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -8 lines) Patch
M chrome/browser/chromeos/system/automatic_reboot_manager.cc View 2 chunks +2 lines, -8 lines 0 comments Download

Messages

Total messages: 6 (3 generated)
hcarmona
Created Revert of Use TaskScheduler instead of blocking pool in automatic_reboot_manager.cc.
3 years, 8 months ago (2017-04-24 20:22:15 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at: https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2842563002/1
3 years, 8 months ago (2017-04-24 20:23:02 UTC) #3
commit-bot: I haz the power
3 years, 8 months ago (2017-04-24 20:23:50 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/764bea2483c240b36b16a1a9196f...

Powered by Google App Engine
This is Rietveld 408576698