cros: Bind password echo with TouchView mode

chrome/browser/ui/views/chrome_views_delegate_chromeos.cc

Index: chrome/browser/ui/views/chrome_views_delegate_chromeos.cc
diff --git a/chrome/browser/ui/views/chrome_views_delegate_chromeos.cc b/chrome/browser/ui/views/chrome_views_delegate_chromeos.cc
index c02b480d014ad3313d2fca93ff61a46fb6927ece..ead72958b877c017a535a9bcd50f0931a2c33582 100644
--- a/chrome/browser/ui/views/chrome_views_delegate_chromeos.cc
+++ b/chrome/browser/ui/views/chrome_views_delegate_chromeos.cc
@@ -4,8 +4,20 @@
 
 #include "chrome/browser/ui/views/chrome_views_delegate.h"
 
+#include "base/time/time.h"
+#include "chrome/browser/chromeos/ui/password_echo_controller.h"
+
 views::Widget::InitParams::WindowOpacity
 ChromeViewsDelegate::GetOpacityForInitParams(
     const views::Widget::InitParams& params) {
   return views::Widget::InitParams::TRANSLUCENT_WINDOW;
 }
+
+base::TimeDelta ChromeViewsDelegate::GetTextfieldPasswordRevealDuration() {
+  const base::TimeDelta kTextfieldPasswordRevealDuration =

[sky, 2016/08/26 22:48:00]

Did you consider doing this if the mouse recent pointer event (which includes
touch and mouse) is a touch event? I think it's worth asking UX about this given
it seems helpful on desktop when using a touch input device too.

[xiyuan, 2016/08/26 23:20:31]

I see your point. The part that I am not sure is how confident we are to enable
this with touch events. People might not want this just because they tap on a
password field.

Will ask UX on the bug.

+      base::TimeDelta::FromSeconds(1);
+  chromeos::PasswordEchoController* const controller =
+      chromeos::PasswordEchoController::Get();
+  return controller && controller->enabled() ? kTextfieldPasswordRevealDuration
+                                             : base::TimeDelta();
+}