Fix Android accessibility for editable text fields

content/browser/accessibility/browser_accessibility_manager_android.cc

Index: content/browser/accessibility/browser_accessibility_manager_android.cc
diff --git a/content/browser/accessibility/browser_accessibility_manager_android.cc b/content/browser/accessibility/browser_accessibility_manager_android.cc
index 4a1b39dc001f2624f6402bb5e8d805d49e0b822c..e7cf912a46d5c9e9043e85cee4c3108d4b775630 100644
--- a/content/browser/accessibility/browser_accessibility_manager_android.cc
+++ b/content/browser/accessibility/browser_accessibility_manager_android.cc
@@ -405,10 +405,11 @@ jboolean BrowserAccessibilityManagerAndroid::PopulateAccessibilityNodeInfo(
       base::android::ConvertUTF8ToJavaString(env, node->GetClassName()).obj());
   if (!node->IsPassword() ||

[David Tseng, 2016/04/11 16:48:34]

How do we currently work with input type password?

[dmazzoni, 2016/04/11 20:49:06]

There's an Android setting that determines whether it's
echoed or not, that's what this checks. So if it is
echoed, we use this API to announce it whether it's a
password field or not. If it is not echoed we don't set
the text.

[David Tseng, 2016/04/11 21:01:41]

I thought it wasn't so much that it was echoed, but the characters were
*/dotted-ed out?

       Java_BrowserAccessibilityManager_shouldExposePasswordText(env, obj)) {
-    Java_BrowserAccessibilityManager_setAccessibilityNodeInfoContentDescription(
+    Java_BrowserAccessibilityManager_setAccessibilityNodeInfoText(
         env, obj, info,
         base::android::ConvertUTF16ToJavaString(env, node->GetText()).obj(),
-        node->IsLink());
+        node->IsLink(),
+        node->IsEditableText());
   }
   base::string16 element_id;
   if (node->GetHtmlAttribute("id", &element_id)) {