Commit d64d6612 authored by Xing Liu's avatar Xing Liu Committed by Commit Bot

Download later: Make "don't show again" checkbox uncheck by default.

This CL makes the "don't show again" checkbox by default unchecked.
This is discussed with PM that the 3+1 approach may be confusing to the
user, and we don't want a IPH on the download later dialog.

Bug: 1117573
Change-Id: I23a95720d031489a4a5ccfa4ec872e9e613279a2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2423044
Commit-Queue: Xing Liu <xingliu@chromium.org>
Reviewed-by: default avatarDavid Trainor <dtrainor@chromium.org>
Cr-Commit-Position: refs/heads/master@{#809445}
parent 6802ee0e
...@@ -168,7 +168,7 @@ public class DownloadLaterDialogTest { ...@@ -168,7 +168,7 @@ public class DownloadLaterDialogTest {
DownloadLaterDialogChoice.ON_WIFI, DownloadLaterPromptStatus.SHOW_INITIAL); DownloadLaterDialogChoice.ON_WIFI, DownloadLaterPromptStatus.SHOW_INITIAL);
showDialog(); showDialog();
assertPositiveButtonText("Download"); assertPositiveButtonText("Download");
assertShowAgainCheckBox(true, View.VISIBLE, true); assertShowAgainCheckBox(true, View.VISIBLE, false);
assertEditText(false); assertEditText(false);
} }
...@@ -191,7 +191,7 @@ public class DownloadLaterDialogTest { ...@@ -191,7 +191,7 @@ public class DownloadLaterDialogTest {
DownloadLaterDialogChoice.DOWNLOAD_LATER, DownloadLaterPromptStatus.SHOW_INITIAL); DownloadLaterDialogChoice.DOWNLOAD_LATER, DownloadLaterPromptStatus.SHOW_INITIAL);
showDialog(); showDialog();
assertPositiveButtonText("Next"); assertPositiveButtonText("Next");
assertShowAgainCheckBox(false, View.VISIBLE, true); assertShowAgainCheckBox(false, View.VISIBLE, false);
assertEditText(false); assertEditText(false);
} }
...@@ -245,7 +245,7 @@ public class DownloadLaterDialogTest { ...@@ -245,7 +245,7 @@ public class DownloadLaterDialogTest {
}); });
assertPositiveButtonText("Next"); assertPositiveButtonText("Next");
assertShowAgainCheckBox(false, View.VISIBLE, true); assertShowAgainCheckBox(false, View.VISIBLE, false);
clickPositiveButton(); clickPositiveButton();
verify(mController, times(0)).onDownloadLaterDialogComplete(anyInt(), anyLong()); verify(mController, times(0)).onDownloadLaterDialogComplete(anyInt(), anyLong());
......
...@@ -119,10 +119,8 @@ public class DownloadLaterDialogView extends ScrollView implements OnCheckedChan ...@@ -119,10 +119,8 @@ public class DownloadLaterDialogView extends ScrollView implements OnCheckedChan
} }
void setCheckbox(@DownloadLaterPromptStatus int promptStatus) { void setCheckbox(@DownloadLaterPromptStatus int promptStatus) {
boolean checked = (promptStatus == DownloadLaterPromptStatus.SHOW_INITIAL)
|| (promptStatus == DownloadLaterPromptStatus.DONT_SHOW);
mCheckBox.setVisibility(VISIBLE); mCheckBox.setVisibility(VISIBLE);
mCheckBox.setChecked(checked); mCheckBox.setChecked(promptStatus == DownloadLaterPromptStatus.DONT_SHOW);
} }
void setCheckboxEnabled(boolean enabled) { void setCheckboxEnabled(boolean enabled) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment