[Password manager] Human readable origins for Android credentials on chrome://settings/passwords

chrome/browser/resources/options/password_manager_list.js

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 cr.define('options.passwordManager', function() {
   /** @const */ var ArrayDataModel = cr.ui.ArrayDataModel;
   /** @const */ var DeletableItemList = options.DeletableItemList;
   /** @const */ var DeletableItem = options.DeletableItem;
   /** @const */ var List = cr.ui.List;
 
   // The following constants should be synchronized with the constants in
   // chrome/browser/ui/webui/options/password_manager_handler.cc.
-  /** @const */ var ORIGIN_FIELD = 'origin';
-  /** @const */ var SHOWN_URL_FIELD = 'shownUrl';
+  /** @const */ var URL_FIELD = 'url';
+  /** @const */ var SHOWN_ORIGIN_FIELD = 'shownOrigin';
   /** @const */ var IS_ANDROID_URI_FIELD = 'isAndroidUri';
+  /** @const */ var IS_CLICKABLE_FIELD = 'isClickable';
   /** @const */ var IS_SECURE_FIELD = 'isSecure';
   /** @const */ var USERNAME_FIELD = 'username';
   /** @const */ var PASSWORD_FIELD = 'password';
   /** @const */ var FEDERATION_FIELD = 'federation';
   /** @const */ var ORIGINAL_INDEX_FIELD = 'index';
 
   /**
    * Creates a new passwords list item.
    * @param {cr.ui.ArrayDataModel} dataModel The data model that contains this
    *     item.
    * @param {Object} entry A dictionary of data on new list item. When the
    *     list has been filtered, one more element [index] may be present.
    * @param {boolean} showPasswords If true, add a button to the element to
    *     allow the user to reveal the saved password.
    * @constructor
    * @extends {options.DeletableItem}
    */
   function PasswordListItem(dataModel, entry, showPasswords) {
     var el = cr.doc.createElement('div');
     el.dataItem = entry;
     el.dataModel = dataModel;
     el.__proto__ = PasswordListItem.prototype;
     el.showPasswords_ = showPasswords;
     el.decorate();
 
     return el;
   }
 
   /**
-   * Returns title for password's origin. If the origin is  Android URI, returns
-   * the origin as it is. Removes the scheme if the url is insecure and removes
-   * trailing punctuation symbols.
+   * Returns title for password's origin. If the origin is not clickable,
+   * returns the origin as it is. For clickable origins, removes the scheme if
+   * the url is insecure and removes trailing punctuation symbols.
    * @param {Object} item A dictionary of data on the list item.
    * @return {string} The title for password's origin.
    */
   function getTitleForPasswordOrigin(item) {
     var title = item.url;
-    if (item.isAndroidUri)
+    if (!item.isClickable)
       return title;
     if (!item.isSecure) {
       var ind = title.indexOf('://');
       if (ind >= 0) {
         title = title.substring(ind + 3);
       }
     }
     return title;
   }
 
   /**
    * Helper function that creates an HTML element for displaying the origin of
    * saved password.
    * @param {Object} item A dictionary of data on the list item.
    * @param {Element} urlDiv div-element that will enclose the created
    *     element.
    * @return {Element} The element for displaying password origin.
    */
   function createUrlLink(item, urlDiv) {
     var urlLink;
-    if (!item.isAndroidUri) {
+    if (item.isClickable) {
       urlLink = item.ownerDocument.createElement('a');
       urlLink.href = item.url;
       urlLink.setAttribute('target', '_blank');
       urlLink.dir = 'ltr';
     } else {
       urlLink = item.ownerDocument.createElement('span');
     }
-    urlLink.textContent = item.shownUrl;
+    urlLink.textContent = item.shownOrigin;
     urlLink.addEventListener('focus', function() {
       item.handleFocus();
     }.bind(item));
     return urlLink;
   }
 
+  /**
+   * Helper function that creates an HTML element that is appended to the
+   * clickable origins of Android credentials.
+   * @return {Element} The element to append Android clickable origins.
+   */
+  function createAndroidSpan() {
+    var androidSpan = cr.doc.createElement('span');
+    androidSpan.textContent = ' (Android)';

[Evan Stade, 2016/03/22 20:13:37]

I think this needs to be left up to translators. Android is probably the same
everywhere but () aren't universal.

Dan, can you put C++-provided translations into a CSS rule? This would be nice:

.android-uri.clickable::after {
  text-content: ${androidUriSuffix};
}

[Dan Beam, 2016/03/22 20:36:49]

content: "$i18n{...}"; might work in the near future but we'd need to escape any
" found in the content of the replacement.  we're hitting similar issues inside
of value="$i18n{...}" in HTML

[kolos1, 2016/03/22 21:00:52]

Where shall I define androidUriSuffix} that will be "(Android)"?

+    return androidSpan;
+  }
+
   PasswordListItem.prototype = {
     __proto__: DeletableItem.prototype,
 
     /** @override */
     decorate: function() {
       DeletableItem.prototype.decorate.call(this);
 
       // The URL of the site.
       var urlDiv = this.ownerDocument.createElement('div');
       urlDiv.className = 'favicon-cell url';
       urlDiv.setAttribute('title', getTitleForPasswordOrigin(this));
       urlDiv.style.backgroundImage = getFaviconImageSet(
           'origin/' + this.url, 16);
 
       this.urlLink = createUrlLink(this, urlDiv);
       urlDiv.appendChild(this.urlLink);
 
+      if (this.isAndroidUri && this.isClickable) {
+        this.androidSpan = createAndroidSpan();
+        urlDiv.appendChild(this.androidSpan);
+      }
+
       this.urlDiv = urlDiv;
       this.contentElement.appendChild(urlDiv);
 
       // The stored username.
       var usernameDiv = this.ownerDocument.createElement('div');
       usernameDiv.className = 'name';
       usernameDiv.title = this.username;
       this.contentElement.appendChild(usernameDiv);
       var usernameInput = this.ownerDocument.createElement('input');
       usernameInput.type = 'text';
@@ skipping 139 matching lines @@
       } else {
         this.hidePassword();
       }
     },
 
     /**
      * Get and set the URL for the entry.
      * @type {string}
      */
     get url() {
-      return this.dataItem[ORIGIN_FIELD];
+      return this.dataItem[URL_FIELD];
     },
     set url(url) {
-      this.dataItem[ORIGIN_FIELD] = url;
+      this.dataItem[URL_FIELD] = url;
     },
 
     /**
-     * Get and set the shown url for the entry.
+     * Get and set the shown origin for the entry.
      * @type {string}
      */
-    get shownUrl() {
-      return this.dataItem[SHOWN_URL_FIELD];
+    get shownOrigin() {
+      return this.dataItem[SHOWN_ORIGIN_FIELD];
     },
-    set shownUrl(shownUrl) {
-      this.dataItem[SHOWN_URL_FIELD] = shownUrl;
+    set shownOrigin(shownOrigin) {
+      this.dataItem[SHOWN_ORIGIN_FIELD] = shownOrigin;
     },
 
     /**
      * Get and set whether the origin is Android URI.
      * @type {boolean}
      */
     get isAndroidUri() {
       return this.dataItem[IS_ANDROID_URI_FIELD];
     },
     set isAndroidUri(isAndroidUri) {

[Evan Stade, 2016/03/22 20:13:37]

is this used?

[kolos1, 2016/03/23 11:59:46]

Yes, we need these properties. 
isAndroidUri to check whether to add android suffix, i.e "(Android)".
isClickable to check whether <span> or <a> should be created to display origin.

[Evan Stade, 2016/03/23 18:54:29]

where?

       this.dataItem[IS_ANDROID_URI_FIELD] = isAndroidUri;
     },
 
     /**
+     * Get and set whether the origin is clickable.
+     * @type {boolean}
+     */
+    get isClickable() {
+      return this.dataItem[IS_CLICKABLE_FIELD];
+    },
+    set isClickable(isClickable) {

[Evan Stade, 2016/03/22 20:13:37]

is this used?

+      this.dataItem[IS_CLICKABLE_FIELD] = isClickable;
+    },
+
+    /**
      * Get and set whether the origin uses secure scheme.
      * @type {boolean}
      */
     get isSecure() {
       return this.dataItem[IS_SECURE_FIELD];
     },
     set isSecure(isSecure) {
       this.dataItem[IS_SECURE_FIELD] = isSecure;
     },
 
@@ skipping 88 matching lines @@
       var tabIndex = focusable ? 0 : -1;
       this.urlLink.tabIndex = tabIndex;
       this.closeButtonElement.tabIndex = tabIndex;
     },
 
     /**
      * Get the url for the entry.
      * @type {string}
      */
     get url() {
-      return this.dataItem[ORIGIN_FIELD];
+      return this.dataItem[URL_FIELD];
     },
     set url(url) {
-      this.dataItem[ORIGIN_FIELD] = url;
+      this.dataItem[URL_FIELD] = url;
     },
 
     /**
-     * Get and set the shown url for the entry.
+     * Get and set the shown origin for the entry.
      * @type {string}
      */
-    get shownUrl() {
-      return this.dataItem[SHOWN_URL_FIELD];
+    get shownOrigin() {
+      return this.dataItem[SHOWN_ORIGIN_FIELD];
     },
-    set shownUrl(shownUrl) {
-      this.dataItem[SHOWN_URL_FIELD] = shownUrl;
+    set shownOrigin(shownOrigin) {
+      this.dataItem[SHOWN_ORIGIN_FIELD] = shownOrigin;
     },
 
     /**
      * Get and set whether the origin is Android URI.
      * @type {boolean}
      */
     get isAndroidUri() {

[Evan Stade, 2016/03/22 20:13:37]

is this used somewhere?

       return this.dataItem[IS_ANDROID_URI_FIELD];
     },
     set isAndroidUri(isAndroidUri) {
       this.dataItem[IS_ANDROID_URI_FIELD] = isAndroidUri;
     },
 
     /**
+     * Get and set whether the origin is clickable.
+     * @type {boolean}
+     */
+    get isClickable() {
+      return this.dataItem[IS_CLICKABLE_FIELD];
+    },
+    set isClickable(isClickable) {
+      this.dataItem[IS_CLICKABLE_FIELD] = isClickable;
+    },
+
+    /**
      * Get and set whether the origin uses secure scheme.
      * @type {boolean}
      */
     get isSecure() {
       return this.dataItem[IS_SECURE_FIELD];
     },
     set isSecure(isSecure) {
       this.dataItem[IS_SECURE_FIELD] = isSecure;
     },
   };
@@ skipping 105 matching lines @@
     get length() {
       return this.dataModel.length;
     },
   };
 
   return {
     PasswordListItem: PasswordListItem,
     PasswordExceptionsListItem: PasswordExceptionsListItem,
     PasswordsList: PasswordsList,
     PasswordExceptionsList: PasswordExceptionsList,
-    ORIGIN_FIELD: ORIGIN_FIELD,
-    SHOWN_URL_FIELD: SHOWN_URL_FIELD,
+    URL_FIELD: URL_FIELD,
+    SHOWN_ORIGIN_FIELD: SHOWN_ORIGIN_FIELD,
+    IS_ANDROID_URI_FIELD: IS_ANDROID_URI_FIELD,
+    IS_CLICKABLE_FIELD: IS_CLICKABLE_FIELD,
     IS_SECURE_FIELD: IS_SECURE_FIELD,
     USERNAME_FIELD: USERNAME_FIELD,
     PASSWORD_FIELD: PASSWORD_FIELD,
     FEDERATION_FIELD: FEDERATION_FIELD,
     ORIGINAL_INDEX_FIELD: ORIGINAL_INDEX_FIELD
   };
 });