72 lines
3.4 KiB
Diff
72 lines
3.4 KiB
Diff
From: csagan5 <32685696+csagan5@users.noreply.github.com>
|
|
Date: Sun, 26 Nov 2017 22:51:43 +0100
|
|
Subject: Skip the first run and metrics
|
|
|
|
---
|
|
.../org/chromium/chrome/browser/firstrun/FirstRunUtils.java | 3 ---
|
|
.../chrome/browser/firstrun/ToSAndUMAFirstRunFragment.java | 6 ------
|
|
.../chromium/chrome/browser/firstrun/FirstRunStatus.java | 6 ++----
|
|
3 files changed, 2 insertions(+), 13 deletions(-)
|
|
|
|
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/firstrun/FirstRunUtils.java b/chrome/android/java/src/org/chromium/chrome/browser/firstrun/FirstRunUtils.java
|
|
--- a/chrome/android/java/src/org/chromium/chrome/browser/firstrun/FirstRunUtils.java
|
|
+++ b/chrome/android/java/src/org/chromium/chrome/browser/firstrun/FirstRunUtils.java
|
|
@@ -43,15 +43,12 @@ public class FirstRunUtils {
|
|
boolean javaPrefValue =
|
|
javaPrefs.readBoolean(ChromePreferenceKeys.FIRST_RUN_CACHED_TOS_ACCEPTED, false);
|
|
boolean nativePrefValue = isFirstRunEulaAccepted();
|
|
- boolean isFirstRunComplete = FirstRunStatus.getFirstRunFlowComplete();
|
|
- if (javaPrefValue || nativePrefValue || isFirstRunComplete) {
|
|
if (!javaPrefValue) {
|
|
javaPrefs.writeBoolean(ChromePreferenceKeys.FIRST_RUN_CACHED_TOS_ACCEPTED, true);
|
|
}
|
|
if (!nativePrefValue) {
|
|
setEulaAccepted();
|
|
}
|
|
- }
|
|
}
|
|
|
|
/**
|
|
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/firstrun/ToSAndUMAFirstRunFragment.java b/chrome/android/java/src/org/chromium/chrome/browser/firstrun/ToSAndUMAFirstRunFragment.java
|
|
--- a/chrome/android/java/src/org/chromium/chrome/browser/firstrun/ToSAndUMAFirstRunFragment.java
|
|
+++ b/chrome/android/java/src/org/chromium/chrome/browser/firstrun/ToSAndUMAFirstRunFragment.java
|
|
@@ -98,12 +98,6 @@ public class ToSAndUMAFirstRunFragment extends Fragment implements FirstRunFragm
|
|
|
|
mAcceptButton.setOnClickListener((v) -> onTosButtonClicked());
|
|
|
|
- mSendReportCheckBox.setChecked(getUmaCheckBoxInitialState());
|
|
- if (!canShowUmaCheckBox()) {
|
|
- mSendReportCheckBox.setVisibility(View.GONE);
|
|
- }
|
|
-
|
|
- mTosAndPrivacy.setMovementMethod(LinkMovementMethod.getInstance());
|
|
|
|
Resources resources = getResources();
|
|
NoUnderlineClickableSpan clickableGoogleTermsSpan =
|
|
diff --git a/chrome/browser/first_run/android/java/src/org/chromium/chrome/browser/firstrun/FirstRunStatus.java b/chrome/browser/first_run/android/java/src/org/chromium/chrome/browser/firstrun/FirstRunStatus.java
|
|
--- a/chrome/browser/first_run/android/java/src/org/chromium/chrome/browser/firstrun/FirstRunStatus.java
|
|
+++ b/chrome/browser/first_run/android/java/src/org/chromium/chrome/browser/firstrun/FirstRunStatus.java
|
|
@@ -43,8 +43,7 @@ public class FirstRunStatus {
|
|
* includes ToS and Sign In pages if necessary.
|
|
*/
|
|
public static boolean getFirstRunFlowComplete() {
|
|
- return SharedPreferencesManager.getInstance().readBoolean(
|
|
- ChromePreferenceKeys.FIRST_RUN_FLOW_COMPLETE, false);
|
|
+ return true;
|
|
}
|
|
|
|
/**
|
|
@@ -61,8 +60,7 @@ public class FirstRunStatus {
|
|
* Checks whether the welcome page should be skipped from the main First Run Experience.
|
|
*/
|
|
public static boolean shouldSkipWelcomePage() {
|
|
- return SharedPreferencesManager.getInstance().readBoolean(
|
|
- ChromePreferenceKeys.FIRST_RUN_SKIP_WELCOME_PAGE, false);
|
|
+ return true;
|
|
}
|
|
|
|
/**
|
|
--
|
|
2.17.1
|
|
|