/irc-logs / mozilla / #accessibility / 2012-04-01 / end

Options:

  1. # Session Start: Sun Apr 01 00:00:00 2012
  2. # Session Ident: #accessibility
  3. # [00:45] * Quits: fxa90id (fxa90id@moz-65280E52.neoplus.adsl.tpnet.pl) (Quit: Leaving)
  4. # [00:50] <@firebot> hub@mozilla.com granted review for attachment 611095 on bug 739193.
  5. # [00:50] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=739193 nor, --, ---, nobody, NEW, nsTextEquivUtils::AppendTextEquivFromContent shouldn't use GetAccService()->GetAccessible
  6. # [01:55] * khuey|away is now known as khuey
  7. # [03:58] * Joins: jprmc (jprmc@moz-7F2FF3EB.cpe.net.cable.rogers.com)
  8. # [03:58] * ChanServ sets mode: +o jprmc
  9. # [04:10] * Quits: @jprmc (jprmc@moz-7F2FF3EB.cpe.net.cable.rogers.com) (Ping timeout)
  10. # [04:16] * Quits: @hub (hub@moz-E2FCA694.figuiere.net) (Ping timeout)
  11. # [04:49] <Mark_Capella> tbsaunde: I forgot to ask you about feedback on Bug 740375 - get rid nsIAccessible innerHTML method
  12. # [04:49] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740375 nor, --, ---, markcapella, ASSI, get rid nsIAccessible innerHTML method
  13. # [04:51] <@tbsaunde> Mark_Capella: yeah, what about it?
  14. # [04:53] <Mark_Capella> just asking for feedback ... review next
  15. # [04:55] <Mark_Capella> oh - ithought you were the mentor on that one
  16. # [04:55] <@tbsaunde> Mark_Capella: not sure who is, but I can look, are you blocked on that?
  17. # [04:56] <Mark_Capella> not blocked ... eitan is the mentor ...
  18. # [04:57] <@tbsaunde> ok, I'll look soon then
  19. # [04:57] <Mark_Capella> tbsaunde: cool thanks
  20. # [04:57] <@tbsaunde> I want to eat soon, so maybe after that
  21. # [04:57] <Mark_Capella> k
  22. # [05:26] <@firebot> trev.saunders@gmail.com granted review for attachment 610983 on bug 740375.
  23. # [05:26] <@firebot> trev.saunders@gmail.com cancelled feedback?(trev.saunders@gma il.com) for attachment 610983 on bug 740375.
  24. # [05:26] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740375 nor, --, ---, markcapella, ASSI, get rid nsIAccessible innerHTML method
  25. # [05:30] <@tbsaunde> Mark_Capella: any problems with the other stuff your working on?
  26. # [05:33] * tbsaunde is now known as tbsaunde|afk
  27. # [05:35] <Mark_Capella> tbsaunde: status on bug739524 ... i need to move the call to TreeViewChanged before the Invalidate
  28. # [05:36] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=739524 nor, --, ---, markcapella, ASSI, replace TreeViewChanged DOM event on direct call from XUL tree
  29. # [06:38] <@tbsaunde|afk> Mark_Capella: yeah, and I'd be a lot happier if you passed mView to that call :)
  30. # [06:40] <Mark_Capella> oh right ... that also ... thanks :)
  31. # [06:40] <Mark_Capella> Im having to fall back and rebase a large patch in another area at the moment ... but i'd like to finish
  32. # [06:40] <Mark_Capella> those two bugs next
  33. # [06:40] <@tbsaunde|afk> ok, sounds good, good luck
  34. # [06:41] <Mark_Capella> i guess alex doesnt work weekends?
  35. # [06:41] <Mark_Capella> whats it like to have a real life? :D
  36. # [06:42] <@tbsaunde|afk> he works a little, but he seems to work all day during the week :p
  37. # [06:48] <Mark_Capella> (and of course the build fails on the large rebase :( )
  38. # [06:48] * khuey is now known as khuey|away
  39. # [06:54] <Mark_Capella> tbsaunde: thanks btw for review+ on bug740375 - will you inbound it, let alex do it, or should I checkin-needed it?
  40. # [06:54] <Mark_Capella> usually he likes to take my stuff forward from this point
  41. # [06:54] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740375 nor, --, ---, markcapella, ASSI, get rid nsIAccessible innerHTML method
  42. # [06:54] <Mark_Capella> though mentors in other areas prefer I steer it through
  43. # [07:21] <@tbsaunde|afk> Mark_Capella: hm, was thinking letting Alex deal with it, but I have some stuff to push anyway I think
  44. # [07:22] <Mark_Capella> Great! I dont like to let my backlog get too big ... things start to bitrot and and then I cant get to new things :)
  45. # [07:23] <@tbsaunde|afk> heh
  46. # [07:36] * tbsaunde|afk is now known as tbsaunde
  47. # [07:43] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Client exited)
  48. # [08:35] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  49. # [08:47] <@tbsaunde> Mark_Capella: when you get a chance can you try and reproduce bug 740958? (I think it's windows only)
  50. # [08:47] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740958 cri, --, ---, nobody, NEW, crash in nsAccessible::GetBounds
  51. # [08:48] <Mark_Capella> k - yah - soons this build finishes
  52. # [08:53] <@tbsaunde> sounds great, make sure you have a11y turned on otherwise you won't find it :)
  53. # [08:54] <Mark_Capella> ummm ... be specific not sure what you meant there
  54. # [08:56] <@tbsaunde> Mark_Capella: well, accessibility is only started if somebody asks for it
  55. # [08:57] <@tbsaunde> so, run var gAccRetrieval = Components.classes["@mozilla.org/accessibleRetrieval;1"]. getService(nsIAccessibleRetrieval);
  56. # [08:58] <@tbsaunde> run that as js in your browser
  57. # [08:58] <Mark_Capella> k - two others pinging me ... let me clear them ... brb
  58. # [08:58] <Mark_Capella> saw ---^ right back ....
  59. # [08:59] <@tbsaunde> I think the error console lets you run chrome js, but not sure that's the right thing
  60. # [09:01] <@tbsaunde> alternatively you can just look at accessible tree in domi
  61. # [09:05] <Mark_Capella> tbsaunde: ok back .... might take me a few minutes ... im still trying to build the rebase for Bug 738380
  62. # [09:05] <Mark_Capella> for jst and hoping it completes this time ... if so then ill jump in here ... will you be around for awhile?
  63. # [09:05] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=738380 nor, --, ---, markcapella, ASSI, Remove nsINode::GetScriptTypeID/SetScriptTypeID
  64. # [09:08] <Mark_Capella> (if no build ill have to clobber ... the patch looks good ... i really dont want to spend all night on that )
  65. # [09:10] <@tbsaunde> Mark_Capella: yeah, I think I'll be up for a bit
  66. # [09:11] <@tbsaunde> couldn't you just push to try and let it deal with the full build and tell you how it goes?
  67. # [09:13] <Mark_Capella> oh ... yah i guess i should start doing that ... ive been making sure my local build works first
  68. # [09:13] <Mark_Capella> its running already
  69. # [09:15] <@tbsaunde> that's reasonable if you can do a full build fast, but if it'll take a while and you cn't do other things while your building then pushing to try makes sense
  70. # [09:26] <@firebot> junky.argonaut@gmail.com requested feedback from the wind for attachment 611249 on bug 737379.
  71. # [09:26] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=737379 tri, --, ---, junky.argonaut, NEW, change Comments to roles::ROLENAME
  72. # [10:05] * Joins: icaaq (Adium@moz-7D09279.cust.bredbandsbolaget.se)
  73. # [10:12] * Quits: icaaq (Adium@moz-7D09279.cust.bredbandsbolaget.se) (Quit: Leaving.)
  74. # [10:13] * Joins: victorporof (victorporo@5425BC5E.62A0413.79933D60.IP)
  75. # [10:19] <@firebot> trev.saunders@gmail.com changed the Target Milestone on bug 740375 from --- to mozilla14.
  76. # [10:19] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740375 nor, --, mozilla14, markcapella, ASSI, get rid nsIAccessible innerHTML method
  77. # [10:23] * Quits: victorporof (victorporo@5425BC5E.62A0413.79933D60.IP) (Connection reset by peer)
  78. # [10:32] * Joins: icaaq (Adium@moz-7D09279.cust.bredbandsbolaget.se)
  79. # [11:00] <@tbsaunde> Mark_Capella: want nything else from me? I'm probably heading towards sleep
  80. # [11:01] <Mark_Capella> nah im good to go for a bit ...
  81. # [11:02] <Mark_Capella> looks like the IsDefunct() bug caused the crashes?
  82. # [11:02] <@tbsaunde> ok, cool
  83. # [11:02] * tbsaunde is now known as tbsaunde|afk
  84. # [11:13] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  85. # [11:28] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  86. # [12:23] * khuey|away is now known as khuey
  87. # [12:27] * Joins: victorporof (victorporo@5425BC5E.62A0413.79933D60.IP)
  88. # [13:18] * khuey is now known as khuey|away
  89. # [13:52] * Joins: muralisr92 (chatzilla@moz-FC0DC146.dynip.nus.edu.sg)
  90. # [14:19] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  91. # [14:44] * Quits: victorporof (victorporo@5425BC5E.62A0413.79933D60.IP) (Connection reset by peer)
  92. # [14:49] * Joins: victorporof (victorporo@5425BC5E.62A0413.79933D60.IP)
  93. # [14:51] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  94. # [14:57] * Quits: victorporof (victorporo@5425BC5E.62A0413.79933D60.IP) (Connection reset by peer)
  95. # [15:18] * Joins: jprmc (jprmc@moz-7F2FF3EB.cpe.net.cable.rogers.com)
  96. # [15:18] * ChanServ sets mode: +o jprmc
  97. # [15:27] * Quits: @jprmc (jprmc@moz-7F2FF3EB.cpe.net.cable.rogers.com) (Ping timeout)
  98. # [15:38] <@firebot> maxli@maxli.ca requested feedback from eitan@monotonous. org for attachment 611270 on bug 670853.
  99. # [15:38] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=670853 nor, --, ---, nobody, NEW, undetermined progressmeters should expose mixed state
  100. # [16:00] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  101. # [17:00] * Joins: leo (leo@66FB507B.FDDCF6D.3CA12E.IP)
  102. # [17:00] <leo> hey folks!
  103. # [17:00] <leo> I've got someone asking if Firefox works with JAWS 13... does it?
  104. # [17:08] * Joins: fxa90id (fxa90id@moz-88432F70.neoplus.adsl.tpnet.pl)
  105. # [17:28] * Quits: fxa90id (fxa90id@moz-88432F70.neoplus.adsl.tpnet.pl) (Quit: Leaving)
  106. # [17:42] * Joins: fer (fherrera@moz-26B29C62.dynamic.jazztel.es)
  107. # [17:43] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  108. # [17:44] * Quits: muralisr92 (chatzilla@moz-FC0DC146.dynip.nus.edu.sg) (Quit: ChatZilla 0.9.88.1 [Firefox 11.0/20120310010316])
  109. # [18:07] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  110. # [18:13] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  111. # [18:14] * Joins: victorporof (victorporo@81BEB511.1C233438.79933D60.IP)
  112. # [18:16] <@firebot> markcapella@twcny.rr.com requested review from surkov.alexander@gm ail.com for attachment 611249 on bug 737379.
  113. # [18:16] <@firebot> markcapella@twcny.rr.com granted feedback for attachment 611249 on bug 737379.
  114. # [18:16] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=737379 tri, --, ---, junky.argonaut, NEW, change Comments to roles::ROLENAME
  115. # [18:25] <@firebot> nickyekaiqi@gmail.com requested feedback from the wind for attachment 611284 on bug 739253.
  116. # [18:25] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=739253 nor, --, ---, nobody, NEW, optimize AccEvent::GetDocAccessible
  117. # [18:32] * Joins: muralisr92 (chatzilla@moz-FC0DC146.dynip.nus.edu.sg)
  118. # [18:33] * khuey|away is now known as khuey
  119. # [18:39] <@firebot> nickyekaiqi@gmail.com requested feedback from the wind for attachment 611287 on bug 739190.
  120. # [18:39] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=739190 nor, --, ---, nobody, NEW, change AppendTarget(nsIContent* aContent) to take nsDocAccessible* argument
  121. # [18:41] * Parts: leo (leo@66FB507B.FDDCF6D.3CA12E.IP)
  122. # [18:46] * Quits: muralisr92 (chatzilla@moz-FC0DC146.dynip.nus.edu.sg) (Quit: ChatZilla 0.9.88.1 [Firefox 11.0/20120310010316])
  123. # [19:08] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  124. # [20:40] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  125. # [20:43] <@firebot> trev.saunders@gmail.com cancelled feedback? for attachment 611287 on bug 739190.
  126. # [20:43] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=739190 nor, --, ---, nobody, NEW, change AppendTarget(nsIContent* aContent) to take nsDocAccessible* argument
  127. # [20:43] * tbsaunde|afk is now known as tbsaunde
  128. # [20:45] * Quits: Mark_Capella (chatzilla@moz-DD0C7E4F.twcny.res.rr.com) (Quit: ChatZilla 0.9.88.1 [Firefox 13.0a2/20120330042010])
  129. # [20:48] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  130. # [20:48] * Joins: Mark_Capella (chatzilla@moz-DD0C7E4F.twcny.res.rr.com)
  131. # [21:02] <Mark_Capella> tbsaunde: welcome back!
  132. # [21:02] <Mark_Capella> re: bug739524 I see point #1 now ... call accessibilityservice before firing event -both still- occurring after the Invalidate ()
  133. # [21:03] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=739524 nor, --, ---, markcapella, ASSI, replace TreeViewChanged DOM event on direct call from XUL tree
  134. # [21:07] <@tbsaunde> Mark_Capella: well, you can notify a11y after the invalidate, but before may be safer
  135. # [21:08] <Mark_Capella> So in order 1) call accessible service
  136. # [21:08] <Mark_Capella> 2) Invalidate()
  137. # [21:08] <Mark_Capella> 3) FireDOM
  138. # [21:08] <Mark_Capella> ??
  139. # [21:09] <@tbsaunde> yeah, I think so, but let me look
  140. # [21:09] <Mark_Capella> k - sorry, i dont understand all the implications
  141. # [21:11] * khuey is now known as khuey|away
  142. # [21:11] <@tbsaunde> well, I don't completely off hand either, but I'm figuring out what Invalidate() actually does
  143. # [21:19] <Mark_Capella> ah ok - then for the final trick you might tell me what passing that third arg does for the accessibility service that
  144. # [21:19] <Mark_Capella> its not already doing ... :(
  145. # [21:20] <@tbsaunde> Mark_Capella: well, you can pass it to nsXULTreeAccessible::TrevViewChanged() so it doesn't need to call GetView()
  146. # [21:21] <@tbsaunde> make sense?
  147. # [21:21] <Mark_Capella> ah ! thats my disconnect ... you want to pass it another stpe downstream ... and then there'd have to be changes there to take advantage of it
  148. # [21:22] <@tbsaunde> yeah, but they should be pretty small :)
  149. # [21:22] <Mark_Capella> oh well ! :)
  150. # [21:22] <@tbsaunde> just take an argument and change the line setting mView
  151. # [21:22] <@tbsaunde> it shouldn't take more than 10 minutes to change
  152. # [21:23] <Mark_Capella> see, at this point if ive got a working patch im like lets go !!! but ok, i know i can handle that :P
  153. # [21:25] <@firebot> asqueella@gmail.com changed the Component on bug 741189 from Untriaged to Disability Access APIs.
  154. # [21:25] <@tbsaunde> yeah, but while your working on it you might as well get it as good as you can easily
  155. # [21:25] <@firebot> asqueella@gmail.com changed the Product on bug 741189 from Firefox to Core.
  156. # [21:25] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741189 nor, --, ---, nobody, UNCO, Print window -> fatal crash
  157. # [21:31] * Joins: fxa90id (fxa90id@moz-88432F70.neoplus.adsl.tpnet.pl)
  158. # [21:33] <Mark_Capella> so down in TreeViewChanged we'd do what with mTree->GetView(getter_AddRefs(mTreeView));
  159. # [21:33] <Mark_Capella> change it like mTree = aView;
  160. # [21:33] <Mark_Capella> ??
  161. # [21:39] <@tbsaunde> yup
  162. # [21:39] <@tbsaunde> well, no, mView = aView
  163. # [21:49] <Mark_Capella> mTreeView = aView ?
  164. # [21:50] <Mark_Capella> looking at GetView ... trying to follow
  165. # [21:52] <@tbsaunde> Mark_Capella: yes, never trust my variable names to be exactly correct ;-)
  166. # [21:52] * khuey|away is now known as khuey
  167. # [21:52] <Mark_Capella> i >thought< i noticed that ... well talk about comments in "@londesc" sometime next :D
  168. # [21:53] <Mark_Capella> laughs
  169. # [21:53] <Mark_Capella> I had mTree, you had mView, together we got mTreeView :D
  170. # [21:55] <@tbsaunde> well, mTree was clearly not it because that member has a different type, and nsTreeBodyFrame uses mView...
  171. # [22:10] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  172. # [22:18] * Quits: victorporof (victorporo@81BEB511.1C233438.79933D60.IP) (Ping timeout)
  173. # [22:19] * Quits: icaaq (Adium@moz-7D09279.cust.bredbandsbolaget.se) (Quit: Leaving.)
  174. # [22:29] <@firebot> maxli@maxli.ca requested review from trev.saunders@gmail .com for attachment 611311 on bug 735645.
  175. # [22:29] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=735645 nor, --, ---, nobody, NEW, expose sub and sup elements in text attributes
  176. # [22:33] <@firebot> orangedaylily@gmail.com requested review from trev.saunders@gmail .com for attachment 611312 on bug 723424.
  177. # [22:33] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=723424 nor, --, ---, orangedaylily, NEW, dexpcom GetUnignoredParent()
  178. # [22:40] * Quits: khuey (khuey@moz-B77DEAEB.mozilla.org) (Ping timeout)
  179. # [22:41] * Joins: khuey (khuey@moz-B77DEAEB.mozilla.org)
  180. # [22:45] * Joins: margle (margle@moz-9542A19B.dsl.mweb.co.za)
  181. # [23:40] * Quits: margle (margle@moz-9542A19B.dsl.mweb.co.za) (Client exited)
  182. # [23:58] * Quits: fxa90id (fxa90id@moz-88432F70.neoplus.adsl.tpnet.pl) (Quit: Leaving)
  183. # [23:59] * khuey is now known as khuey|away
  184. # Session Close: Mon Apr 02 00:00:00 2012

The end :)