/irc-logs / mozilla / #accessibility / 2012-04-10 / end

Options:

  1. # Session Start: Tue Apr 10 00:00:00 2012
  2. # Session Ident: #accessibility
  3. # [00:14] <@firebot> markcapella@twcny.rr.com requested feedback from hub@mozilla.com for attachment 613410 on bug 741707.
  4. # [00:14] <@firebot> markcapella@twcny.rr.com changed the Status on bug 741707 from NEW to ASSIGNED.
  5. # [00:14] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741707 nor, --, ---, markcapella, ASSI, rm unused a11y CIDs
  6. # [00:22] * Quits: shorlander (shorlander@moz-853043D6.dhcp.insightbb.com) (Ping timeout)
  7. # [00:22] * Joins: shorlander_ (shorlander@moz-853043D6.dhcp.insightbb.com)
  8. # [00:27] <@firebot> hub@mozilla.com denied review for attachment 613245 on bug 741683.
  9. # [00:27] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741683 nor, --, ---, markcapella, ASSI, move nsAccessNode::ScrollTo to nsCoreUtils
  10. # [00:34] * Quits: victorporof (victorporo@BD5B5B26.602BD6C.79933D60.IP) (Connection reset by peer)
  11. # [00:49] <Mark_Capella> hub: im working on the rebase and will post it in once it clean builds here locally
  12. # [00:52] <@hub> Mark_Capella: to be honest I might have misunderstood what surkov said. Otherwise it would be r+
  13. # [00:52] <Mark_Capella> cool B) yah there was a lot going on in that patch, and you kinda got dragged in late ... ill post the rebase asap and ask for review again
  14. # [00:59] * Joins: jkiss (jkiss@5248D1C6.C5B5659C.8732AABF.IP)
  15. # [01:04] <jkiss> The W3C A11y API task force is looking for help with the HTML to Platform APIs Implentation Guide at http://dvcs.w3.org/hg/html-api-map/raw-file/tip/Overview.html
  16. # [01:04] <jkiss> If you've got some time, please let us know what you think should be addded or changed
  17. # [01:05] <jkiss> You can also the bug adding feature built into the page if you have a W3C bugzilla account
  18. # [01:05] <jkiss> All feedback appreciated
  19. # [01:30] <@hub> maybe I'm missing something, but do we have an event like "page loaded"
  20. # [01:41] * Quits: ehsan (ehsan@F2D29657.F60B0462.67AC9B1.IP) (Input/output error)
  21. # [01:53] * Quits: nhirata (nhirata.bu@moz-C03D0C61.vlan426.asr1.sfo1.gblx.net) (Quit: nhirata)
  22. # [01:58] * Joins: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP)
  23. # [01:58] <Mark_Capella> hub: rebase of bug741683 is building ... any ideas on the bug741701 ?
  24. # [01:58] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741683 nor, --, ---, markcapella, ASSI, move nsAccessNode::ScrollTo to nsCoreUtils
  25. # [01:58] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741701 nor, --, mozilla14, markcapella, ASSI, CAccessibleHyperlink::get_anchor shouldn't use nsIWinAccessNode
  26. # [02:00] <Mark_Capella> er --- im mean bug741707 ??
  27. # [02:00] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741707 nor, --, ---, markcapella, ASSI, rm unused a11y CIDs
  28. # [02:00] <Mark_Capella> (typed it wrong)
  29. # [02:06] <@hub> I didn't really look yet
  30. # [02:07] <@hub> as for the Windows build error, I have no idea
  31. # [02:07] <@hub> let me check on Linux
  32. # [02:07] <Mark_Capella> k :)
  33. # [02:07] <Mark_Capella> not familiar with these concepts yet so debugging is going slow ...
  34. # [02:09] * Quits: jimm (jmathies@moz-7F164CA1.pn.at.cox.net) (Quit: )
  35. # [02:09] * Quits: @jprmc (jprmc@F2D29657.F60B0462.67AC9B1.IP) (Ping timeout)
  36. # [02:27] * Joins: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp)
  37. # [02:28] * Joins: surkov (surkov@B9538DF7.B7015738.33A1AC3C.IP)
  38. # [02:28] * ChanServ sets mode: +o surkov
  39. # [02:29] * Quits: @surkov (surkov@B9538DF7.B7015738.33A1AC3C.IP) (Quit: surkov)
  40. # [02:35] <@hub> Mark_Capella: it fails to build on Linux too
  41. # [02:35] <@hub> let me finish rebuilding without the patch so that I can get my browser back
  42. # [02:35] <@hub> (same errors)
  43. # [02:35] <Mark_Capella> yah ... odd ... those macros are new to me and there must be an interdependancy that Im unaware of ...
  44. # [02:36] <@hub> yeah it is XPCOM
  45. # [02:36] <@hub> it is full of pain
  46. # [02:36] <Mark_Capella> >ouchies<
  47. # [02:37] <Mark_Capella> part of the patch caused a secondary problem that I was able to track down / correct ... this one has me stuck
  48. # [02:37] <Mark_Capella> im trying to fix it without asking for too much help :P
  49. # [02:46] <@firebot> markcapella@twcny.rr.com requested review from hub@mozilla.com for attachment 613450 on bug 741683.
  50. # [02:47] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741683 nor, --, ---, markcapella, ASSI, move nsAccessNode::ScrollTo to nsCoreUtils
  51. # [02:47] <@firebot> hub@mozilla.com denied feedback for attachment 613410 on bug 741707.
  52. # [02:47] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741707 nor, --, ---, markcapella, ASSI, rm unused a11y CIDs
  53. # [02:54] <Mark_Capella> hub: ok forgotten : but I still have >issues< :D
  54. # [02:55] <Mark_Capella> polite way to say im confused !
  55. # [03:09] <Mark_Capella> hub: ill have to wait then ... alex has a buncha my stuff and othes coming through inbound yet maybe by tomorrow
  56. # [03:15] <@hub> ok, gotta go nw
  57. # [03:15] <@hub> now
  58. # [03:16] * Quits: @hub (hub@21B7B9F2.B87E9213.6E712CE2.IP) (Ping timeout)
  59. # [03:17] * ctalbert is now known as ctalbert|afk
  60. # [03:38] <@firebot> markcapella@twcny.rr.com changed the Status on bug 741709 from NEW to ASSIGNED.
  61. # [03:38] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741709 nor, --, ---, markcapella, ASSI, nsAccessibleWrap::get_accParent() shouldn't use do_QueryObject to get a nsDocAccessible
  62. # [03:59] * Joins: nickc (Mibbit@moz-91D453DB.network.nus.edu.sg)
  63. # [04:17] * Joins: surkov (surkov@B9538DF7.B7015738.33A1AC3C.IP)
  64. # [04:17] * ChanServ sets mode: +o surkov
  65. # [04:25] <@firebot> markcapella@twcny.rr.com requested feedback from trev.saunders@gma il.com for attachment 613470 on bug 741709.
  66. # [04:25] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741709 nor, --, ---, markcapella, ASSI, nsAccessibleWrap::get_accParent() shouldn't use do_QueryObject to get a nsDocAccessible
  67. # [04:38] <@tbsaunde> Mark_Capella: what were these build errors? (just came back from being away and haven't read mail yet)
  68. # [04:39] <Mark_Capella> i found one of the problems
  69. # [04:39] <Mark_Capella> there was a dependency on another bug that I hadnt done yet
  70. # [04:40] <@tbsaunde> Mark_Capella: yeah, but that wouldn't have effected linux
  71. # [04:41] <Mark_Capella> well bug 741709 has to be done before bug741707 can be done ....
  72. # [04:41] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741709 nor, --, ---, markcapella, ASSI, nsAccessibleWrap::get_accParent() shouldn't use do_QueryObject to get a nsDocAccessible
  73. # [04:41] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741707 nor, --, ---, markcapella, ASSI, rm unused a11y CIDs
  74. # [04:41] <Mark_Capella> I didnt see the dependency in the bugzilla / blocking / depends on thing
  75. # [04:46] <@tbsaunde> Mark_Capella: yes, bug the dependancy should only effect windows
  76. # [04:47] <nickc> Hi mark
  77. # [04:48] <Mark_Capella> well i coded / tested and posted patch for bug741709 ... to get it out of the way
  78. # [04:48] <Mark_Capella> now I can go back and look at bug740717
  79. # [04:48] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740717 nor, P3, ---, nobody, NEW, Add language switcher dropdown to Marketplace
  80. # [04:48] <Mark_Capella> nickc: hi
  81. # [04:48] <Mark_Capella> 741707 that is
  82. # [04:48] <Mark_Capella> too many numbers and its late :)
  83. # [04:48] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  84. # [04:56] <@tbsaunde> heh
  85. # [04:58] <@firebot> orangedaylily@gmail.com requested review from hub@mozilla.com for attachment 613477 on bug 741681.
  86. # [04:58] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741681 nor, --, ---, orangedaylily, NEW, replace nsAccessNode::GetPresContext by nsDocAccessible::PresContext
  87. # [05:04] <@tbsaunde> Mark_Capella: you could just ask for review if you think the patch is good like bug 741709
  88. # [05:04] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741709 nor, --, ---, markcapella, ASSI, nsAccessibleWrap::get_accParent() shouldn't use do_QueryObject to get a nsDocAccessible
  89. # [05:04] <Mark_Capella> oh! did I do a feedback?
  90. # [05:05] <@firebot> trev.saunders@gmail.com granted review for attachment 613470 on bug 741709.
  91. # [05:05] <@firebot> trev.saunders@gmail.com cancelled feedback?(trev.saunders@gma il.com) for attachment 613470 on bug 741709.
  92. # [05:08] * Quits: @surkov (surkov@B9538DF7.B7015738.33A1AC3C.IP) (Quit: surkov)
  93. # [06:02] * khuey is now known as khuey|away
  94. # [06:07] * Joins: victorporof (victorporo@BD5B5B26.602BD6C.79933D60.IP)
  95. # [06:36] * Quits: brambles_ (brambles@4CBAB088.F3076E90.1822ACA6.IP) (Client exited)
  96. # [06:37] * Joins: brambles (brambles@4CBAB088.F3076E90.1822ACA6.IP)
  97. # [06:43] * Quits: jkiss (jkiss@5248D1C6.C5B5659C.8732AABF.IP) (Ping timeout)
  98. # [06:58] * Joins: hub (hub@moz-E2FCA694.figuiere.net)
  99. # [06:58] * ChanServ sets mode: +o hub
  100. # [06:58] <@firebot> markcapella@twcny.rr.com requested feedback from hub@mozilla.com for attachment 613492 on bug 741707.
  101. # [06:58] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741707 nor, --, ---, markcapella, ASSI, rm unused a11y CIDs
  102. # [07:15] <@firebot> hub@mozilla.com granted feedback for attachment 613492 on bug 741707.
  103. # [07:15] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741707 nor, --, ---, markcapella, ASSI, rm unused a11y CIDs
  104. # [07:18] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Client exited)
  105. # [07:29] <@firebot> hub@mozilla.com granted review for attachment 613477 on bug 741681.
  106. # [07:29] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741681 nor, --, ---, orangedaylily, NEW, replace nsAccessNode::GetPresContext by nsDocAccessible::PresContext
  107. # [07:37] <Mark_Capella> yee haa ... i love it when a plan comes together :)
  108. # [07:38] * Quits: Mark_Capella (chatzilla@moz-DD0C7E4F.twcny.res.rr.com) (Connection reset by peer)
  109. # [07:38] * Joins: Mark_Capella (chatzilla@moz-DD0C7E4F.twcny.res.rr.com)
  110. # [07:43] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  111. # [07:55] * Joins: surkov (surkov@B9538DF7.B7015738.33A1AC3C.IP)
  112. # [07:55] * ChanServ sets mode: +o surkov
  113. # [08:03] <@firebot> surkov.alexander@gmail.com changed the Target Milestone on bug 741709 from --- to mozilla14.
  114. # [08:03] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741709 nor, --, mozilla14, markcapella, ASSI, nsAccessibleWrap::get_accParent() shouldn't use do_QueryObject to get a nsDocAccessible
  115. # [08:12] * Joins: MarcoZ (Daily@moz-C84B5E86.dip.t-dialin.net)
  116. # [08:12] * ChanServ sets mode: +o MarcoZ
  117. # [08:17] * Quits: victorporof (victorporo@BD5B5B26.602BD6C.79933D60.IP) (Quit: Leaving...)
  118. # [08:17] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  119. # [08:18] * Quits: @hub (hub@moz-E2FCA694.figuiere.net) (Ping timeout)
  120. # [08:37] <@firebot> nickyekaiqi@gmail.com requested feedback from the wind for attachment 613504 on bug 740764.
  121. # [08:37] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740764 nor, --, ---, nickyekaiqi, NEW, merge GetAttributes and GetAttributesInternal
  122. # [08:52] * Mark_Capella is now known as Mark_Capella|around
  123. # [09:04] <@firebot> surkov.alexander@gmail.com cancelled feedback? for attachment 613504 on bug 740764.
  124. # [09:05] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740764 nor, --, ---, nickyekaiqi, NEW, merge GetAttributes and GetAttributesInternal
  125. # [09:58] <jhk> this is how we can get rid of xpcom GetParent? http://pastebin.mozilla.org/1561853
  126. # [09:59] <@surkov> jhk: what is acc?
  127. # [09:59] <jhk> nsAccessibleWrap* acc = static_cast<nsAccessibleWrap>(this);
  128. # [10:01] <jhk> surkov: ^
  129. # [10:01] <@surkov> jhk: acc->Parent()
  130. # [10:01] <@surkov> so you code looks correct
  131. # [10:02] <jhk> surkov: thanks!
  132. # [10:02] <@surkov> yw
  133. # [10:07] * Parts: icaaq1 (Adium@moz-7D09279.cust.bredbandsbolaget.se)
  134. # [10:46] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  135. # [10:51] * Joins: victorporof (victorporo@moz-FDB8B824.info.uaic.ro)
  136. # [10:52] * Mark_Capella|around is now known as Mark_Capella
  137. # [11:10] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  138. # [11:18] * Quits: victorporof (victorporo@moz-FDB8B824.info.uaic.ro) (Quit: Leaving...)
  139. # [11:20] * Joins: rdt (Adium@moz-3D54E27E.nycmny.fios.verizon.net)
  140. # [11:22] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  141. # [11:33] * Parts: rdt (Adium@moz-3D54E27E.nycmny.fios.verizon.net)
  142. # [11:59] * Joins: jhk (jigneshhk1@8E6C34C1.A3F9767A.1C37C358.IP)
  143. # [12:17] * Quits: @MarcoZ (Daily@moz-C84B5E86.dip.t-dialin.net) (Input/output error)
  144. # [12:17] * Joins: MarcoZ (Daily@moz-C84B5E86.dip.t-dialin.net)
  145. # [12:17] * ChanServ sets mode: +o MarcoZ
  146. # [12:18] <@MarcoZ> Hi all!
  147. # [12:19] <@MarcoZ> Mark_Capella: Hi, am getting to your review now, was on public Easter holidays.
  148. # [12:19] <Mark_Capella> marcoz: thanks! Welcome back :)
  149. # [12:20] <Mark_Capella> lots of people have been out .... real quiet around here
  150. # [12:23] <@surkov> yeah, try server builds are fast as never
  151. # [12:24] <@surkov> usually I'm waiting about 15 minutes before it does a push
  152. # [12:24] <@surkov> these days it's about half of minute ;)
  153. # [12:25] * Quits: jhk (jigneshhk1@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  154. # [12:28] * Joins: jhk (jigneshhk1@8E6C34C1.A3F9767A.1C37C358.IP)
  155. # [12:29] * Quits: jhk (jigneshhk1@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  156. # [12:29] * Joins: jhk (jigneshhk1@8E6C34C1.A3F9767A.1C37C358.IP)
  157. # [12:30] <Mark_Capella> its fast now that they got it running ... it was down from Friday night to Sunday morning
  158. # [12:30] <Mark_Capella> or maybe that was just the autloand portion
  159. # [12:36] <@firebot> marco.zehe@googlemail.com granted review for attachment 612984 on bug 739612.
  160. # [12:36] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=739612 nor, --, ---, markcapella, ASSI, Cleanup A11y tests and test-suite organization
  161. # [12:38] <@MarcoZ> Mark_Capella: To be absolutely certain, you should throw the updated patch for bug 739612 against try and watch the results to make sure the tests still pass on all platforms.
  162. # [12:39] <Mark_Capella> yah - ok - ill do that here this morning - thankx !
  163. # [12:41] * Joins: victorporof (victorporo@4B0D9BBA.6D2FB7A4.79933D60.IP)
  164. # [13:29] <@firebot> surkov.alexander@gmail.com granted review for attachment 613541 on bug 741707.
  165. # [13:29] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741707 nor, --, ---, markcapella, ASSI, rm unused a11y CIDs
  166. # [13:31] <Mark_Capella> Woo Hoo :)
  167. # [13:33] <Mark_Capella> finally figured out why the out of place getRole() test worked (instead of failing as expected)
  168. # [13:36] * Quits: @surkov (surkov@B9538DF7.B7015738.33A1AC3C.IP) (Quit: surkov)
  169. # [13:49] * Quits: victorporof (victorporo@4B0D9BBA.6D2FB7A4.79933D60.IP) (Connection reset by peer)
  170. # [13:49] * Joins: victorporo (victorporo@4B0D9BBA.6D2FB7A4.79933D60.IP)
  171. # [13:53] * Quits: victorporo (victorporo@4B0D9BBA.6D2FB7A4.79933D60.IP) (Connection reset by peer)
  172. # [13:53] * Joins: victorporof (victorporo@4B0D9BBA.6D2FB7A4.79933D60.IP)
  173. # [13:54] * Mark_Capella is now known as Mark_Capella|away
  174. # [13:57] * Quits: victorporof (victorporo@4B0D9BBA.6D2FB7A4.79933D60.IP) (Connection reset by peer)
  175. # [13:57] * Joins: victorporo (victorporo@4B0D9BBA.6D2FB7A4.79933D60.IP)
  176. # [14:01] * Quits: victorporo (victorporo@4B0D9BBA.6D2FB7A4.79933D60.IP) (Ping timeout)
  177. # [14:03] * Joins: victorporof (victorporo@4B0D9BBA.6D2FB7A4.79933D60.IP)
  178. # [14:08] <nickc> Anyone to help?
  179. # [14:08] * Quits: victorporof (victorporo@4B0D9BBA.6D2FB7A4.79933D60.IP) (Connection reset by peer)
  180. # [14:08] * Joins: victorporo (victorporo@4B0D9BBA.6D2FB7A4.79933D60.IP)
  181. # [14:09] <nickc> Need some clarification of a bug
  182. # [14:09] <@tbsaunde> nickc: yeah?
  183. # [14:11] <@tbsaunde> nickc: what is your issue?
  184. # [14:12] * Quits: victorporo (victorporo@4B0D9BBA.6D2FB7A4.79933D60.IP) (Connection reset by peer)
  185. # [14:12] * Joins: victorporof (victorporo@4B0D9BBA.6D2FB7A4.79933D60.IP)
  186. # [14:13] * Quits: victorporof (victorporo@4B0D9BBA.6D2FB7A4.79933D60.IP) (Connection reset by peer)
  187. # [14:13] <nickc> I sent it to you through personal talking
  188. # [14:13] <nickc> You did not receive the messages?
  189. # [14:13] * Joins: victorporo (victorporo@4B0D9BBA.6D2FB7A4.79933D60.IP)
  190. # [14:15] * Joins: jprmc (jprmc@C3A2FAF3.5BCEC6DB.DA78B690.IP)
  191. # [14:15] * ChanServ sets mode: +o jprmc
  192. # [14:15] * victorporo is now known as victorporof
  193. # [14:16] * Quits: victorporof (victorporo@4B0D9BBA.6D2FB7A4.79933D60.IP) (Connection reset by peer)
  194. # [14:18] <@MarcoZ> nick: Perhaps if you post your question to the channel, any of the team members wo are currently here can try to answer.
  195. # [14:19] <@MarcoZ> nickc: ^
  196. # [14:19] <nickc> Yup
  197. # [14:19] <nickc> I tried to copy
  198. # [14:19] * Quits: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP) (Input/output error)
  199. # [14:19] <nickc> Now I am working on this bug 740764
  200. # [14:19] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740764 nor, --, ---, nickyekaiqi, NEW, merge GetAttributes and GetAttributesInternal
  201. # [14:19] <nickc> surkov is not here
  202. # [14:20] <nickc> I need some more explanation about the review part on nsAccessible.cpp
  203. # [14:20] * Joins: victorporof (victorporo@830D7731.AB163232.79933D60.IP)
  204. # [14:21] <nickc> There exists a function with the prototype NS_IMETHODIMP nsAccessible::GetAttributes(nsIPersistentProperties** aAttributes) in nsAccessibility
  205. # [14:22] <nickc> In the review, I am told to create another NS_IMETHODIMP nsAccessible::GetAttributes(nsIPersistentProperties** aAttributes) function. What does it mean?
  206. # [14:22] <nickc> I talked with surkov before he left.
  207. # [14:22] <@tbsaunde> nickc: I don't think you should be creating a second one
  208. # [14:22] <nickc> It seems that they are the same one.
  209. # [14:22] <@tbsaunde> but the existing one should be NS_IMETHODIMP not virtual nsresult
  210. # [14:23] <nickc> NS_IMETHODIMP and virtual nsresult are types?
  211. # [14:23] <@tbsaunde> but other than that I don't really see what Alex wants
  212. # [14:24] <@tbsaunde> he might be around in a bit, or you can work on something else
  213. # [14:24] <nickc> Is there any other recommended bug about accessibility
  214. # [14:25] <nickc> I want to do one more
  215. # [14:25] <nickc> Some bug which is not that hard, since I am still trying to be more familiar with it
  216. # [14:26] * Quits: logbot (logbot@moz-622AFC27.glob.com.au) (Client exited)
  217. # [14:26] * Joins: logbot (logbot@moz-622AFC27.glob.com.au)
  218. # [14:31] * Quits: @jprmc (jprmc@C3A2FAF3.5BCEC6DB.DA78B690.IP) (Ping timeout)
  219. # [14:33] * Joins: icaaq (Adium@714E29CB.13DB46CE.3B93FF6D.IP)
  220. # [14:36] * Joins: surkov (surkov@B9538DF7.B7015738.33A1AC3C.IP)
  221. # [14:36] * ChanServ sets mode: +o surkov
  222. # [14:41] * Joins: shorlander (shorlander@moz-853043D6.dhcp.insightbb.com)
  223. # [14:41] * Quits: shorlander_ (shorlander@moz-853043D6.dhcp.insightbb.com) (Ping timeout)
  224. # [14:44] <@tbsaunde> nickc: there are a bunch of other good first bugs, you could ask about the difficulty of one of those in here and we can tell you, or surkov seems to be around
  225. # [14:44] * tbsaunde is now known as tbsaunde|shower
  226. # [14:45] <@surkov> getAccessible is not trivial one but iirc nickc has good results on it
  227. # [14:46] <nickc> Hi surkov
  228. # [14:46] <@surkov> hi, nickc
  229. # [14:46] <nickc> Still a little blur
  230. # [14:46] <nickc> Sorry for that
  231. # [14:46] <@surkov> it's ok
  232. # [14:47] <nickc> Where should I put that part of code in?
  233. # [14:49] <nickc> For NS_IMETHODIMP nsAccessible::GetAttributes(nsIPersistentProperties** aAttributes)
  234. # [14:49] <nickc> In this function, I should call it again.
  235. # [14:49] <nickc> Then it runs into an infinite loop...:(
  236. # [14:49] <nickc> To me...
  237. # [14:50] <nickc> I must misunderstand your meaning
  238. # [14:50] <@surkov> nickc: you should put it somewhere in nsAccessible (you can put it near to existing getAttributes implementation)
  239. # [14:52] <nickc> I will put my understanding into pasterbin and let you check it. Is it OK?
  240. # [14:52] <@surkov> it's not infinite loop, it's two different methods, one of them takes nsIPersistentProperties** (nsIAccessible interface implementation) and another one takes nsIPersistentProperties*
  241. # [14:52] <@surkov> sure
  242. # [14:53] <nickc> Two methods!
  243. # [14:53] <nickc> One para is pointer and the other one is pointer to pointer
  244. # [14:53] <@surkov> nickc: it's called overloading functions http://publib.boulder.ibm.com/infocenter/comphelp/v8v101/index.jsp?topic=%2Fcom.ibm.xlcpp8a.doc%2Flanguage%2Fref%2Fcplr312.htm
  245. # [14:54] <@surkov> right
  246. # [14:54] <nickc> Oh! That's why I got confused. I thought the same function is called by itself. :)
  247. # [14:55] <nickc> Then I just define 2 functions
  248. # [14:55] <nickc> One is called in the other one
  249. # [14:56] <@surkov> right
  250. # [14:56] <nickc> OK! I thought they had the same para. :(
  251. # [14:56] <nickc> Clear now!
  252. # [14:56] <nickc> Thx
  253. # [15:00] <@MarcoZ> Hi surkov!
  254. # [15:00] <@MarcoZ> surkov: Am I correct in assuming that bug 729831 is in Mozilla13?
  255. # [15:00] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=729831 cri, --, mozilla13, surkov.alexander, RESO FIXED, Crash @ mozilla::a11y::StyleInfo::Margin
  256. # [15:03] * Joins: davidb (davidb@F2D29657.F60B0462.67AC9B1.IP)
  257. # [15:03] * ChanServ sets mode: +qo davidb davidb
  258. # [15:04] <@MarcoZ> Hey davidb!
  259. # [15:04] <@davidb> hi MarcoZ! Nice holiday?
  260. # [15:08] <@davidb> tbsaunde|shower: I see results are in on the talos try.
  261. # [15:09] <@surkov> MarcoZ: hi, sounds so
  262. # [15:09] <@MarcoZ> davidb: Yes, we went to Petra's Mom's place and spent the rest of Easter Sunday and yesterday at home, clearing up more stuff. Almost all boxes unpacked now. \-O-/
  263. # [15:10] <@davidb> nice
  264. # [15:10] <@MarcoZ> surkov: Thanks, then I don't think any more action is needed on the bug, as i said in my e-mail.
  265. # [15:10] <@davidb> BTW I've not checked twitter in a few days
  266. # [15:10] <@davidb> which bug?
  267. # [15:11] <@MarcoZ> davidb: bug 729831. The one you pinged me in e-mail about yesterday.
  268. # [15:11] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=729831 cri, --, mozilla13, surkov.alexander, RESO FIXED, Crash @ mozilla::a11y::StyleInfo::Margin
  269. # [15:11] <@MarcoZ> davidb: Nothing to uplift, just to verify.
  270. # [15:11] * khuey|away is now known as khuey
  271. # [15:11] <@davidb> ah ok
  272. # [15:12] <@davidb> MarcoZ: I haven't caught up on the inspect label bug… did that get resolved?
  273. # [15:12] <@MarcoZ> davidb: Oh then you didn't see my comment about me agreeing with a big German ecomonomy newspaper that stated that Silicon Valley is really going insane? ;)
  274. # [15:12] <@surkov> MarcoZ: but you might want to double check that
  275. # [15:12] <@davidb> MarcoZ: haha
  276. # [15:12] <@surkov> because target13 was set for first patch
  277. # [15:12] <@MarcoZ> I mean this 1 billion dollar Instagram deal just screams "Bubble!!!" to me.
  278. # [15:13] <@eeejay> yep, bubble
  279. # [15:13] <@MarcoZ> surkov: I believe second patch was landed *before* the merge on March 13.
  280. # [15:13] <@eeejay> and that new silicon valley reality show...
  281. # [15:13] <@surkov> ok
  282. # [15:13] <@MarcoZ> davidb: No activity at all on that bug in the last six days.
  283. # [15:15] <@MarcoZ> surkov: Commented.
  284. # [15:15] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  285. # [15:15] <@davidb> MarcoZ: one of us should help drive it.
  286. # [15:22] <@MarcoZ> davidb: TBH, I'm not sure where that bug should go. I'm unsure of the feature/how it works, and also what the exact interaction model is.
  287. # [15:22] <@MarcoZ> The one thing I *do* know is that hiding it from accessibility APIs is the worst of ideas.
  288. # [15:22] <@MarcoZ> But if you'd ask me what its label should be.....I have *no* idea. ;)
  289. # [15:23] <@davidb> MarcoZ: I think we just need a good label. The button in question sets the Inspect mode to mouse interaction sub-mode.
  290. # [15:23] <@davidb> So something like "rollover mode"
  291. # [15:24] <@davidb> MarcoZ: what does the button name say in nightly?
  292. # [15:24] * tbsaunde|shower is now known as tbsaunde
  293. # [15:26] <@MarcoZ> Nothin.
  294. # [15:26] <@MarcoZ> Labelless.
  295. # [15:27] <@davidb> MarcoZ: not even the tooltip?
  296. # [15:29] <@tbsaunde> davidb: yeah, haven't looked at them yet though
  297. # [15:30] * Joins: jprmc (jprmc@F2D29657.F60B0462.67AC9B1.IP)
  298. # [15:30] * ChanServ sets mode: +o jprmc
  299. # [15:31] * Quits: jhk (jigneshhk1@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  300. # [15:34] <icaaq> Hi, I asked yesterday about keyboard and video, and why I need javascript for it to be keyboard accessible. have a look at http://icaaq.com/sandbox/a11y/video.htm it's the simplest of all videos but it is not focusable with the keyboard when javascript is disabled.
  301. # [15:35] <@davidb> icaaq: http://mindforks.blogspot.ca/2009/09/keyboard-control-of-html5-video.html
  302. # [15:37] <@davidb> icaaq: it might be time to make the controls themselves keyboard navigable, we just need someone to work on it
  303. # [15:39] <@MarcoZ> davidb: No.
  304. # [15:39] <@MarcoZ> Not even the tooltip.
  305. # [15:39] <icaaq> davidb: yes, other browsers already have it implemented
  306. # [15:39] <@MarcoZ> davidb: At least I didn't find it.
  307. # [15:39] <icaaq> davidb: opera for example
  308. # [15:41] <@tbsaunde> davidb: did you have a chance to look yet?
  309. # [15:45] <@davidb> MarcoZ: that seems odd. so you encounter a nameless "button"?
  310. # [15:46] <@davidb> tbsaunde: I looked briefly… all the T's are green, but I'm not sure if that means we're ok or not since usually the talos regression algorithm needs multiple data points
  311. # [15:46] <@MarcoZ> davidb: I see some unlabelled buttons here and there, I am not even sure *where* the right one is.
  312. # [15:47] <@davidb> ok
  313. # [15:47] <@davidb> i need to fix my windows build
  314. # [15:47] <@tbsaunde> davidb: ok, so, do we have a way to compare that run against the m-c changeset it is on top of?
  315. # [15:47] <@tbsaunde> or do we need to rerun tests first?
  316. # [15:48] * @tbsaunde not really sure how to use talos
  317. # [15:49] <@davidb> tbsaunde: we probably need to look at the actual numerical values and compare manually
  318. # [15:50] <@davidb> tbsaunde: it might be easier to create tests like we did for previous perf work
  319. # [15:50] <@davidb> actually, almost certainly easier
  320. # [15:50] <@tbsaunde> davidb: not sure I follow
  321. # [15:51] <@tbsaunde> it seems crazy that we don't have a way to test perf of a patch without landing it on a rel branch, I'm not sure I believe that
  322. # [15:51] <@davidb> tbsaunde: I'll back up. Why are you doing this talos try push?
  323. # [15:52] <@tbsaunde> davidb: so, mostly what want to look at is how much worse turning on a11y
  324. # [15:52] <@tbsaunde> err, sorry
  325. # [15:53] <@davidb> it is an interesting thing to try out, I just want to be clear I understand your goal.
  326. # [15:53] <@tbsaunde> mostly I want to have some real idea how much worse we are with a11y on, and where in particular we are bad
  327. # [15:53] <@davidb> ok
  328. # [15:53] <@tbsaunde> davidb: were you thinking of a different goal?
  329. # [15:53] <@davidb> There is also the desire to have a test for our speech rec goal, which might be a manual test like we created for previous perf work.
  330. # [15:53] <@davidb> Yes I was conflating the two.
  331. # [15:54] <@davidb> tbsaunde: I would hope green T's mean we are really performant for the regular Talos tests we have but it is worth confirming with Talos people.
  332. # [15:54] <@davidb> anode is one of them but on temporary leave i think
  333. # [15:55] <@davidb> !seen anode
  334. # [15:55] <@firebot> anode was last seen 27 weeks, 3 days, 16 hours, 47 minutes and 47 seconds ago, changing nick to anode|away.
  335. # [15:55] <@davidb> catlee is probably a good person to poke
  336. # [15:56] <@tbsaunde> davidb: ok, I'll try
  337. # [15:56] <@davidb> awesome
  338. # [15:56] <@tbsaunde> davidb: its pretty suprising we don't regress dom tests at all
  339. # [15:56] <@davidb> right. I'm skeptical.
  340. # [15:57] <@davidb> tbsaunde: although, the Talos tests for DOM probably don't time our 'subsequent' churn.
  341. # [15:57] <@davidb> when I wrote our Talos tests I had to be tricky
  342. # [15:58] <@davidb> force reflow to complete, then set a content timer.
  343. # [15:58] <@davidb> blah blah
  344. # [15:58] <@tbsaunde> davidb: sure, but it would still imply that having a11y on when you don't need it wouldn't hurt responsiveness, only mean some background cpu time would be used
  345. # [15:58] <@davidb> tbsaunde: not necessarily
  346. # [15:58] <@tbsaunde> davidb: why?
  347. # [15:59] <@davidb> we can produce a lot of churn when the refresh driver says 'go ahead'
  348. # [15:59] <@davidb> probably mostly around events
  349. # [16:01] <@davidb> i.e. WillRefresh is called
  350. # [16:01] <nickc> Hi surkov! Before I start to fix it, could you have a look at my potential method? http://pastebin.mozilla.org/1562265
  351. # [16:01] <@tbsaunde> davidb: sure, but presambly somebody else tests that if they change content what is actually gets painted in a set amount of time
  352. # [16:01] <@davidb> tbsaunde: i wouldn't make any assumptions about precisely what they are timing.
  353. # [16:02] <@tbsaunde> davidb: I guess, but it seems like somebody real should test that
  354. # [16:02] <@davidb> yes
  355. # [16:02] <@tbsaunde> any way class
  356. # [16:02] * tbsaunde is now known as tbsaunde|afk
  357. # [16:02] <@davidb> saved by the bell :)
  358. # [16:08] <@surkov> nickc: for nsAccessible::GetAttributes(nsIPersistentProperties **aAttributes) don't forget to create attributes
  359. # [16:08] <@surkov> nickc: see attributes = do_CreateInstance(NS_PERSISTENTPROPERTIES_CONTRACTID); part
  360. # [16:09] <nickc> Yes. So it is what you want?
  361. # [16:09] <@surkov> yep
  362. # [16:27] * Joins: hub (hub@moz-E2FCA694.figuiere.net)
  363. # [16:27] * ChanServ sets mode: +o hub
  364. # [16:38] * Joins: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP)
  365. # [16:42] <@firebot> nickyekaiqi@gmail.com requested feedback from the wind for attachment 613599 on bug 740764.
  366. # [16:42] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740764 nor, --, ---, nickyekaiqi, NEW, merge GetAttributes and GetAttributesInternal
  367. # [16:42] <nickc> Just submit a patch. Hope it improves. Come back later.
  368. # [16:42] <nickc> Leave for a while
  369. # [16:42] * Quits: nickc (Mibbit@moz-91D453DB.network.nus.edu.sg) (Quit: http://www.mibbit.com ajax IRC Client)
  370. # [16:42] * Quits: ehsan (ehsan@F0B20A8D.8458880F.57F33CED.IP) (Input/output error)
  371. # [16:51] <@davidb> icaaq: feel free to comment on bug 494175
  372. # [16:51] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=494175 nor, --, ---, surkov.alexander, ASSI, Make <video> and <audio> controls keyboard navigable
  373. # [16:51] <@davidb> icaaq: I recommend reading the whole discussion first.
  374. # [16:52] * Joins: nhirata (nhirata.bu@moz-C03D0C61.vlan426.asr1.sfo1.gblx.net)
  375. # [17:00] * Quits: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp) (Quit: mike5w3c)
  376. # [17:05] * Joins: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp)
  377. # [17:09] * Joins: ehsan (ehsan@F2D29657.F60B0462.67AC9B1.IP)
  378. # [17:10] <@firebot> surkov.alexander@gmail.com granted review for attachment 613332 on bug 739498.
  379. # [17:10] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=739498 nor, --, ---, eitan, NEW, Add Javascript layer for mobile accessibility
  380. # [17:11] <@MarcoZ> Yay!
  381. # [17:15] <@davidb> Yes epic. I'm going to review it next.
  382. # [17:15] <@davidb> (I began yesterday)
  383. # [17:16] <@MarcoZ> Have fun! :D
  384. # [17:16] <@davidb> heheh
  385. # [17:16] <@davidb> I'll try not to ask for dojo-isms
  386. # [17:18] <@firebot> surkov.alexander@gmail.com requested review from trev.saunders@gmail .com for attachment 613599 on bug 740764.
  387. # [17:18] <@firebot> surkov.alexander@gmail.com granted feedback for attachment 613599 on bug 740764.
  388. # [17:18] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740764 nor, --, ---, nickyekaiqi, NEW, merge GetAttributes and GetAttributesInternal
  389. # [17:22] * Joins: askalski (akuda@moz-6A36EC49.ip.abpl.pl)
  390. # [17:22] * ChanServ sets mode: +o askalski
  391. # [17:33] <@firebot> ehsan@mozilla.com changed the Status on bug 740758 from ASSIGNED to RESOLVED.
  392. # [17:33] <@firebot> ehsan@mozilla.com set the Resolution field on bug 740758 to FIXED.
  393. # [17:34] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740758 nor, --, mozilla14, markcapella, RESO FIXED, dexpcom nsAccessible::GetValue
  394. # [17:34] <@firebot> ehsan@mozilla.com changed the Status on bug 742695 from ASSIGNED to RESOLVED.
  395. # [17:34] <@firebot> ehsan@mozilla.com set the Resolution field on bug 742695 to FIXED.
  396. # [17:34] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=742695 nor, --, mozilla14, markcapella, RESO FIXED, densify nsARIAGridAccessible
  397. # [17:34] <@firebot> ehsan@mozilla.com changed the Status on bug 741701 from ASSIGNED to RESOLVED.
  398. # [17:34] <@firebot> ehsan@mozilla.com set the Resolution field on bug 741701 to FIXED.
  399. # [17:34] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741701 nor, --, mozilla14, markcapella, RESO FIXED, CAccessibleHyperlink::get_anchor shouldn't use nsIWinAccessNode
  400. # [17:35] <@firebot> ehsan@mozilla.com changed the Status on bug 539683 from ASSIGNED to RESOLVED.
  401. # [17:35] <@firebot> ehsan@mozilla.com set the Resolution field on bug 539683 to FIXED.
  402. # [17:35] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=539683 nor, --, mozilla14, surkov.alexander, RESO FIXED, defunct object methods should return CO_E_OBJECTNOTCONNECTED
  403. # [17:42] <@firebot> surkov.alexander@gmail.com changed the Target Milestone on bug 707488 from --- to mozilla14.
  404. # [17:42] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=707488 cri, --, mozilla14, nobody, NEW, Crash in nsHTMLTextFieldAccessible::NativeRole or nsXULTextFieldAccessible::NativeRole
  405. # [17:43] * Quits: @askalski (akuda@moz-6A36EC49.ip.abpl.pl) (Quit: Wychodzi)
  406. # [17:54] <@firebot> ehsan@mozilla.com changed the Status on bug 741709 from ASSIGNED to RESOLVED.
  407. # [17:54] <@firebot> ehsan@mozilla.com set the Resolution field on bug 741709 to FIXED.
  408. # [17:54] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741709 nor, --, mozilla14, markcapella, RESO FIXED, nsAccessibleWrap::get_accParent() shouldn't use do_QueryObject to get a nsDocAccessible
  409. # [18:01] * Quits: @surkov (surkov@B9538DF7.B7015738.33A1AC3C.IP) (Quit: surkov)
  410. # [18:04] * Joins: nickc (Mibbit@8A748F18.D0521ABA.428E23B0.IP)
  411. # [18:05] <@firebot> nickyekaiqi@gmail.com cancelled review?(trev.saunders@gmail .com) for attachment 613599 on bug 740764.
  412. # [18:05] <@firebot> nickyekaiqi@gmail.com requested review from trev.saunders@gmail .com for attachment 613625 on bug 740764.
  413. # [18:05] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740764 nor, --, ---, nickyekaiqi, NEW, merge GetAttributes and GetAttributesInternal
  414. # [18:16] * Joins: jhk (jigneshhk1@8E6C34C1.A3F9767A.1C37C358.IP)
  415. # [18:18] * khuey is now known as khuey|away
  416. # [18:22] * Quits: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp) (Quit: mike5w3c)
  417. # [18:26] * Joins: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp)
  418. # [18:31] <icaaq> davidb: thanks, *eveningread*!
  419. # [18:31] <@davidb> :)
  420. # [18:33] * Quits: @hub (hub@moz-E2FCA694.figuiere.net) (Ping timeout)
  421. # [18:37] * Joins: fxa90id (fxa90id@moz-C6D6DA75.neoplus.adsl.tpnet.pl)
  422. # [18:51] * @MarcoZ is off. See you tomorrow!
  423. # [18:51] * Quits: @MarcoZ (Daily@moz-C84B5E86.dip.t-dialin.net) (Quit: l8er)
  424. # [18:54] * @eeejay heads to the airport
  425. # [18:54] <@eeejay> davidb, i'll have a new patch ready en route
  426. # [18:54] * Quits: fxa90id (fxa90id@moz-C6D6DA75.neoplus.adsl.tpnet.pl) (Connection reset by peer)
  427. # [18:54] <@davidb> sweet
  428. # [18:55] * khuey|away is now known as khuey
  429. # [18:56] * Joins: icaaq1 (Adium@moz-7D09279.cust.bredbandsbolaget.se)
  430. # [18:57] * Quits: icaaq (Adium@714E29CB.13DB46CE.3B93FF6D.IP) (Ping timeout)
  431. # [19:15] * Joins: hub (hub@21B7B9F2.B87E9213.6E712CE2.IP)
  432. # [19:15] * ChanServ sets mode: +o hub
  433. # [19:17] <@firebot> hub@mozilla.com granted review for attachment 613450 on bug 741683.
  434. # [19:17] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741683 nor, --, ---, markcapella, ASSI, move nsAccessNode::ScrollTo to nsCoreUtils
  435. # [19:19] <@firebot> hub@mozilla.com granted review for attachment 613499 on bug 741681.
  436. # [19:19] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741681 nor, --, ---, orangedaylily, NEW, replace nsAccessNode::GetPresContext by nsDocAccessible::PresContext
  437. # [19:19] <@hub> Mark_Capella|away: I'll rebase and checkin
  438. # [19:21] * ctalbert|afk is now known as ctalbert
  439. # [19:21] <Mark_Capella|away> hub: thanks :)
  440. # [19:23] <@davidb> khuey: i build on windows using regular make because pymake was failing after many trials and permutations. how do i convert my objdir back to using pymake? (MSYS-style srcdir being users with Pymake error)
  441. # [19:28] * Mark_Capella|away is now known as Mark_Capella
  442. # [19:28] <khuey> davidb: rm -rf objdir
  443. # [19:29] <khuey> and then build with pymake
  444. # [19:29] <Mark_Capella> python -OO build/pymake/make.py -f client.mk
  445. # [19:31] <jhk> and what if I only want to compile accessibility part?
  446. # [19:31] <@davidb> khuey: umm… that sort of defeats the purpose
  447. # [19:32] <@davidb> i've done my experiments… pymake just wouldn't take for me
  448. # [19:32] <@davidb> Mark_Capella: tried that
  449. # [19:32] <Mark_Capella> whats it do?
  450. # [19:32] <@davidb> one thing i haven't tried is changing my mozconfig… but regular make worked :(
  451. # [19:33] <@davidb> Mark_Capella: i'd tell you but a number of different fails are twirling in my head
  452. # [19:33] <Mark_Capella> for mozconfig ... mk_add_options MOZ_OBJDIR=@TOPSRCDIR@/obj
  453. # [19:33] <@davidb> i have that
  454. # [19:34] <@davidb> i've pounded through issues with pymake before but never took this long
  455. # [19:34] <Mark_Capella> where is the mozconfig located? in the repo?
  456. # [19:34] <@davidb> .mozconfig
  457. # [19:34] <Mark_Capella> @topsrcdir@? yah .mozconfig
  458. # [19:35] <@davidb> when was the last time you guys did a clean build?
  459. # [19:35] <@davidb> using pymake
  460. # [19:35] <Mark_Capella> couple days ago had issues
  461. # [19:35] <Mark_Capella> seems theres a build problem known issue
  462. # [19:35] <@davidb> i think khuey fixed that yesterday
  463. # [19:36] <@davidb> and regarding pyc files… i fixed them
  464. # [19:37] <Mark_Capella> :) didnt nwo they were broken whats your situation do when you try?
  465. # [19:37] <@davidb> i'm trying a full clobber again
  466. # [19:37] <Mark_Capella> yah .... 2hours for me
  467. # [19:38] <@davidb> i was using pymake from my objdir… so let's see if from root works better
  468. # [19:38] <@davidb> (always used to use it from objdir)
  469. # [19:38] <Mark_Capella> from root !!!!
  470. # [19:38] <@davidb> Mark_Capella: that's too long
  471. # [19:38] <@davidb> from topsrc
  472. # [19:38] <Mark_Capella> right
  473. # [19:38] <Mark_Capella> repo
  474. # [19:39] <Mark_Capella> has need started from objdir ... my custom batch file switches me into topsrc
  475. # [19:39] <@davidb> fail
  476. # [19:39] <@davidb> Mark_Capella: do you use parallel build switch?
  477. # [19:39] <Mark_Capella> how does it fail? that fast has seveal easy reasons
  478. # [19:41] <@davidb> khuey, Mark_Capella: pastebin.mozilla.org/1562565
  479. # [19:41] <@davidb> http://pastebin.mozilla.org/1562565
  480. # [19:42] <Mark_Capella> thats the known issue i mentioned - restart the run and pray
  481. # [19:42] <@davidb> known? i tried a bug search...
  482. # [19:43] <@davidb> Mark_Capella: 2 hours is too long
  483. # [19:43] <khuey> davidb: pull
  484. # [19:44] <@davidb> khuey: now? i have a trunk for 2 hrs ago
  485. # [19:44] <@davidb> for/from
  486. # [19:44] <khuey> from m-c?
  487. # [19:44] <@davidb> inbound
  488. # [19:44] <khuey> ah
  489. # [19:44] <khuey> the fix may not have made it to inbound ...
  490. # [19:44] <@davidb> crap!
  491. # [19:44] <@davidb> although… my build seems to have gotten further
  492. # [19:45] <khuey> oh, it did
  493. # [19:45] <khuey> at 8:12:35 AM PDT
  494. # [19:45] <khuey> so clearly your pull is more than 2 hours old
  495. # [19:45] <@firebot> nickyekaiqi@gmail.com requested feedback from the wind for attachment 613663 on bug 743654.
  496. # [19:45] <@davidb> yeah i don't hit that particular bug anymore - the one you landed a fix for
  497. # [19:45] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=743654 nor, --, ---, nobody, NEW, replace nsIWinAccessNode on static_cast<nsAccessibleWrap*>
  498. # [19:45] <@davidb> crap
  499. # [19:46] <@davidb> fail
  500. # [19:46] <@davidb> ok pulling
  501. # [19:46] <Mark_Capella> https://bugzilla.mozilla.org/show_bug.cgi?id=741248
  502. # [19:46] <@davidb> yeah i know the bug :)
  503. # [19:46] <@firebot> Bug 741248 cri, --, mozilla14, khuey, RESO FIXED, pymake builds fail in nsIDOMBatteryManager.h or nsIDOMNavigatorBattery.h since the Paris bindings la
  504. # [19:46] <Mark_Capella> same thing?
  505. # [19:46] <@davidb> this time i hit mozIStorageVacuumParticipant.xpt failed
  506. # [19:46] <@davidb> i don't get the dom battery error anymore
  507. # [19:47] * @davidb tries again
  508. # [19:47] <Mark_Capella> what are you mozconfig options? i forgot the parallel builds keywords, i tired and stopped using soething
  509. # [19:47] <@davidb> -j8
  510. # [19:47] <@davidb> -j[n]
  511. # [19:48] <khuey> mk_add_options MOZ_MAKE_FLAGS=-jalot
  512. # [19:48] <@davidb> heh
  513. # [19:48] <Mark_Capella> yep - i dont go like that
  514. # [19:48] <@davidb> although with pymake we need to specify on command line no?
  515. # [19:48] <Mark_Capella> tried using the -j parm
  516. # [19:48] <Mark_Capella> stopped
  517. # [19:48] <khuey> no, you want to stick it in the makefile
  518. # [19:48] <Mark_Capella> in the .mozconfig
  519. # [19:48] <@davidb> khuey: i have it in the makefile for good measure
  520. # [19:48] <khuey> er, yes
  521. # [19:48] <khuey> in the mozconfig
  522. # [19:48] <@davidb> err yes mozconfig
  523. # [19:49] <@davidb> maybe the command line req is gone now
  524. # [19:49] * Joins: drexler (chatzilla@moz-BD26ECCD.hsd1.vt.comcast.net)
  525. # [19:49] * Quits: drexler (chatzilla@moz-BD26ECCD.hsd1.vt.comcast.net) (Quit: ChatZilla 0.9.88-rdmsoft [XULRunner 1.9.0.17/2009122204])
  526. # [19:49] <nickc> Sorry no need for the feedback, it has some error for that bug
  527. # [19:50] <nickc> I forgot to do something
  528. # [19:50] <@davidb> Mark_Capella: it is worth it if you can get parallel builds going
  529. # [19:50] <Mark_Capella> yah - i wanted to get something done so i stopped playing with it
  530. # [19:50] * @davidb nods
  531. # [19:51] <Mark_Capella> but it looks worth it - i tryied -j4 maybe something larger
  532. # [19:51] <@davidb> Mark_Capella: i'll just say that once you've done a full build in less than 12 minutes, you'll never go back
  533. # [19:51] <Mark_Capella> 12 minutes !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
  534. # [19:51] <Mark_Capella> no
  535. # [19:51] <Mark_Capella> friggin
  536. # [19:51] <Mark_Capella> way
  537. # [19:51] <@davidb> well, certainly less than 20
  538. # [19:52] <@davidb> khuey: what's your full clobber build time?
  539. # [19:52] <Mark_Capella> less than 20 on a small change is best i ever get :(
  540. # [19:52] <khuey> davidb: 25-30 minutes on my laptop
  541. # [19:52] <@davidb> khuey: that's not an MBP then
  542. # [19:53] <khuey> no, its not
  543. # [19:53] <@davidb> ok
  544. # [19:53] <@davidb> I'm building on a box these day
  545. # [19:53] <@davidb> s
  546. # [19:53] <khuey> my laptop is good enough usually
  547. # [19:54] <@davidb> yeah
  548. # [19:54] <khuey> except when doing android work
  549. # [19:54] <@davidb> oof yeah
  550. # [19:54] <@davidb> do you have 8 or 16?
  551. # [19:54] <@davidb> probably 8
  552. # [19:57] <@davidb> Mark_Capella: around the time of my first patch… i used to start a compile before bed, then wake up and it would still be going.
  553. # [19:57] <Mark_Capella> :O
  554. # [19:57] <@davidb> Things have improved.
  555. # [19:57] <@hub> just out: http://android-developers.blogspot.ca/2012/04/accessibility-are-you-serving-all-your.html <- eeejay not sure you saw that
  556. # [19:57] <@hub> it is a bit thin
  557. # [19:57] <Mark_Capella> ive got time now - maybe ill try a -j8 and se what happens
  558. # [19:57] <@davidb> go for it
  559. # [19:58] <@davidb> Mark_Capella: how many CPUs do you have?
  560. # [19:58] <Mark_Capella> needs to do a pull anyhow 2 cores
  561. # [19:58] <@davidb> what does Windows Resource Monitor show you?
  562. # [19:58] <@davidb> (Accessories->System Tools)
  563. # [19:59] <Mark_Capella> yah - two of them cpu0 and cpu1
  564. # [19:59] <@davidb> ok
  565. # [19:59] <@davidb> -j2 is enough i think
  566. # [19:59] <@davidb> if that's how it works
  567. # [20:00] <@davidb> it doesn't hurt to specify too many though
  568. # [20:00] <@firebot> nickyekaiqi@gmail.com cancelled feedback? for attachment 613663 on bug 743654.
  569. # [20:00] <@firebot> nickyekaiqi@gmail.com requested feedback from the wind for attachment 613665 on bug 743654.
  570. # [20:00] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=743654 nor, --, ---, nobody, NEW, replace nsIWinAccessNode on static_cast<nsAccessibleWrap*>
  571. # [20:00] <@davidb> hub: i think eeejay is on a plane
  572. # [20:00] <Mark_Capella> actually i thought i read somewhere that you should try to limit it not do -j9999
  573. # [20:01] <@davidb> well that probably causes some upfront churn
  574. # [20:01] <@hub> davidb: good point
  575. # [20:01] <@davidb> my build built!
  576. # [20:01] <@davidb> hub: but still… his phantom is here so nick'ing him makes sense
  577. # [20:01] <Mark_Capella> yah!!
  578. # [20:01] <@davidb> \o/ i can do some real work!
  579. # [20:01] <Mark_Capella> re-run the build when fail .. see that bug above
  580. # [20:01] <@davidb> oh meeting in 7 minutes
  581. # [20:02] <Mark_Capella> was my problem
  582. # [20:02] <@davidb> Mark_Capella: yeah that is my bad
  583. # [20:02] <@davidb> although… could have sworn i tried that
  584. # [20:02] <Mark_Capella> does a little danve - got 5 patchs moved in today
  585. # [20:02] * @davidb claps
  586. # [20:03] <nickc> claps too
  587. # [20:03] <@davidb> those aren't golf claps ;)
  588. # [20:03] <Mark_Capella> personal best - goes to disney world
  589. # [20:03] <@davidb> Mark_Capella: clearly we need to challenge you
  590. # [20:03] <Mark_Capella> thanks his mom .... surkov ... the academy
  591. # [20:04] <Mark_Capella> yah cahllenge ... talk to me after the meeting :)
  592. # [20:05] <@davidb> :)
  593. # [20:08] <Mark_Capella> hub: cool! thanks! That was the weirdest patch ive worked on
  594. # [20:09] <Mark_Capella> i still dont know how half of it works
  595. # [20:09] <Mark_Capella> :O
  596. # [20:12] * Joins: drexler (chatzilla@moz-BD26ECCD.hsd1.vt.comcast.net)
  597. # [20:19] <@firebot> hub@mozilla.com changed the Target Milestone on bug 741707 from --- to mozilla14.
  598. # [20:19] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741707 nor, --, mozilla14, markcapella, ASSI, rm unused a11y CIDs
  599. # [20:19] <@firebot> hub@mozilla.com changed the Target Milestone on bug 741681 from --- to mozilla14.
  600. # [20:19] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741681 nor, --, mozilla14, orangedaylily, NEW, replace nsAccessNode::GetPresContext by nsDocAccessible::PresContext
  601. # [20:20] <@firebot> hub@mozilla.com changed the Target Milestone on bug 741683 from --- to mozilla14.
  602. # [20:20] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741683 nor, --, mozilla14, markcapella, ASSI, move nsAccessNode::ScrollTo to nsCoreUtils
  603. # [20:22] <@hub> Mark_Capella: my pleasure
  604. # [20:27] * Quits: nickc (Mibbit@8A748F18.D0521ABA.428E23B0.IP) (Quit: http://www.mibbit.com ajax IRC Client)
  605. # [20:28] * Quits: jhk (jigneshhk1@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  606. # [20:41] * Joins: jhk (jigneshhk1@8E6C34C1.A3F9767A.1C37C358.IP)
  607. # [20:59] * Quits: drexler (chatzilla@moz-BD26ECCD.hsd1.vt.comcast.net) (Ping timeout)
  608. # [20:59] <@eeejay> yo yo
  609. # [21:01] <@hub> hey eeejay
  610. # [21:03] <@davidb> eeejay: what country are you in?
  611. # [21:03] <@eeejay> davidb, at the gate in israel
  612. # [21:03] <@davidb> ok
  613. # [21:05] * Quits: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp) (Quit: mike5w3c)
  614. # [21:05] * Joins: jkiss (jkiss@5248D1C6.C5B5659C.8732AABF.IP)
  615. # [21:07] * Quits: jhk (jigneshhk1@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  616. # [21:18] * Joins: askalski (akuda@moz-6A36EC49.ip.abpl.pl)
  617. # [21:18] * ChanServ sets mode: +o askalski
  618. # [21:25] * Quits: @jprmc (jprmc@F2D29657.F60B0462.67AC9B1.IP) (Ping timeout)
  619. # [21:26] * Joins: jprmc (jprmc@F2D29657.F60B0462.67AC9B1.IP)
  620. # [21:26] * ChanServ sets mode: +o jprmc
  621. # [21:26] * Joins: drexler (chatzilla@moz-BD26ECCD.hsd1.vt.comcast.net)
  622. # [21:30] <@firebot> eitan@monotonous.org requested review from dbolter@mozilla.com for attachment 613709 on bug 739498.
  623. # [21:30] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=739498 nor, --, ---, eitan, NEW, Add Javascript layer for mobile accessibility
  624. # [21:32] * Joins: jhk (jigneshhk1@8E6C34C1.A3F9767A.1C37C358.IP)
  625. # [21:33] * Quits: jhk (jigneshhk1@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  626. # [21:33] <@davidb> hub: I guess Lavina didn't push to try
  627. # [21:33] * Joins: jhk (jigneshhk1@8E6C34C1.A3F9767A.1C37C358.IP)
  628. # [21:33] <@davidb> (happens)
  629. # [21:34] <@hub> davidb: and I didn'
  630. # [21:34] <@hub> and I didn't wait long enough
  631. # [21:34] <@hub> :-(
  632. # [21:34] <@hub> davidb: I have 3 patches
  633. # [21:34] <@davidb> happens to the best of us
  634. # [21:34] <@hub> actually I believe it is worse than that
  635. # [21:34] <@hub> it is changes colliding
  636. # [21:35] <@davidb> oh
  637. # [21:35] * Quits: nhirata (nhirata.bu@moz-C03D0C61.vlan426.asr1.sfo1.gblx.net) (Quit: nhirata)
  638. # [21:36] <Mark_Capella> build -j2 elapsed time 1:18 still running
  639. # [21:36] <@hub> davidb: I'll push Mark_Capella's patched first
  640. # [21:37] <@firebot> jigneshhk1992@gmail.com requested review from trev.saunders@gmail .com for attachment 613717 on bug 741697.
  641. # [21:37] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741697 nor, --, ---, jigneshhk1992, ASSI, dexpcomify CAccessibleComponent
  642. # [21:37] <@davidb> ok
  643. # [21:37] * Quits: @jprmc (jprmc@F2D29657.F60B0462.67AC9B1.IP) (Ping timeout)
  644. # [21:38] * Quits: jhk (jigneshhk1@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  645. # [21:38] * Joins: jhk (jigneshhk1@8E6C34C1.A3F9767A.1C37C358.IP)
  646. # [21:39] * Joins: jprmc (jprmc@F2D29657.F60B0462.67AC9B1.IP)
  647. # [21:39] * ChanServ sets mode: +o jprmc
  648. # [21:40] * Quits: jhk (jigneshhk1@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  649. # [21:40] * Joins: nhirata (nhirata.bu@moz-C03D0C61.vlan426.asr1.sfo1.gblx.net)
  650. # [21:40] * Joins: jhk (jigneshhk1@8E6C34C1.A3F9767A.1C37C358.IP)
  651. # [21:53] <@hub> davidb: so shall I in that case remove the r+ flag and request the person to fix it?
  652. # [21:53] <Mark_Capella> build -j2 total runtime 1:36 :( >whah<
  653. # [21:54] <@davidb> hub: your call
  654. # [21:59] * Joins: fxa90id (fxa90id@moz-C6D6DA75.neoplus.adsl.tpnet.pl)
  655. # [22:01] * Quits: fxa90id (fxa90id@moz-C6D6DA75.neoplus.adsl.tpnet.pl) (Quit: Leaving)
  656. # [22:08] * Quits: victorporof (victorporo@830D7731.AB163232.79933D60.IP) (Ping timeout)
  657. # [22:09] * Joins: andym (andym@3A0929D2.B9EE6B50.1ECA502.IP)
  658. # [22:09] <drexler> Mark_Capella: i was pushing 5 hrs + with my core solo + 1GB RAM...the dark days...:(
  659. # [22:10] <andym> looking at web site design and worried a little bit about the font / colour choice, that could be a problem for some users
  660. # [22:10] <andym> is that something i should worry about and who would be the best to ask about?
  661. # [22:21] <@firebot> New Core - Disability Access APIs bug 744144 filed by markcapella@twcny.rr.com.
  662. # [22:21] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=744144 nor, --, ---, nobody, NEW, Create / Use a heuristic for @longdesc where no valid UIR exists
  663. # [22:22] * Quits: jkiss (jkiss@5248D1C6.C5B5659C.8732AABF.IP) (Client exited)
  664. # [22:23] <@davidb> andym: yeah we need to track that
  665. # [22:23] <@davidb> andym: you could file bugs, I think satdav is the one to ping
  666. # [22:23] <andym> davidb: it feels rather subjective so it's hard for me to say
  667. # [22:23] <@davidb> andym: there are some analyzers out there
  668. # [22:25] <@davidb> andym: you could also ask for community help here https://lists.mozilla.org/listinfo/accessibility
  669. # [22:25] <@davidb> lots of industry experts would probably jump in
  670. # [22:25] <@davidb> failing that i can help you
  671. # [22:25] <@davidb> but i need to juggle priorities
  672. # [22:26] <andym> ok, i'lll see how it goes with the designs
  673. # [22:26] <andym> thanks
  674. # [22:26] <@davidb> anytime
  675. # [22:27] <@davidb> andym: there is a contrast analyzer addon
  676. # [22:27] <andym> https://addons.mozilla.org/en-US/firefox/addon/juicy-studio-accessibility-too/ ?
  677. # [22:28] <@davidb> maybe - i can't recall for sure
  678. # [22:28] <andym> that gives the site pretty bad ratings :)
  679. # [22:28] <andym> cool
  680. # [22:28] <@davidb> andym: are you in webdev?
  681. # [22:29] <andym> yes
  682. # [22:29] <@davidb> we need to be upfront about a11y as part of Ux
  683. # [22:29] * Joins: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp)
  684. # [22:29] <andym> we do, i've filed bugs on this the past and they haven't gone far
  685. # [22:29] <@hub> andym: bug #?
  686. # [22:29] <andym> i'd like to push it in the marketplace, which i'm working on
  687. # [22:29] <@davidb> ok you have my support
  688. # [22:30] <andym> cool, it's more getting people to do it
  689. # [22:30] <@davidb> my/our
  690. # [22:30] <@davidb> yes
  691. # [22:30] <@davidb> i can help with that
  692. # [22:30] <andym> https://bugzilla.mozilla.org/show_bug.cgi?id=686499
  693. # [22:30] <@firebot> Bug 686499 nor, --, ---, nobody, NEW, Let's be accessible
  694. # [22:30] <andym> gosh thats old
  695. # [22:31] * @davidb comments
  696. # [22:32] * Quits: @askalski (akuda@moz-6A36EC49.ip.abpl.pl) (Quit: Wychodzi)
  697. # [22:33] <@hub> davidb: maybe we should make sure that sort of bug gets CC to us
  698. # [22:33] <@hub> or something
  699. # [22:34] <@davidb> hub: I just added the access keyword
  700. # [22:34] <@davidb> I'd like to see a Mozilla community group gather around this.
  701. # [22:35] <andym> i'd like to see webdev be part of that
  702. # [22:36] <@davidb> andym: I like that you said that :)
  703. # [22:36] <@davidb> andym: maybe talk to your team, and if you like you can help us do it via the mailing list
  704. # [22:36] <@davidb> you can drive it
  705. # [22:36] * Joins: askalski (akuda@moz-6A36EC49.ip.abpl.pl)
  706. # [22:36] * ChanServ sets mode: +o askalski
  707. # [22:36] <andym> there's a webdev off site at the end of the month
  708. # [22:37] <@davidb> but definitely connect with satdav
  709. # [22:37] <@davidb> andym: where?
  710. # [22:37] <andym> santa clara,
  711. # [22:37] <andym> maybe we can get someone to come over talk and start to kick things off
  712. # [22:38] <@davidb> yeah
  713. # [22:38] <@davidb> andym: can you email me the dates
  714. # [22:38] <@davidb> or a link
  715. # [22:38] <@davidb> or ping me here with that if you can before i pack up :)
  716. # [22:39] <andym> https://intranet.mozilla.org/Webdev:Offsite
  717. # [22:39] <@davidb> andym: you might ask around on your team if anyone cows WCAG
  718. # [22:39] <@davidb> cows/knows
  719. # [22:39] <andym> Santa Cruz even
  720. # [22:39] <@davidb> yeah tempting :)
  721. # [22:40] <@davidb> ok i gotta run
  722. # [22:40] <@davidb> should be back on line tonight (in ~4 hours) or tomorrow
  723. # [22:41] * Quits: @davidb (davidb@F2D29657.F60B0462.67AC9B1.IP) (Quit: davidb)
  724. # [22:43] * Quits: @jprmc (jprmc@F2D29657.F60B0462.67AC9B1.IP) (Ping timeout)
  725. # [23:03] * Quits: drexler (chatzilla@moz-BD26ECCD.hsd1.vt.comcast.net) (Quit: ChatZilla 0.9.88-rdmsoft [XULRunner 1.9.0.17/2009122204])
  726. # [23:11] * Joins: jprmc (jprmc@F2D29657.F60B0462.67AC9B1.IP)
  727. # [23:11] * ChanServ sets mode: +o jprmc
  728. # [23:31] * tbsaunde|afk is now known as tbsaunde
  729. # Session Close: Wed Apr 11 00:00:01 2012

The end :)