Lock screen pin keyboard ui upgrades.

chrome/browser/resources/settings/people_page/setup_pin_dialog.js

Index: chrome/browser/resources/settings/people_page/setup_pin_dialog.js
diff --git a/chrome/browser/resources/settings/people_page/setup_pin_dialog.js b/chrome/browser/resources/settings/people_page/setup_pin_dialog.js
index a3985a55711829ab38557e69f1cde0a838b5b418..d60e0e82aaca5aa788ceb0027ca9c8ca54b49077 100644
--- a/chrome/browser/resources/settings/people_page/setup_pin_dialog.js
+++ b/chrome/browser/resources/settings/people_page/setup_pin_dialog.js
@@ -44,17 +44,6 @@ Polymer({
     initialPin_: String,
 
     /**
-     * The actual problem message to display.
-     * @private
-     */
-    problemMessage_: String,
-
-    /**
-     * The type of problem class to show (warning or error).
-     */
-    problemClass_: String,
-
-    /**
      * Should the step-specific submit button be displayed?
      * @private
      */
@@ -88,6 +77,14 @@ Polymer({
   },
 
   /**
+   * Gets the pin keyboard element.
+   * @type {!HTMLDivElement}
+   */
+  get pinKeyboard() {
+    return this.$$('#pinKeyboard');
+  },
+
+  /**
    * Resets the element to the initial state.
    * @private
    */
@@ -157,15 +154,13 @@ Polymer({
    * @param {string} problemClass
    */
   showProblem_: function(messageId, problemClass) {
-    this.problemMessage_ = this.i18n(messageId);
-    this.problemClass_ = problemClass;
+    this.pinKeyboard.setProblem(this.i18n(messageId), true /*isError*/);

[jdufault, 2016/08/24 01:20:18]

Refactor the callers of showProblem_ to also use true/false.

problemClass is not being propagated right now which breaks all of the warnings.
If we want to eliminate the difference between warning and errors, we should
eliminate the problemClass parameter entirely.

[sammiequon, 2016/08/25 23:50:02]

No more problem message.

     this.updateStyles();
   },
 
   /** @private */
   hideProblem_: function() {
-    this.problemMessage_ = '';
-    this.problemClass_ = '';
+    this.pinKeyboard.setProblem('', true /*isError*/);

[jdufault, 2016/08/24 01:20:18]

Add a clearProblem method?

[sammiequon, 2016/08/25 23:50:02]

No more problem message.

   },
 
   /** @private */