Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(2218)

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/preferences/password/SavePasswordsPreferences.java

Issue 2067323004: Allow copying and viewing account credentials in PasswordEntryEditor (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address comments Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/preferences/password/SavePasswordsPreferences.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/preferences/password/SavePasswordsPreferences.java b/chrome/android/java/src/org/chromium/chrome/browser/preferences/password/SavePasswordsPreferences.java
index 27a1e58e70519d956456dae6fd85e6d267ed35a1..3b5d3fefb5576b7013ee5ebaec515e3c20608c99 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/preferences/password/SavePasswordsPreferences.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/preferences/password/SavePasswordsPreferences.java
@@ -45,6 +45,7 @@ public class SavePasswordsPreferences extends PreferenceFragment
// Keys for name/password dictionaries.
public static final String PASSWORD_LIST_URL = "url";
public static final String PASSWORD_LIST_NAME = "name";
+ public static final String PASSWORD_LIST_PASSWORD = "password";
// Used to pass the password id into a new activity.
public static final String PASSWORD_LIST_ID = "id";
@@ -73,6 +74,19 @@ public class SavePasswordsPreferences extends PreferenceFragment
private ChromeSwitchPreference mSavePasswordsSwitch;
private ChromeBaseCheckBoxPreference mAutoSignInSwitch;
+ // Used for verifying if 60 seconds have passed since last authenticating, its value is set
+ // in PasswordReauthentication using System.currentTimeMillis() (milliseconds since the
+ // _UNIX_ epoch)
Bernhard Bauer 2016/08/18 18:57:05 I used the underscores for emphasis in my comment.
dozsa 2016/08/19 08:36:07 Ooh okay, sorry. Removed.
+ private static long sLastReauthTimeMillis = 0;
+
+ public static void setLastReauthTimeMillis(long value) {
+ sLastReauthTimeMillis = value;
+ }
+
+ public static long getLastReauthTimeMillis() {
+ return sLastReauthTimeMillis;
+ }
+
@Override
public void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
@@ -111,6 +125,12 @@ public class SavePasswordsPreferences extends PreferenceFragment
return true;
}
+ @Override
+ public void onDetach() {
+ super.onDetach();
+ sLastReauthTimeMillis = 0;
+ }
+
void rebuildPasswordLists() {
mNoPasswords = false;
mNoPasswordExceptions = false;
@@ -154,12 +174,14 @@ public class SavePasswordsPreferences extends PreferenceFragment
PreferenceScreen screen = getPreferenceManager().createPreferenceScreen(getActivity());
String url = saved.getUrl();
String name = saved.getUserName();
+ String password = saved.getPassword();
screen.setTitle(url);
screen.setOnPreferenceClickListener(this);
screen.setSummary(name);
Bundle args = screen.getExtras();
args.putString(PASSWORD_LIST_NAME, name);
args.putString(PASSWORD_LIST_URL, url);
+ args.putString(PASSWORD_LIST_PASSWORD, password);
args.putInt(PASSWORD_LIST_ID, i);
profileCategory.addPreference(screen);
}

Powered by Google App Engine
This is Rietveld 408576698