Issue 501916 : Add data type counts to profile deletion flow

ui/login/account_picker/user_pod_row.js

 // Copyright 2014 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.
 
 /**
  * @fileoverview User pod row implementation.
  */
 
 cr.define('login', function() {
   /**
@@ skipping 935 matching lines @@
     },
 
     /**
      * Gets action box menu, remove user command item div.
      * @type {!HTMLInputElement}
      */
     get actionBoxMenuRemoveElement() {
       return this.querySelector('.action-box-menu-remove');
     },
 
-    /**
-     * Gets action box menu, remove user warning text div.
-     * @type {!HTMLInputElement}
-     */
-    get actionBoxRemoveUserWarningTextElement() {
-      return this.querySelector('.action-box-remove-user-warning-text');
-    },
+    // /**
+     // * Gets action box menu, remove user warning text div.

[msramek, 2015/07/29 13:55:01]

Please remove the unused code.

[lwchkg, 2015/07/29 14:46:05]

Oh it is embarrassing...

+     // * @type {!HTMLInputElement}
+     // */
+    // get actionBoxRemoveUserWarningTextElement() {
+      // return this.querySelector('.action-box-remove-user-warning-text');
+    // },
 
     /**
      * Gets action box menu, remove legacy supervised user warning text div.
      * @type {!HTMLInputElement}
      */
     get actionBoxRemoveLegacySupervisedUserWarningTextElement() {
       return this.querySelector(
           '.action-box-remove-legacy-supervised-user-warning-text');
     },
 
@@ skipping 445 matching lines @@
       // Move up the menu if it overlaps shelf.
       var maxHeight = cr.ui.LoginUITools.getMaxHeightBeforeShelfOverlapping(
           this.actionBoxMenu);
       var actualHeight = parseInt(
           window.getComputedStyle(this.actionBoxMenu).height);
       if (maxHeight < actualHeight) {
         this.actionBoxMenu.classList.add('menu-moved-up');
         this.actionBoxAreaElement.classList.add('menu-moved-up');
       }
       chrome.send('logRemoveUserWarningShown');
+
+      // Show extra statistics information for desktop users
+      if (this.user.isDesktopUser) {
+        // set a global handler for the callback
+        window.updateRemoveWarningDialog =
+            this.updateRemoveWarningDialog_.bind(this);
+        chrome.send('removeUserWarningLoadStats', [this.user.profilePath]);
+      }
     },
 
     /**
+     * Refresh the statistics in the remove user warning dialog.
+     * @param {string} profilePath the filepath of the URL (must be verified)
+     * @param {Object} profileStats Statistics associated with profileURL.
+     */
+    updateRemoveWarningDialog_: function(profilePath, profileStats) {
+      if (profilePath === this.user.profilePath) {
+        // Converting profileStats into id attribute by an object.
+        var stats_id_map = {
+          'BrowsingHistory': 'action-box-remove-user-warning-history',
+          'Passwords': 'action-box-remove-user-warning-passwords',
+          'Bookmarks': 'action-box-remove-user-warning-bookmarks',
+          'Settings': 'action-box-remove-user-warning-settings',
+        }
+        // Load individual statistics
+        var num_stats_loaded = 0;
+        var total_count = 0;
+        for (var key in profileStats) {
+          if (stats_id_map.hasOwnProperty(key)) {
+            this.querySelector("." + stats_id_map[key]).textContent
+                = profileStats[key];
+            num_stats_loaded++;
+            total_count += profileStats[key];
+          }
+        }
+        // Write total number if all statistics are loaded.
+        if (num_stats_loaded === Object.keys(stats_id_map).length) {
+          elements = this.getElementsByClassName('total-count');
+          for (var i = 0; i < elements.length; i++) {
+            elements[i].textContent = total_count;
+          }
+        }
+      }
+    },
+
+    /**
      * Handles a click event on remove user confirmation button.
      * @param {Event} e Click event.
      */
     handleRemoveUserConfirmationClick_: function(e) {
       if (this.isActionBoxMenuActive) {
         this.isActionBoxMenuActive = false;
         this.removeUser(this.user);
         e.stopPropagation();
       }
     },
@@ skipping 498 matching lines @@
 
     /** @override */
     update: function() {
       this.imageElement.src = this.user.userImage;
       this.nameElement.textContent = this.user.displayName;
       this.reauthNameHintElement.textContent = this.user.displayName;
 
       var isLockedUser = this.user.needsSignin;
       var isLegacySupervisedUser = this.user.legacySupervisedUser;
       var isChildUser = this.user.childUser;
+      var isSynced = this.user.emailAddress !== "";
       this.classList.toggle('locked', isLockedUser);
       this.classList.toggle('legacy-supervised', isLegacySupervisedUser);
       this.classList.toggle('child', isChildUser);
 
       if (this.isAuthTypeUserClick)
         this.passwordLabelElement.textContent = this.authValue;
 
-      this.actionBoxRemoveUserWarningTextElement.hidden =
-          isLegacySupervisedUser;
+      this.querySelector('.action-box-remove-user-warning-text-nonsync')

[msramek, 2015/07/29 13:55:01]

These classes seem to refer to a particular element - shouldn't they be IDs
then? Or if there are more elements with that class, querySelector will only
pick the first one, which is probably not the desired result.

[lwchkg, 2015/07/29 14:46:05]

All tags in this file should NOT have IDs because the tags are duplicated once
for each user in the profile selection screen.

Actually I found that out in the hard way... (And OMG Chromium crash every time
I reload chrome://user-manager and crash very often when I inspect any running
JS in that url.)

[msramek, 2015/07/29 18:10:25]

If that's the case, then your selectors will always pick the first matching
element, which means always show the data in the first user pod, no?

+          .hidden = isLegacySupervisedUser || isSynced;
+      this.querySelector('.action-box-remove-user-warning-table-nonsync')
+          .hidden = isLegacySupervisedUser || isSynced;
+      this.querySelector('.action-box-remove-user-warning-text-sync')
+          .hidden = isLegacySupervisedUser || !isSynced;
+      this.querySelector('.action-box-remove-user-warning-text-sync2')
+          .hidden = isLegacySupervisedUser || !isSynced;
+      this.querySelector('.action-box-remove-user-warning-text-nostats')
+          .hidden = true;//isLegacySupervisedUser;
       this.actionBoxRemoveLegacySupervisedUserWarningTextElement.hidden =
           !isLegacySupervisedUser;
 
+      // Convert HTML textContent into HTML. Done only once.
+      var tag =
+          this.querySelector('.action-box-remove-user-warning-text-nonsync');
+      if (!tag.childElementCount) {
+          this.querySelector('.action-box-remove-user-warning-text-nonsync')
+              .innerHTML = loadTimeData.getString(
+                  'removeUserWarningTextNonSync');
+          this.querySelector('.action-box-remove-user-warning-text-sync')
+              .innerHTML = loadTimeData.getString(
+                  'removeUserWarningTextSync');
+          this.querySelector('.action-box-remove-user-warning-text-sync2')
+              .innerHTML = loadTimeData.getString(
+                  'removeUserWarningTextSync2');
+      }
+
       this.passwordElement.setAttribute('aria-label', loadTimeData.getStringF(
         'passwordFieldAccessibleName', this.user_.emailAddress));
 
       UserPod.prototype.updateActionBoxArea.call(this);
     },
 
     /** @override */
     focusInput: function() {
       // Move tabIndex from the whole pod to the main input.
       this.tabIndex = -1;
@@ skipping 1192 matching lines @@
        if (pod && pod.multiProfilesPolicyApplied) {
          pod.userTypeBubbleElement.classList.remove('bubble-shown');
        }
      }
   };
 
   return {
     PodRow: PodRow
   };
 });