/irc-logs / mozilla / #accessibility / 2011-12-06 / end
Options:
- # Session Start: Tue Dec 06 00:00:00 2011
- # Session Ident: #accessibility
- # [00:02] <@tbsaunde|afk> hub: yamer?
- # [00:03] <@hub> tbsaunde|afk: yammmer. it is some sort of Facebook we use internally
- # [00:04] * tbsaunde|afk is now known as tbsaunde
- # [00:04] * Quits: richardschwerdtfeger (richardsch@moz-130F1494.lightspeed.austtx.sbcglobal.net) (Quit: richardschwerdtfeger)
- # [00:04] <@tbsaunde> hub: oh, lovely
- # [00:06] * Quits: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com) (Quit: nhirata)
- # [00:09] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Connection reset by peer)
- # [00:09] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # [00:11] * Joins: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com)
- # [00:38] * Joins: hhillen (hhillen@moz-2A42E9B3.upc-d.chello.nl)
- # [00:52] * iamjessklein-away is now known as jess
- # [01:07] * Quits: @jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com) (Ping timeout)
- # [01:08] * jess is now known as iamjessklein-away
- # [01:09] * Joins: jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com)
- # [01:09] * ChanServ sets mode: +o jprmc
- # [01:28] * Quits: hhillen (hhillen@moz-2A42E9B3.upc-d.chello.nl) (Quit: hhillen)
- # [01:46] * Quits: Flyingmana (Mana@moz-12184E52.adsl.alicedsl.de) (Client exited)
- # [02:01] * Quits: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com) (Quit: nhirata)
- # [02:08] * Joins: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com)
- # [02:09] * Quits: victorporof (victorporo@DCE93940.AB163232.79933D60.IP) (Quit: victorporof)
- # [02:10] * Quits: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com) (Quit: nhirata)
- # [02:11] * Quits: @jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com) (Ping timeout)
- # [02:18] * Joins: jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com)
- # [02:18] * ChanServ sets mode: +o jprmc
- # [02:28] * Joins: firewolfbot (firebot@moz-67BF9FE6.carolina.res.rr.com)
- # [02:29] * Quits: ehsan (ehsan@F2D29657.F60B0462.67AC9B1.IP) (sand.mozilla.org concrete.mozilla.org)
- # [02:29] * Quits: khuey|pto (khuey@moz-B77DEAEB.mozilla.org) (sand.mozilla.org concrete.mozilla.org)
- # [02:29] * Quits: @eeejay (eitan@moz-EE2D6C10.monotonous.org) (sand.mozilla.org concrete.mozilla.org)
- # [02:29] * Quits: @firebot (firebot@moz-67BF9FE6.carolina.res.rr.com) (sand.mozilla.org concrete.mozilla.org)
- # [02:29] * Joins: khuey|pto (khuey@moz-B77DEAEB.mozilla.org)
- # [02:36] * Joins: eeejay (eitan@moz-EE2D6C10.monotonous.org)
- # [02:44] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
- # [02:49] <firewolfbot> hub@mozilla.com requested checkin from the wind for attachment 579167 on bug 689105.
- # [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
- # [02:54] <firewolfbot> hub@mozilla.com requested review from surkov.alexander@gm ail.com for attachment 579169 on bug 703770.
- # [02:54] <firewolfbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=703770 nor, --, ---, hub, NEW, Mac Accessibility Verifier errors
- # [02:56] <firewolfbot> hub@mozilla.com cancelled review? for attachment 577354 on bug 455443.
- # [02:56] <firewolfbot> hub@mozilla.com requested review from surkov.alexander@gm ail.com for attachment 579172 on bug 455443.
- # [02:56] <firewolfbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=455443 nor, --, ---, hub, ASSI, cache the parent for the accessibilityAttributeValue(NSAccessibilityParentAttribute)
- # [03:00] <firewolfbot> New Core - Disability Access APIs bug 707846 filed by hub@mozilla.com.
- # [03:00] <firewolfbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=707846 nor, --, ---, nobody, NEW, [mac] remove unused nsAccessibleWrap::GetUnignoredChildCount()
- # [03:01] <firewolfbot> hub@mozilla.com requested review from surkov.alexander@gm ail.com for attachment 579175 on bug 707846.
- # [03:04] * Joins: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP)
- # [03:04] * Quits: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP) (Connection reset by peer)
- # [03:04] * Joins: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP)
- # [03:06] <@tbsaunde> man, I don't get objective C++
- # [03:07] <@hub> tbsaunde: it is just a mix between Objective-C and C++. So one can call the other.
- # [03:07] <@hub> tbsaunde: since their syntax do not conflict
- # [03:08] <@tbsaunde> hub: sure, but I don't get objective C either
- # [03:08] <@tbsaunde> (I have spent all of five minutes trying by reading patches)
- # [03:08] <@hub> tbsaunde: that's what the Mac API use. Not like there is choice.
- # [03:08] <@hub> it give a dynamic object model and easier ABI compatibility
- # [03:09] <@hub> but the performance hit is not negligible
- # [03:09] <@tbsaunde> hub: sure, I'm just saying I don't get it :)
- # [03:09] <@hub> I sometime wish Gtk had used Objective-C instead of reinventing an object model
- # [03:10] <@hub> tbsaunde: basically [obj foo:param] means to call method foo on obj with param
- # [03:11] <@tbsaunde> hub: ... ok
- # [03:12] <@hub> @interface / @end is the class declaration
- # [03:12] <@hub> @implementation / @end is the class implementation
- # [03:12] <@tbsaunde> hub: ok, weird
- # [03:14] <firewolfbot> New Core - Disability Access APIs bug 707852 filed by hub@mozilla.com.
- # [03:14] <firewolfbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=707852 nor, --, ---, nobody, NEW, [mac] remove the Objective-C wrapper
- # [03:16] <@tbsaunde> hub: thanks
- # [03:19] <@tbsaunde> hub: I can't think of a reason to keep putting (mac) in the summary of bugs
- # [03:23] <@hub> not sure. I'll stop.
- # [03:32] * Quits: @hub (hub@21B7B9F2.B87E9213.6E712CE2.IP) (Ping timeout)
- # [03:34] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # [03:37] * Quits: @jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com) (Ping timeout)
- # [03:47] * Joins: nhirata (nhirata.bu@moz-2A9C9106.hsd1.ca.comcast.net)
- # [03:51] * Quits: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP) (Ping timeout)
- # [03:51] * Joins: ehsan (ehsan@moz-104CC309.mv.mozilla.com)
- # [04:07] * Joins: hub (hub@83874EA1.EB7C1AF9.6F478678.IP)
- # [04:07] * ChanServ sets mode: +o hub
- # [05:14] * Joins: surkov (surkov@EAF8AE41.26ED71FF.34044A7F.IP)
- # [05:14] * ChanServ sets mode: +o surkov
- # [05:35] <@surkov> hi, hub, nice progress!
- # [05:36] <@hub> thanks
- # [05:36] <@hub> I have lot of work still
- # [05:37] <@hub> and today bugzilla being down didn't help
- # [05:42] <firewolfbot> surkov.alexander@gmail.com granted review for attachment 579175 on bug 707846.
- # [05:42] <firewolfbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=707846 nor, --, ---, hub, NEW, [mac] remove unused nsAccessibleWrap::GetUnignoredChildCount()
- # [05:43] <@surkov> btw, does parent patch helps perf?
- # [05:47] <@hub> surkov: I think so.
- # [05:47] <@surkov> cool
- # [05:47] <@surkov> hub, what do you think about creation mac tree (the bug I filed)?
- # [05:49] <@surkov> bug 705404
- # [05:49] <firewolfbot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=705404 nor, --, ---, hub, NEW, build separate accessible tree on mac
- # [05:51] <@hub> yeah
- # [05:51] <@hub> technically this is what we do
- # [06:00] <@surkov> hub, this approach allows to avoid creation of mac wrappers when we don't need it
- # [06:01] <@tbsaunde> surkov: objects to killing nsAccessNode::GetPresShell()? as far as I can see we might as well just use do_QueryReferent(mWeakShell)
- # [06:01] <@tbsaunde> *objections
- # [06:02] <@surkov> tbsaunde: if we keep document reference instead of mWeakShell then we wouldn't need press shell in most cases
- # [06:03] <@surkov> document can provide GetPresShell and we wouldn't deal with weak shell likely
- # [06:03] <@tbsaunde> surkov: yeah, forgot about that
- # [06:03] <@tbsaunde> yeah
- # [06:18] * Quits: ehsan (ehsan@moz-104CC309.mv.mozilla.com) (Input/output error)
- # [06:59] * firewolfbot is now known as firebot
- # [07:18] <firebot> surkov.alexander@gmail.com cancelled checkin? for attachment 579167 on bug 689105.
- # [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
- # [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
- # [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
- # [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
- # [07:21] <@tbsaunde> but I'm not really sure what we'd do in the case of two changes to the same nsIContent*
- # [07:22] <@tbsaunde> but I'm not sure if that's a real posibility or not
- # [07:22] <@surkov> it's possible I think
- # [07:22] <@surkov> why do you think queue more suitable?
- # [07:23] <@surkov> about IsFromUserInputFlag I guess it's sort of late to calculate it on text reflow
- # [07:23] <@tbsaunde> surkov: be back in 10, sorry have to go do something
- # [07:23] <@surkov> sure
- # [07:39] <@tbsaunde> ok, back
- # [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
- # [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
- # [07:41] <@surkov> this might be nice
- # [07:41] <@surkov> but I'm not sure if anybody cares
- # [07:42] <@surkov> since we describe what was changed, not user actions actually
- # [07:44] <@tbsaunde> surkov: even so I think its nice to have the events be in the order the changes happen
- # [07:44] <@surkov> it's nice, agree
- # [07:44] <@tbsaunde> surkov: I suppose we can't compute what changed until reflow has finished?
- # [07:45] <@surkov> how would you remove dupes?
- # [07:45] <@surkov> for specific node - yes
- # [07:45] <@tbsaunde> surkov: not sure about dups yet
- # [07:46] <@tbsaunde> I'm not clear what you mean by for specific nodes
- # [07:58] <@tbsaunde> surkov: ?
- # [07:58] <firebot> surkov.alexander@gmail.com denied review for attachment 579169 on bug 703770.
- # [07:58] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=703770 nor, --, ---, hub, NEW, Mac Accessibility Verifier errors
- # [07:59] <@surkov> tbsaunde: I mean reflow in general might be in process but some pieces were processed already
- # [07:59] <@surkov> so if text is pending for reflow then we can't get its future text
- # [08:00] <@tbsaunde> surkov: yeah, ok
- # [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?
- # [08:05] <@surkov> tbsaunde: first of all you should check if we can calculate ifFromUserInput at text reflow at all
- # [08:07] <@tbsaunde> surkov: yeah, I suppose that's a good idea :)
- # [08:08] <@surkov> yep, it may happen synchronously for editing then we are ok, otherwise we're not
- # [08:20] <firebot> surkov.alexander@gmail.com granted review for attachment 579172 on bug 455443.
- # [08:20] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=455443 nor, --, ---, hub, ASSI, cache the parent for the accessibilityAttributeValue(NSAccessibilityParentAttribute)
- # [08:26] <@tbsaunde> surkov: it'll be fun if its async
- # [09:03] * Joins: MarcoZ (marco.zehe@moz-404D9980.dip.t-dialin.net)
- # [09:03] * ChanServ sets mode: +o MarcoZ
- # [09:04] <@MarcoZ> Good morning all!
- # [09:04] <@MarcoZ> hub: surkov: Hi! Good work! I'm marvelling!
- # [09:05] <@surkov> yeah, hub wakes up ;)
- # [09:17] <@hub> and I might have a trybuild with that
- # [09:26] <@MarcoZ> YAY! I'd love to try it!!
- # [09:30] * Joins: atulagrwl (atul@moz-D64D2953.ts.2iij.net)
- # [09:33] * Quits: @hub (hub@83874EA1.EB7C1AF9.6F478678.IP) (Ping timeout)
- # [09:42] * Joins: victorporof (victorporo@DCE93940.AB163232.79933D60.IP)
- # [09:44] * Joins: Flyingmana (Mana@moz-12184E52.adsl.alicedsl.de)
- # [10:06] <@MarcoZ> surkov: So, how complicated will bug 686909 going to be?
- # [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
- # [10:06] <@MarcoZ> s/will/is/
- # [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
- # [10:08] * @MarcoZ nods.
- # [10:10] <@tbsaunde> it would help if my local builds actually start with --no-remote not just exit 0 :/
- # [10:18] * Quits: @surkov (surkov@EAF8AE41.26ED71FF.34044A7F.IP) (Ping timeout)
- # [10:18] * Joins: surkov (surkov@EAF8AE41.26ED71FF.34044A7F.IP)
- # [10:18] * ChanServ sets mode: +o surkov
- # [10:21] <@MarcoZ> tbsaunde: Huh?
- # [10:22] <@tbsaunde> MarcoZ: I'm trying totest stuff, but its hard when builds don't start
- # [10:28] <@MarcoZ> tbsaunde: Hm, strange! Regular builds or local ones?
- # [10:31] <@tbsaunde> MarcoZ: local ones, not much point otherwise
- # [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()
- # [11:11] <@tbsaunde> which I guess makes sense if you think about it
- # [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?
- # [11:18] <@tbsaunde> surkov: that would mean having the existing hashtable and a new hashmap right?
- # [11:18] <@tbsaunde> that seems really heavy weight, but I'm not sure if there's a better approach
- # [11:19] <@surkov> yeah, right, that might be too havvy
- # [11:19] <@surkov> but I don't think layout folks will do that for us
- # [11:19] <@surkov> so maybe we should integrate into layout code hardly
- # [11:19] <@tbsaunde> yeah, I can't see why layout folks would care what caused it and doing the update async makes sense probably
- # [11:20] <@surkov> tbsaunde: which existing hash table is suitable for this?
- # [11:20] <@tbsaunde> hardly?
- # [11:20] <@surkov> they won't care but they may allow us to add extra bit for their structures, dunno
- # [11:20] <@tbsaunde> oh, hmm
- # [11:20] <@surkov> add a11y propose info into their structures
- # [11:21] <@tbsaunde> yeah, I got you
- # [11:21] <@surkov> they keep somewhere all frames insertion/removals
- # [11:21] <@tbsaunde> by which existing hash table do you mean which of the nsHashtable types do we want?
- # [11:21] <@surkov> but it's likely they don't do that for text changes unfortunately
- # [11:21] <@surkov> I thought you suffused to reuse some existing hash table in our code
- # [11:22] <@tbsaunde> no
- # [11:24] <@surkov> got it
- # [11:24] <@surkov> yep, it's just hasmap
- # [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?
- # [11:24] <@surkov> we need to keep "the presence of the node"
- # [11:25] <@surkov> DOM doesn't give us enough information but it may be enough to address the problem we have
- # [11:25] <@tbsaunde> ok
- # [11:25] <@surkov> if all we care about is text editing
- # [11:26] <@tbsaunde> what else would we care about for text change events?
- # [11:27] <@surkov> other kinds of text change events are used rather for vb update than to announce something
- # [11:28] <@surkov> dunno, I don't know all usecases
- # [11:32] <@tbsaunde> surkov: I think the important question is if its from user editing or something else
- # [11:38] <@surkov> that's sort of wide treatment, because user editing can trigger other things
- # [11:40] * Quits: @MarcoZ (marco.zehe@moz-404D9980.dip.t-dialin.net) (Input/output error)
- # [11:41] <@tbsaunde> surkov: yeah, I think probably only direct effects
- # [11:41] <@tbsaunde> but the whole thing is kind of tricky
- # [11:41] <@surkov> righty
- # [11:59] * Quits: @surkov (surkov@EAF8AE41.26ED71FF.34044A7F.IP) (Connection reset by peer)
- # [12:02] * Joins: surkov (surkov@7F8B251F.331F467.5D3F4C44.IP)
- # [12:02] * ChanServ sets mode: +o surkov
- # [12:12] * Quits: @surkov (surkov@7F8B251F.331F467.5D3F4C44.IP) (Ping timeout)
- # [12:35] * Quits: victorporof (victorporo@DCE93940.AB163232.79933D60.IP) (Quit: victorporof)
- # [12:45] * khuey|pto is now known as khuey
- # [12:47] * Quits: khuey (khuey@moz-B77DEAEB.mozilla.org) (Quit: leaving)
- # [12:48] * Joins: khuey (khuey@moz-B77DEAEB.mozilla.org)
- # [13:46] * Joins: MarcoZ (marco.zehe@moz-404D9980.dip.t-dialin.net)
- # [13:47] * ChanServ sets mode: +o MarcoZ
- # [14:03] * khuey is now known as khuey|away
- # [14:06] * Joins: surkov (surkov@7F8B251F.331F467.5D3F4C44.IP)
- # [14:06] * ChanServ sets mode: +o surkov
- # [14:20] * Joins: surkov_ (surkov@178F4FC2.D2B288F8.5D3F4C44.IP)
- # [14:21] * Quits: @surkov (surkov@7F8B251F.331F467.5D3F4C44.IP) (Ping timeout)
- # [14:21] * surkov_ is now known as surkov
- # [14:32] * Quits: bwinton_away (bwinton@D7D3C4B5.88764A66.72A31D6.IP) (Ping timeout)
- # [14:34] * Joins: bwinton_away (bwinton@D7D3C4B5.88764A66.72A31D6.IP)
- # [14:44] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
- # [14:44] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
- # [14:53] * iamjessklein-away is now known as jess
- # [14:57] <@MarcoZ> surkov: Does mozilla.com e-mail also not work for you? I haven't received any mail since yesterday my evening.
- # [14:57] <surkov> MarcoZ: yep, it doesn't
- # [14:58] <@MarcoZ> surkov: And Bugzilla is also down for me. Geez...
- # [14:58] <surkov> yeah, same
- # [15:05] * Joins: davidb (davidb@F2D29657.F60B0462.67AC9B1.IP)
- # [15:05] * ChanServ sets mode: +qo davidb davidb
- # [15:06] <@MarcoZ> Hi davidb!
- # [15:08] <@davidb> hi MarcoZ!
- # [15:08] <@davidb> surely tbsaunde isn't awake?
- # [15:09] <@davidb> firebot: accessibility?
- # [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
- # [15:09] <@davidb> hmm
- # [15:11] <@MarcoZ> davidb: He was earlier, but probably is asleep now.
- # [15:11] <@davidb> yeah
- # [15:11] <@MarcoZ> davidb: And with Bugzilla down, there's not much to do.
- # [15:12] <@davidb> Perhaps write tests, or update the our websites
- # [15:12] <@davidb> I'm sad our web face is so confusing
- # [15:12] <@davidb> oh surkov is here?
- # [15:12] <surkov> I am
- # [15:12] <@davidb> hi
- # [15:13] * davidb sets mode: +o surkov
- # [15:13] <@surkov> hi
- # [15:15] <@davidb> i sure do rely on email a lot
- # [15:15] * @MarcoZ too.
- # [15:15] * @davidb hunts for a bookmark
- # [15:17] * Joins: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
- # [15:19] * Joins: ehsan (ehsan@moz-104CC309.mv.mozilla.com)
- # [15:24] <@MarcoZ> Sheesh, nntp is also down.
- # [15:24] * Quits: bwinton_away (bwinton@D7D3C4B5.88764A66.72A31D6.IP) (Ping timeout)
- # [15:27] * Joins: richardschwerdtfeger (richardsch@BE0AFBFF.9DD4DBBF.6A7A197.IP)
- # [15:29] * @davidb reviews http://www.w3.org/html/wg/wiki/ChangeProposals/FocusRingTextBaseline
- # [15:38] * khuey|away is now known as khuey
- # [15:39] <@davidb> man, can't even get to my personal planning etherpad
- # [15:39] <@davidb> nooooooooooooooo.com
- # [15:43] * Joins: bwinton (bwinton@F2D29657.F60B0462.67AC9B1.IP)
- # [15:43] * ChanServ sets mode: +h bwinton
- # [15:49] * Joins: jprmc (jprmc@moz-9CE00FC0.holiday-inn-express.mozilla.hq)
- # [15:49] * ChanServ sets mode: +o jprmc
- # [15:50] * Quits: @MarcoZ (marco.zehe@moz-404D9980.dip.t-dialin.net) (Quit: l8er)
- # [15:59] * Joins: ehsan_ (ehsan@F2D29657.F60B0462.67AC9B1.IP)
- # [16:00] * Quits: ehsan (ehsan@moz-104CC309.mv.mozilla.com) (Ping timeout)
- # [16:11] * Quits: nhirata (nhirata.bu@moz-2A9C9106.hsd1.ca.comcast.net) (Quit: nhirata)
- # [16:15] * Quits: atulagrwl (atul@moz-D64D2953.ts.2iij.net) (Client exited)
- # [16:17] * khuey is now known as khuey|away
- # [16:19] <@davidb> how is safari 5 a11y?
- # [16:23] <firebot> josh@joshmatthews.net requested review from surkov.alexander@gm ail.com for attachment 579280 on bug 706369.
- # [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
- # [16:28] <@davidb> yaya
- # [16:28] <@davidb> bugzilla!
- # [16:29] * Joins: bwinton_away (bwinton@D7D3C4B5.88764A66.72A31D6.IP)
- # [16:34] <@davidb> surkov: did dbaron get back to you?
- # [16:34] <@surkov> davidb: dunno, mozilla mail doesn't work for me
- # [16:34] <@davidb> oh hmmm mail is still down yeah
- # [16:35] * Joins: MarcoZ (marco.zehe@moz-404D9980.dip.t-dialin.net)
- # [16:35] * ChanServ sets mode: +o MarcoZ
- # [16:36] * Joins: atulagrwl (atul@4386A3CB.E762D0A2.A3D1B221.IP)
- # [16:38] <@davidb> surkov: why does fig caption get the extra whitespace at the end?
- # [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?
- # [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
- # [16:39] <@davidb> a loop is a loop :)
- # [16:39] <@surkov> davidb: gaps or name calculation algorithm impl
- # [16:39] <@davidb> hm
- # [16:39] * @surkov forgot to file bug about this
- # [16:45] * Quits: atulagrwl (atul@4386A3CB.E762D0A2.A3D1B221.IP) (Ping timeout)
- # [16:47] <@tbsaunde> MarcoZ: I don't see anything particularly scary in that patch
- # [16:47] * @tbsaunde notes they added a couple uses of GetChildAt() :/
- # [16:48] <@davidb> surkov: what is the current status of bug https://www.w3.org/Bugs/Public/show_bug.cgi?id=14320
- # [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 :)
- # [16:50] <@surkov> it appears making aria-controls to allow aria-activedescendant to control the focus sounds good
- # [16:50] <@surkov> that's what I commented there
- # [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)
- # [16:51] <@surkov> in this light there's nothing bad in aria-owns
- # [16:51] <@davidb> i'm curious what textual change we want to the impl guide
- # [16:52] <@davidb> it is probably best if you propose something (in the bug) and we can polish it
- # [16:52] <@surkov> it might be not enough to change impl guide only unfortunately
- # [16:52] <@davidb> one goal is to make sure the guide doesn't block good potential solutions
- # [16:52] <@surkov> I still need to play with autocomplete implementation and ARIAs
- # [16:52] <@davidb> ok
- # [16:53] <@surkov> that's great, but ARIA spec can do that
- # [16:53] <@davidb> right
- # [16:53] <@surkov> and the funny things these specs are controlled by different groups
- # [16:54] <@davidb> agreed
- # [16:55] * Joins: victorporof (victorporo@E8764A15.F4308271.4873E54D.IP)
- # [17:01] * Joins: atulagrwl (atul@944C5E56.71A613B7.A3D1B221.IP)
- # [17:04] <@davidb> surkov: do you have an xpi of the latest domi somewhere?
- # [17:04] <@surkov> nope
- # [17:04] <@surkov> I build it from sources
- # [17:05] <@davidb> ok and you just build it straight into your FF install dir?
- # [17:05] <@surkov> I suggest you to do the same if you are going to run it on nightlies you build
- # [17:05] <@davidb> yeah
- # [17:05] <@davidb> surkov: how do i do this?
- # [17:05] <@surkov> clone http://hg.mozilla.org/dom-inspector/
- # [17:06] <@davidb> oh and i give it a path to my install?
- # [17:06] <@surkov> then copy it into extensions folder of your future build
- # [17:06] <@davidb> oh
- # [17:06] <@surkov> and then add ac_add_options --enable-extensions=default,inspector into your .mozconfig
- # [17:06] <@davidb> thanks!
- # [17:06] <@surkov> if you have one tree for everything then you can clone into extensions folder
- # [17:07] <@surkov> I usually copy sources for new build
- # [17:07] <@davidb> i'm not sure i got you
- # [17:07] <@surkov> so I can continue to work on patches without affecting on the build
- # [17:07] <@davidb> ah
- # [17:07] * Quits: %bwinton (bwinton@F2D29657.F60B0462.67AC9B1.IP) (Quit: Leaving...)
- # [17:07] * bwinton_away is now known as bwinton
- # [17:12] <@davidb> surkov: is there a make step?
- # [17:13] * Joins: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com)
- # [17:14] <@tbsaunde> now, if you want to have fun write a macro atomically such that atomically { ... } does the right thing
- # [17:15] <@surkov> davidb: no, just add option into .mozconfig
- # [17:16] <@surkov> tbsaunde: example?
- # [17:17] <@tbsaunde> surkov: you doo #define atomically blah such that atomically { do_something_useful_that_needs_to_be_atomic(); |} is fine
- # [17:17] <@MarcoZ> Bah, Bugzilla is down again!
- # [17:17] <@tbsaunde> not sure how else you can explain that problem with out giving away the answer
- # [17:18] <@tbsaunde> there all telling you to go work on your existing patches :)
- # [17:19] * Quits: @jprmc (jprmc@moz-9CE00FC0.holiday-inn-express.mozilla.hq) (Ping timeout)
- # [17:20] * @davidb builds with inspector
- # [17:40] <@davidb> MarcoZ: were you pinged at all to check accessibility of mozilla.org/story ?
- # [17:40] <@davidb> just curious
- # [17:44] <@davidb> it could be that webdev has absorbed some a11y qa
- # [17:50] <@MarcoZ> davidb: No I wasn't pinged about mozilla.org/story at all.
- # [17:51] <firebot> marco.zehe@googlemail.com changed the Status on bug 658272 from RESOLVED to VERIFIED.
- # [17:51] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=658272 nor, --, mozilla11, surkov.alexander, VERI FIXED, Implement figure and figcaption accessibility
- # [17:51] <firebot> maglione.k@gmail.com changed the Status on bug 705564 from UNCONFIRMED to RESOLVED.
- # [17:52] <firebot> maglione.k@gmail.com set the Resolution field on bug 705564 to WORKSFORME.
- # [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
- # [17:52] * Joins: jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com)
- # [17:52] * ChanServ sets mode: +o jprmc
- # [17:56] * Joins: Flyingmana_ (Mana@moz-9C0B5A6C.adsl.alicedsl.de)
- # [17:57] * Quits: Flyingmana (Mana@moz-12184E52.adsl.alicedsl.de) (Ping timeout)
- # [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.
- # [18:00] <@davidb> i was more interested in whether you were pinged
- # [18:03] <@MarcoZ> davidb: As I said...nope.
- # [18:04] <@davidb> MarcoZ: ah yes, sorry
- # [18:04] * @davidb dials into w3 call
- # [18:15] <firebot> jigneshhk1992@gmail.com cancelled review?(surkov.alexander@gm ail.com) for attachment 579280 on bug 706369.
- # [18:15] <firebot> jigneshhk1992@gmail.com requested review from surkov.alexander@gm ail.com for attachment 579326 on bug 706369.
- # [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
- # [18:21] * Parts: richardschwerdtfeger (richardsch@BE0AFBFF.9DD4DBBF.6A7A197.IP)
- # [18:25] <firebot> surkov.alexander@gmail.com denied review for attachment 579326 on bug 706369.
- # [18:44] <firebot> jigneshhk1992@gmail.com requested review from surkov.alexander@gm ail.com for attachment 579342 on bug 706369.
- # [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
- # [18:48] * davidb changes topic to 'we are experiencing service outages | "build the web for everyone" | release schedule: http://mzl.la/LNc0W'
- # [18:49] * davidb sets mode: +o eeejay
- # [18:50] <firebot> surkov.alexander@gmail.com granted review for attachment 579342 on bug 706369.
- # [18:55] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Ping timeout)
- # [18:58] <firebot> jigneshhk1992@gmail.com requested review from surkov.alexander@gm ail.com for attachment 579347 on bug 706369.
- # [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
- # [19:02] * jess is now known as iamjessklein-away
- # [19:02] * iamjessklein-away is now known as jess
- # [19:03] <firebot> josh@joshmatthews.net cancelled review?(surkov.alexander@gm ail.com) for attachment 579347 on bug 706369.
- # [19:06] * Joins: hhillen (hhillen@moz-2A42E9B3.upc-d.chello.nl)
- # [19:11] <firebot> bmo@edmorley.co.uk changed the Status on bug 707846 from NEW to RESOLVED.
- # [19:11] <firebot> bmo@edmorley.co.uk set the Resolution field on bug 707846 to FIXED.
- # [19:11] <firebot> bmo@edmorley.co.uk changed the Target Milestone on bug 707846 from --- to mozilla11.
- # [19:11] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=707846 nor, --, mozilla11, hub, RESO FIXED, [mac] remove unused nsAccessibleWrap::GetUnignoredChildCount()
- # [19:11] <firebot> bmo@edmorley.co.uk changed the Status on bug 689105 from NEW to RESOLVED.
- # [19:11] <firebot> bmo@edmorley.co.uk set the Resolution field on bug 689105 to FIXED.
- # [19:11] <firebot> bmo@edmorley.co.uk changed the Target Milestone on bug 689105 from --- to mozilla11.
- # [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
- # [19:11] <firebot> bmo@edmorley.co.uk changed the Status on bug 704852 from ASSIGNED to RESOLVED.
- # [19:12] <firebot> bmo@edmorley.co.uk set the Resolution field on bug 704852 to FIXED.
- # [19:12] <firebot> bmo@edmorley.co.uk changed the Target Milestone on bug 704852 from --- to mozilla11.
- # [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
- # [19:15] * Joins: hub (hub@21B7B9F2.B87E9213.6E712CE2.IP)
- # [19:15] * ChanServ sets mode: +o hub
- # [19:21] * Quits: atulagrwl (atul@944C5E56.71A613B7.A3D1B221.IP) (Quit: Leaving)
- # [19:22] * Quits: victorporof (victorporo@E8764A15.F4308271.4873E54D.IP) (Ping timeout)
- # [19:22] * Quits: @surkov (surkov@178F4FC2.D2B288F8.5D3F4C44.IP) (Quit: surkov)
- # [19:41] * ehsan_ is now known as ehsan
- # [19:44] <@MarcoZ> hub: Congratulations on the first landed Mac patches!
- # [19:45] * khuey|away is now known as khuey
- # [19:46] * Quits: @hub (hub@21B7B9F2.B87E9213.6E712CE2.IP) (Ping timeout)
- # [19:47] <@davidb> woot!
- # [19:47] * Joins: hub (hub@21B7B9F2.B87E9213.6E712CE2.IP)
- # [19:47] * ChanServ sets mode: +o hub
- # [19:47] <firebot> jigneshhk1992@gmail.com requested review from surkov.alexander@gm ail.com for attachment 579368 on bug 706369.
- # [19:47] <@davidb> and so it begins
- # [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
- # [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.
- # [19:50] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=707852 nor, --, ---, nobody, NEW, [mac] remove the Objective-C wrapper
- # [19:50] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=703770 nor, --, ---, hub, NEW, Mac Accessibility Verifier errors
- # [19:50] <@MarcoZ> unless surkov and hub think this is not ready for that kind of exposure yet.
- # [19:54] * Quits: hhillen (hhillen@moz-2A42E9B3.upc-d.chello.nl) (Quit: hhillen)
- # [19:54] <@davidb> well, your testing can help that decision
- # [19:56] * khuey is now known as khuey|away
- # [20:05] <@MarcoZ> davidb: Yes, hope Hub gets me that try server build soon, would love to give it a spin!
- # [20:05] <@MarcoZ> But for today, I'm off. See you tomorrow!
- # [20:05] <@davidb> ciao@!
- # [20:05] <@davidb> -@
- # [20:06] * Quits: @MarcoZ (marco.zehe@moz-404D9980.dip.t-dialin.net) (Quit: See you tomorrow!)
- # [20:08] <@hub> yeah I have to figure out where it located :-/
- # [20:21] <@davidb> hard without email i imagine
- # [20:22] <@davidb> hub: perhaps here somewhere http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/
- # [20:23] <@eeejay> YES!
- # [20:23] <@davidb> eeejay: that sounds positive
- # [20:24] * khuey|away is now known as khuey
- # [20:24] * @eeejay is happy about birch moving to mc
- # [20:27] <@davidb> ah yes
- # [20:29] <@hub> davidb: yeah.
- # [20:29] <@hub> eeejay: that's awesome
- # [20:33] <@hub> ok I found them
- # [20:33] <@hub> the do
- # [20:34] <@hub> the have a11y in it
- # [20:34] <@hub> that's a start
- # [20:36] <@davidb> coolio
- # [20:38] * Quits: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com) (Quit: nhirata)
- # [20:39] <@hub> I should file a bz to remind us to enable a11y in the mac build
- # [20:44] * Joins: nhirata (nhirata.bu@moz-BBE3ABD.mv.mozilla.com)
- # [20:50] <@davidb> yep
- # [20:51] * Quits: @jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com) (Input/output error)
- # [20:52] * Joins: jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com)
- # [20:52] * ChanServ sets mode: +o jprmc
- # [21:19] * khuey is now known as khuey|away
- # [21:26] <@hub> ok, I had a scare
- # [21:26] <@hub> that the bug resurfaced
- # [21:27] <@hub> but no
- # [21:27] <@hub> try build didn't pickup the mozconfig change
- # [21:27] <@davidb> aha
- # [21:28] <@hub> the suffix is apparently "osx64" not "macosx64"
- # [21:39] <firebot> hub@mozilla.com requested checkin from the wind for attachment 579413 on bug 455443.
- # [21:39] <firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=455443 nor, --, ---, hub, ASSI, cache the parent for the accessibilityAttributeValue(NSAccessibilityParentAttribute)
- # [21:40] * khuey|away is now known as khuey
- # [21:50] * Quits: jaws (u2871@moz-160C58C6.com) (Max SendQ exceeded)
- # [21:50] * Joins: jaws (u2871@moz-160C58C6.com)
- # [21:57] <@tbsaunde> hub: + mParent = GetClosestInterestingAccessible(nativeParent);
- # [21:57] <@tbsaunde> + return mParent;
- # [21:57] <@tbsaunde> really?
- # [21:58] <@hub> tbsaunde: what's wrong?
- # [21:58] <@tbsaunde> hub: why not return mParent = GetClosestInterestAccessible()?
- # [21:59] <@hub> tbsaunde: to be honest? because that even less readable. and it does nothing to the output machine code
- # [22:00] <@tbsaunde> hub: I'd disagree
- # [22:01] <@tbsaunde> not with the result, but just as readable and less lines
- # [22:03] <@tbsaunde> hub: why do you think using lvalues as rvalues is unreadable?
- # [22:05] <@hub> doing TWO statement in one line just blunder readability
- # [22:05] <@hub> what if one really meant return mParent == GetClosestInterestingAccessible(nativeParent);
- # [22:05] <@hub> it would immediately pop
- # [22:05] <@tbsaunde> pop?
- # [22:06] <@tbsaunde> it returns a pointer not a bool so that's a type error for one thing
- # [22:06] <@tbsaunde> also don't be bad
- # [22:07] <@hub> I have seen so many bugs caused by sloppy programming style like combining statements, etc
- # [22:07] <@hub> that I tend to try to be strict on it
- # [22:07] <@hub> including mandating { and }
- # [22:07] <@hub> which not everybody seems to agree
- # [22:09] <@tbsaunde> because is silly :)
- # [22:09] <@tbsaunde> I can't honestly think of that many bugs that are related to syntax
- # [22:10] <@tbsaunde> (even when talking to ta's grading horrible kernels)
- # [22:10] * Quits: @eeejay (eitan@moz-EE2D6C10.monotonous.org) (Input/output error)
- # [22:11] * Joins: eeejay (eitan@moz-EE2D6C10.monotonous.org)
- # [22:26] * Quits: @davidb (davidb@F2D29657.F60B0462.67AC9B1.IP) (Quit: davidb)
- # [22:26] * khuey is now known as khuey|away
- # [22:53] * khuey|away is now known as khuey
- # [22:53] * Parts: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
- # [23:02] * jess is now known as iamjessklein-away
- # [23:03] * Quits: @jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com) (Ping timeout)
- # [23:04] * khuey is now known as khuey|away
- # [23:06] * Joins: jprmc (jprmc@moz-BBE3ABD.mv.mozilla.com)
- # [23:06] * ChanServ sets mode: +o jprmc
- # [23:11] * Joins: davidb (davidb@moz-A4A01B28.eng.wind.ca)
- # [23:11] * ChanServ sets mode: +qo davidb davidb
- # [23:29] * bwinton is now known as bwinton_away
- # [23:29] * iamjessklein-away is now known as jess
- # [23:52] * Joins: davidb_ (davidb@moz-976CD2.dsl.bell.ca)
- # [23:52] * Quits: davidb_ (davidb@moz-976CD2.dsl.bell.ca) (Input/output error)
- # Session Close: Wed Dec 07 00:00:00 2011
The end :)