/irc-logs / mozilla / #accessibility / 2013-03-18 / end
Options:
- # Session Start: Mon Mar 18 00:00:00 2013
- # Session Ident: #accessibility
- # [02:23] <@firebot> surkov.alexander@gmail.com cancelled review?(surkov.alexander@gm ail.com) for attachment 720484 on bug 847233.
- # [02:23] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=847233 nor, --, ---, mounir, ASSI, Fix a11y with <input type='file'> changes
- # [02:43] * Joins: Jamie (Instantbir@moz-CA26021.jantrid.net)
- # [02:52] <@firebot> surkov.alexander@gmail.com cancelled review?(surkov.alexander@gm ail.com) for attachment 724788 on bug 850981.
- # [02:52] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=850981 nor, --, ---, trev.saunders, NEW, make AppendChild() inline around InsertChildAt()
- # [02:57] * Quits: Hixie (ianh@C6D27F7B.5EFFBB68.81BC061B.IP) (Ping timeout)
- # [02:57] * Joins: Hixie (ianh@C6D27F7B.5EFFBB68.81BC061B.IP)
- # [03:02] <@firebot> trev.saunders@gmail.com granted review for attachment 725255 on bug 833256.
- # [03:02] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=833256 nor, --, ---, joshyyuan, NEW, role note shouldn't pick up the name from subtree
- # [03:02] * Joins: surkov (surkov@511E724B.E00BF004.E17943EE.IP)
- # [03:02] * ChanServ sets mode: +o surkov
- # [03:20] * Joins: scott_gonzalez (scott_gonz@moz-9A6B491C.nycmny.fios.verizon.net)
- # [03:34] <tbsaunde> surkov: so, what does the embedded object collector thing have to do with tree update other than you need to be sure to invalidate it when needed?
- # [03:34] <@surkov> just invalidate
- # [03:40] <tbsaunde> surkov: so, i looked at those flush during accessible tree update asserts and it looks like ProcessLoad() does happen between when we make NotificationController::mState eRefreshProcessingForUpdate and when we change it away
- # [03:41] <tbsaunde> I suspect that processing generic notifications that aren't for content insertions can also cause script to run because of sync events
- # [03:42] <@surkov> tbsaunde: a11y sync events?
- # [03:42] <tbsaunde> surkov: yes
- # [03:45] <tbsaunde> surkov: DocAccessible::ProcessLoad() calls nsEventShell::FireEvent() and so does RootAccessible::PRocessDOMEvent()
- # [03:45] <@surkov> tbsaunde: should we change that mState before generic notification processing?
- # [03:46] <tbsaunde> surkov: maybe, but are we sure that running js there is ok?
- # [03:46] <tbsaunde> especially before we call DocAccessible::ProcessInvalidationList()
- # [03:47] <@surkov> basically we need to make sure that we don't run script when we build the tree, right?
- # [03:47] <tbsaunde> surkov: atleast that
- # [03:47] <@surkov> if notification processing can't affect on the tree then it's seems ok
- # [03:48] <tbsaunde> surkov: so should we maybe handle the invalidation list before ProcessLoad / notifications?
- # [03:48] <@surkov> seems reasoanble
- # [03:50] <tbsaunde> surkov: so, why to notifications need to fire events sync?
- # [03:51] <@surkov> I'd say that should stay in line to keep events in sync
- # [03:52] <tbsaunde> surkov: what about cases where they should be coallesced with async evnet?
- # [03:53] <@surkov> you meant shouldn't?
- # [03:53] <tbsaunde> surkov: no
- # [03:54] <@surkov> tbsaunde: if they will be coalesced then it's fine
- # [03:54] <tbsaunde> surkov: afaik now they are not
- # [03:55] <@surkov> tbsaunde: correct, and I said it should be ok if they will
- # [03:55] <tbsaunde> surkov: since nsEventShell::FireVent() never touches the event queue
- # [03:58] <tbsaunde> surkov: then I don't get what your saying
- # [03:59] * Quits: Jamie (Instantbir@moz-CA26021.jantrid.net) (Client exited)
- # [03:59] <@surkov> tbsaunde: I meant putting them into queue seems reasonable
- # [04:00] * Joins: Jamie (Instantbir@moz-CA26021.jantrid.net)
- # [04:00] <tbsaunde> surkov: ah, so your saying using IFreDelayedAccEvent() is fine then?
- # [04:00] <@surkov> tbsaunde: yes
- # [04:01] * Quits: Jamie (Instantbir@moz-CA26021.jantrid.net) (Quit: Instantbird 1.4a1pre -- http://www.instantbird.com)
- # [04:01] <@surkov> tbsaunde: but if no coalescence will happen then probably it doesn't make huge sense
- # [04:02] <tbsaunde> surkov: yeah, but doesn't really hurt much either
- # [04:02] <@surkov> seems so
- # [04:07] * Joins: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP)
- # [04:21] * Quits: scott_gonzalez (scott_gonz@moz-9A6B491C.nycmny.fios.verizon.net) (Quit: scott_gonzalez)
- # [04:24] <yzen> hi everyone, would anybody know where I can find accessfu unit tests inside the accessible tests?
- # [04:28] * Joins: scott_gonzalez (scott_gonz@moz-9A6B491C.nycmny.fios.verizon.net)
- # [04:36] * Joins: Jamie (Instantbir@moz-CA26021.jantrid.net)
- # [04:45] * Quits: scott_gonzalez (scott_gonz@moz-9A6B491C.nycmny.fios.verizon.net) (Quit: scott_gonzalez)
- # [04:48] <yzen> never mind i think i found this bug 811307
- # [04:48] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=811307 nor, --, ---, dbolter, NEW, [AccessFu] Add mochitest for enabling
- # [05:16] <@firebot> yura.zenevich@gmail.com requested feedback from eitan@monotonous. org for attachment 725996 on bug 753984.
- # [05:16] <@firebot> yura.zenevich@gmail.com requested needinfo from eitan@monotonous. org on bug 753984.
- # [05:16] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=753984 nor, --, ---, yura.zenevich, NEW, [AccessFu] Make utterance order configurable
- # [05:27] <@firebot> New Core - Disability Access APIs bug 852021 filed by surkov.alexander@gmail.com.
- # [05:28] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=852021 nor, --, ---, nobody, NEW, add getText* at caret offset mochitest
- # [05:37] <@firebot> surkov.alexander@gmail.com changed the Assignee on bug 852021 from nobody@mozilla.org to surkov.alexander@gmail.com.
- # [05:48] <@firebot> trev.saunders@gmail.com requested review from surkov.alexander@gm ail.com for attachment 725999 on bug 850981.
- # [05:48] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=850981 nor, --, ---, trev.saunders, NEW, make AppendChild() inline around InsertChildAt()
- # [05:55] * Quits: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP) (Quit: Leaving.)
- # [06:17] * Quits: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com) (Ping timeout)
- # [06:53] <@firebot> New Core - Disability Access APIs bug 852044 filed by trev.saunders@gmail.com.
- # [06:53] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=852044 nor, --, ---, trev.saunders, NEW, don't fire events from DocAccessible::ProcessLoad() sync
- # [06:57] <@firebot> trev.saunders@gmail.com requested review from surkov.alexander@gm ail.com for attachment 726005 on bug 852044.
- # [06:58] <tbsaunde> surkov: do you remember why event coalescing happens on queue insertion instead of jsut before firing?
- # [06:58] <@surkov> tbsaunde: some coalescence uses an accessible tree
- # [06:59] <tbsaunde> surkov: oh, yeah ok
- # [06:59] <tbsaunde> surkov: I wonder if its worth to only coalesce the stuff we have to then and do other stuff later
- # [07:00] <@surkov> this sounds complicated
- # [07:01] <tbsaunde> surkov: I don't think its too bad if you just break it up on the level of cases in the current switch
- # [07:02] <@surkov> btw, what's for that?
- # [07:03] <tbsaunde> surkov: don't think I get the question
- # [07:03] <@surkov> tbsaunde: why do you want to delay coalescence?
- # [07:04] <tbsaunde> surkov: I think its faster (atleast algorithmically)
- # [07:05] <@surkov> ok, it's worth to investigate
- # [07:07] * Joins: icaaq (Adium@moz-D8A76420.cust.bredbandsbolaget.se)
- # [07:25] * Quits: nhirata (anonymous@moz-1A50F7F8.hsd1.ca.comcast.net) (Quit: nhirata)
- # [08:01] <@firebot> surkov.alexander@gmail.com granted review for attachment 725999 on bug 850981.
- # [08:01] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=850981 nor, --, ---, trev.saunders, NEW, make AppendChild() inline around InsertChildAt()
- # [08:17] * Quits: Jesse (jruderman@moz-537BCF9.hsd1.ca.comcast.net) (Quit: Jesse)
- # [08:20] * Quits: Jamie (Instantbir@moz-CA26021.jantrid.net) (Quit: Instantbird 1.4a1pre -- http://www.instantbird.com)
- # [08:33] * Joins: Jesse (jruderman@moz-537BCF9.hsd1.ca.comcast.net)
- # [09:15] * Joins: marcoz (marco.zehe@moz-B9E01AB9.dip.t-dialin.net)
- # [09:15] * ChanServ sets mode: +o marcoz
- # [09:30] * Quits: icaaq (Adium@moz-D8A76420.cust.bredbandsbolaget.se) (Quit: Leaving.)
- # [09:43] * Joins: icaaq (Adium@moz-D8A76420.cust.bredbandsbolaget.se)
- # [10:30] * surkov sets mode: +o tbsaunde
- # [10:30] <@surkov> tbsaunde: ping
- # [10:41] * Quits: vd (vd@E355CC6C.D7B7DA9B.43EBE9B3.IP) (Quit: Reconnecting)
- # [10:41] * Joins: vd (vd@E355CC6C.D7B7DA9B.43EBE9B3.IP)
- # [10:42] * Quits: vd (vd@E355CC6C.D7B7DA9B.43EBE9B3.IP) (Quit: leaving)
- # [11:40] * Quits: @marcoz (marco.zehe@moz-B9E01AB9.dip.t-dialin.net) (Quit: Leaving.)
- # [11:40] * Joins: marcoz (marco.zehe@moz-B9E01AB9.dip.t-dialin.net)
- # [11:41] * ChanServ sets mode: +o marcoz
- # [12:12] <@tbsaunde> surkov: hi
- # [12:13] <@surkov> tbsaunde: good morning, what do you think if we keep all managers as members of accessibilityService
- # [12:14] <@tbsaunde> surkov: instead of inheriting from them?
- # [12:14] <@surkov> tbsaunde: yes
- # [12:14] <@surkov> tbsaunde: we will still have globals like SelectionMgr() and etc
- # [12:14] <@tbsaunde> surkov: I gues that's fine, but why not just have seperate globals?
- # [12:15] <@tbsaunde> surkov: what problem are you trying to solve?
- # [12:15] <@surkov> AccService will manage them anyway
- # [12:15] <@surkov> tbsaunde: it'd be a nice way to add cc to them
- # [12:16] <@tbsaunde> surkov: is that a good idea? they can't possibly die till xpcom-shutdown, and they will certainly die then
- # [12:17] <@tbsaunde> if we think they don't we could just assert that
- # [12:17] <@surkov> tbsaunde: I suppose they can keep things alive that could lead to leaks after shutdown (sometimes I can see firefox.exe handing after app is closed)
- # [12:19] <@tbsaunde> surkov: so why don't you assert it? or we could just not really ref count it at all
- # [12:19] <@surkov> tbsaunde: they are used as listeners
- # [12:19] <@surkov> so they should be refcounted
- # [12:19] <@surkov> (except FocusManager)
- # [12:20] <@tbsaunde> surkov: nobody should hold refs or be notifying of anything we care about nearly as late as xpcom-shutdown afaik
- # [12:21] <@surkov> true, sometimes that happens
- # [12:21] <@tbsaunde> what happens?
- # [12:22] <@surkov> remember our popular leak, before I removed DocManager from page hide listeners we leaked DocManager
- # [12:22] <@surkov> because event target kept it alive
- # [12:24] <@tbsaunde> surkov: sure, but if cc would fix that at all I think its just wallpaper, and you'd still leak stuff till xpcom-shutdown
- # [12:24] <@surkov> tbsaunde: wouldn't it possible to leak, if SelectionManager keeps DOM element and DOM element keeps AccessibilityService
- # [12:25] <@surkov> good point
- # [12:26] <@tbsaunde> surkov: it shouldn't be bad to fake ref count accessibility service if your worried about it, or we could just assert it gets deleted in shutdown
- # [12:27] <@tbsaunde> seperating managers might help with header inclussion / be a good idea, I just don't think it solves your problem :)
- # [12:28] <@surkov> ok
- # [12:42] <@tbsaunde> marcoz: I'd bet you broke ke testcaret/test_general.html
- # [12:52] <@marcoz> tbsaunde: I didn't. The push was davidb's patch, pushed by eeejay. I just analyzed the failures and commented. For lack of anything more useful to do. :)
- # [12:53] <@tbsaunde> I think leaving unstared is a better idea than blaiming some random orange bug that's been hit once in half a year
- # [12:54] <@marcoz> tbsaunde: Moreover, the patch itself doesn't touch that file, so it does something wrong that subsequently affects it and causes bug 781120 to surface.
- # [12:54] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=781120 nor, --, ---, nobody, RESO WORKSFORME, Intermittent accessible/textcaret/test_general.html | Wrong caret offset for [object HTMLDocument @
- # [12:55] <@marcoz> tbsaunde: The failure was consistent across platforms with this particular checkin. It may be random, but it also indicates something worth noting. I also added a manual comment explicitly stating that this series of stars was the result of a particular try push.
- # [12:57] * Joins: scott_gonzalez (scott_gonz@moz-9A6B491C.nycmny.fios.verizon.net)
- # [12:57] <@tbsaunde> marcoz: the chances that's random are \epsilon
- # [12:58] <@tbsaunde> I don't know how the patch broke other tests but it did, and given it broke things I don't see what we get from pretending it didn't by starring it
- # [13:12] * Joins: Justin_o (Justin_o@67828CC7.C1A51174.9D42CF23.IP)
- # [13:13] * Quits: @surkov (surkov@511E724B.E00BF004.E17943EE.IP) (Quit: surkov)
- # [13:33] * Joins: hub (hub@F2D29657.F60B0462.67AC9B1.IP)
- # [13:33] * ChanServ sets mode: +o hub
- # [13:43] * Quits: @marcoz (marco.zehe@moz-B9E01AB9.dip.t-dialin.net) (Quit: Leaving.)
- # [13:51] * Quits: brambles (xymox@moz-969AAE9B.barwen.ch) (Ping timeout)
- # [13:52] * Joins: brambles (xymox@moz-969AAE9B.barwen.ch)
- # [13:53] * Joins: marcoz (marco.zehe@moz-B9E01AB9.dip.t-dialin.net)
- # [13:55] <marcoz> tbsaunde: I didn't imply they didn't. These stars were meant as an FYI for the impact this patch had. I also documented in bug 811307 that all of the failures pointed to the same already reported orange.
- # [13:55] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=811307 nor, --, ---, dbolter, NEW, [AccessFu] Add mochitest for enabling
- # [13:55] * ChanServ sets mode: +o marcoz
- # [14:00] <@tbsaunde> marcoz: I think the nice orange tree does that just fine, but whatever
- # [14:04] * Joins: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP)
- # [14:07] * Joins: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
- # [14:08] * clown is now known as clown_mtg
- # [14:10] * Joins: davidb (davidb@F2D29657.F60B0462.67AC9B1.IP)
- # [14:10] * ChanServ sets mode: +qo davidb davidb
- # [14:11] <@davidb> heyo
- # [14:12] * Quits: Justin_o (Justin_o@67828CC7.C1A51174.9D42CF23.IP) (Quit: Justin_o)
- # [14:12] <@tbsaunde> morning
- # [14:14] <@hub> heya
- # [14:25] <@davidb> marcoz: that text caret failures are a bummer
- # [14:28] <@marcoz> davidb: Hi! I think the debug ones of those are even more interesting below those obvious oranges, because I think they indicate a leak. At least the Windows and Linux ones did, IIRC.
- # [14:28] <@marcoz> But yeah, the fact that all of these runs had the same tests fail is remarkable.
- # [14:29] * Joins: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com)
- # [14:39] <@marcoz> davidb: To be sure we are not dealing with some other random fluke, I would think it makes sense to push that patch to try once again and watch the results. If they show something similar, we know we still have a real problem.
- # [14:39] <@davidb> marcoz: sounds like you are volunteering :)
- # [14:40] <@marcoz> davidb: Um, not explicitly, no, I was just suggesting a course of action. ;)
- # [14:40] <@davidb> heheh
- # [14:57] <@marcoz> davidb: 1:1?
- # [14:57] <@firebot> New Core - Disability Access APIs bug 852125 filed by jruderman@gmail.com.
- # [14:57] <@davidb> marcoz: yes! 1 min
- # [14:57] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=852125 nor, --, ---, nobody, NEW, "ASSERTION: Unbinding the unbound accessible!" with <iframe>
- # [14:59] <@davidb> marcoz: ready when you are
- # [15:02] * Joins: jongunderson (chatzilla@moz-689FC632.ptr.us.xo.net)
- # [15:13] <@firebot> New Core - Disability Access APIs bug 852129 filed by jruderman@gmail.com.
- # [15:13] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=852129 nor, --, ---, nobody, NEW, "ASSERTION: Text leaf parent is not hypertext" with text node child of <img>
- # [15:25] <@davidb> marcoz: https://wiki.mozilla.org/Firefox/Goals/2013Q2#Firefox_for_Metro
- # [15:25] <@davidb> nothing there
- # [15:31] * Joins: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net)
- # [15:34] * Quits: jongunderson (chatzilla@moz-689FC632.ptr.us.xo.net) (Quit: ChatZilla 0.9.90 [Firefox 19.0.2/20130307023931])
- # [15:42] <@firebot> New Core - Disability Access APIs bug 852138 filed by jruderman@gmail.com.
- # [15:42] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=852138 nor, --, ---, nobody, NEW, "ASSERTION: Adopting child!" in Accessible::BindToParent
- # [15:44] <@davidb> interesting
- # [15:51] <@marcoz> Yeah where do al these new assertions come from?
- # [15:51] <@marcoz> davidb: Did we get rid of try syntax again? I no longer see a reference to it on the tbpl page.
- # [15:52] <@davidb> we still use it
- # [15:53] <@marcoz> Darn why did they take away the link to it then….. *sighs*
- # [15:56] <@davidb> marcoz: http://trychooser.pub.build.mozilla.org/
- # [15:56] <@tbsaunde> marcoz: davidb I believe jesse was ignoring assertions coming from accessible/ because he was getting too many that weren't getting fixed
- # [15:56] <@davidb> yeah
- # [15:56] <@davidb> i find that last bug test case rather simple
- # [15:58] <@tbsaunde> davidb: haven't looked yet, but plan to today
- # [15:58] <@davidb> awesome
- # [15:59] <Jesse> yep. now that https://bugzilla.mozilla.org/show_bug.cgi?id=404077 is catching new assertions in accessibility tests, i can trust that bugs i find aren't a complete waste of time
- # [15:59] <@firebot> Bug 404077 nor, --, mozilla22, dbaron, RESO FIXED, Make assertions cause test failures in Mochitest
- # [15:59] <Jesse> and tbsaunde helped me understand the current failures in https://bugzilla.mozilla.org/show_bug.cgi?id=571613
- # [15:59] <@firebot> Bug 571613 nor, --, ---, nobody, NEW, Pass accessibility tests (mochitest-a11y) without assertion failures
- # [16:00] <@davidb> excellent
- # [16:00] <Jesse> and yeah, there was also some "the accessibility team seems to not be ignoring my bugs any more" :)
- # [16:01] <Jesse> btw, do you ever run the other test suites with accessibility enabled?
- # [16:03] <@tbsaunde> Jesse: not regularly, I think I might have tried once but don't remember the results
- # [16:04] <@tbsaunde> I think its a decent idea though :)
- # [16:05] <Jesse> that would probably find some of the same bugs as fuzzing, as well as bugs i'd miss
- # [16:07] * Quits: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com) (Input/output error)
- # [16:08] <@tbsaunde> Jesse: yeah, shouldn't be hard to atleast try it
- # [16:17] <@firebot> New Core - Disability Access APIs bug 852150 filed by jruderman@gmail.com.
- # [16:17] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=852150 nor, --, ---, nobody, NEW, "ASSERTION: No accessible parent?!" with table
- # [16:20] * Quits: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP) (Quit: Leaving.)
- # [16:26] <@davidb> tbsaunde, hub note http://mxr.mozilla.org/mozilla-central/source/accessible/tests/mochitest/common.js#58
- # [16:26] <@davidb> if we get something better we can get rid of those
- # [16:27] * Quits: clown_mtg (clown@67828CC7.C1A51174.9D42CF23.IP) (Quit: Leaving.)
- # [16:36] * Joins: nhirata (anonymous@moz-1A50F7F8.hsd1.ca.comcast.net)
- # [16:37] * Joins: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
- # [16:43] * khuey|away is now known as khuey
- # [16:51] * Joins: Justin_o (Justin_o@67828CC7.C1A51174.9D42CF23.IP)
- # [16:56] * Joins: victorporof (victorporo@DC82DE00.39508509.6A4F8DA2.IP)
- # [17:04] <@marcoz> davidb: I pushed the patch of bug 811307 to try once again to make sure we're dealing with a real problem.
- # [17:04] * Quits: Justin_o (Justin_o@67828CC7.C1A51174.9D42CF23.IP) (Quit: Justin_o)
- # [17:04] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=811307 nor, --, ---, dbolter, NEW, [AccessFu] Add mochitest for enabling
- # [17:08] * Quits: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net) (Connection reset by peer)
- # [17:08] * Joins: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net)
- # [17:12] * Joins: Justin_o (Justin_o@67828CC7.C1A51174.9D42CF23.IP)
- # [17:13] <@davidb> ty
- # [17:42] <@marcoz> OK the tests will fail on this try-run, too, then. Thinking of cancelling it.
- # [17:42] <@marcoz> davidb: eeejay: Agreed to cancel the try run I started?
- # [17:42] <@eeejay> marcoz, yeah. go ahead
- # [17:42] * @davidb ndos
- # [17:43] <@eeejay> no idea about the leak. it might have to do with the other failing tests
- # [17:43] * @davidb nods even
- # [17:45] <@marcoz> davidb: eeejay: "Cancellation request for revision 2a90f51410a2 failed. ([object XMLHttpRequestProgressEvent])
- # [17:45] <@marcoz> You need to enable JavaScript in order to use Tinderboxpushlog.
- # [17:45] <@marcoz> "
- # [17:45] <@eeejay> pff
- # [17:45] <@marcoz> eeejay: However, it *is* enabled.
- # [17:45] <@davidb> i would hope so
- # [17:46] <@marcoz> eeejay: davidb: Perhaps you can stop it without a stupid error? https://tbpl.mozilla.org/?tree=Try&rev=2a90f51410a2
- # [17:47] <@davidb> i think i did
- # [17:47] <@marcoz> davidb: Thanks!
- # [17:47] <@eeejay> me too
- # [17:47] <@davidb> np
- # [17:49] <@marcoz> davidb: eeejay: BTW: Hell is freezing over! I am going to start an experiment: Migrating to Android away from an iPhone for my mobile device. A NEXUS 4 is ordered.
- # [17:49] <@davidb> fuck yeah
- # [17:49] * @eeejay checks outside for flying pigs
- # [17:50] <@marcoz> LOL!
- # [17:50] <@eeejay> marcoz, we have you as a full-time dogfooder that is good
- # [17:51] * @eeejay is migrating to "the old reader"
- # [17:51] <@eeejay> i imported my opml file and i am number 49247 in the waiting list
- # [17:51] <@marcoz> eeejay: Yes, Firefox is one main reason I want to migrate. I also find that Android 4.2.2 fixed a few gesture bugs that were bothering me previously. So I decided to collect some more info, and last Thursday ordered the NEXUS 4. Hope it'll arrive any day now.
- # [17:52] <@eeejay> marcoz, did you get a chance to hold a nexus 4?
- # [17:52] <@marcoz> eeejay: Nope. But I read up on it, and from what I read, it sounded like a device I would enjoy.
- # [17:53] <@eeejay> cool
- # [17:53] <@marcoz> I would have loved to go for an HTC One instead, but I am afraid of the HTC Sense skin.
- # [17:53] <@eeejay> marcoz, after holding your ipad mini, it is hard to take android hardware as seriously
- # [17:54] <@marcoz> eeejay: Well, you know, some of the makers are realizing that. LGee, for one. Their Google device is made of aluminum and glass, not polycarbonite. The HTC One is aluminum unibody, too. The ones that are really crap are the ones Samsung builds.
- # [17:54] <@marcoz> I could not use the Galaxy Nexus as my main phone, for example.
- # [17:55] <@eeejay> and asus apparently, the nexus 7
- # [17:55] * @eeejay might go for an htc one
- # [17:55] <@marcoz> eeejay: Yes that is very unfortunate! They could have made a much better job of that!
- # [17:55] <@eeejay> although i might just go to firefox os from here
- # [17:56] <@eeejay> i can't deal with those screen sizes. i think it is the most absurd industry sometimes
- # [17:57] <@marcoz> eeejay: I don't think Firefox OS is ready for my personal use case yet. For one thing, I'm missing a way to keep my contacts and calendar in sync. And because I no longer trust Google to keep services around that millions of people rely on, I just recently switched to a managed Exchange server on my domain provider's end, which syncs nicely between iOS and Android and OS X and even Windows.
- # [17:57] <@eeejay> for example, in apple's lineup. spec follows features. meaning when they refresh the hardware you see new user features
- # [17:58] <@marcoz> Firefox OS needs to learn to speak anything other than Facebook before it can really be called mature in that particular sense.
- # [17:58] <@eeejay> marcoz, cool
- # [17:59] <@eeejay> otoh, in the android world. spec is king, and it has nothing to do with features. they give you beast cpus that have no real world gain besides making a smoother ui (which means the ui is flawed to begin with because it is not smooth on average specs)
- # [17:59] <@marcoz> eeejay: Well, they did make the screen size bigger with the iPhone 5 and newest generation of the iPod touch. And according to my partner, this one centimeter longer screen makes a huge difference visually. Whenever she has to use her iPhone 4S, she cringes because the iPod Touch we have is so much nicer to her eyes.
- # [18:00] * Joins: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net)
- # [18:00] <@marcoz> BTW she's getting an HTC One. She's sighted, so doesn't need it to work with Talkback.
- # [18:00] <@eeejay> marcoz, yeah, but they had the wisdom of making it longer, and not just bigger. the longer form factor still works well with one hand operation
- # [18:01] * Joins: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com)
- # [18:01] * Joins: jongunderson (chatzilla@15338043.BA1AA609.8AEF6256.IP)
- # [18:01] <@eeejay> i can hardly use my gs2 with one hand because my thumb won't reach the far corner
- # [18:02] <@marcoz> eeejay: Yes, Apple seem to know what's best for many average users. However, the recent tone their marketing chief Phil Schiller has been taking up with Android was much more noise than actually convincing people of his own product. When a company has to turn to such measures, it tastes not so good, IMO. Previously, Apple was playing on its strengths, but for whatever reason, they changed tactics. That tone doesn't sit with me very w
- # [18:04] <@marcoz> Dunno if a company starts bashing the competition instead of playing at its own strengths to distinguish itself, it feels like executives run out of strategy.
- # [18:05] * Quits: logbot (logbot@moz-58CB32ED.glob.com.au) (Ping timeout)
- # [18:08] <@marcoz> OK, I'm off for the evening. See you tomorrow!
- # [18:09] * Quits: @marcoz (marco.zehe@moz-B9E01AB9.dip.t-dialin.net) (Quit: Leaving.)
- # [18:09] * Quits: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com) (Client exited)
- # [18:11] * Joins: logbot (logbot@moz-58CB32ED.glob.com.au)
- # [18:15] <@davidb> hmmm vmware fusion update
- # [18:20] <@firebot> hub@mozilla.com requested review from dbolter@mozilla.com for attachment 726221 on bug 845095.
- # [18:20] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=845095 nor, --, ---, nobody, NEW, Intermittent Mac states/test_tree.xul | Test timed out. followed by states/test_visibility.html fail
- # [18:22] <@firebot> dbolter@mozilla.com granted review for attachment 726221 on bug 845095.
- # [18:26] <@hub> davidb: I'll wait for the try build to finish
- # [18:26] <@davidb> i trust that :)
- # [18:26] <@davidb> i mean i trust you to do that
- # [18:28] * Joins: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP)
- # [18:39] * Quits: scott_gonzalez (scott_gonz@moz-9A6B491C.nycmny.fios.verizon.net) (Ping timeout)
- # [18:40] * Quits: jongunderson (chatzilla@15338043.BA1AA609.8AEF6256.IP) (Quit: ChatZilla 0.9.90 [Firefox 19.0.2/20130307023931])
- # [18:40] * Joins: scott_gonzalez (scott_gonz@moz-9A6B491C.nycmny.fios.verizon.net)
- # [18:49] * Joins: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com)
- # [19:17] * @davidb builds ff os
- # [19:27] * Quits: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net) (Quit: habber)
- # [19:32] * Quits: icaaq (Adium@moz-D8A76420.cust.bredbandsbolaget.se) (Quit: Leaving.)
- # [19:37] * Joins: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net)
- # [19:40] * Quits: Justin_o (Justin_o@67828CC7.C1A51174.9D42CF23.IP) (Quit: Justin_o)
- # [19:42] * Quits: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com) (Ping timeout)
- # [19:57] * Quits: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net) (Quit: habber)
- # [20:13] * Joins: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com)
- # [20:13] * Joins: icaaq (Adium@moz-D8A76420.cust.bredbandsbolaget.se)
- # [20:16] * Quits: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net) (Quit: lizzard)
- # [20:22] <@tbsaunde> man, I sometimes hate how we treat <body> specially
- # [20:24] <@davidb> tbsaunde: what's happening?
- # [20:24] <@davidb> document life cycle stuff?
- # [20:24] <@davidb> or ownership
- # [20:26] <@tbsaunde> davidb: looking at the last bug jesse filed, its a tree update bug
- # [20:26] <@davidb> ok
- # [20:27] <@tbsaunde> basically what happens is this first they have <body><div style=display:none></div><table></table><tr></tr></body> append the table as a child of the div
- # [20:27] * Joins: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net)
- # [20:28] <@tbsaunde> layout calls DocAccessible::ContentRemoved() with the child as the body for some reason and then the fallback of usingthe treewalker fails because the table has already been removed from the children of the body
- # [20:29] <@tbsaunde> I guess its notthat related, but still
- # [20:49] * Quits: @davidb (davidb@F2D29657.F60B0462.67AC9B1.IP) (Ping timeout)
- # [20:50] * Joins: davidb (davidb@F2D29657.F60B0462.67AC9B1.IP)
- # [20:50] * Quits: Jesse (jruderman@moz-537BCF9.hsd1.ca.comcast.net) (Quit: Jesse)
- # [20:50] * ChanServ sets mode: +qo davidb davidb
- # [21:11] <@davidb> dunno
- # [21:13] * Joins: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net)
- # [21:13] <@firebot> emorley@mozilla.com changed the Resolution on bug 833256 from --- to FIXED.
- # [21:13] <@firebot> emorley@mozilla.com changed the Status on bug 833256 from NEW to RESOLVED.
- # [21:13] <@firebot> emorley@mozilla.com changed the Target Milestone on bug 833256 from --- to mozilla22.
- # [21:13] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=833256 nor, --, mozilla22, joshyyuan, RESO FIXED, role note shouldn't pick up the name from subtree
- # [21:19] * Quits: @davidb (davidb@F2D29657.F60B0462.67AC9B1.IP) (Quit: davidb)
- # [21:22] * Quits: victorporof (victorporo@DC82DE00.39508509.6A4F8DA2.IP) (Quit: victorporof)
- # [21:25] * Joins: victorporof (victorporo@DC82DE00.39508509.6A4F8DA2.IP)
- # [21:25] * Joins: Jesse (jruderman@2557E599.66715431.D25A875A.IP)
- # [21:36] <yzen> eeejay: hi, I was wondering if you had a chance to take a look at the patch for bug 753984 . davidb mentioned I should add new unit tests for that, perhaps you have some suggestions there as well ?
- # [21:37] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=753984 nor, --, ---, yura.zenevich, NEW, [AccessFu] Make utterance order configurable
- # [21:37] <@hub> btw, how did the bug day work out?
- # [21:40] * Quits: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net) (Quit: lizzard)
- # [21:41] <@eeejay> yzen, i'll get back to you on that one soon. sorry
- # [21:41] <yzen> eeejay: no worries, whenever you have time
- # [21:49] <@firebot> eitan@monotonous.org granted feedback for attachment 725996 on bug 753984.
- # [21:49] <@firebot> eitan@monotonous.org cancelled needinfo?(eitan@monotonous. org) on bug 753984.
- # [21:49] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=753984 nor, --, ---, yura.zenevich, NEW, [AccessFu] Make utterance order configurable
- # [21:54] * Joins: lizzard (ehenry@moz-605588F2.tmodns.net)
- # [21:56] * Parts: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
- # [21:57] * Quits: victorporof (victorporo@DC82DE00.39508509.6A4F8DA2.IP) (Quit: victorporof)
- # [22:10] * Quits: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP) (Quit: Leaving.)
- # [22:31] * Quits: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com) (Ping timeout)
- # [22:33] * Joins: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com)
- # [22:38] * Quits: icaaq (Adium@moz-D8A76420.cust.bredbandsbolaget.se) (Quit: Leaving.)
- # [22:43] * Quits: lizzard (ehenry@moz-605588F2.tmodns.net) (Ping timeout)
- # [22:44] * Joins: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP)
- # [22:54] * Quits: @hub (hub@F2D29657.F60B0462.67AC9B1.IP) (Ping timeout)
- # [22:55] * Quits: Jesse (jruderman@2557E599.66715431.D25A875A.IP) (Connection reset by peer)
- # [22:56] * Joins: victorporof (victorporo@DC82DE00.39508509.6A4F8DA2.IP)
- # [22:56] * Joins: Jesse_ (jruderman@2557E599.66715431.D25A875A.IP)
- # [22:57] * Quits: Jesse_ (jruderman@2557E599.66715431.D25A875A.IP) (Ping timeout)
- # [23:01] * Joins: Jesse (jruderman@2557E599.66715431.D25A875A.IP)
- # [23:13] * Quits: victorporof (victorporo@DC82DE00.39508509.6A4F8DA2.IP) (Quit: victorporof)
- # [23:28] * Quits: scott_gonzalez (scott_gonz@moz-9A6B491C.nycmny.fios.verizon.net) (Quit: scott_gonzalez)
- # [23:49] * Joins: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net)
- # Session Close: Tue Mar 19 00:00:01 2013
The end :)