/irc-logs / mozilla / #accessibility / 2012-11-09 / end

Options:

  1. # Session Start: Fri Nov 09 00:00:00 2012
  2. # Session Ident: #accessibility
  3. # [00:00] * Quits: icaaq (Adium@moz-30948322.bb.sky.com) (Quit: Leaving.)
  4. # [00:07] * Quits: @hub (hub@F2D29657.F60B0462.67AC9B1.IP) (Ping timeout)
  5. # [00:23] * Quits: nhirata (anonymous@52A1524D.4E7EDEC9.632B8C24.IP) (Connection reset by peer)
  6. # [00:23] * Joins: nhirata (anonymous@52A1524D.4E7EDEC9.632B8C24.IP)
  7. # [00:43] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  8. # [01:42] * Quits: nhirata (anonymous@52A1524D.4E7EDEC9.632B8C24.IP) (Ping timeout)
  9. # [01:43] * Joins: nhirata (anonymous@52A1524D.4E7EDEC9.632B8C24.IP)
  10. # [01:49] * Joins: Jamie (Instantbir@moz-CA26021.jantrid.net)
  11. # [01:55] * Quits: nhirata (anonymous@52A1524D.4E7EDEC9.632B8C24.IP) (Ping timeout)
  12. # [02:17] * Quits: @jprmc (jprmc@moz-542588AA.cpe.net.cable.rogers.com) (Ping timeout)
  13. # [02:31] * Quits: Jamie (Instantbir@moz-CA26021.jantrid.net) (Ping timeout)
  14. # [02:44] * Quits: habber (habber@moz-2DC46876.nyc.res.rr.com) (Connection reset by peer)
  15. # [02:44] * Joins: habber (habber@moz-2DC46876.nyc.res.rr.com)
  16. # [02:45] * Quits: habber (habber@moz-2DC46876.nyc.res.rr.com) (Quit: habber)
  17. # [02:54] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
  18. # [02:57] * Joins: Jamie (Instantbir@moz-CA26021.jantrid.net)
  19. # [02:57] * Joins: jprmc (jprmc@moz-542588AA.cpe.net.cable.rogers.com)
  20. # [02:57] * ChanServ sets mode: +o jprmc
  21. # [03:01] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  22. # [03:02] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
  23. # [03:04] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  24. # [03:06] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
  25. # [03:11] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  26. # [03:14] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
  27. # [03:24] * Joins: surkov (surkov@D08E2EFF.E35E3D12.E17943EE.IP)
  28. # [03:24] * ChanServ sets mode: +o surkov
  29. # [03:36] * Quits: scott_gonzalez (scott_gonz@moz-C35301AF.lightspeed.rlghnc.sbcglobal.net) (Quit: scott_gonzalez)
  30. # [03:46] * Joins: drexler (chatzilla@moz-AD4BBA18.hsd1.vt.comcast.net)
  31. # [03:47] <tbsaunde> surkov: hey
  32. # [03:47] <@surkov> hey, tbsaunde
  33. # [03:48] <tbsaunde> surkov: so, I'm trying to understand bug 809871 but it seems like nsTreeBoxObject::GetTreeBody() should be called with the default arg value of false, and so reflow shouldn't happen
  34. # [03:48] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=809871 nor, --, ---, surkov.alexander, ASSI, XUL tree accessible creation may flush layout
  35. # [03:48] <@surkov> funny
  36. # [03:48] * Joins: scott_gonzalez (scott_gonz@moz-6B7D4671.nc.res.rr.com)
  37. # [03:48] <@surkov> I missed that
  38. # [03:48] * Joins: scottgonzalez (scott_gonz@moz-6B7D4671.nc.res.rr.com)
  39. # [03:48] <@surkov> I assumed it was true
  40. # [03:49] <tbsaunde> surkov: msvc debugger can't show you value of arguments?
  41. # [03:49] * Quits: Jamie (Instantbir@moz-CA26021.jantrid.net) (Quit: Instantbird 1.3a1pre -- http://www.instantbird.com)
  42. # [03:50] <@surkov> I didn't look at them
  43. # [03:50] <@surkov> but I don't have reliable way to reproduce it
  44. # [03:50] * Quits: scott_gonzalez (scott_gonz@moz-6B7D4671.nc.res.rr.com) (Ping timeout)
  45. # [03:50] <tbsaunde> surkov: I see, how did you find it?
  46. # [03:50] <@surkov> tbsaunde: via DOMi
  47. # [03:50] <@surkov> and NVDA running
  48. # [03:51] <@surkov> tbsaunde: I think I should try to catch it one more time before we get the patch
  49. # [03:51] <tbsaunde> surkov: I see, we crashed or what?
  50. # [03:51] <@surkov> nope, just asserted
  51. # [03:51] <tbsaunde> surkov: ah
  52. # [03:52] <tbsaunde> fwiw in general I think using concrete types for that stuff is a good idea, just want to understand how it fixes the bug
  53. # [03:53] <@firebot> surkov.alexander@gmail.com cancelled review?(trev.saunders@gmail .com) for attachment 679692 on bug 809871.
  54. # [03:53] <@surkov> tbsaunde: it doesn't I think
  55. # [03:54] <tbsaunde> surkov: that would be my guess to :)
  56. # [03:54] <@surkov> :)
  57. # [03:54] <@surkov> it seems nsBoxObject::GetFrame(false) can flush layout anyway
  58. # [03:54] <tbsaunde> surkov: any reason you used the nsPIBoxObject thing instead of making nsITreeBoxObject [builtinclass]
  59. # [03:54] <@surkov> tbsaunde: see http://mxr.mozilla.org/mozilla-central/source/layout/xul/base/src/nsBoxObject.cpp#117
  60. # [03:55] <@surkov> tbsaunde: it seems I should pass 'true' instead 'false' :)
  61. # [03:57] <@surkov> tbsaunde: nope, it will flush it in either case
  62. # [03:57] <tbsaunde> surkov: wouldn't that still have reflow happen in some cases?
  63. # [03:57] <tbsaunde> don't you need to fix layout stuff?
  64. # [03:57] <@surkov> maybe, dunno how
  65. # [03:57] <tbsaunde> surkov: yeah, well looks like not our bug anyway :)
  66. # [03:58] <tbsaunde> surkov: ask roc / bz or someone?
  67. # [03:59] <@surkov> agree, we need cc them
  68. # [04:04] <@firebot> surkov.alexander@gmail.com changed the Assignee on bug 809871 from surkov.alexander@gmail.com to nobody@mozilla.org.
  69. # [04:04] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=809871 nor, --, ---, nobody, ASSI, XUL tree accessible creation may flush layout
  70. # [04:09] <tbsaunde> surkov: btw I bet that bug can be exploited somehow or another...
  71. # [04:10] <@surkov> quite possible
  72. # [04:25] <@surkov> tbsaunde: about for (uint32_t idx = count - 1; idx < count; idx--)
  73. # [04:25] <@surkov> not sure how to make it working
  74. # [04:26] <@firebot> surkov.alexander@gmail.com changed the Assignee on bug 789245 from icepyramid@gmail.com to nobody@mozilla.org.
  75. # [04:26] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=789245 nor, --, ---, nobody, NEW, add TableCellAccessible::Row()
  76. # [04:33] * surkov sets mode: +o scottgonzalez
  77. # [04:33] * surkov sets mode: -o scottgonzalez
  78. # [04:34] * surkov sets mode: +o tbsaunde
  79. # [04:47] * Quits: ehsan (ehsan@F2D29657.F60B0462.67AC9B1.IP) (Input/output error)
  80. # [05:01] * Joins: Jamie (Instantbir@moz-CA26021.jantrid.net)
  81. # [05:07] * Quits: scottgonzalez (scott_gonz@moz-6B7D4671.nc.res.rr.com) (Quit: scottgonzalez)
  82. # [05:08] <@surkov> Jamie: hello
  83. # [05:09] <Jamie> surkov: hi!
  84. # [05:09] <@surkov> Jamie: a question, if we would fire state change for traversed state on the link then how would it be?
  85. # [05:09] <@surkov> Jamie: https://bugzilla.mozilla.org/show_bug.cgi?id=757774#c24
  86. # [05:09] <@firebot> Bug 757774 nor, --, ---, nobody, NEW, Intermittent TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/states/test_link.htm
  87. # [05:11] <Jamie> heh, sorry, I didn't realise I was copied on that bug because I get open/closed notifications for all a11y bugs and just assumed it wasn't relevant to me
  88. # [05:11] <Jamie> so deleted the emails without reading them :)
  89. # [05:11] <@surkov> Jamie: yeah, sometimes it happens :)
  90. # [05:12] <@surkov> especially on spamming bugs like intermittent failures
  91. # [05:13] <Jamie> hold on a second
  92. # [05:14] <Jamie> surkov: commented, but in short, yes, it's a good idea.
  93. # [05:14] <@surkov> Jamie: cool, thank you
  94. # [05:14] <Jamie> sorry about ignoring the bug :)
  95. # [05:17] <@surkov> nm :)
  96. # [05:47] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  97. # [05:48] <Jamie> surkov: something seems to be really broken with Firefox's calculation of invisible and offscreen states lately
  98. # [05:48] <Jamie> surkov: I'm assuming this is the point of that "expose intended visibility" bug?
  99. # [05:48] <@surkov> Jamie: do you mean nightly?
  100. # [05:48] <Jamie> surkov: yes, though I think it's been happening fo ra few months, so probably not just nightly now
  101. # [05:49] <@surkov> Jamie: iirc we caught one good bug and fixed it in Firefox 18 so I thought we should be in good shape
  102. # [05:49] <@surkov> Jamie: can you try to catch it?
  103. # [05:49] <Jamie> surkov: actually, hmm
  104. # [05:50] <Jamie> surkov: should offscreen (e.g. left at -9000px) be exposed with offscreen state? that is controversial I guess, since some people use that to hide text but make it appear for sr users
  105. # [05:50] <@surkov> Jamie: it seems like it should be offscreen
  106. # [05:50] <@surkov> it's not I guess?
  107. # [05:50] <Jamie> surkov: arrrg
  108. # [05:50] <Jamie> surkov: sorry, I think I'm just braindead
  109. # [05:51] * Jamie must have been looking at the wrong node or something <embarrassed>
  110. # [05:51] <@surkov> if you get a test case then please file a bug
  111. # [05:52] <Jamie> surkov: okay, so one example that *is* valid though: in the Help -> About dialog, there are these property pages for the various phases of update. They do have the offscreen state, but I would have thought they should isnteaed be invisible
  112. # [05:52] <Jamie> they're not just "in the background", as such; they're not visible at all
  113. # [05:53] <@surkov> Jamie: they aren't, perhaps that fall into the case of tabs
  114. # [05:54] <@surkov> Jamie: file a bug? ;)
  115. # [05:54] <@surkov> big changes always give regressions
  116. # [05:55] * Jamie nods
  117. # [05:55] <Jamie> sorry about the first false alarm :)
  118. # [05:56] <Jamie> surkov: Finally, there's one edge case I'm not sure how to handle. Imagine someone has done an offscreen live region so that it will be spoken by sr but not seen by sighted users.
  119. # [05:56] <Jamie> surkov: We don't want to speak live regions in background tabs, so we ignore offscreen. However, this one is *always* offscreen because the author put it there
  120. # [05:57] <Jamie> so we end up with a nasty lose lose situation :)
  121. # [05:57] <@surkov> Jamie: it seems we need in-background-tab state :)
  122. # [05:57] * Jamie nods; I was heading in that direction of thought as well
  123. # [05:58] <Jamie> or yet another kind of "for sr users only" :)
  124. # [05:58] <@surkov> if this visibility state maps into accessible states well at all
  125. # [05:58] <@surkov> for example, invisible + offscreen doesn't make a lot of sense
  126. # [05:58] <Jamie> the positioning text off-screen thing is a really nasty hack that kinda needs to die, but it exists and we're stuck with it
  127. # [05:59] <Jamie> imo, -5000px isn't really off-screen. off-screen to me means "scrolled out of view by the user" (including background tab) or similar
  128. # [06:00] <@surkov> Jamie: or can you filter them by tab documents? if it's in this tab document then you announce it, if it's not then not
  129. # [06:00] <Jamie> surkov: we could, but that'd be very firefox specific code. currently, our live regions code is "IA2 generic", if that makes sense
  130. # [06:00] <Jamie> I'd prefer to keep it that way if I can
  131. # [06:00] <@surkov> I see
  132. # [06:00] <Jamie> Nm, I'll have to think about it
  133. # [06:01] * Joins: ehsan (ehsan@moz-C48D29C4.cable.teksavvy.com)
  134. # [06:01] <Jamie> Arguably, off-screen live regions are a really bad idea anyway, but I've just noticed Facebook are doing it
  135. # [06:19] <@surkov> Jamie: btw, don't you use firefox keyword, it'd handy to see needsExternalFix + firefox + major bugs
  136. # [06:35] <@firebot> surkov.alexander@gmail.com cancelled review?(trev.saunders@gmail .com) for attachment 677685 on bug 807911.
  137. # [06:35] <@firebot> surkov.alexander@gmail.com requested review from trev.saunders@gmail .com for attachment 679970 on bug 807911.
  138. # [06:35] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=807911 nor, --, ---, surkov.alexander, ASSI, whittle mutation events processing
  139. # [06:39] <@firebot> surkov.alexander@gmail.com cancelled review?(trev.saunders@gmail .com) for attachment 679970 on bug 807911.
  140. # [06:39] <@firebot> surkov.alexander@gmail.com requested review from trev.saunders@gmail .com for attachment 679971 on bug 807911.
  141. # [06:46] * Quits: drexler (chatzilla@moz-AD4BBA18.hsd1.vt.comcast.net) (Quit: ChatZilla 0.9.89-rdmsoft [XULRunner 1.9.0.17/2009122204])
  142. # [07:06] <@firebot> surkov.alexander@gmail.com changed the Assignee on bug 757774 from nobody@mozilla.org to surkov.alexander@gmail.com.
  143. # [07:06] <@firebot> surkov.alexander@gmail.com requested review from trev.saunders@gmail .com for attachment 679975 on bug 757774.
  144. # [07:06] <@firebot> surkov.alexander@gmail.com changed the Status on bug 757774 from NEW to ASSIGNED.
  145. # [07:06] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=757774 nor, --, ---, nobody, NEW, Intermittent TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/states/test_link.htm
  146. # [07:14] <@tbsaunde> surkov: still confused about the for () thing?
  147. # [07:14] <@surkov> tbsaunde: nope, I commented in the bug
  148. # [07:23] <@surkov> tbsaunde: btw, it'd be nice to answer my comment in fixed bug 770564 ;)
  149. # [07:23] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=770564 maj, --, mozilla19, trev.saunders, RESO FIXED, The more text in a textarea, the more time it takes to update the content
  150. # [07:23] <@surkov> and file good first bugs :)
  151. # [07:27] * Quits: Jamie (Instantbir@moz-CA26021.jantrid.net) (Quit: Instantbird 1.3a1pre -- http://www.instantbird.com)
  152. # [07:34] * Quits: ehsan (ehsan@moz-C48D29C4.cable.teksavvy.com) (Connection reset by peer)
  153. # [07:34] * Joins: ehsan_ (ehsan@moz-C48D29C4.cable.teksavvy.com)
  154. # [07:41] * Quits: gavin (gavin@moz-59682A1.dmz.scl3.mozilla.com) (Input/output error)
  155. # [07:41] * Joins: gavin (gavin@moz-59682A1.dmz.scl3.mozilla.com)
  156. # [07:44] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Client exited)
  157. # [08:01] * Quits: @tbsaunde (tbsaunde@moz-59682A1.dmz.scl3.mozilla.com) (Ping timeout)
  158. # [08:01] * Joins: tbsaunde (tbsaunde@moz-59682A1.dmz.scl3.mozilla.com)
  159. # [08:06] <tbsaunde> surkov: yeah, I remember, sorry I half let it slip in queue
  160. # [08:10] <@surkov> I see, sometimes things hangs in queue
  161. # [08:10] * Joins: victorporof (victorporo@311CA19C.477C3F8E.79933D60.IP)
  162. # [08:20] * Joins: Stevef (chatzilla@moz-CD0F47B5.cable.virginmedia.com)
  163. # [08:57] <@firebot> surkov.alexander@gmail.com cancelled review?(trev.saunders@gmail .com) for attachment 679971 on bug 807911.
  164. # [08:57] <@firebot> surkov.alexander@gmail.com requested review from trev.saunders@gmail .com for attachment 679995 on bug 807911.
  165. # [08:57] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=807911 nor, --, ---, surkov.alexander, ASSI, whittle mutation events processing
  166. # [09:05] * Quits: Stevef (chatzilla@moz-CD0F47B5.cable.virginmedia.com) (Ping timeout)
  167. # [09:18] * Joins: Stevef (chatzilla@moz-CD0F47B5.cable.virginmedia.com)
  168. # [09:33] * Joins: margle (margle@E3F0FA38.AA35ED30.A77EFE04.IP)
  169. # [10:20] * Joins: icaaq (Adium@CA8A6033.7DCD925.CE255B90.IP)
  170. # [10:24] * Quits: icaaq (Adium@CA8A6033.7DCD925.CE255B90.IP) (Ping timeout)
  171. # [10:24] * Joins: icaaq (Adium@CA8A6033.7DCD925.CE255B90.IP)
  172. # [10:32] * Quits: Stevef (chatzilla@moz-CD0F47B5.cable.virginmedia.com) (Ping timeout)
  173. # [10:33] * Joins: Stevef (chatzilla@moz-CD0F47B5.cable.virginmedia.com)
  174. # [10:36] * Quits: icaaq (Adium@CA8A6033.7DCD925.CE255B90.IP) (Ping timeout)
  175. # [10:38] * Joins: icaaq (Adium@7C3FFB6A.7DCD925.CE255B90.IP)
  176. # [10:48] * Quits: @surkov (surkov@D08E2EFF.E35E3D12.E17943EE.IP) (Quit: surkov)
  177. # [10:51] * Quits: icaaq (Adium@7C3FFB6A.7DCD925.CE255B90.IP) (Ping timeout)
  178. # [10:53] * Joins: icaaq (Adium@moz-D110ACAA.cust.telenor.se)
  179. # [11:01] * Quits: icaaq (Adium@moz-D110ACAA.cust.telenor.se) (Quit: Leaving.)
  180. # [11:19] * Quits: victorporof (victorporo@311CA19C.477C3F8E.79933D60.IP) (Connection reset by peer)
  181. # [11:19] * Joins: victorporof_ (victorporo@311CA19C.477C3F8E.79933D60.IP)
  182. # [11:20] * Quits: margle (margle@E3F0FA38.AA35ED30.A77EFE04.IP) (Quit: Computer has gone to sleep.)
  183. # [11:23] * Quits: victorporof_ (victorporo@311CA19C.477C3F8E.79933D60.IP) (Connection reset by peer)
  184. # [11:23] * Joins: victorporof (victorporo@311CA19C.477C3F8E.79933D60.IP)
  185. # [12:02] * Joins: surkov (surkov@D08E2EFF.E35E3D12.E17943EE.IP)
  186. # [12:02] * ChanServ sets mode: +o surkov
  187. # [12:08] * Joins: margle (margle@E3F0FA38.AA35ED30.A77EFE04.IP)
  188. # [12:59] <@firebot> New Core - Disability Access APIs bug 810260 filed by surkov.alexander@gmail.com.
  189. # [13:00] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=810260 nor, --, ---, nobody, NEW, xul:deck hidden pages shouldn't be offscreen
  190. # [13:11] * Quits: khuey|away (khuey@moz-59682A1.dmz.scl3.mozilla.com) (Input/output error)
  191. # [13:11] * Joins: khuey|away (khuey@moz-59682A1.dmz.scl3.mozilla.com)
  192. # [13:11] * Quits: gavin (gavin@moz-59682A1.dmz.scl3.mozilla.com) (Input/output error)
  193. # [13:11] * Joins: gavin (gavin@moz-59682A1.dmz.scl3.mozilla.com)

The end :)