/irc-logs / mozilla / #accessibility / 2011-12-06 / end

Options:

  1. # Session Start: Tue Dec 06 00:00:00 2011
  2. # Session Ident: #accessibility
  3. # [00:02] <@tbsaunde|afk> hub: yamer?
  4. # [00:03] <@hub> tbsaunde|afk: yammmer. it is some sort of Facebook we use internally
  5. # [00:04] * tbsaunde|afk is now known as tbsaunde
  6. # [00:04] * Quits: richardschwerdtfeger (richardsch@moz-130F1494.lightspeed.austtx.sbcglobal.net) (Quit: richardschwerdtfeger)
  7. # [00:04] <@tbsaunde> hub: oh, lovely
  8. # [00:06] * Quits: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com) (Quit: nhirata)
  9. # [00:09] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Connection reset by peer)
  10. # [00:09] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  11. # [00:11] * Joins: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com)
  12. # [00:38] * Joins: hhillen (hhillen@moz-2A42E9B3.upc-d.chello.nl)
  13. # [00:52] * iamjessklein-away is now known as jess
  14. # [01:07] * Quits: @jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com) (Ping timeout)
  15. # [01:08] * jess is now known as iamjessklein-away
  16. # [01:09] * Joins: jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com)
  17. # [01:09] * ChanServ sets mode: +o jprmc
  18. # [01:28] * Quits: hhillen (hhillen@moz-2A42E9B3.upc-d.chello.nl) (Quit: hhillen)
  19. # [01:46] * Quits: Flyingmana (Mana@moz-12184E52.adsl.alicedsl.de) (Client exited)
  20. # [02:01] * Quits: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com) (Quit: nhirata)
  21. # [02:08] * Joins: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com)
  22. # [02:09] * Quits: victorporof (victorporo@DCE93940.AB163232.79933D60.IP) (Quit: victorporof)
  23. # [02:10] * Quits: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com) (Quit: nhirata)
  24. # [02:11] * Quits: @jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com) (Ping timeout)
  25. # [02:18] * Joins: jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com)
  26. # [02:18] * ChanServ sets mode: +o jprmc
  27. # [02:28] * Joins: firewolfbot (firebot@moz-67BF9FE6.carolina.res.rr.com)
  28. # [02:29] * Quits: ehsan (ehsan@F2D29657.F60B0462.67AC9B1.IP) (sand.mozilla.org concrete.mozilla.org)
  29. # [02:29] * Quits: khuey|pto (khuey@moz-B77DEAEB.mozilla.org) (sand.mozilla.org concrete.mozilla.org)
  30. # [02:29] * Quits: @eeejay (eitan@moz-EE2D6C10.monotonous.org) (sand.mozilla.org concrete.mozilla.org)
  31. # [02:29] * Quits: @firebot (firebot@moz-67BF9FE6.carolina.res.rr.com) (sand.mozilla.org concrete.mozilla.org)
  32. # [02:29] * Joins: khuey|pto (khuey@moz-B77DEAEB.mozilla.org)
  33. # [02:36] * Joins: eeejay (eitan@moz-EE2D6C10.monotonous.org)
  34. # [02:44] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
  35. # [02:49] <firewolfbot> hub@mozilla.com requested checkin from the wind for attachment 579167 on bug 689105.
  36. # [02:49] <firewolfbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=689105 maj, --, ---, hub, NEW, Accessibility in main window broken for VoiceOver, VO doesn't see anything but the title bar and its
  37. # [02:54] <firewolfbot> hub@mozilla.com requested review from surkov.alexander@gm ail.com for attachment 579169 on bug 703770.
  38. # [02:54] <firewolfbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=703770 nor, --, ---, hub, NEW, Mac Accessibility Verifier errors
  39. # [02:56] <firewolfbot> hub@mozilla.com cancelled review? for attachment 577354 on bug 455443.
  40. # [02:56] <firewolfbot> hub@mozilla.com requested review from surkov.alexander@gm ail.com for attachment 579172 on bug 455443.
  41. # [02:56] <firewolfbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=455443 nor, --, ---, hub, ASSI, cache the parent for the accessibilityAttributeValue(NSAccessibilityParentAttribute)
  42. # [03:00] <firewolfbot> New Core - Disability Access APIs bug 707846 filed by hub@mozilla.com.
  43. # [03:00] <firewolfbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=707846 nor, --, ---, nobody, NEW, [mac] remove unused nsAccessibleWrap::GetUnignoredChildCount()
  44. # [03:01] <firewolfbot> hub@mozilla.com requested review from surkov.alexander@gm ail.com for attachment 579175 on bug 707846.
  45. # [03:04] * Joins: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP)
  46. # [03:04] * Quits: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP) (Connection reset by peer)
  47. # [03:04] * Joins: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP)
  48. # [03:06] <@tbsaunde> man, I don't get objective C++
  49. # [03:07] <@hub> tbsaunde: it is just a mix between Objective-C and C++. So one can call the other.
  50. # [03:07] <@hub> tbsaunde: since their syntax do not conflict
  51. # [03:08] <@tbsaunde> hub: sure, but I don't get objective C either
  52. # [03:08] <@tbsaunde> (I have spent all of five minutes trying by reading patches)
  53. # [03:08] <@hub> tbsaunde: that's what the Mac API use. Not like there is choice.
  54. # [03:08] <@hub> it give a dynamic object model and easier ABI compatibility
  55. # [03:09] <@hub> but the performance hit is not negligible
  56. # [03:09] <@tbsaunde> hub: sure, I'm just saying I don't get it :)
  57. # [03:09] <@hub> I sometime wish Gtk had used Objective-C instead of reinventing an object model
  58. # [03:10] <@hub> tbsaunde: basically [obj foo:param] means to call method foo on obj with param
  59. # [03:11] <@tbsaunde> hub: ... ok
  60. # [03:12] <@hub> @interface / @end is the class declaration
  61. # [03:12] <@hub> @implementation / @end is the class implementation
  62. # [03:12] <@tbsaunde> hub: ok, weird
  63. # [03:14] <firewolfbot> New Core - Disability Access APIs bug 707852 filed by hub@mozilla.com.
  64. # [03:14] <firewolfbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=707852 nor, --, ---, nobody, NEW, [mac] remove the Objective-C wrapper
  65. # [03:16] <@tbsaunde> hub: thanks
  66. # [03:19] <@tbsaunde> hub: I can't think of a reason to keep putting (mac) in the summary of bugs
  67. # [03:23] <@hub> not sure. I'll stop.
  68. # [03:32] * Quits: @hub (hub@21B7B9F2.B87E9213.6E712CE2.IP) (Ping timeout)
  69. # [03:34] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  70. # [03:37] * Quits: @jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com) (Ping timeout)
  71. # [03:47] * Joins: nhirata (nhirata.bu@moz-2A9C9106.hsd1.ca.comcast.net)
  72. # [03:51] * Quits: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP) (Ping timeout)
  73. # [03:51] * Joins: ehsan (ehsan@moz-104CC309.mv.mozilla.com)
  74. # [04:07] * Joins: hub (hub@83874EA1.EB7C1AF9.6F478678.IP)
  75. # [04:07] * ChanServ sets mode: +o hub
  76. # [05:14] * Joins: surkov (surkov@EAF8AE41.26ED71FF.34044A7F.IP)
  77. # [05:14] * ChanServ sets mode: +o surkov
  78. # [05:35] <@surkov> hi, hub, nice progress!
  79. # [05:36] <@hub> thanks
  80. # [05:36] <@hub> I have lot of work still
  81. # [05:37] <@hub> and today bugzilla being down didn't help
  82. # [05:42] <firewolfbot> surkov.alexander@gmail.com granted review for attachment 579175 on bug 707846.
  83. # [05:42] <firewolfbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=707846 nor, --, ---, hub, NEW, [mac] remove unused nsAccessibleWrap::GetUnignoredChildCount()
  84. # [05:43] <@surkov> btw, does parent patch helps perf?
  85. # [05:47] <@hub> surkov: I think so.
  86. # [05:47] <@surkov> cool
  87. # [05:47] <@surkov> hub, what do you think about creation mac tree (the bug I filed)?
  88. # [05:49] <@surkov> bug 705404
  89. # [05:49] <firewolfbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=705404 nor, --, ---, hub, NEW, build separate accessible tree on mac
  90. # [05:51] <@hub> yeah
  91. # [05:51] <@hub> technically this is what we do
  92. # [06:00] <@surkov> hub, this approach allows to avoid creation of mac wrappers when we don't need it
  93. # [06:01] <@tbsaunde> surkov: objects to killing nsAccessNode::GetPresShell()? as far as I can see we might as well just use do_QueryReferent(mWeakShell)
  94. # [06:01] <@tbsaunde> *objections
  95. # [06:02] <@surkov> tbsaunde: if we keep document reference instead of mWeakShell then we wouldn't need press shell in most cases
  96. # [06:03] <@surkov> document can provide GetPresShell and we wouldn't deal with weak shell likely
  97. # [06:03] <@tbsaunde> surkov: yeah, forgot about that
  98. # [06:03] <@tbsaunde> yeah
  99. # [06:18] * Quits: ehsan (ehsan@moz-104CC309.mv.mozilla.com) (Input/output error)
  100. # [06:59] * firewolfbot is now known as firebot
  101. # [07:18] <firebot> surkov.alexander@gmail.com cancelled checkin? for attachment 579167 on bug 689105.
  102. # [07:18] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=689105 maj, --, ---, hub, NEW, Accessibility in main window broken for VoiceOver, VO doesn't see anything but the title bar and its
  103. # [07:20] <@tbsaunde> surkov: so, I was thinking about bug 686909 for a minute, and considering what we do with the mTexthash thing to stuff the IsFromUserInputFlag into it
  104. # [07:20] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=686909 nor, --, ---, nobody, NEW, The system suffix is for system generated events only
  105. # [07:21] <@tbsaunde> I'm not completely sure what I think yet, but I tend to think a queue makes more sense than hash table
  106. # [07:21] <@tbsaunde> but I'm not really sure what we'd do in the case of two changes to the same nsIContent*
  107. # [07:22] <@tbsaunde> but I'm not sure if that's a real posibility or not
  108. # [07:22] <@surkov> it's possible I think
  109. # [07:22] <@surkov> why do you think queue more suitable?
  110. # [07:23] <@surkov> about IsFromUserInputFlag I guess it's sort of late to calculate it on text reflow
  111. # [07:23] <@tbsaunde> surkov: be back in 10, sorry have to go do something
  112. # [07:23] <@surkov> sure
  113. # [07:39] <@tbsaunde> ok, back
  114. # [07:40] <@tbsaunde> surkov: well, it seems like it would be nice to fire the events in the order they actually happen instead of a random one
  115. # [07:41] <@tbsaunde> otherwise a hash table just seems more heavy weight than we need and gives us no better properties I can think of
  116. # [07:41] <@surkov> this might be nice
  117. # [07:41] <@surkov> but I'm not sure if anybody cares
  118. # [07:42] <@surkov> since we describe what was changed, not user actions actually
  119. # [07:44] <@tbsaunde> surkov: even so I think its nice to have the events be in the order the changes happen
  120. # [07:44] <@surkov> it's nice, agree
  121. # [07:44] <@tbsaunde> surkov: I suppose we can't compute what changed until reflow has finished?
  122. # [07:45] <@surkov> how would you remove dupes?
  123. # [07:45] <@surkov> for specific node - yes
  124. # [07:45] <@tbsaunde> surkov: not sure about dups yet
  125. # [07:46] <@tbsaunde> I'm not clear what you mean by for specific nodes
  126. # [07:58] <@tbsaunde> surkov: ?
  127. # [07:58] <firebot> surkov.alexander@gmail.com denied review for attachment 579169 on bug 703770.
  128. # [07:58] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=703770 nor, --, ---, hub, NEW, Mac Accessibility Verifier errors
  129. # [07:59] <@surkov> tbsaunde: I mean reflow in general might be in process but some pieces were processed already
  130. # [07:59] <@surkov> so if text is pending for reflow then we can't get its future text
  131. # [08:00] <@tbsaunde> surkov: yeah, ok
  132. # [08:05] <@tbsaunde> surkov: so, if we get notified of two text reflows involving the same nsIContent* do we use if it was from user input the first time or the second?
  133. # [08:05] <@surkov> tbsaunde: first of all you should check if we can calculate ifFromUserInput at text reflow at all
  134. # [08:07] <@tbsaunde> surkov: yeah, I suppose that's a good idea :)
  135. # [08:08] <@surkov> yep, it may happen synchronously for editing then we are ok, otherwise we're not
  136. # [08:20] <firebot> surkov.alexander@gmail.com granted review for attachment 579172 on bug 455443.
  137. # [08:20] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=455443 nor, --, ---, hub, ASSI, cache the parent for the accessibilityAttributeValue(NSAccessibilityParentAttribute)
  138. # [08:26] <@tbsaunde> surkov: it'll be fun if its async
  139. # [09:03] * Joins: MarcoZ (marco.zehe@moz-404D9980.dip.t-dialin.net)
  140. # [09:03] * ChanServ sets mode: +o MarcoZ
  141. # [09:04] <@MarcoZ> Good morning all!
  142. # [09:04] <@MarcoZ> hub: surkov: Hi! Good work! I'm marvelling!
  143. # [09:05] <@surkov> yeah, hub wakes up ;)
  144. # [09:17] <@hub> and I might have a trybuild with that
  145. # [09:26] <@MarcoZ> YAY! I'd love to try it!!
  146. # [09:30] * Joins: atulagrwl (atul@moz-D64D2953.ts.2iij.net)
  147. # [09:33] * Quits: @hub (hub@83874EA1.EB7C1AF9.6F478678.IP) (Ping timeout)
  148. # [09:42] * Joins: victorporof (victorporo@DCE93940.AB163232.79933D60.IP)
  149. # [09:44] * Joins: Flyingmana (Mana@moz-12184E52.adsl.alicedsl.de)
  150. # [10:06] <@MarcoZ> surkov: So, how complicated will bug 686909 going to be?
  151. # [10:06] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=686909 nor, --, ---, nobody, NEW, The system suffix is for system generated events only
  152. # [10:06] <@MarcoZ> s/will/is/
  153. # [10:07] <@surkov> MarcoZ: if idea from my last comments work then it's not too complicated but it should be tested well, for example, perf
  154. # [10:08] * @MarcoZ nods.
  155. # [10:10] <@tbsaunde> it would help if my local builds actually start with --no-remote not just exit 0 :/
  156. # [10:18] * Quits: @surkov (surkov@EAF8AE41.26ED71FF.34044A7F.IP) (Ping timeout)
  157. # [10:18] * Joins: surkov (surkov@EAF8AE41.26ED71FF.34044A7F.IP)
  158. # [10:18] * ChanServ sets mode: +o surkov
  159. # [10:21] <@MarcoZ> tbsaunde: Huh?
  160. # [10:22] <@tbsaunde> MarcoZ: I'm trying totest stuff, but its hard when builds don't start
  161. # [10:28] <@MarcoZ> tbsaunde: Hm, strange! Regular builds or local ones?
  162. # [10:31] <@tbsaunde> MarcoZ: local ones, not much point otherwise
  163. # [11:10] <@tbsaunde> surkov: ok, the experimental answer seems to be the CaptureIsFromUserInput() style of detecting if something is from user input doesn't work if you check in ScheduleTextUpdate()
  164. # [11:11] <@tbsaunde> which I guess makes sense if you think about it
  165. # [11:12] <@surkov> ok, what do you think about idea I posted into the bug, keep a hash of changed nodes and storing isfromuserinput flag?
  166. # [11:18] <@tbsaunde> surkov: that would mean having the existing hashtable and a new hashmap right?
  167. # [11:18] <@tbsaunde> that seems really heavy weight, but I'm not sure if there's a better approach
  168. # [11:19] <@surkov> yeah, right, that might be too havvy
  169. # [11:19] <@surkov> but I don't think layout folks will do that for us
  170. # [11:19] <@surkov> so maybe we should integrate into layout code hardly
  171. # [11:19] <@tbsaunde> yeah, I can't see why layout folks would care what caused it and doing the update async makes sense probably
  172. # [11:20] <@surkov> tbsaunde: which existing hash table is suitable for this?
  173. # [11:20] <@tbsaunde> hardly?
  174. # [11:20] <@surkov> they won't care but they may allow us to add extra bit for their structures, dunno
  175. # [11:20] <@tbsaunde> oh, hmm
  176. # [11:20] <@surkov> add a11y propose info into their structures
  177. # [11:21] <@tbsaunde> yeah, I got you
  178. # [11:21] <@surkov> they keep somewhere all frames insertion/removals
  179. # [11:21] <@tbsaunde> by which existing hash table do you mean which of the nsHashtable types do we want?
  180. # [11:21] <@surkov> but it's likely they don't do that for text changes unfortunately
  181. # [11:21] <@surkov> I thought you suffused to reuse some existing hash table in our code
  182. # [11:22] <@tbsaunde> no
  183. # [11:24] <@surkov> got it
  184. # [11:24] <@surkov> yep, it's just hasmap
  185. # [11:24] <@tbsaunde> surkov: do the DOM mutation events give us enough information to just use them? or would we still need to use layout?
  186. # [11:24] <@surkov> we need to keep "the presence of the node"
  187. # [11:25] <@surkov> DOM doesn't give us enough information but it may be enough to address the problem we have
  188. # [11:25] <@tbsaunde> ok
  189. # [11:25] <@surkov> if all we care about is text editing
  190. # [11:26] <@tbsaunde> what else would we care about for text change events?
  191. # [11:27] <@surkov> other kinds of text change events are used rather for vb update than to announce something
  192. # [11:28] <@surkov> dunno, I don't know all usecases
  193. # [11:32] <@tbsaunde> surkov: I think the important question is if its from user editing or something else
  194. # [11:38] <@surkov> that's sort of wide treatment, because user editing can trigger other things
  195. # [11:40] * Quits: @MarcoZ (marco.zehe@moz-404D9980.dip.t-dialin.net) (Input/output error)
  196. # [11:41] <@tbsaunde> surkov: yeah, I think probably only direct effects
  197. # [11:41] <@tbsaunde> but the whole thing is kind of tricky
  198. # [11:41] <@surkov> righty
  199. # [11:59] * Quits: @surkov (surkov@EAF8AE41.26ED71FF.34044A7F.IP) (Connection reset by peer)
  200. # [12:02] * Joins: surkov (surkov@7F8B251F.331F467.5D3F4C44.IP)
  201. # [12:02] * ChanServ sets mode: +o surkov
  202. # [12:12] * Quits: @surkov (surkov@7F8B251F.331F467.5D3F4C44.IP) (Ping timeout)
  203. # [12:35] * Quits: victorporof (victorporo@DCE93940.AB163232.79933D60.IP) (Quit: victorporof)
  204. # [12:45] * khuey|pto is now known as khuey
  205. # [12:47] * Quits: khuey (khuey@moz-B77DEAEB.mozilla.org) (Quit: leaving)
  206. # [12:48] * Joins: khuey (khuey@moz-B77DEAEB.mozilla.org)
  207. # [13:46] * Joins: MarcoZ (marco.zehe@moz-404D9980.dip.t-dialin.net)
  208. # [13:47] * ChanServ sets mode: +o MarcoZ
  209. # [14:03] * khuey is now known as khuey|away
  210. # [14:06] * Joins: surkov (surkov@7F8B251F.331F467.5D3F4C44.IP)
  211. # [14:06] * ChanServ sets mode: +o surkov
  212. # [14:20] * Joins: surkov_ (surkov@178F4FC2.D2B288F8.5D3F4C44.IP)
  213. # [14:21] * Quits: @surkov (surkov@7F8B251F.331F467.5D3F4C44.IP) (Ping timeout)
  214. # [14:21] * surkov_ is now known as surkov
  215. # [14:32] * Quits: bwinton_away (bwinton@D7D3C4B5.88764A66.72A31D6.IP) (Ping timeout)
  216. # [14:34] * Joins: bwinton_away (bwinton@D7D3C4B5.88764A66.72A31D6.IP)
  217. # [14:44] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
  218. # [14:44] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  219. # [14:53] * iamjessklein-away is now known as jess
  220. # [14:57] <@MarcoZ> surkov: Does mozilla.com e-mail also not work for you? I haven't received any mail since yesterday my evening.
  221. # [14:57] <surkov> MarcoZ: yep, it doesn't
  222. # [14:58] <@MarcoZ> surkov: And Bugzilla is also down for me. Geez...
  223. # [14:58] <surkov> yeah, same
  224. # [15:05] * Joins: davidb (davidb@F2D29657.F60B0462.67AC9B1.IP)
  225. # [15:05] * ChanServ sets mode: +qo davidb davidb
  226. # [15:06] <@MarcoZ> Hi davidb!
  227. # [15:08] <@davidb> hi MarcoZ!
  228. # [15:08] <@davidb> surely tbsaunde isn't awake?
  229. # [15:09] <@davidb> firebot: accessibility?
  230. # [15:09] <firebot> davidb: Accessibility in Firefox can be enhanced using extensions like http://www.firevox.clcworld.net/downloads.html and http://www.webvisum.com/en/main/download and https://addons.mozilla.org/en-US/firefox/addon/6489
  231. # [15:09] <@davidb> hmm
  232. # [15:11] <@MarcoZ> davidb: He was earlier, but probably is asleep now.
  233. # [15:11] <@davidb> yeah
  234. # [15:11] <@MarcoZ> davidb: And with Bugzilla down, there's not much to do.
  235. # [15:12] <@davidb> Perhaps write tests, or update the our websites
  236. # [15:12] <@davidb> I'm sad our web face is so confusing
  237. # [15:12] <@davidb> oh surkov is here?
  238. # [15:12] <surkov> I am
  239. # [15:12] <@davidb> hi
  240. # [15:13] * davidb sets mode: +o surkov
  241. # [15:13] <@surkov> hi
  242. # [15:15] <@davidb> i sure do rely on email a lot
  243. # [15:15] * @MarcoZ too.
  244. # [15:15] * @davidb hunts for a bookmark
  245. # [15:17] * Joins: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
  246. # [15:19] * Joins: ehsan (ehsan@moz-104CC309.mv.mozilla.com)
  247. # [15:24] <@MarcoZ> Sheesh, nntp is also down.
  248. # [15:24] * Quits: bwinton_away (bwinton@D7D3C4B5.88764A66.72A31D6.IP) (Ping timeout)
  249. # [15:27] * Joins: richardschwerdtfeger (richardsch@BE0AFBFF.9DD4DBBF.6A7A197.IP)
  250. # [15:29] * @davidb reviews http://www.w3.org/html/wg/wiki/ChangeProposals/FocusRingTextBaseline
  251. # [15:38] * khuey|away is now known as khuey
  252. # [15:39] <@davidb> man, can't even get to my personal planning etherpad
  253. # [15:39] <@davidb> nooooooooooooooo.com
  254. # [15:43] * Joins: bwinton (bwinton@F2D29657.F60B0462.67AC9B1.IP)
  255. # [15:43] * ChanServ sets mode: +h bwinton
  256. # [15:49] * Joins: jprmc (jprmc@moz-9CE00FC0.holiday-inn-express.mozilla.hq)
  257. # [15:49] * ChanServ sets mode: +o jprmc
  258. # [15:50] * Quits: @MarcoZ (marco.zehe@moz-404D9980.dip.t-dialin.net) (Quit: l8er)
  259. # [15:59] * Joins: ehsan_ (ehsan@F2D29657.F60B0462.67AC9B1.IP)
  260. # [16:00] * Quits: ehsan (ehsan@moz-104CC309.mv.mozilla.com) (Ping timeout)
  261. # [16:11] * Quits: nhirata (nhirata.bu@moz-2A9C9106.hsd1.ca.comcast.net) (Quit: nhirata)
  262. # [16:15] * Quits: atulagrwl (atul@moz-D64D2953.ts.2iij.net) (Client exited)
  263. # [16:17] * khuey is now known as khuey|away
  264. # [16:19] <@davidb> how is safari 5 a11y?
  265. # [16:23] <firebot> josh@joshmatthews.net requested review from surkov.alexander@gm ail.com for attachment 579280 on bug 706369.
  266. # [16:23] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=706369 nor, --, ---, jigneshhk1992, NEW, don't use nsIContent::GetChildAt to iterate through children
  267. # [16:28] <@davidb> yaya
  268. # [16:28] <@davidb> bugzilla!
  269. # [16:29] * Joins: bwinton_away (bwinton@D7D3C4B5.88764A66.72A31D6.IP)
  270. # [16:34] <@davidb> surkov: did dbaron get back to you?
  271. # [16:34] <@surkov> davidb: dunno, mozilla mail doesn't work for me
  272. # [16:34] <@davidb> oh hmmm mail is still down yeah
  273. # [16:35] * Joins: MarcoZ (marco.zehe@moz-404D9980.dip.t-dialin.net)
  274. # [16:35] * ChanServ sets mode: +o MarcoZ
  275. # [16:36] * Joins: atulagrwl (atul@4386A3CB.E762D0A2.A3D1B221.IP)
  276. # [16:38] <@davidb> surkov: why does fig caption get the extra whitespace at the end?
  277. # [16:38] * @MarcoZ looks at the patch for bug 706369 and is officially scared. Is it really possible to use a for loop like this?
  278. # [16:38] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=706369 nor, --, ---, jigneshhk1992, NEW, don't use nsIContent::GetChildAt to iterate through children
  279. # [16:39] <@davidb> a loop is a loop :)
  280. # [16:39] <@surkov> davidb: gaps or name calculation algorithm impl
  281. # [16:39] <@davidb> hm
  282. # [16:39] * @surkov forgot to file bug about this
  283. # [16:45] * Quits: atulagrwl (atul@4386A3CB.E762D0A2.A3D1B221.IP) (Ping timeout)
  284. # [16:47] <@tbsaunde> MarcoZ: I don't see anything particularly scary in that patch
  285. # [16:47] * @tbsaunde notes they added a couple uses of GetChildAt() :/
  286. # [16:48] <@davidb> surkov: what is the current status of bug https://www.w3.org/Bugs/Public/show_bug.cgi?id=14320
  287. # [16:49] <@surkov> honestly I'm not sure, there's a vary of autocompletes implementation on the web and they all do different things and likely not accessible in Firefox :)
  288. # [16:50] <@surkov> it appears making aria-controls to allow aria-activedescendant to control the focus sounds good
  289. # [16:50] <@surkov> that's what I commented there
  290. # [16:50] <@surkov> I still think aria-owns is not very good option (however we have examples of autocomplete where autocomplete popup is a real child of autocomplete widget)
  291. # [16:51] <@surkov> in this light there's nothing bad in aria-owns
  292. # [16:51] <@davidb> i'm curious what textual change we want to the impl guide
  293. # [16:52] <@davidb> it is probably best if you propose something (in the bug) and we can polish it
  294. # [16:52] <@surkov> it might be not enough to change impl guide only unfortunately
  295. # [16:52] <@davidb> one goal is to make sure the guide doesn't block good potential solutions
  296. # [16:52] <@surkov> I still need to play with autocomplete implementation and ARIAs
  297. # [16:52] <@davidb> ok
  298. # [16:53] <@surkov> that's great, but ARIA spec can do that
  299. # [16:53] <@davidb> right
  300. # [16:53] <@surkov> and the funny things these specs are controlled by different groups
  301. # [16:54] <@davidb> agreed
  302. # [16:55] * Joins: victorporof (victorporo@E8764A15.F4308271.4873E54D.IP)
  303. # [17:01] * Joins: atulagrwl (atul@944C5E56.71A613B7.A3D1B221.IP)
  304. # [17:04] <@davidb> surkov: do you have an xpi of the latest domi somewhere?
  305. # [17:04] <@surkov> nope
  306. # [17:04] <@surkov> I build it from sources
  307. # [17:05] <@davidb> ok and you just build it straight into your FF install dir?
  308. # [17:05] <@surkov> I suggest you to do the same if you are going to run it on nightlies you build
  309. # [17:05] <@davidb> yeah
  310. # [17:05] <@davidb> surkov: how do i do this?
  311. # [17:05] <@surkov> clone http://hg.mozilla.org/dom-inspector/
  312. # [17:06] <@davidb> oh and i give it a path to my install?
  313. # [17:06] <@surkov> then copy it into extensions folder of your future build
  314. # [17:06] <@davidb> oh
  315. # [17:06] <@surkov> and then add ac_add_options --enable-extensions=default,inspector into your .mozconfig
  316. # [17:06] <@davidb> thanks!
  317. # [17:06] <@surkov> if you have one tree for everything then you can clone into extensions folder
  318. # [17:07] <@surkov> I usually copy sources for new build
  319. # [17:07] <@davidb> i'm not sure i got you
  320. # [17:07] <@surkov> so I can continue to work on patches without affecting on the build
  321. # [17:07] <@davidb> ah
  322. # [17:07] * Quits: %bwinton (bwinton@F2D29657.F60B0462.67AC9B1.IP) (Quit: Leaving...)
  323. # [17:07] * bwinton_away is now known as bwinton
  324. # [17:12] <@davidb> surkov: is there a make step?
  325. # [17:13] * Joins: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com)
  326. # [17:14] <@tbsaunde> now, if you want to have fun write a macro atomically such that atomically { ... } does the right thing
  327. # [17:15] <@surkov> davidb: no, just add option into .mozconfig
  328. # [17:16] <@surkov> tbsaunde: example?
  329. # [17:17] <@tbsaunde> surkov: you doo #define atomically blah such that atomically { do_something_useful_that_needs_to_be_atomic(); |} is fine
  330. # [17:17] <@MarcoZ> Bah, Bugzilla is down again!
  331. # [17:17] <@tbsaunde> not sure how else you can explain that problem with out giving away the answer
  332. # [17:18] <@tbsaunde> there all telling you to go work on your existing patches :)
  333. # [17:19] * Quits: @jprmc (jprmc@moz-9CE00FC0.holiday-inn-express.mozilla.hq) (Ping timeout)
  334. # [17:20] * @davidb builds with inspector
  335. # [17:40] <@davidb> MarcoZ: were you pinged at all to check accessibility of mozilla.org/story ?
  336. # [17:40] <@davidb> just curious
  337. # [17:44] <@davidb> it could be that webdev has absorbed some a11y qa
  338. # [17:50] <@MarcoZ> davidb: No I wasn't pinged about mozilla.org/story at all.
  339. # [17:51] <firebot> marco.zehe@googlemail.com changed the Status on bug 658272 from RESOLVED to VERIFIED.
  340. # [17:51] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=658272 nor, --, mozilla11, surkov.alexander, VERI FIXED, Implement figure and figcaption accessibility
  341. # [17:51] <firebot> maglione.k@gmail.com changed the Status on bug 705564 from UNCONFIRMED to RESOLVED.
  342. # [17:52] <firebot> maglione.k@gmail.com set the Resolution field on bug 705564 to WORKSFORME.
  343. # [17:52] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=705564 nor, --, ---, nobody, RESO WORKSFORME, Keybindings for keys F16, F25 are unavailable to addons
  344. # [17:52] * Joins: jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com)
  345. # [17:52] * ChanServ sets mode: +o jprmc
  346. # [17:56] * Joins: Flyingmana_ (Mana@moz-9C0B5A6C.adsl.alicedsl.de)
  347. # [17:57] * Quits: Flyingmana (Mana@moz-12184E52.adsl.alicedsl.de) (Ping timeout)
  348. # [17:58] <@MarcoZ> davidb: I just looked, and this looks all pretty standard. The embedded Youtube video is the only bit which may give trouble. The standard is inaccessible Flash, but if you have HTML5 enabled on Youtube, you get video playback controls.
  349. # [18:00] <@davidb> i was more interested in whether you were pinged
  350. # [18:03] <@MarcoZ> davidb: As I said...nope.
  351. # [18:04] <@davidb> MarcoZ: ah yes, sorry
  352. # [18:04] * @davidb dials into w3 call
  353. # [18:15] <firebot> jigneshhk1992@gmail.com cancelled review?(surkov.alexander@gm ail.com) for attachment 579280 on bug 706369.
  354. # [18:15] <firebot> jigneshhk1992@gmail.com requested review from surkov.alexander@gm ail.com for attachment 579326 on bug 706369.
  355. # [18:15] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=706369 nor, --, ---, jigneshhk1992, NEW, don't use nsIContent::GetChildAt to iterate through children
  356. # [18:21] * Parts: richardschwerdtfeger (richardsch@BE0AFBFF.9DD4DBBF.6A7A197.IP)
  357. # [18:25] <firebot> surkov.alexander@gmail.com denied review for attachment 579326 on bug 706369.
  358. # [18:44] <firebot> jigneshhk1992@gmail.com requested review from surkov.alexander@gm ail.com for attachment 579342 on bug 706369.
  359. # [18:44] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=706369 nor, --, ---, jigneshhk1992, NEW, don't use nsIContent::GetChildAt to iterate through children
  360. # [18:48] * davidb changes topic to 'we are experiencing service outages | "build the web for everyone" | release schedule: http://mzl.la/LNc0W'
  361. # [18:49] * davidb sets mode: +o eeejay
  362. # [18:50] <firebot> surkov.alexander@gmail.com granted review for attachment 579342 on bug 706369.
  363. # [18:55] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
  364. # [18:58] <firebot> jigneshhk1992@gmail.com requested review from surkov.alexander@gm ail.com for attachment 579347 on bug 706369.
  365. # [18:58] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=706369 nor, --, ---, jigneshhk1992, NEW, don't use nsIContent::GetChildAt to iterate through children
  366. # [19:02] * jess is now known as iamjessklein-away
  367. # [19:02] * iamjessklein-away is now known as jess
  368. # [19:03] <firebot> josh@joshmatthews.net cancelled review?(surkov.alexander@gm ail.com) for attachment 579347 on bug 706369.
  369. # [19:06] * Joins: hhillen (hhillen@moz-2A42E9B3.upc-d.chello.nl)
  370. # [19:11] <firebot> bmo@edmorley.co.uk changed the Status on bug 707846 from NEW to RESOLVED.
  371. # [19:11] <firebot> bmo@edmorley.co.uk set the Resolution field on bug 707846 to FIXED.
  372. # [19:11] <firebot> bmo@edmorley.co.uk changed the Target Milestone on bug 707846 from --- to mozilla11.
  373. # [19:11] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=707846 nor, --, mozilla11, hub, RESO FIXED, [mac] remove unused nsAccessibleWrap::GetUnignoredChildCount()
  374. # [19:11] <firebot> bmo@edmorley.co.uk changed the Status on bug 689105 from NEW to RESOLVED.
  375. # [19:11] <firebot> bmo@edmorley.co.uk set the Resolution field on bug 689105 to FIXED.
  376. # [19:11] <firebot> bmo@edmorley.co.uk changed the Target Milestone on bug 689105 from --- to mozilla11.
  377. # [19:11] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=689105 maj, --, mozilla11, hub, RESO FIXED, Accessibility in main window broken for VoiceOver, VO doesn't see anything but the title bar and its
  378. # [19:11] <firebot> bmo@edmorley.co.uk changed the Status on bug 704852 from ASSIGNED to RESOLVED.
  379. # [19:12] <firebot> bmo@edmorley.co.uk set the Resolution field on bug 704852 to FIXED.
  380. # [19:12] <firebot> bmo@edmorley.co.uk changed the Target Milestone on bug 704852 from --- to mozilla11.
  381. # [19:12] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=704852 nor, --, mozilla11, surkov.alexander, RESO FIXED, Calculate compatibility mode when accessibility starts
  382. # [19:15] * Joins: hub (hub@21B7B9F2.B87E9213.6E712CE2.IP)
  383. # [19:15] * ChanServ sets mode: +o hub
  384. # [19:21] * Quits: atulagrwl (atul@944C5E56.71A613B7.A3D1B221.IP) (Quit: Leaving)
  385. # [19:22] * Quits: victorporof (victorporo@E8764A15.F4308271.4873E54D.IP) (Ping timeout)
  386. # [19:22] * Quits: @surkov (surkov@178F4FC2.D2B288F8.5D3F4C44.IP) (Quit: surkov)
  387. # [19:41] * ehsan_ is now known as ehsan
  388. # [19:44] <@MarcoZ> hub: Congratulations on the first landed Mac patches!
  389. # [19:45] * khuey|away is now known as khuey
  390. # [19:46] * Quits: @hub (hub@21B7B9F2.B87E9213.6E712CE2.IP) (Ping timeout)
  391. # [19:47] <@davidb> woot!
  392. # [19:47] * Joins: hub (hub@21B7B9F2.B87E9213.6E712CE2.IP)
  393. # [19:47] * ChanServ sets mode: +o hub
  394. # [19:47] <firebot> jigneshhk1992@gmail.com requested review from surkov.alexander@gm ail.com for attachment 579368 on bug 706369.
  395. # [19:47] <@davidb> and so it begins
  396. # [19:47] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=706369 nor, --, ---, jigneshhk1992, NEW, don't use nsIContent::GetChildAt to iterate through children
  397. # [19:50] <@MarcoZ> Yeah, once Bug 707852 and bug 703770 are also addressed, and we have a try-server or other build we can distribute, I'd like to do extensive testing myself, and if this checks out, also give something to a couple of users.
  398. # [19:50] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=707852 nor, --, ---, nobody, NEW, [mac] remove the Objective-C wrapper
  399. # [19:50] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=703770 nor, --, ---, hub, NEW, Mac Accessibility Verifier errors
  400. # [19:50] <@MarcoZ> unless surkov and hub think this is not ready for that kind of exposure yet.
  401. # [19:54] * Quits: hhillen (hhillen@moz-2A42E9B3.upc-d.chello.nl) (Quit: hhillen)
  402. # [19:54] <@davidb> well, your testing can help that decision
  403. # [19:56] * khuey is now known as khuey|away
  404. # [20:05] <@MarcoZ> davidb: Yes, hope Hub gets me that try server build soon, would love to give it a spin!
  405. # [20:05] <@MarcoZ> But for today, I'm off. See you tomorrow!
  406. # [20:05] <@davidb> ciao@!
  407. # [20:05] <@davidb> -@
  408. # [20:06] * Quits: @MarcoZ (marco.zehe@moz-404D9980.dip.t-dialin.net) (Quit: See you tomorrow!)
  409. # [20:08] <@hub> yeah I have to figure out where it located :-/
  410. # [20:21] <@davidb> hard without email i imagine
  411. # [20:22] <@davidb> hub: perhaps here somewhere http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/
  412. # [20:23] <@eeejay> YES!
  413. # [20:23] <@davidb> eeejay: that sounds positive
  414. # [20:24] * khuey|away is now known as khuey
  415. # [20:24] * @eeejay is happy about birch moving to mc
  416. # [20:27] <@davidb> ah yes
  417. # [20:29] <@hub> davidb: yeah.
  418. # [20:29] <@hub> eeejay: that's awesome
  419. # [20:33] <@hub> ok I found them
  420. # [20:33] <@hub> the do
  421. # [20:34] <@hub> the have a11y in it
  422. # [20:34] <@hub> that's a start
  423. # [20:36] <@davidb> coolio
  424. # [20:38] * Quits: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com) (Quit: nhirata)
  425. # [20:39] <@hub> I should file a bz to remind us to enable a11y in the mac build
  426. # [20:44] * Joins: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com)
  427. # [20:50] <@davidb> yep
  428. # [20:51] * Quits: @jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com) (Input/output error)
  429. # [20:52] * Joins: jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com)
  430. # [20:52] * ChanServ sets mode: +o jprmc
  431. # [21:19] * khuey is now known as khuey|away
  432. # [21:26] <@hub> ok, I had a scare
  433. # [21:26] <@hub> that the bug resurfaced
  434. # [21:27] <@hub> but no
  435. # [21:27] <@hub> try build didn't pickup the mozconfig change
  436. # [21:27] <@davidb> aha
  437. # [21:28] <@hub> the suffix is apparently "osx64" not "macosx64"
  438. # [21:39] <firebot> hub@mozilla.com requested checkin from the wind for attachment 579413 on bug 455443.
  439. # [21:39] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=455443 nor, --, ---, hub, ASSI, cache the parent for the accessibilityAttributeValue(NSAccessibilityParentAttribute)
  440. # [21:40] * khuey|away is now known as khuey
  441. # [21:50] * Quits: jaws (u2871@moz-160C58C6.com) (Max SendQ exceeded)
  442. # [21:50] * Joins: jaws (u2871@moz-160C58C6.com)
  443. # [21:57] <@tbsaunde> hub: + mParent = GetClosestInterestingAccessible(nativeParent);
  444. # [21:57] <@tbsaunde> + return mParent;
  445. # [21:57] <@tbsaunde> really?
  446. # [21:58] <@hub> tbsaunde: what's wrong?
  447. # [21:58] <@tbsaunde> hub: why not return mParent = GetClosestInterestAccessible()?
  448. # [21:59] <@hub> tbsaunde: to be honest? because that even less readable. and it does nothing to the output machine code
  449. # [22:00] <@tbsaunde> hub: I'd disagree
  450. # [22:01] <@tbsaunde> not with the result, but just as readable and less lines
  451. # [22:03] <@tbsaunde> hub: why do you think using lvalues as rvalues is unreadable?
  452. # [22:05] <@hub> doing TWO statement in one line just blunder readability
  453. # [22:05] <@hub> what if one really meant return mParent == GetClosestInterestingAccessible(nativeParent);
  454. # [22:05] <@hub> it would immediately pop
  455. # [22:05] <@tbsaunde> pop?
  456. # [22:06] <@tbsaunde> it returns a pointer not a bool so that's a type error for one thing
  457. # [22:06] <@tbsaunde> also don't be bad
  458. # [22:07] <@hub> I have seen so many bugs caused by sloppy programming style like combining statements, etc
  459. # [22:07] <@hub> that I tend to try to be strict on it
  460. # [22:07] <@hub> including mandating { and }
  461. # [22:07] <@hub> which not everybody seems to agree
  462. # [22:09] <@tbsaunde> because is silly :)
  463. # [22:09] <@tbsaunde> I can't honestly think of that many bugs that are related to syntax
  464. # [22:10] <@tbsaunde> (even when talking to ta's grading horrible kernels)
  465. # [22:10] * Quits: @eeejay (eitan@moz-EE2D6C10.monotonous.org) (Input/output error)
  466. # [22:11] * Joins: eeejay (eitan@moz-EE2D6C10.monotonous.org)
  467. # [22:26] * Quits: @davidb (davidb@F2D29657.F60B0462.67AC9B1.IP) (Quit: davidb)
  468. # [22:26] * khuey is now known as khuey|away
  469. # [22:53] * khuey|away is now known as khuey
  470. # [22:53] * Parts: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
  471. # [23:02] * jess is now known as iamjessklein-away
  472. # [23:03] * Quits: @jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com) (Ping timeout)
  473. # [23:04] * khuey is now known as khuey|away
  474. # [23:06] * Joins: jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com)
  475. # [23:06] * ChanServ sets mode: +o jprmc
  476. # [23:11] * Joins: davidb (davidb@moz-A4A01B28.eng.wind.ca)
  477. # [23:11] * ChanServ sets mode: +qo davidb davidb
  478. # [23:29] * bwinton is now known as bwinton_away
  479. # [23:29] * iamjessklein-away is now known as jess
  480. # [23:52] * Joins: davidb_ (davidb@moz-976CD2.dsl.bell.ca)
  481. # [23:52] * Quits: davidb_ (davidb@moz-976CD2.dsl.bell.ca) (Input/output error)
  482. # Session Close: Wed Dec 07 00:00:00 2011

The end :)