/irc-logs / mozilla / #accessibility / 2013-05-15 / end

Options:

  1. # Session Start: Wed May 15 00:00:00 2013
  2. # Session Ident: #accessibility
  3. # [00:02] * khuey is now known as khuey|away
  4. # [00:02] * Quits: webatou1 (Instantbir@455F2973.12C6E01D.BF9F5BB8.IP) (Quit: Instantbird 1.3 -- http://www.instantbird.com)
  5. # [00:10] * Joins: maxli (maxli@C03F2759.C70D537E.2FA4EA88.IP)
  6. # [00:11] * Joins: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP)
  7. # [00:19] <yzen> eeejay: ping
  8. # [00:31] * Quits: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net) (Quit: lizzard)
  9. # [01:02] * Joins: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net)
  10. # [01:08] <eeejay> yzen, yo!
  11. # [01:27] * Quits: maxli (maxli@C03F2759.C70D537E.2FA4EA88.IP) (Connection reset by peer)
  12. # [01:27] * Joins: maxli (maxli@C03F2759.C70D537E.2FA4EA88.IP)
  13. # [01:34] * Quits: maxli (maxli@C03F2759.C70D537E.2FA4EA88.IP) (Quit: Leaving.)
  14. # [01:48] * Quits: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net) (Quit: habber)
  15. # [02:00] * Joins: maxli (maxli@C03F2759.C70D537E.2FA4EA88.IP)
  16. # [02:05] <yzen> eeejay: if you still there, i was just wondering if you think my changes are good in the last patch
  17. # [02:12] <eeejay> yzen, your patch uncovered a whole bunch of other issues that need to be resolved
  18. # [02:13] <@davidb> darn patches
  19. # [02:15] <yzen> eeejay: interesting anything i can help with ?
  20. # [02:15] <eeejay> yzen, there will be stuff for you to review :)
  21. # [02:15] <yzen> alright :)
  22. # [02:16] <eeejay> yzen, we were cheating a lot with in-process content. the origin message manager was always the top/first one
  23. # [02:16] <eeejay> yzen, your patch fixes that, and now we get messages from the actual targets
  24. # [02:16] <yzen> cool
  25. # [02:18] <yzen> eeejay: should i be still using your a11y-devtool branch ?
  26. # [02:19] <eeejay> yzen, i have a rebased version here i could rebase
  27. # [02:19] <eeejay> i mean push
  28. # [02:20] <yzen> eeejay: sure that would be helpful :)
  29. # [02:27] <eeejay> yzen, pushed
  30. # [02:28] <yzen> eeejay: nice , thanks ill pull
  31. # [02:29] <eeejay> yzen, this New World Order needs to be tested in android too, didn't do that
  32. # [02:41] * Quits: @davidb (davidb@2557E599.66715431.D25A875A.IP) (Quit: davidb)
  33. # [02:44] * Joins: scott_gonzalez (scott_gonz@moz-91C81A39.hrbgpa.fios.verizon.net)
  34. # [02:53] <yzen> eeejay: the extension is called screen-reader-simulator right ?
  35. # [02:53] <eeejay> yzen, yeah
  36. # [02:53] <yzen> eeejay: for some reason it's disabled by default
  37. # [02:55] <eeejay> yzen, the extension?
  38. # [02:55] <eeejay> yzen, or in the dev panel?
  39. # [02:55] <yzen> eeejay: the extension
  40. # [02:56] <yzen> eeejay: let me try rebuilding it again
  41. # [02:56] <yzen> eeejay: got it working now :)
  42. # [02:57] <eeejay> yzen, cool :)
  43. # [02:57] <eeejay> yzen, you could also install the extension standalone if you don't want to test it with gaia
  44. # [02:58] <yzen> eeejay: i ll try that too
  45. # [03:01] * Quits: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP) (Quit: Leaving.)
  46. # [03:15] <@firebot> New Core - Disability Access APIs bug 872338 filed by eitan@monotonous.org.
  47. # [03:15] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=872338 nor, --, ---, nobody, NEW, iframe documents should support nsIAccessibleCursorable too
  48. # [03:24] <@firebot> eitan@monotonous.org requested review from surkov.alexander@gm ail.com for attachment 749607 on bug 872338.
  49. # [03:30] <@firebot> surkov.alexander@gmail.com granted review for attachment 746184 on bug 869280.
  50. # [03:30] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=869280 nor, --, ---, eitan, NEW, Add temporary modal subtree to pivot API
  51. # [03:32] * Quits: scott_gonzalez (scott_gonz@moz-91C81A39.hrbgpa.fios.verizon.net) (Quit: scott_gonzalez)
  52. # [03:33] * Joins: maxli1 (maxli@C03F2759.C70D537E.2FA4EA88.IP)
  53. # [03:33] * Quits: maxli (maxli@C03F2759.C70D537E.2FA4EA88.IP) (Connection reset by peer)
  54. # [03:37] * Joins: maxli (maxli@C03F2759.C70D537E.2FA4EA88.IP)
  55. # [03:37] * Quits: maxli1 (maxli@C03F2759.C70D537E.2FA4EA88.IP) (Connection reset by peer)
  56. # [03:53] * Joins: davidb (davidb@moz-AD5202B6.static.nextweb.net)
  57. # [03:53] * ChanServ sets mode: +qo davidb davidb
  58. # [03:57] * Joins: hyeonseok (Thunderbir@CE402FF1.2CC2EE93.E0CF5FF1.IP)
  59. # [04:03] * Quits: maxli (maxli@C03F2759.C70D537E.2FA4EA88.IP) (Quit: Leaving.)
  60. # [04:04] <@firebot> eitan@monotonous.org cancelled review?(yura.zenevich@gmail .com) for attachment 749093 on bug 871859.
  61. # [04:04] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=871859 nor, --, ---, nobody, NEW, [AccessFu] Fix coordinate translation when getting bounds to display
  62. # [04:05] <@firebot> eitan@monotonous.org changed the Resolution on bug 871859 from --- to INVALID.
  63. # [04:05] <@firebot> eitan@monotonous.org changed the Status on bug 871859 from NEW to RESOLVED.
  64. # [04:15] * Parts: hyeonseok (Thunderbir@CE402FF1.2CC2EE93.E0CF5FF1.IP)
  65. # [04:28] <@firebot> New Core - Disability Access APIs bug 872355 filed by eitan@monotonous.org.
  66. # [04:28] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=872355 nor, --, ---, nobody, NEW, [AccessFu] Always use vc of nested iframes
  67. # [04:32] <@firebot> eitan@monotonous.org requested review from yura.zenevich@gmail .com for attachment 749617 on bug 872355.
  68. # [04:33] * Parts: eeejay (eeejay@moz-B3973587.xen.prgmr.com) (Ex-Chat)
  69. # [04:33] * Joins: eeejay (eeejay@moz-B3973587.xen.prgmr.com)
  70. # [04:33] * ChanServ sets mode: +o eeejay
  71. # [04:33] <@eeejay> goinghomebyebye
  72. # [04:34] <@davidb> ciao
  73. # [04:36] * Joins: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP)
  74. # [04:40] * Quits: @davidb (davidb@moz-AD5202B6.static.nextweb.net) (Quit: blast off!)
  75. # [04:51] * Quits: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net) (Quit: lizzard)
  76. # [04:55] <yzen> eeejay: ping
  77. # [04:57] * Joins: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net)
  78. # [05:36] * Quits: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net) (Quit: lizzard)
  79. # [05:48] * Joins: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net)
  80. # [06:02] * Quits: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net) (Quit: lizzard)
  81. # [06:28] <@firebot> surkov.alexander@gmail.com granted review for attachment 749607 on bug 872338.
  82. # [06:28] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=872338 nor, --, ---, nobody, NEW, iframe documents should support nsIAccessibleCursorable too
  83. # [06:29] * Quits: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com) (Client exited)
  84. # [06:29] * Joins: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net)
  85. # [06:50] * Joins: xuku (Mibbit@66B2896A.CBF8C476.41967585.IP)
  86. # [06:51] * Quits: fxa90id (fxa90id@moz-D1DD986C.dsl.dynamic.t-mobile.pl) (Connection reset by peer)
  87. # [06:51] * Quits: Jesse (jruderman@moz-537BCF9.hsd1.ca.comcast.net) (Connection reset by peer)
  88. # [06:53] * Joins: fxa90id (fxa90id@moz-D1DD986C.dsl.dynamic.t-mobile.pl)
  89. # [07:04] * Quits: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net) (Quit: lizzard)
  90. # [07:04] * Joins: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net)
  91. # [07:05] * Quits: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net) (Quit: lizzard)
  92. # [07:06] * Joins: Jesse (jruderman@moz-537BCF9.hsd1.ca.comcast.net)
  93. # [07:18] * Joins: fxa90id_ (fxa90id@moz-D1DD986C.dsl.dynamic.t-mobile.pl)
  94. # [07:18] * Quits: fxa90id (fxa90id@moz-D1DD986C.dsl.dynamic.t-mobile.pl) (Ping timeout)
  95. # [08:12] * Quits: fxa90id_ (fxa90id@moz-D1DD986C.dsl.dynamic.t-mobile.pl) (Connection reset by peer)
  96. # [08:14] <@firebot> surkov.alexander@gmail.com cancelled review?(trev.saunders@gmail .com) for attachment 742208 on bug 678429.
  97. # [08:14] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=678429 nor, --, ---, surkov.alexander, ASSI, implement weak reference for accessible objects
  98. # [08:14] * Joins: fxa90id (fxa90id@moz-D1DD986C.dsl.dynamic.t-mobile.pl)
  99. # [08:15] <@firebot> surkov.alexander@gmail.com changed the Assignee on bug 678429 from surkov.alexander@gmail.com to nobody@mozilla.org.
  100. # [08:27] * Quits: nhirata (anonymous@moz-1A50F7F8.hsd1.ca.comcast.net) (Quit: nhirata)
  101. # [08:30] <fxa90id> :)
  102. # [08:32] * Quits: @hub (hub@E639FEDB.F04051C4.DC1243F8.IP) (Ping timeout)
  103. # [08:36] * Joins: hub (hub@E639FEDB.F04051C4.DC1243F8.IP)
  104. # [08:36] * ChanServ sets mode: +o hub
  105. # [08:44] * Quits: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP) (Quit: Leaving.)
  106. # [08:44] * Quits: xuku (Mibbit@66B2896A.CBF8C476.41967585.IP) (Quit: http://www.mibbit.com ajax IRC Client)
  107. # [09:11] <@firebot> trev.saunders@gmail.com granted review for attachment 749133 on bug 864224.
  108. # [09:11] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=864224 nor, --, ---, surkov.alexander, ASSI, Support nested ARIA listitems structured by role='group'
  109. # [09:28] * Quits: fxa90id (fxa90id@moz-D1DD986C.dsl.dynamic.t-mobile.pl) (Ping timeout)
  110. # [09:31] * Joins: fxa90id (fxa90id@moz-D1DD986C.dsl.dynamic.t-mobile.pl)
  111. # [09:33] * Joins: icaaq (Adium@moz-3B892381.cust.tele2.se)
  112. # [09:36] * Quits: logbot (logbot@moz-58CB32ED.glob.com.au) (Ping timeout)
  113. # [09:37] * Joins: logbot (logbot@moz-58CB32ED.glob.com.au)
  114. # [09:39] * Quits: fxa90id (fxa90id@moz-D1DD986C.dsl.dynamic.t-mobile.pl) (Quit: Leaving)
  115. # [09:54] * Joins: victorporof (victorporo@10AA7997.880C8D58.79933D60.IP)
  116. # [10:25] <@firebot> surkov.alexander@gmail.com granted in-testsuite on bug 861062.
  117. # [10:25] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=861062 nor, --, ---, surkov.alexander, ASSI, getTextAfterOffset for word boundaries: still evolving
  118. # [10:29] * Joins: Gijs (gijs@moz-C11B0461.dsl.alice.nl)
  119. # [10:48] * Quits: icaaq (Adium@moz-3B892381.cust.tele2.se) (Quit: Leaving.)
  120. # [10:49] <@firebot> hobinjk@gmail.com requested feedback from bmcbride@mozilla. com for attachment 749753 on bug 492557.
  121. # [10:49] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=492557 nor, --, ---, mdeboer, ASSI, Create a keyboard shortcut HUD
  122. # [10:55] * Joins: surkov (surkov@4F778CCB.95871A73.E17943EE.IP)
  123. # [10:55] * ChanServ sets mode: +o surkov
  124. # [11:11] <@tbsaunde> surkov: testTextAt(0, BOUNDARY_WORD_START, "@", 0, 1) on <div>@private</div> seem right?
  125. # [11:12] <@surkov> tbsaunde: it seems that @ should be part of word
  126. # [11:12] <@surkov> at least that's a behavior on os x
  127. # [11:14] <@tbsaunde> surkov: ok
  128. # [11:15] <@tbsaunde> surkov: so then "@private", 0, 7?
  129. # [11:16] <@surkov> tbsaunde: yes
  130. # [11:16] <@surkov> tbsaunde: you know chrome things @ is a word
  131. # [11:16] <@surkov> separator
  132. # [11:17] <@surkov> IE has the same behavior as Firefox
  133. # [11:18] <@surkov> on os x (safari, chrome, firefox) @ is part of word
  134. # [11:18] <@tbsaunde> surkov: so it depends on os? :/
  135. # [11:19] <@surkov> it looks so
  136. # [11:19] <@surkov> maybe it's worth to ping roc
  137. # [11:22] <@tbsaunde> surkov: seems currently we say private, 1, 8 on linux
  138. # [11:24] <@surkov> tbsaunde: I will try different exapmles on win and os x and will comment into bug
  139. # [11:25] <@surkov> tbsaunde: at 0 offset?
  140. # [11:25] <@tbsaunde> surkov: yeah, 0 offset
  141. # [11:25] <@tbsaunde> surkov: interestingly adding that test makes some other random tests fail
  142. # [11:25] <@surkov> hm
  143. # [11:27] <@tbsaunde> surkov: I haven't rebsed on top of bug 861062 yet, but diff is http://paste.debian.net/4293/
  144. # [11:28] <@surkov> I don't spot anything
  145. # [11:28] * Quits: @firebot (firebot@moz-F8C1AF63.carolina.res.rr.com) (Ping timeout)
  146. # [11:37] * Joins: firebot (firebot@moz-F8C1AF63.carolina.res.rr.com)
  147. # [11:37] * ChanServ sets mode: +o firebot
  148. # [11:46] <@tbsaunde> surkov: ugh, is FindWordBoundary() going to stop using GetPosAndText() at some point?
  149. # [11:46] <@surkov> tbsaunde: yes, at some point
  150. # [11:47] <@surkov> tbsaunde: we need to figure out correct behavior, FindWordBoundary returns wrong results because movement form left-to-right and right-to-left is not consistnt
  151. # [11:47] <@surkov> if we figure out consistent behavior on layout side then I think we don't need to fix anything on a11y side
  152. # [11:48] <@tbsaunde> surkov: ok, well right now I'm not clear on what layout stuff this is that needs to be fixed
  153. # [11:49] <@surkov> tbsaunde: yes, first of all we need to decide what behavior is wanted
  154. # [11:49] <@tbsaunde> surkov: so, is there something you wanted me to look into?
  155. # [11:50] <@surkov> tbsaunde: I thought I knew what behavior is wanted, so let's wait until figure out the correct behavior
  156. # [11:50] <@surkov> and after that yes
  157. # [11:52] <@tbsaunde> surkov: ok
  158. # [12:16] * Joins: icaaq (Adium@D2069856.FE797095.222B27F0.IP)
  159. # [13:20] * Joins: maxli (maxli@C03F2759.C70D537E.2FA4EA88.IP)
  160. # [13:22] <@tbsaunde> surkov: those overloaded virtual warnings for GroupPosition() are kind of anoying do you have any real objection to renaming the xpcom method to ScriptableGroupPosition() ?
  161. # [13:26] * Quits: maxli (maxli@C03F2759.C70D537E.2FA4EA88.IP) (Ping timeout)
  162. # [13:48] <@surkov> tbsaunde: disable warning instead?
  163. # [13:49] <@surkov> API change because of compilation warning seems crazy
  164. # [13:50] <@tbsaunde> surkov: I think its an api we'd explicitly prefer people not use
  165. # [13:51] <@surkov> tbsaunde: why? access fu is one example
  166. # [13:52] <@tbsaunde> surkov: I meant only change name in C++
  167. # [13:52] <@surkov> tbsaunde: you mean internal GroupPostion()?
  168. # [13:53] <@surkov> anyway I think I'd prefer to disable warning rather than change method names
  169. # [13:53] <@tbsaunde> surkov: no, I mean change the name of the xpcom method but only for C++
  170. # [13:53] <@surkov> how does it work?
  171. # [13:54] <@tbsaunde> surkov: its just what binaryname() does in xpidl
  172. # [13:54] <@tbsaunde> surkov: js only cares about the index of the function in the vtable not what its called in C++
  173. # [13:54] <@surkov> tbsaunde: why do you want to not disable warning?
  174. # [13:55] <@tbsaunde> surkov: because over loading virtual functions is confusing
  175. # [13:55] <@surkov> I though it's a general practice in c++
  176. # [13:56] <@tbsaunde> surkov: I'd sort of like to rename all the xpcom methods that have better internal methods to discurage they're use
  177. # [13:56] <@tbsaunde> surkov: not really?
  178. # [13:58] <@surkov> tbsaunde: is the idea of different names still needed when different objects are created for xpcom and internal?
  179. # [14:00] <@tbsaunde> surkov: just to fix the warning?
  180. # [14:00] <@surkov> tbsaunde: no because we wanted to have xpcom, internal, msaa, atk impl live on different objects
  181. # [14:01] <@surkov> like we do for xpcom a11y events
  182. # [14:02] <@tbsaunde> surkov: if you have seperate object to implement xpcom interfaces then you don't have overloads of same function anymore
  183. # [14:02] <@surkov> tbsaunde: correct
  184. # [14:03] <@tbsaunde> surkov: so the warning would obviously go away and people would be less likely to use the xpcom method since its on a different object
  185. # [14:03] <@surkov> yep
  186. # [14:04] * Joins: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP)
  187. # [14:06] <@tbsaunde> surkov: but I don't think we'll finish seperating xpcom objects soon so why does it matter?
  188. # [14:08] <@surkov> tbsaunde: so why to bother if warning is going get away (maybe not soon)? just disable it
  189. # [14:11] <@tbsaunde> surkov: it anoys me, I have to read several lines of garbage to see errors
  190. # [14:11] <@surkov> tbsaunde: why not just disable it? it's easy and works as temporary solution well
  191. # [14:12] <@tbsaunde> surkov: because I'd like to see the warning for other cases
  192. # [14:12] * Joins: scott_gonzalez (scott_gonz@moz-91C81A39.hrbgpa.fios.verizon.net)
  193. # [14:13] * @tbsaunde doesn't see why renaming a method that nobody cares about is a big deal
  194. # [14:16] <@surkov> tbsaunde: that warning seems rather useless since says about function overloading
  195. # [14:16] <@surkov> tbsaunde: do you know potential problems the warning says about?
  196. # [14:18] <@tbsaunde> surkov: well, you could end up with really crazy things if you have say virtual foo(int) and virtual foo(bool) and don't define both of them if define one of them on a class
  197. # [14:18] <@tbsaunde> surkov: which I believe we've been bitten by with the windows QI stuff
  198. # [14:20] <@surkov> tbsaunde: ok, if you really want to change binary name then ok but pls don't change all of them
  199. # [14:22] <@tbsaunde> surkov: ok, I wasn't going to bother with that I don't care that much
  200. # [14:22] <@surkov> ok
  201. # [14:37] * Joins: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com)
  202. # [14:50] * Joins: maxli (maxli@F2D29657.F60B0462.67AC9B1.IP)
  203. # [14:52] * Quits: @surkov (surkov@4F778CCB.95871A73.E17943EE.IP) (Quit: surkov)
  204. # [15:54] * Joins: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
  205. # [15:59] * Joins: marcoz (marco.zehe@moz-4DA86096.dip0.t-ipconnect.de)
  206. # [16:00] * ChanServ sets mode: +o marcoz
  207. # [16:00] <@marcoz> Good day all!
  208. # [16:00] <yzen> marcoz: Hi
  209. # [16:04] <yzen> marcoz: i have another question about bug 845870
  210. # [16:04] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=845870 nor, --, ---, yura.zenevich, NEW, [AccessFu] Trust explicitly associated names when speaking certain elements
  211. # [16:04] <@marcoz> yzen: Go right ahead!
  212. # [16:06] <yzen> marcoz: what happens in case of eNameFromSubtreeIfReqRule, as it seems like just including subtree for eNameFromSubtree and explicit does not handle the test case eeejay was concerned with
  213. # [16:06] <yzen> ?
  214. # [16:12] * Quits: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com) (Ping timeout)
  215. # [16:13] * Joins: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com)
  216. # [16:15] <@marcoz> yzen: Hm, I am trying to get this straight in my head first. For example in the case of a table cell, everything inside is supposed to be the name for the cell, right?
  217. # [16:16] <yzen> marcoz: that's what i thought
  218. # [16:16] <@marcoz> So in theory, it should not be any different than a button. But table cells can contain rich content, like whole paragraphs, although this is more likely the case in layout tables than data tables.
  219. # [16:17] <yzen> marcoz: on the other hand for example pushbutton has the name from subtree rule, which means both the explicit name and the subtree text will be in the name, would that be expected ?
  220. # [16:18] <@marcoz> yzen: No it would not. The explicit name overrides the name from sub tree in case of a button. At least that's what I always thought the explicit name was supposed to do. ;)
  221. # [16:18] <@marcoz> yzen: So in other words, the name from sub tree would be ignored as soon as an explicit name was found.
  222. # [16:19] <@marcoz> yzen: s/ignored/not calculated/
  223. # [16:20] <yzen> marcoz: is that in case of a button specifically ?
  224. # [16:21] <@marcoz> yzen: I don't think so. It should be the case for links, for example, too. As soon as there's an explicit name, sub tree info should be ignored.
  225. # [16:24] <yzen> marcoz: i m just referring to yours and eeejay's earlier comments for the bug (i might've misunderstood it): you said "not prune the sub tree if the name has been provided via an explicit means, for roles whole rule calls for name from subtree calculation otherwise"
  226. # [16:24] <yzen> marcoz: as far as i understand you buttons and links are a separate case ?
  227. # [16:25] <@marcoz> yzen: Do *not* prune the tree if the name has *not* been derived from the sub tree. :)
  228. # [16:26] <@marcoz> yzen: For example: A fieldset (groupbox) element gets its name from the legend element. In that case, it may not be explicit, but still not from the whole sub tree, but from one specific element. That's when we do not want the sub tree to be pruned.
  229. # [16:26] <@marcoz> yzen: So, links and buttons should not be a special case at all, but go with the standard FromSubtree rule.
  230. # [16:29] <yzen> marcoz: got it :)
  231. # [16:29] <yzen> *hopes
  232. # [16:31] <@firebot> trev.saunders@gmail.com requested review from surkov.alexander@gm ail.com for attachment 749877 on bug 448235.
  233. # [16:31] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=448235 nor, --, ---, nobody, NEW, Thunderbird replied and forwarded states not exposed via AT-SPI
  234. # [16:35] * Quits: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com) (Ping timeout)
  235. # [16:50] * Joins: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net)
  236. # [17:03] * Joins: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com)
  237. # [17:09] * Joins: xuku (Mibbit@66B2896A.CBF8C476.41967585.IP)
  238. # [17:10] * Quits: xuku (Mibbit@66B2896A.CBF8C476.41967585.IP) (Quit: http://www.mibbit.com ajax IRC Client)
  239. # [17:18] <@firebot> yura.zenevich@gmail.com requested review from marco.zehe@googlema il.com for attachment 749900 on bug 845870.
  240. # [17:18] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=845870 nor, --, ---, yura.zenevich, NEW, [AccessFu] Trust explicitly associated names when speaking certain elements
  241. # [17:18] * Joins: nhirata (anonymous@moz-BE33DA21.fw1.sfo1.mozilla.net)
  242. # [17:19] <@firebot> yura.zenevich@gmail.com requested review from eitan@monotonous.or g for attachment 749902 on bug 845870.
  243. # [17:28] <@firebot> zach.xuku@gmail.com requested review from trev.saunders@gmail .com for attachment 749908 on bug 869845.
  244. # [17:28] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=869845 nor, --, ---, zach.xuku, NEW, FirePlatformEvent shouldn't be virtual
  245. # [17:29] * Joins: xuku (Mibbit@66B2896A.CBF8C476.41967585.IP)
  246. # [17:41] * Quits: Jesse (jruderman@moz-537BCF9.hsd1.ca.comcast.net) (Quit: Jesse)
  247. # [17:50] <@firebot> marco.zehe@googlemail.com granted review for attachment 749900 on bug 845870.
  248. # [17:50] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=845870 nor, --, ---, yura.zenevich, NEW, [AccessFu] Trust explicitly associated names when speaking certain elements
  249. # [17:50] * Quits: icaaq (Adium@D2069856.FE797095.222B27F0.IP) (Quit: Leaving.)
  250. # [17:51] <yzen> marcoz: as for bug 858130, ill carry over the r+'s since i only update the test cases there, is that ok ?
  251. # [17:51] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=858130 nor, --, ---, yura.zenevich, NEW, [AccessFu] Utterances are not stripped of whitespace
  252. # [17:55] * Quits: victorporof (victorporo@10AA7997.880C8D58.79933D60.IP) (Quit: victorporof)
  253. # [17:56] <@firebot> marco.zehe@googlemail.com granted review for attachment 749902 on bug 845870.
  254. # [17:56] <@marcoz> yzen: Yes that's perfectly fine!
  255. # [17:57] <yzen> marcoz: awesome
  256. # [17:57] * Joins: victorporof (victorporo@10AA7997.880C8D58.79933D60.IP)
  257. # [18:04] * Quits: @marcoz (marco.zehe@moz-4DA86096.dip0.t-ipconnect.de) (Quit: Leaving.)
  258. # [18:07] * Quits: xuku (Mibbit@66B2896A.CBF8C476.41967585.IP) (Quit: http://www.mibbit.com ajax IRC Client)
  259. # [18:16] * Joins: fxa90id (fxa90id@moz-D1DD986C.dsl.dynamic.t-mobile.pl)
  260. # [18:20] * Joins: xuku (Mibbit@66B2896A.CBF8C476.41967585.IP)
  261. # [18:31] * Joins: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net)
  262. # [18:31] * khuey|away is now known as khuey
  263. # [18:52] * Quits: xuku (Mibbit@66B2896A.CBF8C476.41967585.IP) (Quit: http://www.mibbit.com ajax IRC Client)
  264. # [19:03] * Joins: Jesse (jruderman@moz-4BE034AB.ptr.us.xo.net)
  265. # [19:06] * Joins: davidb (davidb@moz-BE33DA21.fw1.sfo1.mozilla.net)
  266. # [19:06] * ChanServ sets mode: +qo davidb davidb
  267. # [19:09] * Quits: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP) (Quit: Leaving.)
  268. # [19:38] * Quits: clown (clown@67828CC7.C1A51174.9D42CF23.IP) (Quit: Leaving.)
  269. # [19:41] * Joins: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
  270. # [19:56] <@eeejay> maxli, hody
  271. # [19:56] <maxli> eeejay, hi
  272. # [19:57] <@eeejay> maxli, ready to meet soon? i'll be on skype
  273. # [19:57] <maxli> eeejay, okay will be there in a couple minutes
  274. # [20:00] * Joins: rednaks (rednaks@D6505675.587EEF59.55FFA9B4.IP)
  275. # [20:13] * Joins: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP)
  276. # [20:16] * Joins: xuku (Mibbit@66B2896A.CBF8C476.41967585.IP)
  277. # [20:16] <@firebot> eitan@monotonous.org changed the Status on bug 812463 from NEW to UNCONFIRMED.
  278. # [20:17] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=812463 maj, --, ---, nobody, UNCO, [AccessFu] Some pages cause freezes in Firefox and speech of TalkBack, may even reboot device
  279. # [20:19] * Quits: maxli (maxli@F2D29657.F60B0462.67AC9B1.IP) (Quit: Leaving.)
  280. # [20:20] * Joins: maxli (maxli@F2D29657.F60B0462.67AC9B1.IP)
  281. # [20:24] * Quits: Gijs (gijs@moz-C11B0461.dsl.alice.nl) (Quit: updates)
  282. # [20:27] * Quits: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net) (Ping timeout)
  283. # [20:29] * Joins: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net)
  284. # [20:33] * Quits: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net) (Ping timeout)
  285. # [20:37] * Joins: Gijs (gijs@moz-C11B0461.dsl.alice.nl)
  286. # [20:48] <@firebot> gavin.sharp@gmail.com changed the Component on bug 872109 from DOM to Keyboard Navigation.
  287. # [20:49] <@firebot> gavin.sharp@gmail.com changed the Product on bug 872109 from Core to Firefox.
  288. # [20:49] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=872109 nor, --, ---, nobody, NEW, Request for fewer keys releasing fullscreen/more consistent fullscreen
  289. # [20:51] <@firebot> maxli@maxli.ca cancelled review?(bugmail.mozilla@sta ktrace.com) for attachment 744599 on bug 785144.
  290. # [20:51] <@firebot> maxli@maxli.ca requested feedback from marco.zehe@google mail.com for attachment 749999 on bug 785144.
  291. # [20:51] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=785144 nor, --, ---, maxli, NEW, [AccessFu] Braille doesn't update when navigating through content
  292. # [20:58] * Quits: xuku (Mibbit@66B2896A.CBF8C476.41967585.IP) (Quit: http://www.mibbit.com ajax IRC Client)
  293. # [21:36] * khuey is now known as khuey|away
  294. # [21:57] * Joins: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net)
  295. # [22:01] * Joins: fxa90id_ (fxa90id@moz-236663A7.dsl.dynamic.t-mobile.pl)
  296. # [22:01] * Quits: fxa90id (fxa90id@moz-D1DD986C.dsl.dynamic.t-mobile.pl) (Ping timeout)
  297. # [22:03] * Quits: Jesse (jruderman@moz-4BE034AB.ptr.us.xo.net) (Quit: Jesse)
  298. # [22:04] * Joins: Jesse (jruderman@moz-4BE034AB.ptr.us.xo.net)
  299. # [22:05] * Joins: icaaq (Adium@moz-200DC1CF.customers.ownit.se)
  300. # [22:26] * Quits: Gijs (gijs@moz-C11B0461.dsl.alice.nl) (Quit: poof)
  301. # [22:34] * Quits: rednaks (rednaks@D6505675.587EEF59.55FFA9B4.IP) (Connection timed out)
  302. # [22:35] * Joins: rednaks (rednaks@D6505675.587EEF59.55FFA9B4.IP)
  303. # [22:40] * Quits: maxli (maxli@F2D29657.F60B0462.67AC9B1.IP) (Quit: Leaving.)
  304. # [22:46] <yzen> eeejay: so i tested that all 3 patches together on android and with gaia, and they work well :) , will try to review the code asap
  305. # [22:47] <@eeejay> yzen, awesome!
  306. # [22:47] * @eeejay has been struggling with compiles all morning
  307. # [22:47] <@eeejay> yzen, i may put up a tweaked patch in 10 minutes
  308. # [22:47] <@eeejay> because we are saying byebye to nsIAccessibleCursorable
  309. # [22:49] * Joins: stefanh (stefanh@moz-3EED0162.customers.ownit.se)
  310. # [22:49] * Quits: @firebot (firebot@moz-F8C1AF63.carolina.res.rr.com) (Ping timeout)
  311. # [22:49] <yzen> eeejay: interesting, would the patch for the bug 872338 be still necessary
  312. # [22:49] <yzen> ?
  313. # [22:50] * Quits: rednaks (rednaks@D6505675.587EEF59.55FFA9B4.IP) (Quit: Téléportation !)
  314. # [22:50] <@eeejay> yzen, yeah. it is an ammendment to that patch, just merging virtualCursor into nsIAccessibleDocument
  315. # [22:50] * Quits: @davidb (davidb@moz-BE33DA21.fw1.sfo1.mozilla.net) (Quit: davidb)
  316. # [22:51] <yzen> eeejay: alright, ill wait for the update..
  317. # [22:51] <@eeejay> yzen, also, we don't need to climb up the doc hierarchy to get a vc in Utils.getVirtualCursor
  318. # [22:51] <@eeejay> so i am tweaking that
  319. # [22:52] * Parts: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
  320. # [22:52] <yzen> eeejay: since it will be present for the content.document ?
  321. # [22:53] <@eeejay> yzen, since all document accessibles will have it
  322. # [22:54] <yzen> eeejay: gotcha
  323. # [23:00] <@eeejay> yzen, k. uploaded.
  324. # [23:00] <@eeejay> bbiabn
  325. # [23:01] <yzen> eeejay: thanks!
  326. # [23:01] * Joins: firebot (firebot@moz-F8C1AF63.carolina.res.rr.com)
  327. # [23:01] * ChanServ sets mode: +o firebot
  328. # [23:06] * Joins: davidb (davidb@moz-BE33DA21.fw1.sfo1.mozilla.net)
  329. # [23:06] * ChanServ sets mode: +qo davidb davidb
  330. # [23:06] <stefanh> tbsaunde: hmm, re https://bugzilla.mozilla.org/show_bug.cgi?id=857461#c17, shouldn't that be XULLink instead of XULText since we're dealing with the .text-link binding?
  331. # [23:06] <@firebot> Bug 857461 nor, --, ---, stefanh, ASSI, pinstripe: non-native focus rings are applied for links in chrome
  332. # [23:07] * Quits: @davidb (davidb@moz-BE33DA21.fw1.sfo1.mozilla.net) (Quit: davidb)
  333. # [23:11] * Joins: davidb (davidb@moz-BE33DA21.fw1.sfo1.mozilla.net)
  334. # [23:11] * ChanServ sets mode: +qo davidb davidb
  335. # [23:11] <@tbsaunde> stefanh: you mean the accessible provider constant?
  336. # [23:13] <stefanh> tbsaunde: the accessible type, yes
  337. # [23:13] <stefanh> http://mxr.mozilla.org/mozilla-central/source/toolkit/content/widgets/text.xml#294
  338. # [23:14] <@tbsaunde> stefanh: seems like a reasonable theory
  339. # [23:14] <@tbsaunde> stefanh: can you point me at the xbl binding
  340. # [23:14] <stefanh> the label value is a link
  341. # [23:14] <stefanh> tbsaunde: see the link 4 lines above you .-)
  342. # [23:15] <stefanh> erm 3 lines
  343. # [23:15] <stefanh> tbsaunde: http://mxr.mozilla.org/mozilla-central/source/toolkit/content/widgets/text.xml#294
  344. # [23:16] <@tbsaunde> stefanh: yeah, I see seems that returns xul link accessible provider type already
  345. # [23:17] * Joins: richardschwerdtfeger (richardsch@moz-23BE07E2.telbonet.an)
  346. # [23:18] <richardschwerdtfeger> If anyone can find Eitan he wanted to come the IndieUI call and we are on the call now.
  347. # [23:20] <stefanh> tbsaunde: hm, I don't get why UpdateLabelValue gets called
  348. # [23:20] * Quits: @firebot (firebot@moz-F8C1AF63.carolina.res.rr.com) (Ping timeout)
  349. # [23:20] <stefanh> tbsaunde: I mean the assertion comes from http://mxr.mozilla.org/mozilla-central/source/accessible/src/base/nsAccessibilityService.cpp#490
  350. # [23:21] <@tbsaunde> stefanh: yeah, that's interesting do you maybe change from a label binding to a text-link binding at some point?
  351. # [23:23] <stefanh> tbsaunde: hm
  352. # [23:28] <stefanh> tbsaunde: you mean because I focus the link?
  353. # [23:29] * stefanh is rather lost
  354. # [23:34] <@tbsaunde> stefanh: that could cause it I suppose
  355. # [23:35] <@tbsaunde> stefanh: I'm pretty tired probably better to talk to alex
  356. # [23:35] <stefanh> tbsaunde: ah, ok. I might do that - thanks
  357. # [23:35] * Joins: xuku (Mibbit@66B2896A.CBF8C476.41967585.IP)
  358. # [23:37] * Quits: xuku (Mibbit@66B2896A.CBF8C476.41967585.IP) (Quit: http://www.mibbit.com ajax IRC Client)
  359. # [23:41] * Joins: firebot (firebot@moz-F8C1AF63.carolina.res.rr.com)
  360. # [23:41] * ChanServ sets mode: +o firebot
  361. # [23:44] * Quits: icaaq (Adium@moz-200DC1CF.customers.ownit.se) (Quit: Leaving.)
  362. # [23:45] * Joins: maxli (maxli@C03F2759.C70D537E.2FA4EA88.IP)
  363. # [23:49] * Quits: Jesse (jruderman@moz-4BE034AB.ptr.us.xo.net) (Quit: Jesse)
  364. # [23:50] * Quits: maxli (maxli@C03F2759.C70D537E.2FA4EA88.IP) (Ping timeout)
  365. # [23:51] * Joins: Jesse (jruderman@moz-4BE034AB.ptr.us.xo.net)
  366. # [23:51] * Joins: maxli (maxli@C03F2759.C70D537E.2FA4EA88.IP)
  367. # [23:56] * Quits: richardschwerdtfeger (richardsch@moz-23BE07E2.telbonet.an) (Quit: richardschwerdtfeger)
  368. # [23:57] * Quits: @eeejay (eeejay@moz-B3973587.xen.prgmr.com) (Quit: Coyote finally caught me)
  369. # [23:57] * Joins: eeejay (eeejay@moz-B3973587.xen.prgmr.com)
  370. # Session Close: Thu May 16 00:00:00 2013

The end :)