/irc-logs / mozilla / #accessibility / 2013-03-18 / end

Options:

  1. # Session Start: Mon Mar 18 00:00:00 2013
  2. # Session Ident: #accessibility
  3. # [02:23] <@firebot> surkov.alexander@gmail.com cancelled review?(surkov.alexander@gm ail.com) for attachment 720484 on bug 847233.
  4. # [02:23] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=847233 nor, --, ---, mounir, ASSI, Fix a11y with <input type='file'> changes
  5. # [02:43] * Joins: Jamie (Instantbir@moz-CA26021.jantrid.net)
  6. # [02:52] <@firebot> surkov.alexander@gmail.com cancelled review?(surkov.alexander@gm ail.com) for attachment 724788 on bug 850981.
  7. # [02:52] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=850981 nor, --, ---, trev.saunders, NEW, make AppendChild() inline around InsertChildAt()
  8. # [02:57] * Quits: Hixie (ianh@C6D27F7B.5EFFBB68.81BC061B.IP) (Ping timeout)
  9. # [02:57] * Joins: Hixie (ianh@C6D27F7B.5EFFBB68.81BC061B.IP)
  10. # [03:02] <@firebot> trev.saunders@gmail.com granted review for attachment 725255 on bug 833256.
  11. # [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
  12. # [03:02] * Joins: surkov (surkov@511E724B.E00BF004.E17943EE.IP)
  13. # [03:02] * ChanServ sets mode: +o surkov
  14. # [03:20] * Joins: scott_gonzalez (scott_gonz@moz-9A6B491C.nycmny.fios.verizon.net)
  15. # [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?
  16. # [03:34] <@surkov> just invalidate
  17. # [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
  18. # [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
  19. # [03:42] <@surkov> tbsaunde: a11y sync events?
  20. # [03:42] <tbsaunde> surkov: yes
  21. # [03:45] <tbsaunde> surkov: DocAccessible::ProcessLoad() calls nsEventShell::FireEvent() and so does RootAccessible::PRocessDOMEvent()
  22. # [03:45] <@surkov> tbsaunde: should we change that mState before generic notification processing?
  23. # [03:46] <tbsaunde> surkov: maybe, but are we sure that running js there is ok?
  24. # [03:46] <tbsaunde> especially before we call DocAccessible::ProcessInvalidationList()
  25. # [03:47] <@surkov> basically we need to make sure that we don't run script when we build the tree, right?
  26. # [03:47] <tbsaunde> surkov: atleast that
  27. # [03:47] <@surkov> if notification processing can't affect on the tree then it's seems ok
  28. # [03:48] <tbsaunde> surkov: so should we maybe handle the invalidation list before ProcessLoad / notifications?
  29. # [03:48] <@surkov> seems reasoanble
  30. # [03:50] <tbsaunde> surkov: so, why to notifications need to fire events sync?
  31. # [03:51] <@surkov> I'd say that should stay in line to keep events in sync
  32. # [03:52] <tbsaunde> surkov: what about cases where they should be coallesced with async evnet?
  33. # [03:53] <@surkov> you meant shouldn't?
  34. # [03:53] <tbsaunde> surkov: no
  35. # [03:54] <@surkov> tbsaunde: if they will be coalesced then it's fine
  36. # [03:54] <tbsaunde> surkov: afaik now they are not
  37. # [03:55] <@surkov> tbsaunde: correct, and I said it should be ok if they will
  38. # [03:55] <tbsaunde> surkov: since nsEventShell::FireVent() never touches the event queue
  39. # [03:58] <tbsaunde> surkov: then I don't get what your saying
  40. # [03:59] * Quits: Jamie (Instantbir@moz-CA26021.jantrid.net) (Client exited)
  41. # [03:59] <@surkov> tbsaunde: I meant putting them into queue seems reasonable
  42. # [04:00] * Joins: Jamie (Instantbir@moz-CA26021.jantrid.net)
  43. # [04:00] <tbsaunde> surkov: ah, so your saying using IFreDelayedAccEvent() is fine then?
  44. # [04:00] <@surkov> tbsaunde: yes
  45. # [04:01] * Quits: Jamie (Instantbir@moz-CA26021.jantrid.net) (Quit: Instantbird 1.4a1pre -- http://www.instantbird.com)
  46. # [04:01] <@surkov> tbsaunde: but if no coalescence will happen then probably it doesn't make huge sense
  47. # [04:02] <tbsaunde> surkov: yeah, but doesn't really hurt much either
  48. # [04:02] <@surkov> seems so
  49. # [04:07] * Joins: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP)
  50. # [04:21] * Quits: scott_gonzalez (scott_gonz@moz-9A6B491C.nycmny.fios.verizon.net) (Quit: scott_gonzalez)
  51. # [04:24] <yzen> hi everyone, would anybody know where I can find accessfu unit tests inside the accessible tests?
  52. # [04:28] * Joins: scott_gonzalez (scott_gonz@moz-9A6B491C.nycmny.fios.verizon.net)
  53. # [04:36] * Joins: Jamie (Instantbir@moz-CA26021.jantrid.net)
  54. # [04:45] * Quits: scott_gonzalez (scott_gonz@moz-9A6B491C.nycmny.fios.verizon.net) (Quit: scott_gonzalez)
  55. # [04:48] <yzen> never mind i think i found this bug 811307
  56. # [04:48] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=811307 nor, --, ---, dbolter, NEW, [AccessFu] Add mochitest for enabling
  57. # [05:16] <@firebot> yura.zenevich@gmail.com requested feedback from eitan@monotonous. org for attachment 725996 on bug 753984.
  58. # [05:16] <@firebot> yura.zenevich@gmail.com requested needinfo from eitan@monotonous. org on bug 753984.
  59. # [05:16] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=753984 nor, --, ---, yura.zenevich, NEW, [AccessFu] Make utterance order configurable
  60. # [05:27] <@firebot> New Core - Disability Access APIs bug 852021 filed by surkov.alexander@gmail.com.
  61. # [05:28] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=852021 nor, --, ---, nobody, NEW, add getText* at caret offset mochitest
  62. # [05:37] <@firebot> surkov.alexander@gmail.com changed the Assignee on bug 852021 from nobody@mozilla.org to surkov.alexander@gmail.com.
  63. # [05:48] <@firebot> trev.saunders@gmail.com requested review from surkov.alexander@gm ail.com for attachment 725999 on bug 850981.
  64. # [05:48] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=850981 nor, --, ---, trev.saunders, NEW, make AppendChild() inline around InsertChildAt()
  65. # [05:55] * Quits: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP) (Quit: Leaving.)
  66. # [06:17] * Quits: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com) (Ping timeout)
  67. # [06:53] <@firebot> New Core - Disability Access APIs bug 852044 filed by trev.saunders@gmail.com.
  68. # [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
  69. # [06:57] <@firebot> trev.saunders@gmail.com requested review from surkov.alexander@gm ail.com for attachment 726005 on bug 852044.
  70. # [06:58] <tbsaunde> surkov: do you remember why event coalescing happens on queue insertion instead of jsut before firing?
  71. # [06:58] <@surkov> tbsaunde: some coalescence uses an accessible tree
  72. # [06:59] <tbsaunde> surkov: oh, yeah ok
  73. # [06:59] <tbsaunde> surkov: I wonder if its worth to only coalesce the stuff we have to then and do other stuff later
  74. # [07:00] <@surkov> this sounds complicated
  75. # [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
  76. # [07:02] <@surkov> btw, what's for that?
  77. # [07:03] <tbsaunde> surkov: don't think I get the question
  78. # [07:03] <@surkov> tbsaunde: why do you want to delay coalescence?
  79. # [07:04] <tbsaunde> surkov: I think its faster (atleast algorithmically)
  80. # [07:05] <@surkov> ok, it's worth to investigate
  81. # [07:07] * Joins: icaaq (Adium@moz-D8A76420.cust.bredbandsbolaget.se)
  82. # [07:25] * Quits: nhirata (anonymous@moz-1A50F7F8.hsd1.ca.comcast.net) (Quit: nhirata)
  83. # [08:01] <@firebot> surkov.alexander@gmail.com granted review for attachment 725999 on bug 850981.
  84. # [08:01] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=850981 nor, --, ---, trev.saunders, NEW, make AppendChild() inline around InsertChildAt()
  85. # [08:17] * Quits: Jesse (jruderman@moz-537BCF9.hsd1.ca.comcast.net) (Quit: Jesse)
  86. # [08:20] * Quits: Jamie (Instantbir@moz-CA26021.jantrid.net) (Quit: Instantbird 1.4a1pre -- http://www.instantbird.com)
  87. # [08:33] * Joins: Jesse (jruderman@moz-537BCF9.hsd1.ca.comcast.net)
  88. # [09:15] * Joins: marcoz (marco.zehe@moz-B9E01AB9.dip.t-dialin.net)
  89. # [09:15] * ChanServ sets mode: +o marcoz
  90. # [09:30] * Quits: icaaq (Adium@moz-D8A76420.cust.bredbandsbolaget.se) (Quit: Leaving.)
  91. # [09:43] * Joins: icaaq (Adium@moz-D8A76420.cust.bredbandsbolaget.se)
  92. # [10:30] * surkov sets mode: +o tbsaunde
  93. # [10:30] <@surkov> tbsaunde: ping
  94. # [10:41] * Quits: vd (vd@E355CC6C.D7B7DA9B.43EBE9B3.IP) (Quit: Reconnecting)
  95. # [10:41] * Joins: vd (vd@E355CC6C.D7B7DA9B.43EBE9B3.IP)
  96. # [10:42] * Quits: vd (vd@E355CC6C.D7B7DA9B.43EBE9B3.IP) (Quit: leaving)
  97. # [11:40] * Quits: @marcoz (marco.zehe@moz-B9E01AB9.dip.t-dialin.net) (Quit: Leaving.)
  98. # [11:40] * Joins: marcoz (marco.zehe@moz-B9E01AB9.dip.t-dialin.net)
  99. # [11:41] * ChanServ sets mode: +o marcoz
  100. # [12:12] <@tbsaunde> surkov: hi
  101. # [12:13] <@surkov> tbsaunde: good morning, what do you think if we keep all managers as members of accessibilityService
  102. # [12:14] <@tbsaunde> surkov: instead of inheriting from them?
  103. # [12:14] <@surkov> tbsaunde: yes
  104. # [12:14] <@surkov> tbsaunde: we will still have globals like SelectionMgr() and etc
  105. # [12:14] <@tbsaunde> surkov: I gues that's fine, but why not just have seperate globals?
  106. # [12:15] <@tbsaunde> surkov: what problem are you trying to solve?
  107. # [12:15] <@surkov> AccService will manage them anyway
  108. # [12:15] <@surkov> tbsaunde: it'd be a nice way to add cc to them
  109. # [12:16] <@tbsaunde> surkov: is that a good idea? they can't possibly die till xpcom-shutdown, and they will certainly die then
  110. # [12:17] <@tbsaunde> if we think they don't we could just assert that
  111. # [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)
  112. # [12:19] <@tbsaunde> surkov: so why don't you assert it? or we could just not really ref count it at all
  113. # [12:19] <@surkov> tbsaunde: they are used as listeners
  114. # [12:19] <@surkov> so they should be refcounted
  115. # [12:19] <@surkov> (except FocusManager)
  116. # [12:20] <@tbsaunde> surkov: nobody should hold refs or be notifying of anything we care about nearly as late as xpcom-shutdown afaik
  117. # [12:21] <@surkov> true, sometimes that happens
  118. # [12:21] <@tbsaunde> what happens?
  119. # [12:22] <@surkov> remember our popular leak, before I removed DocManager from page hide listeners we leaked DocManager
  120. # [12:22] <@surkov> because event target kept it alive
  121. # [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
  122. # [12:24] <@surkov> tbsaunde: wouldn't it possible to leak, if SelectionManager keeps DOM element and DOM element keeps AccessibilityService
  123. # [12:25] <@surkov> good point
  124. # [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
  125. # [12:27] <@tbsaunde> seperating managers might help with header inclussion / be a good idea, I just don't think it solves your problem :)
  126. # [12:28] <@surkov> ok
  127. # [12:42] <@tbsaunde> marcoz: I'd bet you broke ke testcaret/test_general.html
  128. # [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. :)
  129. # [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
  130. # [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.
  131. # [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 @
  132. # [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.
  133. # [12:57] * Joins: scott_gonzalez (scott_gonz@moz-9A6B491C.nycmny.fios.verizon.net)
  134. # [12:57] <@tbsaunde> marcoz: the chances that's random are \epsilon
  135. # [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
  136. # [13:12] * Joins: Justin_o (Justin_o@67828CC7.C1A51174.9D42CF23.IP)
  137. # [13:13] * Quits: @surkov (surkov@511E724B.E00BF004.E17943EE.IP) (Quit: surkov)
  138. # [13:33] * Joins: hub (hub@F2D29657.F60B0462.67AC9B1.IP)
  139. # [13:33] * ChanServ sets mode: +o hub
  140. # [13:43] * Quits: @marcoz (marco.zehe@moz-B9E01AB9.dip.t-dialin.net) (Quit: Leaving.)
  141. # [13:51] * Quits: brambles (xymox@moz-969AAE9B.barwen.ch) (Ping timeout)
  142. # [13:52] * Joins: brambles (xymox@moz-969AAE9B.barwen.ch)
  143. # [13:53] * Joins: marcoz (marco.zehe@moz-B9E01AB9.dip.t-dialin.net)
  144. # [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.
  145. # [13:55] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=811307 nor, --, ---, dbolter, NEW, [AccessFu] Add mochitest for enabling
  146. # [13:55] * ChanServ sets mode: +o marcoz
  147. # [14:00] <@tbsaunde> marcoz: I think the nice orange tree does that just fine, but whatever
  148. # [14:04] * Joins: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP)
  149. # [14:07] * Joins: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
  150. # [14:08] * clown is now known as clown_mtg
  151. # [14:10] * Joins: davidb (davidb@F2D29657.F60B0462.67AC9B1.IP)
  152. # [14:10] * ChanServ sets mode: +qo davidb davidb
  153. # [14:11] <@davidb> heyo
  154. # [14:12] * Quits: Justin_o (Justin_o@67828CC7.C1A51174.9D42CF23.IP) (Quit: Justin_o)
  155. # [14:12] <@tbsaunde> morning
  156. # [14:14] <@hub> heya
  157. # [14:25] <@davidb> marcoz: that text caret failures are a bummer
  158. # [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.
  159. # [14:28] <@marcoz> But yeah, the fact that all of these runs had the same tests fail is remarkable.
  160. # [14:29] * Joins: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com)
  161. # [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.
  162. # [14:39] <@davidb> marcoz: sounds like you are volunteering :)
  163. # [14:40] <@marcoz> davidb: Um, not explicitly, no, I was just suggesting a course of action. ;)
  164. # [14:40] <@davidb> heheh
  165. # [14:57] <@marcoz> davidb: 1:1?
  166. # [14:57] <@firebot> New Core - Disability Access APIs bug 852125 filed by jruderman@gmail.com.
  167. # [14:57] <@davidb> marcoz: yes! 1 min
  168. # [14:57] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=852125 nor, --, ---, nobody, NEW, "ASSERTION: Unbinding the unbound accessible!" with <iframe>
  169. # [14:59] <@davidb> marcoz: ready when you are
  170. # [15:02] * Joins: jongunderson (chatzilla@moz-689FC632.ptr.us.xo.net)
  171. # [15:13] <@firebot> New Core - Disability Access APIs bug 852129 filed by jruderman@gmail.com.
  172. # [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>
  173. # [15:25] <@davidb> marcoz: https://wiki.mozilla.org/Firefox/Goals/2013Q2#Firefox_for_Metro
  174. # [15:25] <@davidb> nothing there
  175. # [15:31] * Joins: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net)
  176. # [15:34] * Quits: jongunderson (chatzilla@moz-689FC632.ptr.us.xo.net) (Quit: ChatZilla 0.9.90 [Firefox 19.0.2/20130307023931])
  177. # [15:42] <@firebot> New Core - Disability Access APIs bug 852138 filed by jruderman@gmail.com.
  178. # [15:42] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=852138 nor, --, ---, nobody, NEW, "ASSERTION: Adopting child!" in Accessible::BindToParent
  179. # [15:44] <@davidb> interesting
  180. # [15:51] <@marcoz> Yeah where do al these new assertions come from?
  181. # [15:51] <@marcoz> davidb: Did we get rid of try syntax again? I no longer see a reference to it on the tbpl page.
  182. # [15:52] <@davidb> we still use it
  183. # [15:53] <@marcoz> Darn why did they take away the link to it then….. *sighs*
  184. # [15:56] <@davidb> marcoz: http://trychooser.pub.build.mozilla.org/
  185. # [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
  186. # [15:56] <@davidb> yeah
  187. # [15:56] <@davidb> i find that last bug test case rather simple
  188. # [15:58] <@tbsaunde> davidb: haven't looked yet, but plan to today
  189. # [15:58] <@davidb> awesome
  190. # [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
  191. # [15:59] <@firebot> Bug 404077 nor, --, mozilla22, dbaron, RESO FIXED, Make assertions cause test failures in Mochitest
  192. # [15:59] <Jesse> and tbsaunde helped me understand the current failures in https://bugzilla.mozilla.org/show_bug.cgi?id=571613
  193. # [15:59] <@firebot> Bug 571613 nor, --, ---, nobody, NEW, Pass accessibility tests (mochitest-a11y) without assertion failures
  194. # [16:00] <@davidb> excellent
  195. # [16:00] <Jesse> and yeah, there was also some "the accessibility team seems to not be ignoring my bugs any more" :)
  196. # [16:01] <Jesse> btw, do you ever run the other test suites with accessibility enabled?
  197. # [16:03] <@tbsaunde> Jesse: not regularly, I think I might have tried once but don't remember the results
  198. # [16:04] <@tbsaunde> I think its a decent idea though :)
  199. # [16:05] <Jesse> that would probably find some of the same bugs as fuzzing, as well as bugs i'd miss
  200. # [16:07] * Quits: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com) (Input/output error)
  201. # [16:08] <@tbsaunde> Jesse: yeah, shouldn't be hard to atleast try it
  202. # [16:17] <@firebot> New Core - Disability Access APIs bug 852150 filed by jruderman@gmail.com.
  203. # [16:17] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=852150 nor, --, ---, nobody, NEW, "ASSERTION: No accessible parent?!" with table
  204. # [16:20] * Quits: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP) (Quit: Leaving.)
  205. # [16:26] <@davidb> tbsaunde, hub note http://mxr.mozilla.org/mozilla-central/source/accessible/tests/mochitest/common.js#58
  206. # [16:26] <@davidb> if we get something better we can get rid of those
  207. # [16:27] * Quits: clown_mtg (clown@67828CC7.C1A51174.9D42CF23.IP) (Quit: Leaving.)
  208. # [16:36] * Joins: nhirata (anonymous@moz-1A50F7F8.hsd1.ca.comcast.net)
  209. # [16:37] * Joins: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
  210. # [16:43] * khuey|away is now known as khuey
  211. # [16:51] * Joins: Justin_o (Justin_o@67828CC7.C1A51174.9D42CF23.IP)
  212. # [16:56] * Joins: victorporof (victorporo@DC82DE00.39508509.6A4F8DA2.IP)
  213. # [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.
  214. # [17:04] * Quits: Justin_o (Justin_o@67828CC7.C1A51174.9D42CF23.IP) (Quit: Justin_o)
  215. # [17:04] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=811307 nor, --, ---, dbolter, NEW, [AccessFu] Add mochitest for enabling
  216. # [17:08] * Quits: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net) (Connection reset by peer)
  217. # [17:08] * Joins: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net)
  218. # [17:12] * Joins: Justin_o (Justin_o@67828CC7.C1A51174.9D42CF23.IP)
  219. # [17:13] <@davidb> ty
  220. # [17:42] <@marcoz> OK the tests will fail on this try-run, too, then. Thinking of cancelling it.
  221. # [17:42] <@marcoz> davidb: eeejay: Agreed to cancel the try run I started?
  222. # [17:42] <@eeejay> marcoz, yeah. go ahead
  223. # [17:42] * @davidb ndos
  224. # [17:43] <@eeejay> no idea about the leak. it might have to do with the other failing tests
  225. # [17:43] * @davidb nods even
  226. # [17:45] <@marcoz> davidb: eeejay: "Cancellation request for revision 2a90f51410a2 failed. ([object XMLHttpRequestProgressEvent])
  227. # [17:45] <@marcoz> You need to enable JavaScript in order to use Tinderboxpushlog.
  228. # [17:45] <@marcoz> "
  229. # [17:45] <@eeejay> pff
  230. # [17:45] <@marcoz> eeejay: However, it *is* enabled.
  231. # [17:45] <@davidb> i would hope so
  232. # [17:46] <@marcoz> eeejay: davidb: Perhaps you can stop it without a stupid error? https://tbpl.mozilla.org/?tree=Try&rev=2a90f51410a2
  233. # [17:47] <@davidb> i think i did
  234. # [17:47] <@marcoz> davidb: Thanks!
  235. # [17:47] <@eeejay> me too
  236. # [17:47] <@davidb> np
  237. # [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.
  238. # [17:49] <@davidb> fuck yeah
  239. # [17:49] * @eeejay checks outside for flying pigs
  240. # [17:50] <@marcoz> LOL!
  241. # [17:50] <@eeejay> marcoz, we have you as a full-time dogfooder that is good
  242. # [17:51] * @eeejay is migrating to "the old reader"
  243. # [17:51] <@eeejay> i imported my opml file and i am number 49247 in the waiting list
  244. # [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.
  245. # [17:52] <@eeejay> marcoz, did you get a chance to hold a nexus 4?
  246. # [17:52] <@marcoz> eeejay: Nope. But I read up on it, and from what I read, it sounded like a device I would enjoy.
  247. # [17:53] <@eeejay> cool
  248. # [17:53] <@marcoz> I would have loved to go for an HTC One instead, but I am afraid of the HTC Sense skin.
  249. # [17:53] <@eeejay> marcoz, after holding your ipad mini, it is hard to take android hardware as seriously
  250. # [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.
  251. # [17:54] <@marcoz> I could not use the Galaxy Nexus as my main phone, for example.
  252. # [17:55] <@eeejay> and asus apparently, the nexus 7
  253. # [17:55] * @eeejay might go for an htc one
  254. # [17:55] <@marcoz> eeejay: Yes that is very unfortunate! They could have made a much better job of that!
  255. # [17:55] <@eeejay> although i might just go to firefox os from here
  256. # [17:56] <@eeejay> i can't deal with those screen sizes. i think it is the most absurd industry sometimes
  257. # [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.
  258. # [17:57] <@eeejay> for example, in apple's lineup. spec follows features. meaning when they refresh the hardware you see new user features
  259. # [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.
  260. # [17:58] <@eeejay> marcoz, cool
  261. # [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)
  262. # [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.
  263. # [18:00] * Joins: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net)
  264. # [18:00] <@marcoz> BTW she's getting an HTC One. She's sighted, so doesn't need it to work with Talkback.
  265. # [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
  266. # [18:01] * Joins: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com)
  267. # [18:01] * Joins: jongunderson (chatzilla@15338043.BA1AA609.8AEF6256.IP)
  268. # [18:01] <@eeejay> i can hardly use my gs2 with one hand because my thumb won't reach the far corner
  269. # [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
  270. # [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.
  271. # [18:05] * Quits: logbot (logbot@moz-58CB32ED.glob.com.au) (Ping timeout)
  272. # [18:08] <@marcoz> OK, I'm off for the evening. See you tomorrow!
  273. # [18:09] * Quits: @marcoz (marco.zehe@moz-B9E01AB9.dip.t-dialin.net) (Quit: Leaving.)
  274. # [18:09] * Quits: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com) (Client exited)
  275. # [18:11] * Joins: logbot (logbot@moz-58CB32ED.glob.com.au)
  276. # [18:15] <@davidb> hmmm vmware fusion update
  277. # [18:20] <@firebot> hub@mozilla.com requested review from dbolter@mozilla.com for attachment 726221 on bug 845095.
  278. # [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
  279. # [18:22] <@firebot> dbolter@mozilla.com granted review for attachment 726221 on bug 845095.
  280. # [18:26] <@hub> davidb: I'll wait for the try build to finish
  281. # [18:26] <@davidb> i trust that :)
  282. # [18:26] <@davidb> i mean i trust you to do that
  283. # [18:28] * Joins: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP)
  284. # [18:39] * Quits: scott_gonzalez (scott_gonz@moz-9A6B491C.nycmny.fios.verizon.net) (Ping timeout)
  285. # [18:40] * Quits: jongunderson (chatzilla@15338043.BA1AA609.8AEF6256.IP) (Quit: ChatZilla 0.9.90 [Firefox 19.0.2/20130307023931])
  286. # [18:40] * Joins: scott_gonzalez (scott_gonz@moz-9A6B491C.nycmny.fios.verizon.net)
  287. # [18:49] * Joins: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com)
  288. # [19:17] * @davidb builds ff os
  289. # [19:27] * Quits: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net) (Quit: habber)
  290. # [19:32] * Quits: icaaq (Adium@moz-D8A76420.cust.bredbandsbolaget.se) (Quit: Leaving.)
  291. # [19:37] * Joins: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net)
  292. # [19:40] * Quits: Justin_o (Justin_o@67828CC7.C1A51174.9D42CF23.IP) (Quit: Justin_o)
  293. # [19:42] * Quits: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com) (Ping timeout)
  294. # [19:57] * Quits: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net) (Quit: habber)
  295. # [20:13] * Joins: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com)
  296. # [20:13] * Joins: icaaq (Adium@moz-D8A76420.cust.bredbandsbolaget.se)
  297. # [20:16] * Quits: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net) (Quit: lizzard)
  298. # [20:22] <@tbsaunde> man, I sometimes hate how we treat <body> specially
  299. # [20:24] <@davidb> tbsaunde: what's happening?
  300. # [20:24] <@davidb> document life cycle stuff?
  301. # [20:24] <@davidb> or ownership
  302. # [20:26] <@tbsaunde> davidb: looking at the last bug jesse filed, its a tree update bug
  303. # [20:26] <@davidb> ok
  304. # [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
  305. # [20:27] * Joins: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net)
  306. # [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
  307. # [20:29] <@tbsaunde> I guess its notthat related, but still
  308. # [20:49] * Quits: @davidb (davidb@F2D29657.F60B0462.67AC9B1.IP) (Ping timeout)
  309. # [20:50] * Joins: davidb (davidb@F2D29657.F60B0462.67AC9B1.IP)
  310. # [20:50] * Quits: Jesse (jruderman@moz-537BCF9.hsd1.ca.comcast.net) (Quit: Jesse)
  311. # [20:50] * ChanServ sets mode: +qo davidb davidb
  312. # [21:11] <@davidb> dunno
  313. # [21:13] * Joins: habber (habber@moz-BE33DA21.fw1.sfo1.mozilla.net)
  314. # [21:13] <@firebot> emorley@mozilla.com changed the Resolution on bug 833256 from --- to FIXED.
  315. # [21:13] <@firebot> emorley@mozilla.com changed the Status on bug 833256 from NEW to RESOLVED.
  316. # [21:13] <@firebot> emorley@mozilla.com changed the Target Milestone on bug 833256 from --- to mozilla22.
  317. # [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
  318. # [21:19] * Quits: @davidb (davidb@F2D29657.F60B0462.67AC9B1.IP) (Quit: davidb)
  319. # [21:22] * Quits: victorporof (victorporo@DC82DE00.39508509.6A4F8DA2.IP) (Quit: victorporof)
  320. # [21:25] * Joins: victorporof (victorporo@DC82DE00.39508509.6A4F8DA2.IP)
  321. # [21:25] * Joins: Jesse (jruderman@2557E599.66715431.D25A875A.IP)
  322. # [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 ?
  323. # [21:37] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=753984 nor, --, ---, yura.zenevich, NEW, [AccessFu] Make utterance order configurable
  324. # [21:37] <@hub> btw, how did the bug day work out?
  325. # [21:40] * Quits: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net) (Quit: lizzard)
  326. # [21:41] <@eeejay> yzen, i'll get back to you on that one soon. sorry
  327. # [21:41] <yzen> eeejay: no worries, whenever you have time
  328. # [21:49] <@firebot> eitan@monotonous.org granted feedback for attachment 725996 on bug 753984.
  329. # [21:49] <@firebot> eitan@monotonous.org cancelled needinfo?(eitan@monotonous. org) on bug 753984.
  330. # [21:49] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=753984 nor, --, ---, yura.zenevich, NEW, [AccessFu] Make utterance order configurable
  331. # [21:54] * Joins: lizzard (ehenry@moz-605588F2.tmodns.net)
  332. # [21:56] * Parts: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
  333. # [21:57] * Quits: victorporof (victorporo@DC82DE00.39508509.6A4F8DA2.IP) (Quit: victorporof)
  334. # [22:10] * Quits: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP) (Quit: Leaving.)
  335. # [22:31] * Quits: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com) (Ping timeout)
  336. # [22:33] * Joins: peteb-away (ptbrunet@moz-B51E1692.austin.res.rr.com)
  337. # [22:38] * Quits: icaaq (Adium@moz-D8A76420.cust.bredbandsbolaget.se) (Quit: Leaving.)
  338. # [22:43] * Quits: lizzard (ehenry@moz-605588F2.tmodns.net) (Ping timeout)
  339. # [22:44] * Joins: yzen (Adium@AA2C085E.10C8AEF8.9A8C35B4.IP)
  340. # [22:54] * Quits: @hub (hub@F2D29657.F60B0462.67AC9B1.IP) (Ping timeout)
  341. # [22:55] * Quits: Jesse (jruderman@2557E599.66715431.D25A875A.IP) (Connection reset by peer)
  342. # [22:56] * Joins: victorporof (victorporo@DC82DE00.39508509.6A4F8DA2.IP)
  343. # [22:56] * Joins: Jesse_ (jruderman@2557E599.66715431.D25A875A.IP)
  344. # [22:57] * Quits: Jesse_ (jruderman@2557E599.66715431.D25A875A.IP) (Ping timeout)
  345. # [23:01] * Joins: Jesse (jruderman@2557E599.66715431.D25A875A.IP)
  346. # [23:13] * Quits: victorporof (victorporo@DC82DE00.39508509.6A4F8DA2.IP) (Quit: victorporof)
  347. # [23:28] * Quits: scott_gonzalez (scott_gonz@moz-9A6B491C.nycmny.fios.verizon.net) (Quit: scott_gonzalez)
  348. # [23:49] * Joins: lizzard (ehenry@moz-5A4BE8E8.dsl.static.sonic.net)
  349. # Session Close: Tue Mar 19 00:00:01 2013

The end :)