/irc-logs / mozilla / #accessibility / 2012-02-10 / end
Options:
- # Session Start: Fri Feb 10 00:00:00 2012
- # Session Ident: #accessibility
- # [00:03] * Quits: satdav (satdav@moz-1ECB932B.cable.virginmedia.com) (Connection reset by peer)
- # [00:23] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # [00:34] * Quits: aaronlev (aaronlev@moz-CDA191A6.c3-0.arl-ubr1.sbo-arl.ma.cable.rcn.com) (Ping timeout)
- # [00:36] * Quits: ehsan (ehsan@F2D29657.F60B0462.67AC9B1.IP) (Input/output error)
- # [01:04] * Quits: drexler (chatzilla@moz-2C2B7D1F.hsd1.vt.comcast.net) (Quit: ChatZilla 0.9.88 [Firefox 10.0/20120129021758])
- # [01:05] * Joins: davidb (davidb@moz-6F9F653A.dsl.bell.ca)
- # [01:05] * ChanServ sets mode: +qo davidb davidb
- # [01:08] * Quits: @davidb (davidb@moz-6F9F653A.dsl.bell.ca) (Quit: davidb)
- # [01:26] * Quits: aaronlev_ (chatzilla@moz-CDA191A6.c3-0.arl-ubr1.sbo-arl.ma.cable.rcn.com) (Ping timeout)
- # [01:44] * Joins: aaronlev (chatzilla@moz-9058091D.bstnma.fios.verizon.net)
- # [01:50] <@firebot> andrew.quartey@gmail.com requested review from surkov.alexander@gm ail.com for attachment 595909 on bug 634829.
- # [01:50] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=634829 nor, --, ---, andrew.quartey, NEW, remove nsIAccessibleWin32Object interface
- # [02:05] * Joins: surkov (surkov@36D639C1.44A4068D.222B27F0.IP)
- # [02:05] * ChanServ sets mode: +o surkov
- # [02:10] * khuey is now known as khuey|away
- # [02:15] * Joins: davidb (davidb@moz-6F9F653A.dsl.bell.ca)
- # [02:15] * ChanServ sets mode: +qo davidb davidb
- # [02:19] <@hub> davidb: satdav was looking for you, but he is gone
- # [02:20] <@davidb> hub: thanks. do you know what about?
- # [02:21] <@davidb> !seen satdav
- # [02:21] <@firebot> satdav was last seen 2 hours, 37 minutes and 16 seconds ago, saying 'same ' in #accessibility.
- # [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?
- # [02:32] <Jamie> I'm guessing stats on, for example, ISimpleDom* usage are sent at startup, so they're only sent once per session
- # [02:33] <Jamie> so you at least know how often ISimple* is used compared to everything else
- # [02:33] <@hub> davidb: nope.
- # [02:33] <@hub> davidb: he didn'
- # [02:33] <@hub> t say
- # [02:33] <@davidb> Jamie: right
- # [02:33] <@davidb> hub: ok thanks
- # [02:34] <@davidb> Jamie: but the dashboard doesn't show screen reader XYZ uses ISimpleDOM
- # [02:34] <Jamie> davidb: same for screen reader injection; only sent at startup?
- # [02:34] <@davidb> that might be in the database but I have to ask.
- # [02:34] <@davidb> I'm not sure when the telemetry ping is sent
- # [02:34] <Jamie> ok, but only once per session I assume
- # [02:35] <@davidb> but we prepare the data as soon as we detect it
- # [02:35] <Jamie> right
- # [02:35] <@davidb> i'm not sure… it might be every N hours.
- # [02:35] * Jamie nods
- # [02:35] <@davidb> Moz is still sort of newish to this
- # [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
- # [02:36] <@davidb> yeah they'd be unfairly equal
- # [02:36] <Jamie> nm. do you have any idea how Moz gains some idea of how many real firefox "users" there are?
- # [02:37] <@davidb> i think it might have to do with an update or add-on check
- # [02:37] <@davidb> i barely care enough :)
- # [02:37] <Jamie> sure. I'm asking the wrong people. I just don't know who to ask :)
- # [02:37] <@davidb> hmm
- # [02:37] <Jamie> trying to figure out how we can get this sort of data for NVDA
- # [02:37] <@davidb> #metrics
- # [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
- # [02:38] <@davidb> Jamie: it is early days but it looks like about 60% nvda, 40% jaws, and 11% WE
- # [02:38] <@davidb> FF users
- # [02:38] * Quits: aaronlev (chatzilla@moz-9058091D.bstnma.fios.verizon.net) (Client exited)
- # [02:38] <@davidb> and it seems to suggest the presence of more than one sr at times
- # [02:38] <Jamie> interesting
- # [02:39] <@davidb> this is a sampling of early adopters that allow telemetry :)
- # [02:39] <@davidb> it is opt-in
- # [02:39] <@davidb> so I would expect it skewed towards NVDA
- # [02:39] * Jamie nods; I only enabled it after Marco's blog post myself
- # [02:39] <@davidb> spread the word
- # [02:39] <@davidb> if you agree it makes sense
- # [02:39] <Jamie> it does. I did retweet that I believe
- # [02:40] * @davidb nods
- # [02:41] * Joins: ehsan (ehsan@F2D29657.F60B0462.67AC9B1.IP)
- # [02:57] <@firebot> surkov.alexander@gmail.com denied review for attachment 595909 on bug 634829.
- # [02:57] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=634829 nor, --, ---, andrew.quartey, NEW, remove nsIAccessibleWin32Object interface
- # [02:59] * khuey|away is now known as khuey
- # [03:04] * Joins: drexler (chatzilla@moz-2C2B7D1F.hsd1.vt.comcast.net)
- # [03:33] * Quits: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com) (Quit: nhirata)
- # [03:43] * Quits: @hub (hub@21B7B9F2.B87E9213.6E712CE2.IP) (Ping timeout)
- # [03:49] * shorlander is now known as shorlander-away
- # [03:56] <@tbsaunde> surkov: fyi I plan to do your reviews after midnight my time
- # [03:57] <@tbsaunde> and will be around then but have to work till then
- # [03:57] <@surkov> sounds good
- # [04:00] * Quits: @davidb (davidb@moz-6F9F653A.dsl.bell.ca) (Quit: davidb)
- # [04:02] * Joins: hub (hub@moz-E2FCA694.figuiere.net)
- # [04:02] * ChanServ sets mode: +o hub
- # [04:30] <@tbsaunde> surkov: I'm not sure I've ever heard someone call writing sml at high volocity grat but ok ;)
- # [04:34] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
- # [04:37] <@surkov> tbsaunde: I failed to understand: "someone call writing sml at high volocity grat"
- # [04:37] * Quits: shorlander-away (shorlander@moz-853043D6.dhcp.insightbb.com) (Ping timeout)
- # [04:38] <@firebot> andrew.quartey@gmail.com requested review from surkov.alexander@gm ail.com for attachment 595941 on bug 634829.
- # [04:38] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=634829 nor, --, ---, andrew.quartey, NEW, remove nsIAccessibleWin32Object interface
- # [04:40] <@tbsaunde> surkov: its a programing lnaguage I don't really like
- # [04:40] <@tbsaunde> sml is
- # [04:40] <@tbsaunde> and I need to write a lot fast
- # [04:40] <@surkov> got it
- # [04:44] <@firebot> surkov.alexander@gmail.com granted review for attachment 595941 on bug 634829.
- # [04:55] * Joins: shorlander-away (shorlander@moz-853043D6.dhcp.insightbb.com)
- # [04:58] * Joins: ptheriault (ptheriault@moz-4BE034AB.ptr.us.xo.net)
- # [05:08] <@firebot> andrew.quartey@gmail.com requested review from surkov.alexander@gm ail.com for attachment 595949 on bug 634829.
- # [05:08] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=634829 nor, --, ---, andrew.quartey, NEW, remove nsIAccessibleWin32Object interface
- # [05:11] <@firebot> surkov.alexander@gmail.com granted review for attachment 595949 on bug 634829.
- # [05:16] * Quits: ehsan (ehsan@F2D29657.F60B0462.67AC9B1.IP) (Input/output error)
- # [05:18] * Quits: shorlander-away (shorlander@moz-853043D6.dhcp.insightbb.com) (Quit: Quit)
- # [05:40] * Joins: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP)
- # [06:01] * Quits: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP) (Input/output error)
- # [06:16] * Quits: drexler (chatzilla@moz-2C2B7D1F.hsd1.vt.comcast.net) (Quit: ChatZilla 0.9.88 [Firefox 10.0/20120129021758])
- # [06:16] <@tbsaunde> well, that wasn't total loss but pretty bad
- # [06:16] * Joins: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP)
- # [06:24] <@tbsaunde> surkov: btw for bug 634829 I believe MOZ_OVERRIDE is prefered to NS_OVERRIDE
- # [06:24] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=634829 nor, --, ---, andrew.quartey, NEW, remove nsIAccessibleWin32Object interface
- # [06:24] <@surkov> ok, I'll fix it
- # [06:25] <@tbsaunde> ok
- # [06:49] * Joins: aaronlev (aaronlev@moz-9058091D.bstnma.fios.verizon.net)
- # [06:52] * Quits: @surkov (surkov@36D639C1.44A4068D.222B27F0.IP) (Ping timeout)
- # [06:58] * Joins: surkov (surkov@94877EB3.B3F96E9C.3AF1D72D.IP)
- # [06:58] * ChanServ sets mode: +o surkov
- # [07:26] * Quits: aaronlev (aaronlev@moz-9058091D.bstnma.fios.verizon.net) (Quit: aaronlev)
- # [07:30] * khuey is now known as khuey|away
- # [07:35] <Jamie> surkov: any idea why bug 677154 only occurs with certain profiles and not others?
- # [07:35] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=677154 maj, P1, ---, nobody, NEW, Detached document accessibility tree
- # [07:41] <@surkov> Jamie: that depends on view area
- # [07:41] <@surkov> Jamie: in your profile Firefox is running maximized
- # [07:41] <@surkov> so you can see a bug
- # [07:41] <@surkov> I keep Firefox window small to run inspect tool so I wasn't able to reproduce the bug
- # [07:50] <@firebot> trev.saunders@gmail.com granted review for attachment 595671 on bug 724452.
- # [07:50] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=724452 nor, --, ---, jigneshhk1992, NEW, use nsFocusManager::GetFocusManager() more
- # [08:04] <@tbsaunde> surkov: so, your patch for bug 725581 I'm really suprised some of those checks aren't redundant
- # [08:04] <@tbsaunde> especially the state one
- # [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
- # [08:05] <@tbsaunde> also can't you merge the chekc after what you add into it?
- # [08:11] <@surkov> I'm not sure how
- # [08:13] <@tbsaunde> surkov: ugh
- # [08:13] <@tbsaunde> surkov: any idea why have you investigated?
- # [08:14] <@surkov> tbsaunde: tried to enable old mochitests
- # [08:18] <@tbsaunde> surkov: and they don't pass without all of that?
- # [08:20] <@surkov> sure
- # [08:21] <@tbsaunde> surkov: I'm not sure what you mean by that
- # [08:21] <@surkov> tbsaunde: they don't pass without code changes
- # [08:21] <@tbsaunde> surkov: ok, did you debug why?
- # [08:22] <@surkov> I'll put comment into bug for history
- # [08:22] <@tbsaunde> ok
- # [08:23] * @tbsaunde doesnt' really like here is crazy patches without explanation of why they need to be that way
- # [08:23] <@tbsaunde> especially when needing that seems to imply crazy state of the world
- # [08:27] <@surkov> tbsaunde: done
- # [08:27] <@surkov> tbsaunde: crazy patch contains comments
- # [08:27] <@surkov> that means they should be improved and I wait for your feedback how to do that :)
- # [08:28] <@tbsaunde> ok
- # [08:36] <@tbsaunde> surkov: so, buzilla is just hainging for me, but we have channel logs so there's still history :)
- # [08:37] <@surkov> when I digging through bugs then I don't check irc log usually :)
- # [08:37] <@surkov> here's the comment
- # [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).
- # [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
- # [08:38] <@tbsaunde> surkov: yeah, bugzilla being slow is :(
- # [08:39] <@surkov> tbsaunde: I wanted to avoid tree traversal when I don't need it
- # [08:41] <@tbsaunde> surkov: unless I miss something you wouldn't be adding tree traversal sine we already get the FocusDisp
- # [08:42] <@surkov> tbsaunde: exactly, that's why focusdisp is after my check
- # [08:42] <@tbsaunde> surkov: oh, I see
- # [08:42] <@tbsaunde> you want to reduce how often we do it
- # [08:44] * Quits: @firebot (firebot@moz-67BF9FE6.carolina.res.rr.com) (Ping timeout)
- # [08:45] <@surkov> yeah, especially for case of -1
- # [08:45] <@tbsaunde> surkov: ok, gues that makes sense
- # [08:45] <@surkov> good
- # [08:45] <@tbsaunde> surkov: btw it occurs to me we could remove shared code from IsFocused() and FocusedAccessible()
- # [08:46] <@surkov> oh, really, iirc I had some problems
- # [08:46] <@surkov> there's XXX comment there
- # [08:47] <@tbsaunde> surkov: weird, looks like they're the same
- # [08:47] <@tbsaunde> at a quick look
- # [08:47] <@surkov> they are almost the same
- # [08:47] <@surkov> but called at different time may give different results
- # [08:48] * Joins: firebot (firebot@moz-67BF9FE6.carolina.res.rr.com)
- # [08:48] * ChanServ sets mode: +o firebot
- # [08:48] <@tbsaunde> surkov: sure, but can't we define IsFocused() as return aAccessible == FocusedAccessible() ?
- # [08:49] <@tbsaunde> surkov: so, one more question about the patch
- # [08:49] <@surkov> tbsaunde: no, we can't
- # [08:49] <@surkov> that's funny though
- # [08:49] <@tbsaunde> why do you want to continue the function in the case that we aren't focusable?
- # [08:50] <@surkov> like handling HTML:p?
- # [08:51] <@tbsaunde> surkov: oh, I see because FocusedAccessible() doesn't do the same doc check missed that first time through
- # [08:52] <@tbsaunde> surkov: oh, so things like caret nav mod
- # [08:52] <@tbsaunde> I gues that makes sense
- # [08:53] <@surkov> right
- # [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
- # [08:58] <@tbsaunde> but I don't really know which makes me a little sad itself
- # [08:58] <@surkov> tbsaunde: yeah, State() but Im keeping in mind to replace heavy State() on small State() versions like VisibilityState()
- # [08:59] <@surkov> not good reason though :)
- # [09:00] <@tbsaunde> surkov: what isn't a good reason for what?
- # [09:00] <@tbsaunde> surkov: breaking up State() into light ones seems reasonable
- # [09:01] <@surkov> tbsaunde: I meant future State() changes is not good reason to use State() as is right now
- # [09:01] <@tbsaunde> surkov: oh
- # [09:02] * Quits: @firebot (firebot@moz-67BF9FE6.carolina.res.rr.com) (Connection reset by peer)
- # [09:03] <@tbsaunde> surkov: so, having some sort of arg to IsInOrContainsFocus() to only check certain conditions might make sense
- # [09:04] <@tbsaunde> surkov: but I think you've aguasted my would x be nicer supply for that issue though
- # [09:05] <@surkov> sometimes I can't translate you, like last sentence
- # [09:05] <@tbsaunde> surkov: I mean I don't think I have many more ideas for improving your patch
- # [09:06] <@surkov> but if you could put IsINorContainsFocus idea into code snippet then it should be nice
- # [09:06] <@surkov> for me it's quite plain method
- # [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
- # [09:07] <@surkov> we should return valid caret in second case too
- # [09:07] * @tbsaunde tries to write sample
- # [09:08] <@tbsaunde> surkov: I'm not sure what your second case is
- # [09:08] <@surkov> caret inside html:p
- # [09:13] * Joins: firebot (firebot@moz-67BF9FE6.carolina.res.rr.com)
- # [09:13] * ChanServ sets mode: +o firebot
- # [09:14] * Joins: MarcoZ (marco.zehe@moz-76FEC283.dip.t-dialin.net)
- # [09:14] * ChanServ sets mode: +o MarcoZ
- # [09:15] * Quits: @hub (hub@moz-E2FCA694.figuiere.net) (Ping timeout)
- # [09:15] <@tbsaunde> surkov: well, I gues you don't change that case so
- # [09:15] <@tbsaunde> surkov: http://paste.debian.net/155787/ may help with what I was suggesting, but I'm not sure i like it
- # [09:17] <@surkov> sounds like GetFocusDisp(accessible, true) is quite useless
- # [09:18] <@surkov> it's equivalent to FocusMgr()->IsFOcused(accessible)
- # [09:18] <@MarcoZ> Good day all!
- # [09:19] <@tbsaunde> surkov: yeah, I was thinking it was useful if you say didn't care if you were contained by focus
- # [09:19] <@tbsaunde> so, if you only check if you are focused or a kid is
- # [09:19] <@tbsaunde> but that isn't god enough here
- # [09:19] <@tbsaunde> so I am silly
- # [09:20] <@tbsaunde> and the only question is a State() cal better than aIsInOrContainsFocus() call
- # [09:21] <@firebot> marco.zehe@googlemail.com changed the Component on bug 725725 from Disability Access to Untriaged.
- # [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
- # [09:36] <@tbsaunde> surkov: ^
- # [09:36] <@surkov> I don't know
- # [09:50] <@firebot> trev.saunders@gmail.com granted review for attachment 595669 on bug 725581.
- # [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
- # [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 :)
- # [09:57] <Jamie> surkov: re maximised
- # [09:58] <@surkov> Jamie: it wasn't easy for me as well to conculd
- # [09:58] <@surkov> this
- # [10:08] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
- # [10:21] * Joins: victorporof (victorporo@C092FEB2.1C233438.79933D60.IP)
- # [10:22] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
- # [10:23] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
- # [10:49] * Quits: Jamie (jamie@moz-CA26021.jantrid.net) (Ping timeout)
- # [11:03] * Joins: Mic (Benedikt@moz-CFE77E7A.superkabel.de)
- # [11:24] <@firebot> jigneshhk1992@gmail.com requested review from trev.saunders@gmail .com for attachment 595997 on bug 724452.
- # [11:24] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=724452 nor, --, ---, jigneshhk1992, NEW, use nsFocusManager::GetFocusManager() more
- # [11:24] <@tbsaunde> surkov: want to take ^? I'm about to head out
- # [11:25] <@surkov> tbsaunde: I'm fine with that
- # [11:25] <@tbsaunde> thx
- # [11:25] * Parts: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
- # [11:25] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
- # [11:47] * Quits: victorporof (victorporo@C092FEB2.1C233438.79933D60.IP) (Connection reset by peer)
- # [11:47] * Joins: victorporo (victorporo@C092FEB2.1C233438.79933D60.IP)
- # [11:50] * Quits: victorporo (victorporo@C092FEB2.1C233438.79933D60.IP) (Connection reset by peer)
- # [11:50] * Joins: victorporof (victorporo@C092FEB2.1C233438.79933D60.IP)
- # [11:59] * Quits: ptheriault (ptheriault@moz-4BE034AB.ptr.us.xo.net) (Connection reset by peer)
- # [11:59] * Joins: ptheriault_ (ptheriault@moz-4BE034AB.ptr.us.xo.net)
- # [12:00] * Quits: ptheriault_ (ptheriault@moz-4BE034AB.ptr.us.xo.net) (Connection reset by peer)
- # [12:26] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
- # [12:47] * Quits: victorporof (victorporo@C092FEB2.1C233438.79933D60.IP) (Connection reset by peer)
- # [12:47] * Joins: victorporo (victorporo@C092FEB2.1C233438.79933D60.IP)
- # [12:58] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
- # [13:02] * Quits: Mic (Benedikt@moz-CFE77E7A.superkabel.de) (Quit: Instantbird 1.2a1pre)
- # [13:08] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
- # [13:16] * Quits: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP) (Input/output error)
- # [13:25] <@firebot> surkov.alexander@gmail.com requested review from hub@mozilla.com for attachment 595997 on bug 724452.
- # [13:25] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=724452 nor, --, ---, jigneshhk1992, NEW, use nsFocusManager::GetFocusManager() more
- # [13:26] <@surkov> tbsaunde: I stole your review request
- # [13:26] <@MarcoZ> surkov: Do you know when your latest DOMi enhancements will make it into an extension people can actually use?
- # [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
- # [13:27] <@surkov> MarcoZ: ^
- # [13:28] <@surkov> if David will get DOMi contributors then I bet it'll be a great a11y tool
- # [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
- # [13:34] * Quits: @jprmc (jprmc@moz-7F2FF3EB.cpe.net.cable.rogers.com) (Ping timeout)
- # [13:37] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
- # [13:44] * Joins: askalski (akuda@moz-4C8A107E.pool85-48-91.dynamic.orange.es)
- # [13:44] * ChanServ sets mode: +o askalski
- # [13:45] * Quits: @askalski (akuda@moz-4C8A107E.pool85-48-91.dynamic.orange.es) (Quit: Wychodzi)
- # [13:58] * Joins: askalski (akuda@moz-4C8A107E.pool85-48-91.dynamic.orange.es)
- # [13:58] * ChanServ sets mode: +o askalski
- # [13:58] * Quits: @askalski (akuda@moz-4C8A107E.pool85-48-91.dynamic.orange.es) (Quit: Wychodzi)
- # [14:14] <@firebot> bmo@edmorley.co.uk changed the Status on bug 721947 from NEW to RESOLVED.
- # [14:14] <@firebot> bmo@edmorley.co.uk set the Resolution field on bug 721947 to FIXED.
- # [14:14] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=721947 nor, --, mozilla13, hub, RESO FIXED, don't use nsIWeakShell
- # [14:14] <@firebot> bmo@edmorley.co.uk changed the Status on bug 673405 from NEW to RESOLVED.
- # [14:14] <@firebot> bmo@edmorley.co.uk set the Resolution field on bug 673405 to FIXED.
- # [14:14] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=673405 cri, --, mozilla13, hub, NEW, Rename GetDocAccessible() to Document()
- # [14:14] <@firebot> bmo@edmorley.co.uk changed the Status on bug 672504 from ASSIGNED to RESOLVED.
- # [14:14] <@firebot> bmo@edmorley.co.uk set the Resolution field on bug 672504 to FIXED.
- # [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
- # [14:19] <@firebot> vlad.mozbugs@gmail.com set status-firefox11 to verified on bug 693343.
- # [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
- # [14:33] <@MarcoZ> Yay hub's patches finally landed on central!
- # [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.
- # [14:49] <@surkov> MarcoZ: that's right
- # [14:54] * Joins: jprmc (jprmc@29DDBCC5.5BCEC6DB.DA78B690.IP)
- # [14:54] * ChanServ sets mode: +o jprmc
- # [15:02] * Quits: @jprmc (jprmc@29DDBCC5.5BCEC6DB.DA78B690.IP) (Ping timeout)
- # [15:13] * Quits: @surkov (surkov@94877EB3.B3F96E9C.3AF1D72D.IP) (Ping timeout)
- # [15:16] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
- # [15:17] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
- # [15:19] * khuey|away is now known as khuey
- # [15:23] * Joins: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
- # [15:24] * Joins: surkov (surkov@37A43653.44A4068D.222B27F0.IP)
- # [15:24] * ChanServ sets mode: +o surkov
- # [15:27] * Joins: davidb (davidb@F2D29657.F60B0462.67AC9B1.IP)
- # [15:27] * ChanServ sets mode: +qo davidb davidb
- # [15:28] <@davidb> hi all!
- # [15:29] <@MarcoZ> Hi davidb!
- # [15:32] * @MarcoZ is refreshing his Windows and Mac builds after the recent code changes that finally landed. :)
- # [15:35] <@davidb> surkov: nice blog post
- # [15:35] <@surkov> davidb: thanks!
- # [15:35] <@davidb> surkov: i was thinking you could add a "Looking for contributors" at the end :)
- # [15:36] <@surkov> davidb: yeah, maybe next time :) that was supposed to be overview
- # [15:36] <@davidb> makes sense
- # [15:40] <@MarcoZ> I believe a call out for contributors is always worth its own blog post. easily gets missed otherwise, surkov, davidb.
- # [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
- # [15:50] <@davidb> yeah
- # [15:51] <@davidb> it doesn't hurt any of us to blog about how to get involved. details are important.
- # [15:52] * Quits: victorporo (victorporo@C092FEB2.1C233438.79933D60.IP) (Connection reset by peer)
- # [15:52] * Joins: victorporof (victorporo@C092FEB2.1C233438.79933D60.IP)
- # [15:57] * Joins: jprmc (jprmc@F2D29657.F60B0462.67AC9B1.IP)
- # [15:57] * ChanServ sets mode: +o jprmc
- # [15:57] * Quits: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp) (Quit: mike5w3c)
- # [15:58] * Joins: skyler_brungardt (skyler@moz-9FA3C392.washdc.fios.verizon.net)
- # [15:58] <@tbsaunde> surkov: ok, though looks you could just cancel it since its the same patch I already reviewed just only for accessible/
- # [15:59] <@tbsaunde> it might be worth if hub looks if using nsFocusManger instead of nsIFocusManager lets us use nicer methods though
- # [15:59] <@surkov> yep
- # [16:11] * Joins: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp)
- # [16:20] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # [16:39] * Joins: ehsan (ehsan@F2D29657.F60B0462.67AC9B1.IP)
- # [16:45] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
- # [16:51] * Joins: ptheriault (ptheriault@moz-4BE034AB.ptr.us.xo.net)
- # [16:56] * Quits: skyler_brungardt (skyler@moz-9FA3C392.washdc.fios.verizon.net) (Quit: skyler_brungardt)
- # [16:57] <@firebot> New Core - Disability Access APIs bug 725998 filed by surkov.alexander@gmail.com.
- # [16:57] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=725998 nor, --, ---, nobody, NEW, get rid nsAccessibilityService::GetAccessibleOrContainer and GetContainerAccessible
- # [17:01] * @davidb files https://bugs.eclipse.org/bugs/show_bug.cgi?id=371250
- # [17:05] * @davidb adds another comment
- # [17:08] <@firebot> bolterbugz@gmail.com changed the Status on bug 688161 from NEW to RESOLVED.
- # [17:08] <@firebot> bolterbugz@gmail.com set the Resolution field on bug 688161 to WONTFIX.
- # [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.
- # [17:10] <@firebot> New Core - Disability Access APIs bug 726005 filed by surkov.alexander@gmail.com.
- # [17:10] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=726005 nor, --, ---, nobody, NEW, IDRefsIterator::Next() should use nsDocAccessible::GetAccessible()
- # [17:10] * Joins: hub (hub@83874EA1.EB7C1AF9.6F478678.IP)
- # [17:10] * ChanServ sets mode: +o hub
- # [17:10] * Joins: askalski (akuda@moz-104CC309.mv.mozilla.com)
- # [17:10] * ChanServ sets mode: +o askalski
- # [17:12] <@askalski> surkov, I got the final answer on LED
- # [17:13] <@askalski> most approximation algorithms are non-practical
- # [17:13] <@surkov> ok
- # [17:13] <@askalski> there are two available solutions
- # [17:13] <@askalski> one O(min(n,m)*d) where d is the distance (d <= max(n,m))
- # [17:14] <@askalski> in linear memory
- # [17:14] <@askalski> and O(n + d^2) which is good for very similar strings
- # [17:14] <@askalski> and is a bit complex in implementation
- # [17:14] <@askalski> I guess I go with the first one
- # [17:15] <@davidb> askalski: can you dump your findings into the bug?
- # [17:15] <@surkov> I think we should go with 1st
- # [17:16] <@surkov> tbsaunde: ^
- # [17:16] <@askalski> davidb, OK
- # [17:17] <@davidb> askalski: you would enjoy working in the toronto office i think
- # [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
- # [17:19] <@askalski> davidb, I hope this can be handled via e-mail.
- # [17:19] * @MarcoZ needs to go a bit earlier today, have an appointment with a potential housekeeping agency in 20.
- # [17:19] <@askalski> davidb, will keep you posted
- # [17:19] <@davidb> ok
- # [17:19] <@davidb> MarcoZ: roger
- # [17:19] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
- # [17:19] * Quits: @MarcoZ (marco.zehe@moz-76FEC283.dip.t-dialin.net) (Quit: Feierabend, ja des is so a scheeeenes Wort! :))
- # [17:20] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
- # [17:26] * Quits: @askalski (akuda@moz-104CC309.mv.mozilla.com) (Quit: Wychodzi)
- # [17:30] * Joins: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com)
- # [17:39] <@firebot> jigneshhk1992@gmail.com requested review from surkov.alexander@gm ail.com for attachment 596058 on bug 725998.
- # [17:39] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=725998 nor, --, ---, nobody, NEW, get rid nsAccessibilityService::GetAccessibleOrContainer and GetContainerAccessible
- # [17:43] * Quits: ptheriault (ptheriault@moz-4BE034AB.ptr.us.xo.net) (Quit: ptheriault)
- # [17:44] <@hub> aren't bug 725998 and bug 725572 the same ?
- # [17:45] <@hub> ah i'm an idiot
- # [17:45] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=725572 nor, --, ---, nobody, NEW, Remove nsIAccessibilityService::GetAccessible()
- # [17:45] <@hub> almost
- # [17:45] * @hub needs more coffee
- # [17:47] * Joins: drexler (chatzilla@moz-2C2B7D1F.hsd1.vt.comcast.net)
- # [17:48] <@hub> oh and third time was a charm
- # [17:48] * Quits: victorporof (victorporo@C092FEB2.1C233438.79933D60.IP) (Quit: Leaving...)
- # [17:50] <@firebot> surkov.alexander@gmail.com cancelled review?(surkov.alexander@gm ail.com) for attachment 596058 on bug 725998.
- # [17:50] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=725998 nor, --, ---, nobody, NEW, get rid nsAccessibilityService::GetAccessibleOrContainer and GetContainerAccessible
- # [17:51] <@surkov> hi, hub
- # [17:52] <@surkov> you have review request in inbox :)
- # [17:53] * khuey is now known as khuey|away
- # [17:57] * Joins: shorlander (shorlander@moz-853043D6.dhcp.insightbb.com)
- # [17:57] * Joins: ptheriault (ptheriault@moz-4BE034AB.ptr.us.xo.net)
- # [17:59] <@tbsaunde> surkov: algorithm seems fine
- # [17:59] <@surkov> ok, thx
- # [17:59] <@hub> surkov: I saw
- # [17:59] <@hub> thanks
- # [17:59] <@surkov> yw :)
- # [18:06] <@firebot> hub@mozilla.com granted review for attachment 595997 on bug 724452.
- # [18:06] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=724452 nor, --, ---, jigneshhk1992, NEW, use nsFocusManager::GetFocusManager() more
- # [18:06] <@tbsaunde> surkov: your plan for the IDrefsIteraotr isn't quiet clear to me from the bug (I mean how to impleement it)
- # [18:07] <@surkov> IDRefIterator takes nsDocAccessible, keeps as member and then Next() does mDocument->GetAccessible
- # [18:07] <@tbsaunde> surkov: so it would take a third argument?
- # [18:07] <@surkov> tbsaunde: yes
- # [18:09] <@tbsaunde> surkov: feels sort of redundant, but I gues its ok
- # [18:09] <@tbsaunde> the doc accessible and the content document
- # [18:09] <@tbsaunde> but those objects are short lived etc so whatever
- # [18:09] <@surkov> tbsaunde: content document may be null
- # [18:09] <@surkov> if xml document is used
- # [18:09] <@surkov> yeah, not super nice solution
- # [18:09] <@surkov> but can't think better
- # [18:10] <@firebot> jigneshhk1992@gmail.com requested review from surkov.alexander@gm ail.com for attachment 596062 on bug 725998.
- # [18:10] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=725998 nor, --, ---, nobody, NEW, get rid nsAccessibilityService::GetAccessibleOrContainer and GetContainerAccessible
- # [18:10] <@tbsaunde> surkov: true, but then we have an mXBLDocument no?
- # [18:10] <@surkov> right
- # [18:15] <@firebot> surkov.alexander@gmail.com granted review for attachment 595997 on bug 724452.
- # [18:17] <@tbsaunde> surkov: good catch on the GetCurrentFocus() thing I would have sworn we used it in one place somewhere
- # [18:18] <@surkov> tbsaunde: I was surprised too
- # [18:30] <@firebot> surkov.alexander@gmail.com granted review for attachment 596062 on bug 725998.
- # [18:30] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=725998 nor, --, ---, nobody, NEW, get rid nsAccessibilityService::GetAccessibleOrContainer and GetContainerAccessible
- # [18:39] * Quits: shorlander (shorlander@moz-853043D6.dhcp.insightbb.com) (Quit: Linkinus - http://linkinus.com)
- # [18:40] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
- # [18:40] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
- # [18:54] * Joins: shorlander (shorlander@679D38.AF86817F.8A1E5CBC.IP)
- # [19:07] * Quits: drexler (chatzilla@moz-2C2B7D1F.hsd1.vt.comcast.net) (Quit: ChatZilla 0.9.88 [Firefox 10.0/20120129021758])
- # [19:09] * Quits: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com) (Quit: nhirata)
- # [19:09] * Joins: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com)
- # [19:17] * Quits: shorlander (shorlander@679D38.AF86817F.8A1E5CBC.IP) (Ping timeout)
- # [19:19] * Joins: victorporof (victorporo@50B3F39.880C8D58.79933D60.IP)
- # [19:20] * Joins: shorlander (shorlander@moz-853043D6.dhcp.insightbb.com)
- # [19:23] * Quits: richardschwerdtfeger (richardsch@moz-130F1494.lightspeed.austtx.sbcglobal.net) (Quit: richardschwerdtfeger)
- # [19:28] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
- # [19:37] <@firebot> New Core - Disability Access APIs bug 726069 filed by surkov.alexander@gmail.com.
- # [19:37] <@firebot> Bug 726069 was not found.
- # [19:41] * Quits: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp) (Quit: mike5w3c)
- # [19:42] <@firebot> New Core - Disability Access APIs bug 726071 filed by surkov.alexander@gmail.com.
- # [19:42] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=726071 nor, --, ---, nobody, NEW, nsAccUtils::GetPositionAndSizeForXULSelectControlItem should use nsDocAccessible::GetAccessible
- # [19:43] * Quits: @hub (hub@83874EA1.EB7C1AF9.6F478678.IP) (Ping timeout)
- # [19:45] <@firebot> New Core - Disability Access APIs bug 726072 filed by surkov.alexander@gmail.com.
- # [19:45] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=726072 nor, --, ---, nobody, NEW, nsAccUtils::GetMultiSelectableContainer should use nsDocAccessible::GetAccessible
- # [19:47] <@davidb> surkov: I was chatting with Ryan Lortie today, he has started hanging around our office.
- # [19:48] <@davidb> (glib maintainer)
- # [19:48] <@surkov> cool
- # [19:48] <@surkov> davidb: news?
- # [19:48] <@davidb> surkov: he wants to devote 6 months to hacking on a11y
- # [19:49] <@davidb> gnome infra
- # [19:49] <@davidb> needs to convince his boss
- # [19:49] <@davidb> I might talk to his boss :)
- # [19:49] <@surkov> :)
- # [19:49] <@surkov> that's nice
- # [19:49] <@davidb> But… he has serious concerns about the current infra.
- # [19:49] <@davidb> So this would be 'trying something new'
- # [19:50] <@davidb> It is too bad he didn't go to the hackfest
- # [19:50] <@surkov> interesting, I always liked new stuffs :)
- # [19:50] <@tbsaunde> davidb: interesting
- # [19:50] <@davidb> yeah
- # [19:50] <@tbsaunde> davidb: you mean glib the gnome lib with some sort of reasonable stuff and the awful gobject stuff
- # [19:50] <@tbsaunde> not glibc?
- # [19:51] <@davidb> he was describing how dbus is intended to be used.
- # [19:51] <@davidb> the former
- # [19:51] <@davidb> glib and gtk
- # [19:51] <@tbsaunde> davidb: yeah
- # [19:51] <@tbsaunde> davidb: what are is his concerns?
- # [19:51] <@davidb> well I don't want to be the middle man, but he'll be here during our work week i think
- # [19:51] * @tbsaunde not realyl sure this is the right place to discuss or the right medium
- # [19:51] <@davidb> right
- # [19:52] <@tbsaunde> sounds good
- # [19:52] <@davidb> also… i think someone should go to guadec
- # [19:53] <@surkov> one Igalia guy said it's nice event
- # [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.
- # [19:53] <@davidb> should something like that come about :)
- # [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.
- # [19:54] <@surkov> yep, but we can collaborate
- # [19:54] <@davidb> right
- # [19:54] <@davidb> Ryan wants to hack.
- # [19:55] <@davidb> I think we could have a GNOME focussed half day during our work week.
- # [19:56] <@surkov> sounds good
- # [19:56] <@surkov> I would like to learn more
- # [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
- # [20:01] * Quits: @surkov (surkov@37A43653.44A4068D.222B27F0.IP) (Quit: surkov)
- # [20:03] * Joins: hub (hub@21B7B9F2.B87E9213.6E712CE2.IP)
- # [20:03] * ChanServ sets mode: +o hub
- # [20:11] <@davidb> tbsaunde: yep.
- # [20:18] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
- # [20:21] * khuey|away is now known as khuey
- # [20:24] <@firebot> New Core - Disability Access APIs bug 726097 filed by hub@mozilla.com.
- # [20:24] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=726097 blo, --, ---, nobody, NEW, [Mac] Accessibility Inspector unable to access items
- # [20:30] <@tbsaunde> man nsHyperTextAccessible is pretty xpcommy
- # [20:34] * davidb is now known as davidb|mtgh
- # [20:34] * davidb|mtgh is now known as davidb|mtg
- # [20:51] * Quits: ptheriault (ptheriault@moz-4BE034AB.ptr.us.xo.net) (Quit: ptheriault)
- # [21:19] <@tbsaunde> ok, who feels like r+ing http://paste.debian.net/155860/ the fix to surkov's nit in bug 672507
- # [21:19] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=672507 nor, --, mozilla13, trev.saunders, RESO FIXED, merge nsIAccessNode and nsIAccessible
- # [21:22] <@tbsaunde> davidb|mtg: hub eeejay ^
- # [21:22] <@davidb|mtg> tbsaunde: can you point out the nit?
- # [21:24] <@davidb|mtg> oh the whole thing
- # [21:24] <@tbsaunde> davidb|mtg: yeah, remove the function and just directly do_Qi
- # [21:24] <@davidb|mtg> i like it
- # [21:25] <@tbsaunde> davidb|mtg: ok
- # [21:27] <@firebot> trev.saunders@gmail.com requested review from bolterbugz@gmail.co m for attachment 596143 on bug 672507.
- # [21:28] <@hub> ok davidb is taking care of it
- # [21:28] <@hub> sorry I missed the call
- # [21:28] * davidb|mtg is now known as davidb
- # [21:28] <@tbsaunde> hub: its ok
- # [21:28] <@davidb> hub: you can drive by r
- # [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
- # [21:30] <@firebot> bolterbugz@gmail.com granted review for attachment 596143 on bug 672507.
- # [21:30] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=672507 nor, --, mozilla13, trev.saunders, RESO FIXED, merge nsIAccessNode and nsIAccessible
- # [21:31] <@davidb> tbsaunde: what about the nsAccessNode::GetStringBundle nit?
- # [21:32] <@tbsaunde> davidb: yeah, forgot about that
- # [21:32] <@tbsaunde> I'll remove it locally
- # [21:34] <@davidb> ok
- # [21:39] <@hub> tbsaunde: X-chat notification in Gnome3 haven't been very noticeable... actually not only X-chat
- # [21:39] * @hub blame the software
- # [21:42] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # [21:46] * Joins: davidwboswell (chatzilla@moz-679B7357.hsd1.ca.comcast.net)
- # [21:50] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
- # [22:01] <@tbsaunde> davidb: I see what you did thar
- # [22:01] <@davidb> the good first bug?
- # [22:03] <@tbsaunde> davidb: yeah, and making Alex the mentor ;)
- # [22:04] <@davidb> hahaha
- # [22:05] <@firebot> hub@mozilla.com granted review for attachment 596066 on bug 724452.
- # [22:05] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=724452 nor, --, ---, jigneshhk1992, NEW, use nsFocusManager::GetFocusManager() more
- # [22:13] <@davidb> uh oh. i'm beginning to understand nsCSSFrameConstructor
- # [22:13] <@davidb> you know… I should use bz more.
- # [22:16] <@tbsaunde> hehe
- # [22:16] <@davidb> i'm still not sure i like this code, but i guess it is more about fast.
- # [22:19] <@tbsaunde> that isn't really suprising
- # [22:19] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
- # [22:19] <@davidb> nope
- # [22:20] <@davidb> but i'm pretty sure i could extract a helper method here that the compiler would inline as needed.
- # [22:23] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # [22:23] * Quits: geo (geo@moz-BBE3ABD.mv.mozilla.com) (Quit: Computer has gone to sleep.)
- # [22:26] * Parts: davidwboswell (chatzilla@moz-679B7357.hsd1.ca.comcast.net)
- # [22:29] * Joins: geo (geo@moz-E31CD2CB.mozilla.org)
- # [22:30] * Joins: aaronlev (aaronlev@moz-CDA191A6.c3-0.arl-ubr1.sbo-arl.ma.cable.rcn.com)
- # [22:31] <@tbsaunde> davidb: wouldn't suprise me code that code be cleaned up is nothing special
- # [22:32] <@davidb> yeah
- # [22:32] <@davidb> i suspect people dive in and out quickly
- # [22:32] <@davidb> ok have a nice weekend - catch ya later!
- # [22:32] <@davidb> all ^
- # [22:32] * Quits: @davidb (davidb@F2D29657.F60B0462.67AC9B1.IP) (Quit: davidb)
- # [22:32] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
- # [22:36] * Quits: victorporof (victorporo@50B3F39.880C8D58.79933D60.IP) (Quit: Linkinus - http://linkinus.com)
- # [22:44] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # [22:53] * Parts: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
- # [23:12] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
- # [23:26] * Quits: @hub (hub@21B7B9F2.B87E9213.6E712CE2.IP) (Ping timeout)
- # [23:28] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # [23:31] * Quits: sgimeno (santiago.g@A922400D.48362F18.BCC03455.IP) (Ping timeout)
- # [23:33] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
- # [23:38] * Joins: skyler_brungardt (skyler@moz-9FA3C392.washdc.fios.verizon.net)
- # [23:43] * Joins: hub (hub@83874EA1.EB7C1AF9.6F478678.IP)
- # [23:43] * ChanServ sets mode: +o hub
- # [23:47] * Joins: sgimeno (santiago.g@A922400D.48362F18.BCC03455.IP)
- # [23:48] * Quits: aaronlev (aaronlev@moz-CDA191A6.c3-0.arl-ubr1.sbo-arl.ma.cable.rcn.com) (Quit: aaronlev)
- # [23:48] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # Session Close: Sat Feb 11 00:00:00 2012
The end :)