Enable TaskScheduler by default in the browser process |
|||||||||||
Issue descriptionIt's currently behind a field trial. Metrics are looking healthy. This issue is meant to track remaining blockers before making it always on in the browser process.
,
Nov 7 2016
,
Nov 15 2016
,
Nov 15 2016
To anyone following this bug, here's the approval from launch review to proceed with making this default on trunk: https://groups.google.com/a/google.com/d/topic/chrome-launch-review/9Ub8euoKGdc/discussion
,
Nov 15 2016
,
Nov 18 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/35980e3860cd6bb4241e9b4a6213048dcd6920e4 commit 35980e3860cd6bb4241e9b4a6213048dcd6920e4 Author: robliao <robliao@chromium.org> Date: Fri Nov 18 03:23:33 2016 Enable Initialization of the Task Scheduler by Default in the Browser Process This change also centralizes the initialization to components/task_scheduler. BUG= 662053 Review-Url: https://codereview.chromium.org/2506693002 Cr-Commit-Position: refs/heads/master@{#433057} [modify] https://crrev.com/35980e3860cd6bb4241e9b4a6213048dcd6920e4/chrome/app/generated_resources.grd [modify] https://crrev.com/35980e3860cd6bb4241e9b4a6213048dcd6920e4/chrome/browser/chrome_browser_main.cc [modify] https://crrev.com/35980e3860cd6bb4241e9b4a6213048dcd6920e4/components/task_scheduler_util/BUILD.gn [add] https://crrev.com/35980e3860cd6bb4241e9b4a6213048dcd6920e4/components/task_scheduler_util/DEPS [modify] https://crrev.com/35980e3860cd6bb4241e9b4a6213048dcd6920e4/components/task_scheduler_util/initialization_util.cc [modify] https://crrev.com/35980e3860cd6bb4241e9b4a6213048dcd6920e4/components/task_scheduler_util/initialization_util.h [modify] https://crrev.com/35980e3860cd6bb4241e9b4a6213048dcd6920e4/ios/chrome/app/strings/ios_strings.grd [modify] https://crrev.com/35980e3860cd6bb4241e9b4a6213048dcd6920e4/ios/chrome/browser/ios_chrome_main_parts.mm
,
Nov 21 2016
Hi TPMs, r433057 unintentionally barely made the M56 cut. Since it's useless at best and mildly bad at worse without its follow-up CLs we'd like to revert it on M56 branch. CL up @ https://codereview.chromium.org/2521653002 pending your merge approval (not really a merge, just a harmless revert on branch..).
,
Nov 21 2016
[Automated comment] DEPS changes referenced in bugdroid comments, needs manual review.
,
Nov 28 2016
+bustamante: this feature is for M57, we're looking to revert its first unintentional commit on M56 branch as described in #7, ok? It's possible to avoid touching strings/DEPS if that matters.
,
Nov 28 2016
,
Nov 28 2016
Yeah sounds good, approving for merge into M56 so it can get reverted. Thanks!
,
Nov 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/d9b5f3f09b0ac69f9ef4541abaa8d1fa891e35a0 commit d9b5f3f09b0ac69f9ef4541abaa8d1fa891e35a0 Author: Gabriel Charette <gab@chromium.org> Date: Tue Nov 29 15:09:09 2016 [M56] Revert "Enable Initialization of the Task Scheduler by Default in the Browser Process" This reverts commit 35980e3860cd6bb4241e9b4a6213048dcd6920e4 on M56 branch. It unintentionally barely made the M56 cut and is useless without its follow-up CLs. BUG= 662053 TBR=robliao@chromium.org Review URL: https://codereview.chromium.org/2521653002 . Cr-Commit-Position: refs/branch-heads/2924@{#154} Cr-Branched-From: 3a87aecc31cd1ffe751dd72c04e5a96a1fc8108a-refs/heads/master@{#433059} [modify] https://crrev.com/d9b5f3f09b0ac69f9ef4541abaa8d1fa891e35a0/chrome/app/generated_resources.grd [modify] https://crrev.com/d9b5f3f09b0ac69f9ef4541abaa8d1fa891e35a0/chrome/browser/chrome_browser_main.cc [modify] https://crrev.com/d9b5f3f09b0ac69f9ef4541abaa8d1fa891e35a0/components/task_scheduler_util/BUILD.gn [delete] https://crrev.com/146e4c96e3e45a0edb5d04f6b76d44e86cdb24f7/components/task_scheduler_util/DEPS [modify] https://crrev.com/d9b5f3f09b0ac69f9ef4541abaa8d1fa891e35a0/components/task_scheduler_util/initialization_util.cc [modify] https://crrev.com/d9b5f3f09b0ac69f9ef4541abaa8d1fa891e35a0/components/task_scheduler_util/initialization_util.h [modify] https://crrev.com/d9b5f3f09b0ac69f9ef4541abaa8d1fa891e35a0/ios/chrome/app/strings/ios_strings.grd [modify] https://crrev.com/d9b5f3f09b0ac69f9ef4541abaa8d1fa891e35a0/ios/chrome/browser/ios_chrome_main_parts.mm
,
Jan 11 2017
|
|||||||||||
►
Sign in to add a comment |
|||||||||||
Comment 1 by gab@chromium.org
, Nov 3 2016