/irc-logs / mozilla / #accessibility / 2012-11-09 / end
Options:
- # Session Start: Fri Nov 09 00:00:00 2012
- # Session Ident: #accessibility
- # [00:00] * Quits: icaaq (Adium@moz-30948322.bb.sky.com) (Quit: Leaving.)
- # [00:07] * Quits: @hub (hub@F2D29657.F60B0462.67AC9B1.IP) (Ping timeout)
- # [00:23] * Quits: nhirata (anonymous@52A1524D.4E7EDEC9.632B8C24.IP) (Connection reset by peer)
- # [00:23] * Joins: nhirata (anonymous@52A1524D.4E7EDEC9.632B8C24.IP)
- # [00:43] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # [01:42] * Quits: nhirata (anonymous@52A1524D.4E7EDEC9.632B8C24.IP) (Ping timeout)
- # [01:43] * Joins: nhirata (anonymous@52A1524D.4E7EDEC9.632B8C24.IP)
- # [01:49] * Joins: Jamie (Instantbir@moz-CA26021.jantrid.net)
- # [01:55] * Quits: nhirata (anonymous@52A1524D.4E7EDEC9.632B8C24.IP) (Ping timeout)
- # [02:17] * Quits: @jprmc (jprmc@moz-542588AA.cpe.net.cable.rogers.com) (Ping timeout)
- # [02:31] * Quits: Jamie (Instantbir@moz-CA26021.jantrid.net) (Ping timeout)
- # [02:44] * Quits: habber (habber@moz-2DC46876.nyc.res.rr.com) (Connection reset by peer)
- # [02:44] * Joins: habber (habber@moz-2DC46876.nyc.res.rr.com)
- # [02:45] * Quits: habber (habber@moz-2DC46876.nyc.res.rr.com) (Quit: habber)
- # [02:54] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
- # [02:57] * Joins: Jamie (Instantbir@moz-CA26021.jantrid.net)
- # [02:57] * Joins: jprmc (jprmc@moz-542588AA.cpe.net.cable.rogers.com)
- # [02:57] * ChanServ sets mode: +o jprmc
- # [03:01] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # [03:02] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
- # [03:04] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # [03:06] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
- # [03:11] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # [03:14] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
- # [03:24] * Joins: surkov (surkov@D08E2EFF.E35E3D12.E17943EE.IP)
- # [03:24] * ChanServ sets mode: +o surkov
- # [03:36] * Quits: scott_gonzalez (scott_gonz@moz-C35301AF.lightspeed.rlghnc.sbcglobal.net) (Quit: scott_gonzalez)
- # [03:46] * Joins: drexler (chatzilla@moz-AD4BBA18.hsd1.vt.comcast.net)
- # [03:47] <tbsaunde> surkov: hey
- # [03:47] <@surkov> hey, tbsaunde
- # [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
- # [03:48] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=809871 nor, --, ---, surkov.alexander, ASSI, XUL tree accessible creation may flush layout
- # [03:48] <@surkov> funny
- # [03:48] * Joins: scott_gonzalez (scott_gonz@moz-6B7D4671.nc.res.rr.com)
- # [03:48] <@surkov> I missed that
- # [03:48] * Joins: scottgonzalez (scott_gonz@moz-6B7D4671.nc.res.rr.com)
- # [03:48] <@surkov> I assumed it was true
- # [03:49] <tbsaunde> surkov: msvc debugger can't show you value of arguments?
- # [03:49] * Quits: Jamie (Instantbir@moz-CA26021.jantrid.net) (Quit: Instantbird 1.3a1pre -- http://www.instantbird.com)
- # [03:50] <@surkov> I didn't look at them
- # [03:50] <@surkov> but I don't have reliable way to reproduce it
- # [03:50] * Quits: scott_gonzalez (scott_gonz@moz-6B7D4671.nc.res.rr.com) (Ping timeout)
- # [03:50] <tbsaunde> surkov: I see, how did you find it?
- # [03:50] <@surkov> tbsaunde: via DOMi
- # [03:50] <@surkov> and NVDA running
- # [03:51] <@surkov> tbsaunde: I think I should try to catch it one more time before we get the patch
- # [03:51] <tbsaunde> surkov: I see, we crashed or what?
- # [03:51] <@surkov> nope, just asserted
- # [03:51] <tbsaunde> surkov: ah
- # [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
- # [03:53] <@firebot> surkov.alexander@gmail.com cancelled review?(trev.saunders@gmail .com) for attachment 679692 on bug 809871.
- # [03:53] <@surkov> tbsaunde: it doesn't I think
- # [03:54] <tbsaunde> surkov: that would be my guess to :)
- # [03:54] <@surkov> :)
- # [03:54] <@surkov> it seems nsBoxObject::GetFrame(false) can flush layout anyway
- # [03:54] <tbsaunde> surkov: any reason you used the nsPIBoxObject thing instead of making nsITreeBoxObject [builtinclass]
- # [03:54] <@surkov> tbsaunde: see http://mxr.mozilla.org/mozilla-central/source/layout/xul/base/src/nsBoxObject.cpp#117
- # [03:55] <@surkov> tbsaunde: it seems I should pass 'true' instead 'false' :)
- # [03:57] <@surkov> tbsaunde: nope, it will flush it in either case
- # [03:57] <tbsaunde> surkov: wouldn't that still have reflow happen in some cases?
- # [03:57] <tbsaunde> don't you need to fix layout stuff?
- # [03:57] <@surkov> maybe, dunno how
- # [03:57] <tbsaunde> surkov: yeah, well looks like not our bug anyway :)
- # [03:58] <tbsaunde> surkov: ask roc / bz or someone?
- # [03:59] <@surkov> agree, we need cc them
- # [04:04] <@firebot> surkov.alexander@gmail.com changed the Assignee on bug 809871 from surkov.alexander@gmail.com to nobody@mozilla.org.
- # [04:04] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=809871 nor, --, ---, nobody, ASSI, XUL tree accessible creation may flush layout
- # [04:09] <tbsaunde> surkov: btw I bet that bug can be exploited somehow or another...
- # [04:10] <@surkov> quite possible
- # [04:25] <@surkov> tbsaunde: about for (uint32_t idx = count - 1; idx < count; idx--)
- # [04:25] <@surkov> not sure how to make it working
- # [04:26] <@firebot> surkov.alexander@gmail.com changed the Assignee on bug 789245 from icepyramid@gmail.com to nobody@mozilla.org.
- # [04:26] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=789245 nor, --, ---, nobody, NEW, add TableCellAccessible::Row()
- # [04:33] * surkov sets mode: +o scottgonzalez
- # [04:33] * surkov sets mode: -o scottgonzalez
- # [04:34] * surkov sets mode: +o tbsaunde
- # [04:47] * Quits: ehsan (ehsan@F2D29657.F60B0462.67AC9B1.IP) (Input/output error)
- # [05:01] * Joins: Jamie (Instantbir@moz-CA26021.jantrid.net)
- # [05:07] * Quits: scottgonzalez (scott_gonz@moz-6B7D4671.nc.res.rr.com) (Quit: scottgonzalez)
- # [05:08] <@surkov> Jamie: hello
- # [05:09] <Jamie> surkov: hi!
- # [05:09] <@surkov> Jamie: a question, if we would fire state change for traversed state on the link then how would it be?
- # [05:09] <@surkov> Jamie: https://bugzilla.mozilla.org/show_bug.cgi?id=757774#c24
- # [05:09] <@firebot> Bug 757774 nor, --, ---, nobody, NEW, Intermittent TEST-UNEXPECTED-FAIL | chrome://mochitests/content/a11y/accessible/states/test_link.htm
- # [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
- # [05:11] <Jamie> so deleted the emails without reading them :)
- # [05:11] <@surkov> Jamie: yeah, sometimes it happens :)
- # [05:12] <@surkov> especially on spamming bugs like intermittent failures
- # [05:13] <Jamie> hold on a second
- # [05:14] <Jamie> surkov: commented, but in short, yes, it's a good idea.
- # [05:14] <@surkov> Jamie: cool, thank you
- # [05:14] <Jamie> sorry about ignoring the bug :)
- # [05:17] <@surkov> nm :)
- # [05:47] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # [05:48] <Jamie> surkov: something seems to be really broken with Firefox's calculation of invisible and offscreen states lately
- # [05:48] <Jamie> surkov: I'm assuming this is the point of that "expose intended visibility" bug?
- # [05:48] <@surkov> Jamie: do you mean nightly?
- # [05:48] <Jamie> surkov: yes, though I think it's been happening fo ra few months, so probably not just nightly now
- # [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
- # [05:49] <@surkov> Jamie: can you try to catch it?
- # [05:49] <Jamie> surkov: actually, hmm
- # [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
- # [05:50] <@surkov> Jamie: it seems like it should be offscreen
- # [05:50] <@surkov> it's not I guess?
- # [05:50] <Jamie> surkov: arrrg
- # [05:50] <Jamie> surkov: sorry, I think I'm just braindead
- # [05:51] * Jamie must have been looking at the wrong node or something <embarrassed>
- # [05:51] <@surkov> if you get a test case then please file a bug
- # [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
- # [05:52] <Jamie> they're not just "in the background", as such; they're not visible at all
- # [05:53] <@surkov> Jamie: they aren't, perhaps that fall into the case of tabs
- # [05:54] <@surkov> Jamie: file a bug? ;)
- # [05:54] <@surkov> big changes always give regressions
- # [05:55] * Jamie nods
- # [05:55] <Jamie> sorry about the first false alarm :)
- # [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.
- # [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
- # [05:57] <Jamie> so we end up with a nasty lose lose situation :)
- # [05:57] <@surkov> Jamie: it seems we need in-background-tab state :)
- # [05:57] * Jamie nods; I was heading in that direction of thought as well
- # [05:58] <Jamie> or yet another kind of "for sr users only" :)
- # [05:58] <@surkov> if this visibility state maps into accessible states well at all
- # [05:58] <@surkov> for example, invisible + offscreen doesn't make a lot of sense
- # [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
- # [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
- # [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
- # [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
- # [06:00] <Jamie> I'd prefer to keep it that way if I can
- # [06:00] <@surkov> I see
- # [06:00] <Jamie> Nm, I'll have to think about it
- # [06:01] * Joins: ehsan (ehsan@moz-C48D29C4.cable.teksavvy.com)
- # [06:01] <Jamie> Arguably, off-screen live regions are a really bad idea anyway, but I've just noticed Facebook are doing it
- # [06:19] <@surkov> Jamie: btw, don't you use firefox keyword, it'd handy to see needsExternalFix + firefox + major bugs
- # [06:35] <@firebot> surkov.alexander@gmail.com cancelled review?(trev.saunders@gmail .com) for attachment 677685 on bug 807911.
- # [06:35] <@firebot> surkov.alexander@gmail.com requested review from trev.saunders@gmail .com for attachment 679970 on bug 807911.
- # [06:35] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=807911 nor, --, ---, surkov.alexander, ASSI, whittle mutation events processing
- # [06:39] <@firebot> surkov.alexander@gmail.com cancelled review?(trev.saunders@gmail .com) for attachment 679970 on bug 807911.
- # [06:39] <@firebot> surkov.alexander@gmail.com requested review from trev.saunders@gmail .com for attachment 679971 on bug 807911.
- # [06:46] * Quits: drexler (chatzilla@moz-AD4BBA18.hsd1.vt.comcast.net) (Quit: ChatZilla 0.9.89-rdmsoft [XULRunner 1.9.0.17/2009122204])
- # [07:06] <@firebot> surkov.alexander@gmail.com changed the Assignee on bug 757774 from nobody@mozilla.org to surkov.alexander@gmail.com.
- # [07:06] <@firebot> surkov.alexander@gmail.com requested review from trev.saunders@gmail .com for attachment 679975 on bug 757774.
- # [07:06] <@firebot> surkov.alexander@gmail.com changed the Status on bug 757774 from NEW to ASSIGNED.
- # [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
- # [07:14] <@tbsaunde> surkov: still confused about the for () thing?
- # [07:14] <@surkov> tbsaunde: nope, I commented in the bug
- # [07:23] <@surkov> tbsaunde: btw, it'd be nice to answer my comment in fixed bug 770564 ;)
- # [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
- # [07:23] <@surkov> and file good first bugs :)
- # [07:27] * Quits: Jamie (Instantbir@moz-CA26021.jantrid.net) (Quit: Instantbird 1.3a1pre -- http://www.instantbird.com)
- # [07:34] * Quits: ehsan (ehsan@moz-C48D29C4.cable.teksavvy.com) (Connection reset by peer)
- # [07:34] * Joins: ehsan_ (ehsan@moz-C48D29C4.cable.teksavvy.com)
- # [07:41] * Quits: gavin (gavin@moz-59682A1.dmz.scl3.mozilla.com) (Input/output error)
- # [07:41] * Joins: gavin (gavin@moz-59682A1.dmz.scl3.mozilla.com)
- # [07:44] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Client exited)
- # [08:01] * Quits: @tbsaunde (tbsaunde@moz-59682A1.dmz.scl3.mozilla.com) (Ping timeout)
- # [08:01] * Joins: tbsaunde (tbsaunde@moz-59682A1.dmz.scl3.mozilla.com)
- # [08:06] <tbsaunde> surkov: yeah, I remember, sorry I half let it slip in queue
- # [08:10] <@surkov> I see, sometimes things hangs in queue
- # [08:10] * Joins: victorporof (victorporo@311CA19C.477C3F8E.79933D60.IP)
- # [08:20] * Joins: Stevef (chatzilla@moz-CD0F47B5.cable.virginmedia.com)
- # [08:57] <@firebot> surkov.alexander@gmail.com cancelled review?(trev.saunders@gmail .com) for attachment 679971 on bug 807911.
- # [08:57] <@firebot> surkov.alexander@gmail.com requested review from trev.saunders@gmail .com for attachment 679995 on bug 807911.
- # [08:57] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=807911 nor, --, ---, surkov.alexander, ASSI, whittle mutation events processing
- # [09:05] * Quits: Stevef (chatzilla@moz-CD0F47B5.cable.virginmedia.com) (Ping timeout)
- # [09:18] * Joins: Stevef (chatzilla@moz-CD0F47B5.cable.virginmedia.com)
- # [09:33] * Joins: margle (margle@E3F0FA38.AA35ED30.A77EFE04.IP)
- # [10:20] * Joins: icaaq (Adium@CA8A6033.7DCD925.CE255B90.IP)
- # [10:24] * Quits: icaaq (Adium@CA8A6033.7DCD925.CE255B90.IP) (Ping timeout)
- # [10:24] * Joins: icaaq (Adium@CA8A6033.7DCD925.CE255B90.IP)
- # [10:32] * Quits: Stevef (chatzilla@moz-CD0F47B5.cable.virginmedia.com) (Ping timeout)
- # [10:33] * Joins: Stevef (chatzilla@moz-CD0F47B5.cable.virginmedia.com)
- # [10:36] * Quits: icaaq (Adium@CA8A6033.7DCD925.CE255B90.IP) (Ping timeout)
- # [10:38] * Joins: icaaq (Adium@7C3FFB6A.7DCD925.CE255B90.IP)
- # [10:48] * Quits: @surkov (surkov@D08E2EFF.E35E3D12.E17943EE.IP) (Quit: surkov)
- # [10:51] * Quits: icaaq (Adium@7C3FFB6A.7DCD925.CE255B90.IP) (Ping timeout)
- # [10:53] * Joins: icaaq (Adium@moz-D110ACAA.cust.telenor.se)
- # [11:01] * Quits: icaaq (Adium@moz-D110ACAA.cust.telenor.se) (Quit: Leaving.)
- # [11:19] * Quits: victorporof (victorporo@311CA19C.477C3F8E.79933D60.IP) (Connection reset by peer)
- # [11:19] * Joins: victorporof_ (victorporo@311CA19C.477C3F8E.79933D60.IP)
- # [11:20] * Quits: margle (margle@E3F0FA38.AA35ED30.A77EFE04.IP) (Quit: Computer has gone to sleep.)
- # [11:23] * Quits: victorporof_ (victorporo@311CA19C.477C3F8E.79933D60.IP) (Connection reset by peer)
- # [11:23] * Joins: victorporof (victorporo@311CA19C.477C3F8E.79933D60.IP)
- # [12:02] * Joins: surkov (surkov@D08E2EFF.E35E3D12.E17943EE.IP)
- # [12:02] * ChanServ sets mode: +o surkov
- # [12:08] * Joins: margle (margle@E3F0FA38.AA35ED30.A77EFE04.IP)
- # [12:59] <@firebot> New Core - Disability Access APIs bug 810260 filed by surkov.alexander@gmail.com.
- # [13:00] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=810260 nor, --, ---, nobody, NEW, xul:deck hidden pages shouldn't be offscreen
- # [13:11] * Quits: khuey|away (khuey@moz-59682A1.dmz.scl3.mozilla.com) (Input/output error)
- # [13:11] * Joins: khuey|away (khuey@moz-59682A1.dmz.scl3.mozilla.com)
- # [13:11] * Quits: gavin (gavin@moz-59682A1.dmz.scl3.mozilla.com) (Input/output error)
- # [13:11] * Joins: gavin (gavin@moz-59682A1.dmz.scl3.mozilla.com)
The end :)