3 years, 8 months ago
(2017-06-30 21:03:17 UTC)
#1
Patchset #1 (id:1) has been deleted
newcomer
Description was changed from ========== Added flag in chrome://flags to enable the New Launcher UI. ...
3 years, 8 months ago
(2017-06-30 21:04:09 UTC)
#2
Description was changed from
==========
Added flag in chrome://flags to enable the New Launcher UI.
BUG=
==========
to
==========
Added flag in chrome://flags to enable the New Launcher UI.
BUG=738573
==========
https://codereview.chromium.org/2965813002/diff/20001/chrome/browser/about_flags.cc File chrome/browser/about_flags.cc (right): https://codereview.chromium.org/2965813002/diff/20001/chrome/browser/about_flags.cc#newcode1445 chrome/browser/about_flags.cc:1445: SINGLE_VALUE_TYPE(app_list::switches::kEnableNewAppListUI), Is there a better descriptor here than "new"? ...
3 years, 8 months ago
(2017-06-30 21:17:26 UTC)
#7
lgtm w/ one nit/suggestion https://codereview.chromium.org/2965813002/diff/100001/chrome/browser/flag_descriptions.cc File chrome/browser/flag_descriptions.cc (right): https://codereview.chromium.org/2965813002/diff/100001/chrome/browser/flag_descriptions.cc#newcode2934 chrome/browser/flag_descriptions.cc:2934: "features."; 'launcher' is fine in ...
3 years, 8 months ago
(2017-06-30 22:18:59 UTC)
#12
Dry run: Try jobs failed on following builders: win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED, http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_rel_ng/builds/461930)
3 years, 8 months ago
(2017-06-30 23:24:37 UTC)
#16
Issue 2965813002: Added flag in chrome://flags to enable the New Launcher UI.
(Closed)
Created 3 years, 8 months ago by newcomer
Modified 3 years, 7 months ago
Reviewers: stevenjb
Base URL:
Comments: 8