New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 783936 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

[Chrome Home] Allow showing the Chrome Home promo on startup to be disabled

Project Member Reported by mdjones@chromium.org, Nov 10 2017

Issue description

We would like the ability to disable the promo from showing on cold-start for users. This should be controlled by finch.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Nov 11 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/862d24329cf7d5365aca1976ebd179b5a0cd1bba

commit 862d24329cf7d5365aca1976ebd179b5a0cd1bba
Author: Matthew Jones <mdjones@chromium.org>
Date: Sat Nov 11 02:21:16 2017

[Home] Allow the Chrome Home promo to be disabled on startup with finch

Since showing the promo at startup is the current behavior, this feature
is enabled by default. The feature to disable is
"ChromeHomePromoOnStartup".

BUG= 783936 

Change-Id: I827f7a828a484603860e3ce8362e91b50a6d742a
Reviewed-on: https://chromium-review.googlesource.com/764788
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Commit-Queue: Matthew Jones <mdjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#515798}
[modify] https://crrev.com/862d24329cf7d5365aca1976ebd179b5a0cd1bba/chrome/android/java/src/org/chromium/chrome/browser/ChromeFeatureList.java
[modify] https://crrev.com/862d24329cf7d5365aca1976ebd179b5a0cd1bba/chrome/android/java/src/org/chromium/chrome/browser/util/FeatureUtilities.java
[modify] https://crrev.com/862d24329cf7d5365aca1976ebd179b5a0cd1bba/chrome/browser/android/chrome_feature_list.cc
[modify] https://crrev.com/862d24329cf7d5365aca1976ebd179b5a0cd1bba/chrome/browser/android/chrome_feature_list.h

Status: Fixed (was: Started)

Comment 3 by k...@chromium.org, Nov 13 2017

Do we want to merge this to M63?
Labels: Merge-Request-63
Based on how we plan to use this, I imagine we would. But if the larger rollout is happening in 64 it can probably wait. I'll add the tag, we can cancel if we decide we don't need it.
Project Member

Comment 5 by sheriffbot@chromium.org, Nov 13 2017

Labels: -Merge-Request-63 Merge-Review-63 Hotlist-Merge-Review
This bug requires manual review: M63 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), gkihumba@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 6 by cma...@chromium.org, Nov 15 2017

Hey what's the update here? Please remove merge label if not needed anymore.

Comment 7 by cma...@chromium.org, Nov 16 2017

Labels: -Hotlist-Merge-Review -Merge-Review-63 Merge-Rejected-63
I am rejecting the merge since this is not critical and surely late to be merged at this point.

Comment 8 by k...@chromium.org, Nov 16 2017

Labels: -Merge-Rejected-63 Merge-Request-63
Sorry for the delay. I was just about to reply. This may prove critical to an earlier feedback cycle for Chrome Home since we decided not to move forward with the startup promo launch of Chrome Home. Can we allow merge to allow for us to move forward the new launch plan?
Project Member

Comment 9 by sheriffbot@chromium.org, Nov 16 2017

Labels: -Merge-Request-63 Merge-Review-63 Hotlist-Merge-Review
This bug requires manual review: M63 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), gkihumba@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 10 by cmasso@google.com, Nov 17 2017

Labels: -Hotlist-Merge-Review -Merge-Review-63 Merge-Approved-63
Project Member

Comment 11 by bugdroid1@chromium.org, Nov 17 2017

Labels: -merge-approved-63 merge-merged-3239
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/a25b7a7af5e0da04d83388cab60bce2395546e34

commit a25b7a7af5e0da04d83388cab60bce2395546e34
Author: Matthew Jones <mdjones@chromium.org>
Date: Fri Nov 17 21:11:48 2017

[Home] Allow the Chrome Home promo to be disabled on startup with finch

Since showing the promo at startup is the current behavior, this feature
is enabled by default. The feature to disable is
"ChromeHomePromoOnStartup".

BUG= 783936 

Change-Id: I827f7a828a484603860e3ce8362e91b50a6d742a
Reviewed-on: https://chromium-review.googlesource.com/764788
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Commit-Queue: Matthew Jones <mdjones@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#515798}(cherry picked from commit 862d24329cf7d5365aca1976ebd179b5a0cd1bba)
Reviewed-on: https://chromium-review.googlesource.com/777799
Reviewed-by: Matthew Jones <mdjones@chromium.org>
Cr-Commit-Position: refs/branch-heads/3239@{#531}
Cr-Branched-From: adb61db19020ed8ecee5e91b1a0ea4c924ae2988-refs/heads/master@{#508578}
[modify] https://crrev.com/a25b7a7af5e0da04d83388cab60bce2395546e34/chrome/android/java/src/org/chromium/chrome/browser/ChromeFeatureList.java
[modify] https://crrev.com/a25b7a7af5e0da04d83388cab60bce2395546e34/chrome/android/java/src/org/chromium/chrome/browser/util/FeatureUtilities.java
[modify] https://crrev.com/a25b7a7af5e0da04d83388cab60bce2395546e34/chrome/browser/android/chrome_feature_list.cc
[modify] https://crrev.com/a25b7a7af5e0da04d83388cab60bce2395546e34/chrome/browser/android/chrome_feature_list.h

Sign in to add a comment