/irc-logs / mozilla / #accessibility / 2012-02-10 / end

Options:

  1. # Session Start: Fri Feb 10 00:00:00 2012
  2. # Session Ident: #accessibility
  3. # [00:03] * Quits: satdav (satdav@moz-1ECB932B.cable.virginmedia.com) (Connection reset by peer)
  4. # [00:23] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  5. # [00:34] * Quits: aaronlev (aaronlev@moz-CDA191A6.c3-0.arl-ubr1.sbo-arl.ma.cable.rcn.com) (Ping timeout)
  6. # [00:36] * Quits: ehsan (ehsan@F2D29657.F60B0462.67AC9B1.IP) (Input/output error)
  7. # [01:04] * Quits: drexler (chatzilla@moz-2C2B7D1F.hsd1.vt.comcast.net) (Quit: ChatZilla 0.9.88 [Firefox 10.0/20120129021758])
  8. # [01:05] * Joins: davidb (davidb@moz-6F9F653A.dsl.bell.ca)
  9. # [01:05] * ChanServ sets mode: +qo davidb davidb
  10. # [01:08] * Quits: @davidb (davidb@moz-6F9F653A.dsl.bell.ca) (Quit: davidb)
  11. # [01:26] * Quits: aaronlev_ (chatzilla@moz-CDA191A6.c3-0.arl-ubr1.sbo-arl.ma.cable.rcn.com) (Ping timeout)
  12. # [01:44] * Joins: aaronlev (chatzilla@moz-9058091D.bstnma.fios.verizon.net)
  13. # [01:50] <@firebot> andrew.quartey@gmail.com requested review from surkov.alexander@gm ail.com for attachment 595909 on bug 634829.
  14. # [01:50] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=634829 nor, --, ---, andrew.quartey, NEW, remove nsIAccessibleWin32Object interface
  15. # [02:05] * Joins: surkov (surkov@36D639C1.44A4068D.222B27F0.IP)
  16. # [02:05] * ChanServ sets mode: +o surkov
  17. # [02:10] * khuey is now known as khuey|away
  18. # [02:15] * Joins: davidb (davidb@moz-6F9F653A.dsl.bell.ca)
  19. # [02:15] * ChanServ sets mode: +qo davidb davidb
  20. # [02:19] <@hub> davidb: satdav was looking for you, but he is gone
  21. # [02:20] <@davidb> hub: thanks. do you know what about?
  22. # [02:21] <@davidb> !seen satdav
  23. # [02:21] <@firebot> satdav was last seen 2 hours, 37 minutes and 16 seconds ago, saying 'same ' in #accessibility.
  24. # [02:29] <Jamie> Hm. From what I've read, telemetry doesn't uniquely identify users (and therefore copies of Firefox) in any way. so how do you make the data about a11y, for example, useful?
  25. # [02:32] <Jamie> I'm guessing stats on, for example, ISimpleDom* usage are sent at startup, so they're only sent once per session
  26. # [02:33] <Jamie> so you at least know how often ISimple* is used compared to everything else
  27. # [02:33] <@hub> davidb: nope.
  28. # [02:33] <@hub> davidb: he didn'
  29. # [02:33] <@hub> t say
  30. # [02:33] <@davidb> Jamie: right
  31. # [02:33] <@davidb> hub: ok thanks
  32. # [02:34] <@davidb> Jamie: but the dashboard doesn't show screen reader XYZ uses ISimpleDOM
  33. # [02:34] <Jamie> davidb: same for screen reader injection; only sent at startup?
  34. # [02:34] <@davidb> that might be in the database but I have to ask.
  35. # [02:34] <@davidb> I'm not sure when the telemetry ping is sent
  36. # [02:34] <Jamie> ok, but only once per session I assume
  37. # [02:35] <@davidb> but we prepare the data as soon as we detect it
  38. # [02:35] <Jamie> right
  39. # [02:35] <@davidb> i'm not sure… it might be every N hours.
  40. # [02:35] * Jamie nods
  41. # [02:35] <@davidb> Moz is still sort of newish to this
  42. # [02:36] <Jamie> right, so just theorising (so I understand it better), if all ISimple* users happened ot use firefox for only an hour at a time but all IA2 users used it for 15 hours at a time, you'd get very biased stats
  43. # [02:36] <@davidb> yeah they'd be unfairly equal
  44. # [02:36] <Jamie> nm. do you have any idea how Moz gains some idea of how many real firefox "users" there are?
  45. # [02:37] <@davidb> i think it might have to do with an update or add-on check
  46. # [02:37] <@davidb> i barely care enough :)
  47. # [02:37] <Jamie> sure. I'm asking the wrong people. I just don't know who to ask :)
  48. # [02:37] <@davidb> hmm
  49. # [02:37] <Jamie> trying to figure out how we can get this sort of data for NVDA
  50. # [02:37] <@davidb> #metrics
  51. # [02:38] <Jamie> so technically it's not at amozilla question at all. it's just that someone at mozilla clearly has a clue and I don't
  52. # [02:38] <@davidb> Jamie: it is early days but it looks like about 60% nvda, 40% jaws, and 11% WE
  53. # [02:38] <@davidb> FF users
  54. # [02:38] * Quits: aaronlev (chatzilla@moz-9058091D.bstnma.fios.verizon.net) (Client exited)
  55. # [02:38] <@davidb> and it seems to suggest the presence of more than one sr at times
  56. # [02:38] <Jamie> interesting
  57. # [02:39] <@davidb> this is a sampling of early adopters that allow telemetry :)
  58. # [02:39] <@davidb> it is opt-in
  59. # [02:39] <@davidb> so I would expect it skewed towards NVDA
  60. # [02:39] * Jamie nods; I only enabled it after Marco's blog post myself
  61. # [02:39] <@davidb> spread the word
  62. # [02:39] <@davidb> if you agree it makes sense
  63. # [02:39] <Jamie> it does. I did retweet that I believe
  64. # [02:40] * @davidb nods
  65. # [02:41] * Joins: ehsan (ehsan@F2D29657.F60B0462.67AC9B1.IP)
  66. # [02:57] <@firebot> surkov.alexander@gmail.com denied review for attachment 595909 on bug 634829.
  67. # [02:57] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=634829 nor, --, ---, andrew.quartey, NEW, remove nsIAccessibleWin32Object interface
  68. # [02:59] * khuey|away is now known as khuey
  69. # [03:04] * Joins: drexler (chatzilla@moz-2C2B7D1F.hsd1.vt.comcast.net)
  70. # [03:33] * Quits: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com) (Quit: nhirata)
  71. # [03:43] * Quits: @hub (hub@21B7B9F2.B87E9213.6E712CE2.IP) (Ping timeout)
  72. # [03:49] * shorlander is now known as shorlander-away
  73. # [03:56] <@tbsaunde> surkov: fyi I plan to do your reviews after midnight my time
  74. # [03:57] <@tbsaunde> and will be around then but have to work till then
  75. # [03:57] <@surkov> sounds good
  76. # [04:00] * Quits: @davidb (davidb@moz-6F9F653A.dsl.bell.ca) (Quit: davidb)
  77. # [04:02] * Joins: hub (hub@moz-E2FCA694.figuiere.net)
  78. # [04:02] * ChanServ sets mode: +o hub
  79. # [04:30] <@tbsaunde> surkov: I'm not sure I've ever heard someone call writing sml at high volocity grat but ok ;)
  80. # [04:34] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
  81. # [04:37] <@surkov> tbsaunde: I failed to understand: "someone call writing sml at high volocity grat"
  82. # [04:37] * Quits: shorlander-away (shorlander@moz-853043D6.dhcp.insightbb.com) (Ping timeout)
  83. # [04:38] <@firebot> andrew.quartey@gmail.com requested review from surkov.alexander@gm ail.com for attachment 595941 on bug 634829.
  84. # [04:38] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=634829 nor, --, ---, andrew.quartey, NEW, remove nsIAccessibleWin32Object interface
  85. # [04:40] <@tbsaunde> surkov: its a programing lnaguage I don't really like
  86. # [04:40] <@tbsaunde> sml is
  87. # [04:40] <@tbsaunde> and I need to write a lot fast
  88. # [04:40] <@surkov> got it
  89. # [04:44] <@firebot> surkov.alexander@gmail.com granted review for attachment 595941 on bug 634829.
  90. # [04:55] * Joins: shorlander-away (shorlander@moz-853043D6.dhcp.insightbb.com)
  91. # [04:58] * Joins: ptheriault (ptheriault@moz-4BE034AB.ptr.us.xo.net)
  92. # [05:08] <@firebot> andrew.quartey@gmail.com requested review from surkov.alexander@gm ail.com for attachment 595949 on bug 634829.
  93. # [05:08] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=634829 nor, --, ---, andrew.quartey, NEW, remove nsIAccessibleWin32Object interface
  94. # [05:11] <@firebot> surkov.alexander@gmail.com granted review for attachment 595949 on bug 634829.
  95. # [05:16] * Quits: ehsan (ehsan@F2D29657.F60B0462.67AC9B1.IP) (Input/output error)
  96. # [05:18] * Quits: shorlander-away (shorlander@moz-853043D6.dhcp.insightbb.com) (Quit: Quit)
  97. # [05:40] * Joins: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP)
  98. # [06:01] * Quits: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP) (Input/output error)
  99. # [06:16] * Quits: drexler (chatzilla@moz-2C2B7D1F.hsd1.vt.comcast.net) (Quit: ChatZilla 0.9.88 [Firefox 10.0/20120129021758])
  100. # [06:16] <@tbsaunde> well, that wasn't total loss but pretty bad
  101. # [06:16] * Joins: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP)
  102. # [06:24] <@tbsaunde> surkov: btw for bug 634829 I believe MOZ_OVERRIDE is prefered to NS_OVERRIDE
  103. # [06:24] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=634829 nor, --, ---, andrew.quartey, NEW, remove nsIAccessibleWin32Object interface
  104. # [06:24] <@surkov> ok, I'll fix it
  105. # [06:25] <@tbsaunde> ok
  106. # [06:49] * Joins: aaronlev (aaronlev@moz-9058091D.bstnma.fios.verizon.net)
  107. # [06:52] * Quits: @surkov (surkov@36D639C1.44A4068D.222B27F0.IP) (Ping timeout)
  108. # [06:58] * Joins: surkov (surkov@94877EB3.B3F96E9C.3AF1D72D.IP)
  109. # [06:58] * ChanServ sets mode: +o surkov
  110. # [07:26] * Quits: aaronlev (aaronlev@moz-9058091D.bstnma.fios.verizon.net) (Quit: aaronlev)
  111. # [07:30] * khuey is now known as khuey|away
  112. # [07:35] <Jamie> surkov: any idea why bug 677154 only occurs with certain profiles and not others?
  113. # [07:35] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=677154 maj, P1, ---, nobody, NEW, Detached document accessibility tree
  114. # [07:41] <@surkov> Jamie: that depends on view area
  115. # [07:41] <@surkov> Jamie: in your profile Firefox is running maximized
  116. # [07:41] <@surkov> so you can see a bug
  117. # [07:41] <@surkov> I keep Firefox window small to run inspect tool so I wasn't able to reproduce the bug
  118. # [07:50] <@firebot> trev.saunders@gmail.com granted review for attachment 595671 on bug 724452.
  119. # [07:50] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=724452 nor, --, ---, jigneshhk1992, NEW, use nsFocusManager::GetFocusManager() more
  120. # [08:04] <@tbsaunde> surkov: so, your patch for bug 725581 I'm really suprised some of those checks aren't redundant
  121. # [08:04] <@tbsaunde> especially the state one
  122. # [08:04] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=725581 nor, --, ---, surkov.alexander, ASSI, caretOffset for textarea should be -1 when textarea doesn't have a focus
  123. # [08:05] <@tbsaunde> also can't you merge the chekc after what you add into it?
  124. # [08:11] <@surkov> I'm not sure how
  125. # [08:13] <@tbsaunde> surkov: ugh
  126. # [08:13] <@tbsaunde> surkov: any idea why have you investigated?
  127. # [08:14] <@surkov> tbsaunde: tried to enable old mochitests
  128. # [08:18] <@tbsaunde> surkov: and they don't pass without all of that?
  129. # [08:20] <@surkov> sure
  130. # [08:21] <@tbsaunde> surkov: I'm not sure what you mean by that
  131. # [08:21] <@surkov> tbsaunde: they don't pass without code changes
  132. # [08:21] <@tbsaunde> surkov: ok, did you debug why?
  133. # [08:22] <@surkov> I'll put comment into bug for history
  134. # [08:22] <@tbsaunde> ok
  135. # [08:23] * @tbsaunde doesnt' really like here is crazy patches without explanation of why they need to be that way
  136. # [08:23] <@tbsaunde> especially when needing that seems to imply crazy state of the world
  137. # [08:27] <@surkov> tbsaunde: done
  138. # [08:27] <@surkov> tbsaunde: crazy patch contains comments
  139. # [08:27] <@surkov> that means they should be improved and I wait for your feedback how to do that :)
  140. # [08:28] <@tbsaunde> ok
  141. # [08:36] <@tbsaunde> surkov: so, buzilla is just hainging for me, but we have channel logs so there's still history :)
  142. # [08:37] <@surkov> when I digging through bugs then I don't check irc log usually :)
  143. # [08:37] <@surkov> here's the comment
  144. # [08:37] <@surkov> Since elements like input or textarea has own selection controller then selection we get is different from documents one. Since selection is turned into caret position then we can get result different from -1 for these controls if document is focused (see eContainedByFocus check in nsHyperTextAccessible::GetCaretOffset).
  145. # [08:38] <@tbsaunde> surkov: so, I'd be fine with changing the existing focus test in that patch to something like if (focusDisp == eNone || (IsDoc() && focusDisp != eThis)) return NS_OK
  146. # [08:38] <@tbsaunde> surkov: yeah, bugzilla being slow is :(
  147. # [08:39] <@surkov> tbsaunde: I wanted to avoid tree traversal when I don't need it
  148. # [08:41] <@tbsaunde> surkov: unless I miss something you wouldn't be adding tree traversal sine we already get the FocusDisp
  149. # [08:42] <@surkov> tbsaunde: exactly, that's why focusdisp is after my check
  150. # [08:42] <@tbsaunde> surkov: oh, I see
  151. # [08:42] <@tbsaunde> you want to reduce how often we do it
  152. # [08:44] * Quits: @firebot (firebot@moz-67BF9FE6.carolina.res.rr.com) (Ping timeout)
  153. # [08:45] <@surkov> yeah, especially for case of -1
  154. # [08:45] <@tbsaunde> surkov: ok, gues that makes sense
  155. # [08:45] <@surkov> good
  156. # [08:45] <@tbsaunde> surkov: btw it occurs to me we could remove shared code from IsFocused() and FocusedAccessible()
  157. # [08:46] <@surkov> oh, really, iirc I had some problems
  158. # [08:46] <@surkov> there's XXX comment there
  159. # [08:47] <@tbsaunde> surkov: weird, looks like they're the same
  160. # [08:47] <@tbsaunde> at a quick look
  161. # [08:47] <@surkov> they are almost the same
  162. # [08:47] <@surkov> but called at different time may give different results
  163. # [08:48] * Joins: firebot (firebot@moz-67BF9FE6.carolina.res.rr.com)
  164. # [08:48] * ChanServ sets mode: +o firebot
  165. # [08:48] <@tbsaunde> surkov: sure, but can't we define IsFocused() as return aAccessible == FocusedAccessible() ?
  166. # [08:49] <@tbsaunde> surkov: so, one more question about the patch
  167. # [08:49] <@surkov> tbsaunde: no, we can't
  168. # [08:49] <@surkov> that's funny though
  169. # [08:49] <@tbsaunde> why do you want to continue the function in the case that we aren't focusable?
  170. # [08:50] <@surkov> like handling HTML:p?
  171. # [08:51] <@tbsaunde> surkov: oh, I see because FocusedAccessible() doesn't do the same doc check missed that first time through
  172. # [08:52] <@tbsaunde> surkov: oh, so things like caret nav mod
  173. # [08:52] <@tbsaunde> I gues that makes sense
  174. # [08:53] <@surkov> right
  175. # [08:57] <@tbsaunde> surkov: so, the thing is to avoid the tree traversal you call State() which on first gues might be heavier than what you skip
  176. # [08:58] <@tbsaunde> but I don't really know which makes me a little sad itself
  177. # [08:58] <@surkov> tbsaunde: yeah, State() but Im keeping in mind to replace heavy State() on small State() versions like VisibilityState()
  178. # [08:59] <@surkov> not good reason though :)
  179. # [09:00] <@tbsaunde> surkov: what isn't a good reason for what?
  180. # [09:00] <@tbsaunde> surkov: breaking up State() into light ones seems reasonable
  181. # [09:01] <@surkov> tbsaunde: I meant future State() changes is not good reason to use State() as is right now
  182. # [09:01] <@tbsaunde> surkov: oh
  183. # [09:02] * Quits: @firebot (firebot@moz-67BF9FE6.carolina.res.rr.com) (Connection reset by peer)
  184. # [09:03] <@tbsaunde> surkov: so, having some sort of arg to IsInOrContainsFocus() to only check certain conditions might make sense
  185. # [09:04] <@tbsaunde> surkov: but I think you've aguasted my would x be nicer supply for that issue though
  186. # [09:05] <@surkov> sometimes I can't translate you, like last sentence
  187. # [09:05] <@tbsaunde> surkov: I mean I don't think I have many more ideas for improving your patch
  188. # [09:06] <@surkov> but if you could put IsINorContainsFocus idea into code snippet then it should be nice
  189. # [09:06] <@surkov> for me it's quite plain method
  190. # [09:07] <@tbsaunde> surkov: but I wonder since the current focus check should mean that html:p can only give you a real text offset if a parent is focused which I gues sort of makes sense, but I'm not really sure what happens in caret nav mode when you are in a html:p
  191. # [09:07] <@surkov> we should return valid caret in second case too
  192. # [09:07] * @tbsaunde tries to write sample
  193. # [09:08] <@tbsaunde> surkov: I'm not sure what your second case is
  194. # [09:08] <@surkov> caret inside html:p
  195. # [09:13] * Joins: firebot (firebot@moz-67BF9FE6.carolina.res.rr.com)
  196. # [09:13] * ChanServ sets mode: +o firebot
  197. # [09:14] * Joins: MarcoZ (marco.zehe@moz-76FEC283.dip.t-dialin.net)
  198. # [09:14] * ChanServ sets mode: +o MarcoZ
  199. # [09:15] * Quits: @hub (hub@moz-E2FCA694.figuiere.net) (Ping timeout)
  200. # [09:15] <@tbsaunde> surkov: well, I gues you don't change that case so
  201. # [09:15] <@tbsaunde> surkov: http://paste.debian.net/155787/ may help with what I was suggesting, but I'm not sure i like it
  202. # [09:17] <@surkov> sounds like GetFocusDisp(accessible, true) is quite useless
  203. # [09:18] <@surkov> it's equivalent to FocusMgr()->IsFOcused(accessible)
  204. # [09:18] <@MarcoZ> Good day all!
  205. # [09:19] <@tbsaunde> surkov: yeah, I was thinking it was useful if you say didn't care if you were contained by focus
  206. # [09:19] <@tbsaunde> so, if you only check if you are focused or a kid is
  207. # [09:19] <@tbsaunde> but that isn't god enough here
  208. # [09:19] <@tbsaunde> so I am silly
  209. # [09:20] <@tbsaunde> and the only question is a State() cal better than aIsInOrContainsFocus() call
  210. # [09:21] <@firebot> marco.zehe@googlemail.com changed the Component on bug 725725 from Disability Access to Untriaged.
  211. # [09:21] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=725725 nor, --, ---, nobody, RESO DUPLICATE, Javascript Security Error 1000 blocks functionalities like opening links in many website (including
  212. # [09:36] <@tbsaunde> surkov: ^
  213. # [09:36] <@surkov> I don't know
  214. # [09:50] <@firebot> trev.saunders@gmail.com granted review for attachment 595669 on bug 725581.
  215. # [09:50] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=725581 nor, --, ---, surkov.alexander, ASSI, caretOffset for textarea should be -1 when textarea doesn't have a focus
  216. # [09:57] <Jamie> surkov: oh. heh. hard to believe it was just that. of course, I wouldn't have noticed that myself unless I'd thought to check specifically :)
  217. # [09:57] <Jamie> surkov: re maximised
  218. # [09:58] <@surkov> Jamie: it wasn't easy for me as well to conculd
  219. # [09:58] <@surkov> this
  220. # [10:08] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  221. # [10:21] * Joins: victorporof (victorporo@C092FEB2.1C233438.79933D60.IP)
  222. # [10:22] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  223. # [10:23] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  224. # [10:49] * Quits: Jamie (jamie@moz-CA26021.jantrid.net) (Ping timeout)
  225. # [11:03] * Joins: Mic (Benedikt@moz-CFE77E7A.superkabel.de)
  226. # [11:24] <@firebot> jigneshhk1992@gmail.com requested review from trev.saunders@gmail .com for attachment 595997 on bug 724452.
  227. # [11:24] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=724452 nor, --, ---, jigneshhk1992, NEW, use nsFocusManager::GetFocusManager() more
  228. # [11:24] <@tbsaunde> surkov: want to take ^? I'm about to head out
  229. # [11:25] <@surkov> tbsaunde: I'm fine with that
  230. # [11:25] <@tbsaunde> thx
  231. # [11:25] * Parts: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  232. # [11:25] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  233. # [11:47] * Quits: victorporof (victorporo@C092FEB2.1C233438.79933D60.IP) (Connection reset by peer)
  234. # [11:47] * Joins: victorporo (victorporo@C092FEB2.1C233438.79933D60.IP)
  235. # [11:50] * Quits: victorporo (victorporo@C092FEB2.1C233438.79933D60.IP) (Connection reset by peer)
  236. # [11:50] * Joins: victorporof (victorporo@C092FEB2.1C233438.79933D60.IP)
  237. # [11:59] * Quits: ptheriault (ptheriault@moz-4BE034AB.ptr.us.xo.net) (Connection reset by peer)
  238. # [11:59] * Joins: ptheriault_ (ptheriault@moz-4BE034AB.ptr.us.xo.net)
  239. # [12:00] * Quits: ptheriault_ (ptheriault@moz-4BE034AB.ptr.us.xo.net) (Connection reset by peer)
  240. # [12:26] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  241. # [12:47] * Quits: victorporof (victorporo@C092FEB2.1C233438.79933D60.IP) (Connection reset by peer)
  242. # [12:47] * Joins: victorporo (victorporo@C092FEB2.1C233438.79933D60.IP)
  243. # [12:58] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  244. # [13:02] * Quits: Mic (Benedikt@moz-CFE77E7A.superkabel.de) (Quit: Instantbird 1.2a1pre)
  245. # [13:08] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  246. # [13:16] * Quits: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP) (Input/output error)
  247. # [13:25] <@firebot> surkov.alexander@gmail.com requested review from hub@mozilla.com for attachment 595997 on bug 724452.
  248. # [13:25] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=724452 nor, --, ---, jigneshhk1992, NEW, use nsFocusManager::GetFocusManager() more
  249. # [13:26] <@surkov> tbsaunde: I stole your review request
  250. # [13:26] <@MarcoZ> surkov: Do you know when your latest DOMi enhancements will make it into an extension people can actually use?
  251. # [13:27] <@surkov> I use it now and use it for a long time, that really depends on people, its UI is not very friendly and accessible but I should say I implement features that help me in debugging
  252. # [13:27] <@surkov> MarcoZ: ^
  253. # [13:28] <@surkov> if David will get DOMi contributors then I bet it'll be a great a11y tool
  254. # [13:29] <@surkov> MarcoZ: right now DOMi allows you what platform tool don't allow you like custom event handlers and making accessible tree snapshots. And that's really great for debugging of ARIA widgets
  255. # [13:34] * Quits: @jprmc (jprmc@moz-7F2FF3EB.cpe.net.cable.rogers.com) (Ping timeout)
  256. # [13:37] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  257. # [13:44] * Joins: askalski (akuda@moz-4C8A107E.pool85-48-91.dynamic.orange.es)
  258. # [13:44] * ChanServ sets mode: +o askalski
  259. # [13:45] * Quits: @askalski (akuda@moz-4C8A107E.pool85-48-91.dynamic.orange.es) (Quit: Wychodzi)
  260. # [13:58] * Joins: askalski (akuda@moz-4C8A107E.pool85-48-91.dynamic.orange.es)
  261. # [13:58] * ChanServ sets mode: +o askalski
  262. # [13:58] * Quits: @askalski (akuda@moz-4C8A107E.pool85-48-91.dynamic.orange.es) (Quit: Wychodzi)
  263. # [14:14] <@firebot> bmo@edmorley.co.uk changed the Status on bug 721947 from NEW to RESOLVED.
  264. # [14:14] <@firebot> bmo@edmorley.co.uk set the Resolution field on bug 721947 to FIXED.
  265. # [14:14] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=721947 nor, --, mozilla13, hub, RESO FIXED, don't use nsIWeakShell
  266. # [14:14] <@firebot> bmo@edmorley.co.uk changed the Status on bug 673405 from NEW to RESOLVED.
  267. # [14:14] <@firebot> bmo@edmorley.co.uk set the Resolution field on bug 673405 to FIXED.
  268. # [14:14] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=673405 cri, --, mozilla13, hub, NEW, Rename GetDocAccessible() to Document()
  269. # [14:14] <@firebot> bmo@edmorley.co.uk changed the Status on bug 672504 from ASSIGNED to RESOLVED.
  270. # [14:14] <@firebot> bmo@edmorley.co.uk set the Resolution field on bug 672504 to FIXED.
  271. # [14:14] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=672504 nor, --, mozilla13, hub, RESO FIXED, Don't keep pointer to weak presshell in accessible
  272. # [14:19] <@firebot> vlad.mozbugs@gmail.com set status-firefox11 to verified on bug 693343.
  273. # [14:19] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=693343 maj, --, mozilla12, trev.saunders, RESO FIXED, accessibility always disabled in GNOME 3 unless GNOME_ACCESSIBILITY is set
  274. # [14:33] <@MarcoZ> Yay hub's patches finally landed on central!
  275. # [14:34] <@MarcoZ> surkov: OK I understand! So you develop these features in DOMi, test them for a while, and then submit the patches for review so they don't get lost and get in the code.
  276. # [14:49] <@surkov> MarcoZ: that's right
  277. # [14:54] * Joins: jprmc (jprmc@29DDBCC5.5BCEC6DB.DA78B690.IP)
  278. # [14:54] * ChanServ sets mode: +o jprmc
  279. # [15:02] * Quits: @jprmc (jprmc@29DDBCC5.5BCEC6DB.DA78B690.IP) (Ping timeout)
  280. # [15:13] * Quits: @surkov (surkov@94877EB3.B3F96E9C.3AF1D72D.IP) (Ping timeout)
  281. # [15:16] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  282. # [15:17] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  283. # [15:19] * khuey|away is now known as khuey
  284. # [15:23] * Joins: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
  285. # [15:24] * Joins: surkov (surkov@37A43653.44A4068D.222B27F0.IP)
  286. # [15:24] * ChanServ sets mode: +o surkov
  287. # [15:27] * Joins: davidb (davidb@F2D29657.F60B0462.67AC9B1.IP)
  288. # [15:27] * ChanServ sets mode: +qo davidb davidb
  289. # [15:28] <@davidb> hi all!
  290. # [15:29] <@MarcoZ> Hi davidb!
  291. # [15:32] * @MarcoZ is refreshing his Windows and Mac builds after the recent code changes that finally landed. :)
  292. # [15:35] <@davidb> surkov: nice blog post
  293. # [15:35] <@surkov> davidb: thanks!
  294. # [15:35] <@davidb> surkov: i was thinking you could add a "Looking for contributors" at the end :)
  295. # [15:36] <@surkov> davidb: yeah, maybe next time :) that was supposed to be overview
  296. # [15:36] <@davidb> makes sense
  297. # [15:40] <@MarcoZ> I believe a call out for contributors is always worth its own blog post. easily gets missed otherwise, surkov, davidb.
  298. # [15:41] <@surkov> MarcoZ: I agree, I think I will post about DOMi updates and sure I will say about contributions, probably I should do good-first-bugs as well
  299. # [15:50] <@davidb> yeah
  300. # [15:51] <@davidb> it doesn't hurt any of us to blog about how to get involved. details are important.
  301. # [15:52] * Quits: victorporo (victorporo@C092FEB2.1C233438.79933D60.IP) (Connection reset by peer)
  302. # [15:52] * Joins: victorporof (victorporo@C092FEB2.1C233438.79933D60.IP)
  303. # [15:57] * Joins: jprmc (jprmc@F2D29657.F60B0462.67AC9B1.IP)
  304. # [15:57] * ChanServ sets mode: +o jprmc
  305. # [15:57] * Quits: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp) (Quit: mike5w3c)
  306. # [15:58] * Joins: skyler_brungardt (skyler@moz-9FA3C392.washdc.fios.verizon.net)
  307. # [15:58] <@tbsaunde> surkov: ok, though looks you could just cancel it since its the same patch I already reviewed just only for accessible/
  308. # [15:59] <@tbsaunde> it might be worth if hub looks if using nsFocusManger instead of nsIFocusManager lets us use nicer methods though
  309. # [15:59] <@surkov> yep
  310. # [16:11] * Joins: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp)
  311. # [16:20] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  312. # [16:39] * Joins: ehsan (ehsan@F2D29657.F60B0462.67AC9B1.IP)
  313. # [16:45] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
  314. # [16:51] * Joins: ptheriault (ptheriault@moz-4BE034AB.ptr.us.xo.net)
  315. # [16:56] * Quits: skyler_brungardt (skyler@moz-9FA3C392.washdc.fios.verizon.net) (Quit: skyler_brungardt)
  316. # [16:57] <@firebot> New Core - Disability Access APIs bug 725998 filed by surkov.alexander@gmail.com.
  317. # [16:57] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=725998 nor, --, ---, nobody, NEW, get rid nsAccessibilityService::GetAccessibleOrContainer and GetContainerAccessible
  318. # [17:01] * @davidb files https://bugs.eclipse.org/bugs/show_bug.cgi?id=371250
  319. # [17:05] * @davidb adds another comment
  320. # [17:08] <@firebot> bolterbugz@gmail.com changed the Status on bug 688161 from NEW to RESOLVED.
  321. # [17:08] <@firebot> bolterbugz@gmail.com set the Resolution field on bug 688161 to WONTFIX.
  322. # [17:08] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=688161 nor, --, ---, nobody, RESO WONTFIX, Automatically set scratchpad to default text editor if a11y instantiated.
  323. # [17:10] <@firebot> New Core - Disability Access APIs bug 726005 filed by surkov.alexander@gmail.com.
  324. # [17:10] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=726005 nor, --, ---, nobody, NEW, IDRefsIterator::Next() should use nsDocAccessible::GetAccessible()
  325. # [17:10] * Joins: hub (hub@83874EA1.EB7C1AF9.6F478678.IP)
  326. # [17:10] * ChanServ sets mode: +o hub
  327. # [17:10] * Joins: askalski (akuda@moz-104CC309.mv.mozilla.com)
  328. # [17:10] * ChanServ sets mode: +o askalski
  329. # [17:12] <@askalski> surkov, I got the final answer on LED
  330. # [17:13] <@askalski> most approximation algorithms are non-practical
  331. # [17:13] <@surkov> ok
  332. # [17:13] <@askalski> there are two available solutions
  333. # [17:13] <@askalski> one O(min(n,m)*d) where d is the distance (d <= max(n,m))
  334. # [17:14] <@askalski> in linear memory
  335. # [17:14] <@askalski> and O(n + d^2) which is good for very similar strings
  336. # [17:14] <@askalski> and is a bit complex in implementation
  337. # [17:14] <@askalski> I guess I go with the first one
  338. # [17:15] <@davidb> askalski: can you dump your findings into the bug?
  339. # [17:15] <@surkov> I think we should go with 1st
  340. # [17:16] <@surkov> tbsaunde: ^
  341. # [17:16] <@askalski> davidb, OK
  342. # [17:17] <@davidb> askalski: you would enjoy working in the toronto office i think
  343. # [17:18] <@askalski> davidb, yes I would. but just today I received a very bad e-mail from my promotor (thesis supervisor). I panic a bit. I'll go with that to the dean
  344. # [17:19] <@askalski> davidb, I hope this can be handled via e-mail.
  345. # [17:19] * @MarcoZ needs to go a bit earlier today, have an appointment with a potential housekeeping agency in 20.
  346. # [17:19] <@askalski> davidb, will keep you posted
  347. # [17:19] <@davidb> ok
  348. # [17:19] <@davidb> MarcoZ: roger
  349. # [17:19] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  350. # [17:19] * Quits: @MarcoZ (marco.zehe@moz-76FEC283.dip.t-dialin.net) (Quit: Feierabend, ja des is so a scheeeenes Wort! :))
  351. # [17:20] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  352. # [17:26] * Quits: @askalski (akuda@moz-104CC309.mv.mozilla.com) (Quit: Wychodzi)
  353. # [17:30] * Joins: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com)
  354. # [17:39] <@firebot> jigneshhk1992@gmail.com requested review from surkov.alexander@gm ail.com for attachment 596058 on bug 725998.
  355. # [17:39] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=725998 nor, --, ---, nobody, NEW, get rid nsAccessibilityService::GetAccessibleOrContainer and GetContainerAccessible
  356. # [17:43] * Quits: ptheriault (ptheriault@moz-4BE034AB.ptr.us.xo.net) (Quit: ptheriault)
  357. # [17:44] <@hub> aren't bug 725998 and bug 725572 the same ?
  358. # [17:45] <@hub> ah i'm an idiot
  359. # [17:45] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=725572 nor, --, ---, nobody, NEW, Remove nsIAccessibilityService::GetAccessible()
  360. # [17:45] <@hub> almost
  361. # [17:45] * @hub needs more coffee
  362. # [17:47] * Joins: drexler (chatzilla@moz-2C2B7D1F.hsd1.vt.comcast.net)
  363. # [17:48] <@hub> oh and third time was a charm
  364. # [17:48] * Quits: victorporof (victorporo@C092FEB2.1C233438.79933D60.IP) (Quit: Leaving...)
  365. # [17:50] <@firebot> surkov.alexander@gmail.com cancelled review?(surkov.alexander@gm ail.com) for attachment 596058 on bug 725998.
  366. # [17:50] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=725998 nor, --, ---, nobody, NEW, get rid nsAccessibilityService::GetAccessibleOrContainer and GetContainerAccessible
  367. # [17:51] <@surkov> hi, hub
  368. # [17:52] <@surkov> you have review request in inbox :)
  369. # [17:53] * khuey is now known as khuey|away
  370. # [17:57] * Joins: shorlander (shorlander@moz-853043D6.dhcp.insightbb.com)
  371. # [17:57] * Joins: ptheriault (ptheriault@moz-4BE034AB.ptr.us.xo.net)
  372. # [17:59] <@tbsaunde> surkov: algorithm seems fine
  373. # [17:59] <@surkov> ok, thx
  374. # [17:59] <@hub> surkov: I saw
  375. # [17:59] <@hub> thanks
  376. # [17:59] <@surkov> yw :)
  377. # [18:06] <@firebot> hub@mozilla.com granted review for attachment 595997 on bug 724452.
  378. # [18:06] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=724452 nor, --, ---, jigneshhk1992, NEW, use nsFocusManager::GetFocusManager() more
  379. # [18:06] <@tbsaunde> surkov: your plan for the IDrefsIteraotr isn't quiet clear to me from the bug (I mean how to impleement it)
  380. # [18:07] <@surkov> IDRefIterator takes nsDocAccessible, keeps as member and then Next() does mDocument->GetAccessible
  381. # [18:07] <@tbsaunde> surkov: so it would take a third argument?
  382. # [18:07] <@surkov> tbsaunde: yes
  383. # [18:09] <@tbsaunde> surkov: feels sort of redundant, but I gues its ok
  384. # [18:09] <@tbsaunde> the doc accessible and the content document
  385. # [18:09] <@tbsaunde> but those objects are short lived etc so whatever
  386. # [18:09] <@surkov> tbsaunde: content document may be null
  387. # [18:09] <@surkov> if xml document is used
  388. # [18:09] <@surkov> yeah, not super nice solution
  389. # [18:09] <@surkov> but can't think better
  390. # [18:10] <@firebot> jigneshhk1992@gmail.com requested review from surkov.alexander@gm ail.com for attachment 596062 on bug 725998.
  391. # [18:10] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=725998 nor, --, ---, nobody, NEW, get rid nsAccessibilityService::GetAccessibleOrContainer and GetContainerAccessible
  392. # [18:10] <@tbsaunde> surkov: true, but then we have an mXBLDocument no?
  393. # [18:10] <@surkov> right
  394. # [18:15] <@firebot> surkov.alexander@gmail.com granted review for attachment 595997 on bug 724452.
  395. # [18:17] <@tbsaunde> surkov: good catch on the GetCurrentFocus() thing I would have sworn we used it in one place somewhere
  396. # [18:18] <@surkov> tbsaunde: I was surprised too
  397. # [18:30] <@firebot> surkov.alexander@gmail.com granted review for attachment 596062 on bug 725998.
  398. # [18:30] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=725998 nor, --, ---, nobody, NEW, get rid nsAccessibilityService::GetAccessibleOrContainer and GetContainerAccessible
  399. # [18:39] * Quits: shorlander (shorlander@moz-853043D6.dhcp.insightbb.com) (Quit: Linkinus - http://linkinus.com)
  400. # [18:40] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  401. # [18:40] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  402. # [18:54] * Joins: shorlander (shorlander@679D38.AF86817F.8A1E5CBC.IP)
  403. # [19:07] * Quits: drexler (chatzilla@moz-2C2B7D1F.hsd1.vt.comcast.net) (Quit: ChatZilla 0.9.88 [Firefox 10.0/20120129021758])
  404. # [19:09] * Quits: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com) (Quit: nhirata)
  405. # [19:09] * Joins: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com)
  406. # [19:17] * Quits: shorlander (shorlander@679D38.AF86817F.8A1E5CBC.IP) (Ping timeout)
  407. # [19:19] * Joins: victorporof (victorporo@50B3F39.880C8D58.79933D60.IP)
  408. # [19:20] * Joins: shorlander (shorlander@moz-853043D6.dhcp.insightbb.com)
  409. # [19:23] * Quits: richardschwerdtfeger (richardsch@moz-130F1494.lightspeed.austtx.sbcglobal.net) (Quit: richardschwerdtfeger)
  410. # [19:28] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  411. # [19:37] <@firebot> New Core - Disability Access APIs bug 726069 filed by surkov.alexander@gmail.com.
  412. # [19:37] <@firebot> Bug 726069 was not found.
  413. # [19:41] * Quits: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp) (Quit: mike5w3c)
  414. # [19:42] <@firebot> New Core - Disability Access APIs bug 726071 filed by surkov.alexander@gmail.com.
  415. # [19:42] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=726071 nor, --, ---, nobody, NEW, nsAccUtils::GetPositionAndSizeForXULSelectControlItem should use nsDocAccessible::GetAccessible
  416. # [19:43] * Quits: @hub (hub@83874EA1.EB7C1AF9.6F478678.IP) (Ping timeout)
  417. # [19:45] <@firebot> New Core - Disability Access APIs bug 726072 filed by surkov.alexander@gmail.com.
  418. # [19:45] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=726072 nor, --, ---, nobody, NEW, nsAccUtils::GetMultiSelectableContainer should use nsDocAccessible::GetAccessible
  419. # [19:47] <@davidb> surkov: I was chatting with Ryan Lortie today, he has started hanging around our office.
  420. # [19:48] <@davidb> (glib maintainer)
  421. # [19:48] <@surkov> cool
  422. # [19:48] <@surkov> davidb: news?
  423. # [19:48] <@davidb> surkov: he wants to devote 6 months to hacking on a11y
  424. # [19:49] <@davidb> gnome infra
  425. # [19:49] <@davidb> needs to convince his boss
  426. # [19:49] <@davidb> I might talk to his boss :)
  427. # [19:49] <@surkov> :)
  428. # [19:49] <@surkov> that's nice
  429. # [19:49] <@davidb> But… he has serious concerns about the current infra.
  430. # [19:49] <@davidb> So this would be 'trying something new'
  431. # [19:50] <@davidb> It is too bad he didn't go to the hackfest
  432. # [19:50] <@surkov> interesting, I always liked new stuffs :)
  433. # [19:50] <@tbsaunde> davidb: interesting
  434. # [19:50] <@davidb> yeah
  435. # [19:50] <@tbsaunde> davidb: you mean glib the gnome lib with some sort of reasonable stuff and the awful gobject stuff
  436. # [19:50] <@tbsaunde> not glibc?
  437. # [19:51] <@davidb> he was describing how dbus is intended to be used.
  438. # [19:51] <@davidb> the former
  439. # [19:51] <@davidb> glib and gtk
  440. # [19:51] <@tbsaunde> davidb: yeah
  441. # [19:51] <@tbsaunde> davidb: what are is his concerns?
  442. # [19:51] <@davidb> well I don't want to be the middle man, but he'll be here during our work week i think
  443. # [19:51] * @tbsaunde not realyl sure this is the right place to discuss or the right medium
  444. # [19:51] <@davidb> right
  445. # [19:52] <@tbsaunde> sounds good
  446. # [19:52] <@davidb> also… i think someone should go to guadec
  447. # [19:53] <@surkov> one Igalia guy said it's nice event
  448. # [19:53] <@davidb> surkov: Ryan is on the GNOME board, and wants an opportunity to have GNOME back any efforts of an open accessibility group.
  449. # [19:53] <@davidb> should something like that come about :)
  450. # [19:54] <@davidb> now, at the same time, I don't think we can shift a lot of focus to GNOME, at the cost of other areas.
  451. # [19:54] <@surkov> yep, but we can collaborate
  452. # [19:54] <@davidb> right
  453. # [19:54] <@davidb> Ryan wants to hack.
  454. # [19:55] <@davidb> I think we could have a GNOME focussed half day during our work week.
  455. # [19:56] <@surkov> sounds good
  456. # [19:56] <@surkov> I would like to learn more
  457. # [19:58] <@tbsaunde> davidb: true, but I'll probably spend some amount of time on linux issues since a good browser makes me wste less time
  458. # [20:01] * Quits: @surkov (surkov@37A43653.44A4068D.222B27F0.IP) (Quit: surkov)
  459. # [20:03] * Joins: hub (hub@21B7B9F2.B87E9213.6E712CE2.IP)
  460. # [20:03] * ChanServ sets mode: +o hub
  461. # [20:11] <@davidb> tbsaunde: yep.
  462. # [20:18] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  463. # [20:21] * khuey|away is now known as khuey
  464. # [20:24] <@firebot> New Core - Disability Access APIs bug 726097 filed by hub@mozilla.com.
  465. # [20:24] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=726097 blo, --, ---, nobody, NEW, [Mac] Accessibility Inspector unable to access items
  466. # [20:30] <@tbsaunde> man nsHyperTextAccessible is pretty xpcommy
  467. # [20:34] * davidb is now known as davidb|mtgh
  468. # [20:34] * davidb|mtgh is now known as davidb|mtg
  469. # [20:51] * Quits: ptheriault (ptheriault@moz-4BE034AB.ptr.us.xo.net) (Quit: ptheriault)
  470. # [21:19] <@tbsaunde> ok, who feels like r+ing http://paste.debian.net/155860/ the fix to surkov's nit in bug 672507
  471. # [21:19] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=672507 nor, --, mozilla13, trev.saunders, RESO FIXED, merge nsIAccessNode and nsIAccessible
  472. # [21:22] <@tbsaunde> davidb|mtg: hub eeejay ^
  473. # [21:22] <@davidb|mtg> tbsaunde: can you point out the nit?
  474. # [21:24] <@davidb|mtg> oh the whole thing
  475. # [21:24] <@tbsaunde> davidb|mtg: yeah, remove the function and just directly do_Qi
  476. # [21:24] <@davidb|mtg> i like it
  477. # [21:25] <@tbsaunde> davidb|mtg: ok
  478. # [21:27] <@firebot> trev.saunders@gmail.com requested review from bolterbugz@gmail.co m for attachment 596143 on bug 672507.
  479. # [21:28] <@hub> ok davidb is taking care of it
  480. # [21:28] <@hub> sorry I missed the call
  481. # [21:28] * davidb|mtg is now known as davidb
  482. # [21:28] <@tbsaunde> hub: its ok
  483. # [21:28] <@davidb> hub: you can drive by r
  484. # [21:30] <@tbsaunde> hub: I only neded one of you but highleted all of you since I wasn't sure who was freeist / paying attention
  485. # [21:30] <@firebot> bolterbugz@gmail.com granted review for attachment 596143 on bug 672507.
  486. # [21:30] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=672507 nor, --, mozilla13, trev.saunders, RESO FIXED, merge nsIAccessNode and nsIAccessible
  487. # [21:31] <@davidb> tbsaunde: what about the nsAccessNode::GetStringBundle nit?
  488. # [21:32] <@tbsaunde> davidb: yeah, forgot about that
  489. # [21:32] <@tbsaunde> I'll remove it locally
  490. # [21:34] <@davidb> ok
  491. # [21:39] <@hub> tbsaunde: X-chat notification in Gnome3 haven't been very noticeable... actually not only X-chat
  492. # [21:39] * @hub blame the software
  493. # [21:42] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  494. # [21:46] * Joins: davidwboswell (chatzilla@moz-679B7357.hsd1.ca.comcast.net)
  495. # [21:50] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
  496. # [22:01] <@tbsaunde> davidb: I see what you did thar
  497. # [22:01] <@davidb> the good first bug?
  498. # [22:03] <@tbsaunde> davidb: yeah, and making Alex the mentor ;)
  499. # [22:04] <@davidb> hahaha
  500. # [22:05] <@firebot> hub@mozilla.com granted review for attachment 596066 on bug 724452.
  501. # [22:05] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=724452 nor, --, ---, jigneshhk1992, NEW, use nsFocusManager::GetFocusManager() more
  502. # [22:13] <@davidb> uh oh. i'm beginning to understand nsCSSFrameConstructor
  503. # [22:13] <@davidb> you know… I should use bz more.
  504. # [22:16] <@tbsaunde> hehe
  505. # [22:16] <@davidb> i'm still not sure i like this code, but i guess it is more about fast.
  506. # [22:19] <@tbsaunde> that isn't really suprising
  507. # [22:19] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  508. # [22:19] <@davidb> nope
  509. # [22:20] <@davidb> but i'm pretty sure i could extract a helper method here that the compiler would inline as needed.
  510. # [22:23] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  511. # [22:23] * Quits: geo (geo@moz-BBE3ABD.mv.mozilla.com) (Quit: Computer has gone to sleep.)
  512. # [22:26] * Parts: davidwboswell (chatzilla@moz-679B7357.hsd1.ca.comcast.net)
  513. # [22:29] * Joins: geo (geo@moz-E31CD2CB.mozilla.org)
  514. # [22:30] * Joins: aaronlev (aaronlev@moz-CDA191A6.c3-0.arl-ubr1.sbo-arl.ma.cable.rcn.com)
  515. # [22:31] <@tbsaunde> davidb: wouldn't suprise me code that code be cleaned up is nothing special
  516. # [22:32] <@davidb> yeah
  517. # [22:32] <@davidb> i suspect people dive in and out quickly
  518. # [22:32] <@davidb> ok have a nice weekend - catch ya later!
  519. # [22:32] <@davidb> all ^
  520. # [22:32] * Quits: @davidb (davidb@F2D29657.F60B0462.67AC9B1.IP) (Quit: davidb)
  521. # [22:32] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
  522. # [22:36] * Quits: victorporof (victorporo@50B3F39.880C8D58.79933D60.IP) (Quit: Linkinus - http://linkinus.com)
  523. # [22:44] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  524. # [22:53] * Parts: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
  525. # [23:12] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
  526. # [23:26] * Quits: @hub (hub@21B7B9F2.B87E9213.6E712CE2.IP) (Ping timeout)
  527. # [23:28] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  528. # [23:31] * Quits: sgimeno (santiago.g@A922400D.48362F18.BCC03455.IP) (Ping timeout)
  529. # [23:33] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
  530. # [23:38] * Joins: skyler_brungardt (skyler@moz-9FA3C392.washdc.fios.verizon.net)
  531. # [23:43] * Joins: hub (hub@83874EA1.EB7C1AF9.6F478678.IP)
  532. # [23:43] * ChanServ sets mode: +o hub
  533. # [23:47] * Joins: sgimeno (santiago.g@A922400D.48362F18.BCC03455.IP)
  534. # [23:48] * Quits: aaronlev (aaronlev@moz-CDA191A6.c3-0.arl-ubr1.sbo-arl.ma.cable.rcn.com) (Quit: aaronlev)
  535. # [23:48] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  536. # Session Close: Sat Feb 11 00:00:00 2012

The end :)