Last Comment Bug 869322 - Screen can't read the text in input box
: Screen can't read the text in input box
: access
Product: Firefox for Android
Classification: Client Software
Component: Keyboards and IME (show other bugs)
: Trunk
: ARM Android
-- normal (vote)
: Firefox 23
Assigned To: Chen Dai
: Jim Chen [:jchen] [:darchons]
Depends on:
  Show dependency treegraph
Reported: 2013-05-06 23:41 PDT by Chen Dai
Modified: 2013-05-09 10:27 PDT (History)
6 users (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

patch for bug869322 (596 bytes, patch)
2013-05-08 01:59 PDT, Chen Dai
eitan: review+
Details | Diff | Splinter Review

Description User image Chen Dai 2013-05-06 23:41:40 PDT
User Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:20.0) Gecko/20100101 Firefox/20.0
Build ID: 20130329030832

Steps to reproduce:

1. enable google talkback
2. open firefox
3. go to
4. input "aaabbbcccddd" into input box

Actual results:

nothing happened

Expected results:

talkback should read "aaabbbcccddd" after input the text

I download the nightly builds, and I found this version "" works fine, but this" works bad
Comment 1 User image Kevin Brosnan [:kbrosnan] 2013-05-07 14:28:56 PDT
cc'ing a couple accessibility team members
Comment 2 User image Chen Dai 2013-05-07 23:01:48 PDT
I think it is due to bug833164
I update the code to revision 124928, it act properly.
But in revision 124929, it cant read
Comment 3 User image Chen Dai 2013-05-08 00:21:17 PDT
Comment 4 User image Chen Dai 2013-05-08 01:59:20 PDT
Created attachment 746832 [details] [diff] [review]
patch for bug869322

This bug is because:
after Bug 833164, accessible events cpp were replaced by code gen of xpcom.
The original textchange event had a funtion isInserted, but in the gen code, isInserted changed to a bool attribute.
In Eventmanager, it still calls isInserted(), so it throw a error
Comment 5 User image Eitan Isaacson [:eeejay] 2013-05-08 09:55:21 PDT
Comment on attachment 746832 [details] [diff] [review]
patch for bug869322

Review of attachment 746832 [details] [diff] [review]:

Good catch! I'm assuming you tested this?
Comment 6 User image Chen Dai 2013-05-08 18:35:15 PDT
yes, I have tested it.
Comment 7 User image Chen Dai 2013-05-08 18:45:41 PDT
What should I do next?
Comment 8 User image alexander :surkov 2013-05-08 18:47:06 PDT
add 'checking-needed' keyword to get the patch landed and find another bug ;)
Comment 9 User image Ryan VanderMeulen [:RyanVM] 2013-05-08 18:59:19 PDT

Thanks for the patch! In the next day or so, this patch will be merged on to mozilla-central and will be resolved FIXED at that time. Currently, this fix will appear first in Firefox 23. Eitan, should this be nominated for uplift? Seems like a bad bug with a low-risk fix.

One more request, to make life easier for those checking in on your behalf, please make sure that you have Mercurial configured to generate all the necessary metadata for checkin. Instructions are linked below. Thanks again!
Comment 10 User image Ryan VanderMeulen [:RyanVM] 2013-05-09 10:27:03 PDT

Note You need to log in before you can comment on or make changes to this bug.