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

Options:

  1. # Session Start: Mon Apr 09 00:00:00 2012
  2. # Session Ident: #accessibility
  3. # [00:13] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  4. # [00:14] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  5. # [00:58] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  6. # [01:55] * Quits: nhirata (nhirata.bu@moz-585D5EFC.hsd1.ca.comcast.net) (Quit: nhirata)
  7. # [02:07] * Mark_Capella is now known as Mark_Capella|nearby
  8. # [02:35] * Mark_Capella|nearby is now known as Mark_Capella
  9. # [04:25] * Joins: surkov (surkov@B9538DF7.B7015738.33A1AC3C.IP)
  10. # [04:25] * ChanServ sets mode: +o surkov
  11. # [04:32] * khuey is now known as khuey|away
  12. # [04:32] * khuey|away is now known as khuey
  13. # [05:10] * Quits: brambles (brambles@4CBAB088.F3076E90.1822ACA6.IP) (Client exited)
  14. # [06:01] <@firebot> surkov.alexander@gmail.com requested review from trev.saunders@gmail .com for attachment 613222 on bug 740758.
  15. # [06:01] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740758 nor, --, ---, markcapella, ASSI, dexpcom nsAccessible::GetValue
  16. # [06:30] <Mark_Capella> surkov: thanks for putting that GetValue patch to bed :)
  17. # [06:30] <@surkov> Mark_Capella: yw!
  18. # [06:32] <@firebot> surkov.alexander@gmail.com granted feedback for attachment 613168 on bug 742695.
  19. # [06:32] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=742695 nor, --, ---, markcapella, ASSI, densify nsARIAGrid
  20. # [06:39] <Mark_Capella> surkov: You want me to touch up bug742695 densify nsARIAGrid and repost? Or did you intent to finish that one also?
  21. # [06:40] <@surkov> Mark_Capella: I'll finish it
  22. # [06:40] <Mark_Capella> ok thanks again ! I wasnt sure from the email / comments
  23. # [06:40] <@surkov> yw, go ahead on something else :)
  24. # [06:40] <Mark_Capella> GetName() ....
  25. # [06:41] <@surkov> ok, cool but keep in mind that you might need to do merging
  26. # [06:41] <@surkov> since these are big patches
  27. # [06:41] <@surkov> maybe you could do something small until I land those
  28. # [06:41] <Mark_Capella> Ok, ummm ... lemme check status
  29. # [06:42] <@surkov> Mark_Capella: what os do you work on?
  30. # [06:42] <Mark_Capella> WIN7 VC2010 Express
  31. # [06:42] <@surkov> k
  32. # [06:45] <Mark_Capella> surkov: did you notice or chat with tbsaunde about bug 741683 move ScrollTo() frpm nsAccessNode?
  33. # [06:45] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741683 nor, --, ---, markcapella, ASSI, move nsAccessNode::ScrollTo to nsCoreUtils
  34. # [06:45] <@surkov> Mark_Capella: I'll comment it
  35. # [06:45] <@surkov> actually I'm on it (my next item)
  36. # [06:45] <Mark_Capella> K - and we're waiting on review+ for @longdesc
  37. # [06:46] <Mark_Capella> (checking sattus here)
  38. # [06:47] <Mark_Capella> Was a quiet weekend - everyone on holiday
  39. # [06:53] <@firebot> surkov.alexander@gmail.com granted feedback for attachment 613167 on bug 741683.
  40. # [06:53] <@surkov> yep
  41. # [06:54] <@firebot> markcapella@twcny.rr.com changed the Status on bug 741703 from NEW to ASSIGNED.
  42. # [06:54] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741703 nor, --, ---, markcapella, ASSI, stop using QueryInterface in CAccessibleHypertext
  43. # [06:54] <@surkov> Mark_Capella: btw, I don't recall if I asked but when you upload new patch then please mark old one as obsolete and move all review requests from old patch to new one
  44. # [06:55] <Mark_Capella> "move review requests" ??? I can do that?
  45. # [06:55] <@firebot> surkov.alexander@gmail.com requested review from jonas@sicking.cc for attachment 612470 on bug 652635.
  46. # [06:55] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=652635 nor, --, ---, markcapella, ASSI, Fallback missing @longdesc to aria-describedby pointing to <a href>
  47. # [06:55] * Quits: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com) (Client exited)
  48. # [06:55] <@surkov> Mark_Capella: just ask them again on new patch
  49. # [06:56] <@surkov> Mark_Capella: when you mark patch as obsolete then all review requests are canceled
  50. # [06:56] <@firebot> surkov.alexander@gmail.com cancelled review?(jonas@sicking.cc) for attachment 612460 on bug 652635.
  51. # [06:57] <Mark_Capella> ok ... how does this work ... I get a feedback+ with a request to do a fix ... so i post the new pacth, obsolete the old one, and ask for feedback again?
  52. # [07:05] <@surkov> Mark_Capella: that depends how do you feel about the patch, if you think the requestee should look at it again then yes, if no then no
  53. # [07:07] * khuey is now known as khuey|away
  54. # [07:09] <Mark_Capella> Ok, tbsaunde was talking to me about it last night also ... i guess the important part is to make the old
  55. # [07:09] <Mark_Capella> patch obsolete ... I think he asked me to make sure to ask for feedback again so its easier for him to notice
  56. # [07:09] <Mark_Capella> outstanding requests / cases where he needs to pay attention
  57. # [08:02] * Quits: @hub (hub@moz-E2FCA694.figuiere.net) (Ping timeout)
  58. # [08:21] <Mark_Capella> surkov: in other areas of the browser, they have me change IID/CID's when I change header files, is this something
  59. # [08:21] <Mark_Capella> that should be done in a11y also?
  60. # [08:22] <@surkov> well, for interfaces yes (idl files), for other things I don't think it's necessary
  61. # [08:22] <@surkov> because all these other things are considered for internal usage
  62. # [08:23] <Mark_Capella> .idl files ... ok ... was wondering ... !
  63. # [08:29] <@firebot> surkov.alexander@gmail.com granted feedback for attachment 613191 on bug 741701.
  64. # [08:30] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741701 nor, --, ---, markcapella, ASSI, CAccessibleHyperlink::get_anchor shouldn't use nsIWinAccessNode
  65. # [08:43] <@firebot> markcapella@twcny.rr.com requested feedback from trev.saunders@gma il.com for attachment 613237 on bug 741703.
  66. # [08:43] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741703 nor, --, ---, markcapella, ASSI, stop using QueryInterface in CAccessibleHypertext
  67. # [09:05] <Mark_Capella> fyi the autoland - to - TRY process has problems ... it's been down since Friday late ...
  68. # [09:06] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  69. # [09:06] <@tbsaunde> Mark_Capella: you can just hg push
  70. # [09:07] * @tbsaunde doesn't really get what people like about autoland
  71. # [09:07] <Mark_Capella> yah ... I thought it might cleared up by now, and I wasnt in a hurry
  72. # [09:08] <Mark_Capella> I got used to it cause I could do a pushh after a feedback+ before I got my L1 access
  73. # [09:09] <@surkov> hi, tbsaunde
  74. # [09:10] <@tbsaunde> hey surkov
  75. # [09:10] <@tbsaunde> surkov: what's up?
  76. # [09:10] <@surkov> tbsaunde: just hi
  77. # [09:10] <@tbsaunde> ah
  78. # [09:10] <@surkov> tbsaunde: and perhaps review for bug 740758 so I can land it
  79. # [09:10] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740758 nor, --, ---, markcapella, ASSI, dexpcom nsAccessible::GetValue
  80. # [09:12] <@tbsaunde> yeah, going through mail now
  81. # [09:37] <@firebot> surkov.alexander@gmail.com granted feedback for attachment 613074 on bug 741681.
  82. # [09:37] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741681 nor, --, ---, orangedaylily, NEW, replace nsAccessNode::GetPresContext by nsDocAccessible::PresContext
  83. # [09:43] <@firebot> markcapella@twcny.rr.com requested review from hub@mozilla.com for attachment 613245 on bug 741683.
  84. # [09:43] <@tbsaunde> surkov: are the xforms people ever going to update there extenssion? or are we just going to keep making sure the xforms code builds when nobody can possibly use it?
  85. # [09:43] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741683 nor, --, ---, markcapella, ASSI, move nsAccessNode::ScrollTo to nsCoreUtils
  86. # [09:43] * Joins: victorporof (victorporo@BD5B5B26.602BD6C.79933D60.IP)
  87. # [09:44] <@surkov> tbsaunde: 1) I think they are 2) yes
  88. # [09:45] <@tbsaunde> surkov: ok
  89. # [09:45] <@tbsaunde> there should have been a forever in 2 fwiw
  90. # [09:56] * Quits: victorporof (victorporo@BD5B5B26.602BD6C.79933D60.IP) (Connection reset by peer)
  91. # [10:26] <@surkov> tbsaunde: wouldn't you like to answer question in bug 740764?
  92. # [10:26] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740764 nor, --, ---, nickyekaiqi, NEW, merge GetAttributes and GetAttributesInternal
  93. # [10:30] <@firebot> surkov.alexander@gmail.com requested review from trev.saunders@gmail .com for attachment 613249 on bug 539683.
  94. # [10:30] <@firebot> surkov.alexander@gmail.com changed the Status on bug 539683 from NEW to ASSIGNED.
  95. # [10:30] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=539683 nor, --, ---, nobody, NEW, defunct object methods should return CO_E_OBJECTNOTCONNECTED
  96. # [10:33] * Joins: victorporof (victorporo@BD5B5B26.602BD6C.79933D60.IP)
  97. # [10:35] <@surkov> tbsaunde: try server for bug 740758 is finished, need you r to land it
  98. # [10:35] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740758 nor, --, ---, markcapella, ASSI, dexpcom nsAccessible::GetValue
  99. # [10:36] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  100. # [11:04] <Mark_Capella> surkov: I can help Ye with 740764 ... hes on #introduction chatting about it with me
  101. # [11:09] <@tbsaunde> surkov: ok, was about half doen will finish up now
  102. # [11:09] <@tbsaunde> went to have breakfast and started talking to some people
  103. # [11:10] <@tbsaunde> surkov: I think you still need to check the return values in ProgressMeterAccessible::GetValue() for GetMaximumValue() / GetMinimumValue()
  104. # [11:11] <Mark_Capella> nope ... if fails safe - i checked
  105. # [11:12] <@tbsaunde> Mark_Capella: explain what you mean by that
  106. # [11:13] <Mark_Capella> well .. the two values are preloaded with zeroes, if either or both calls fail, both return either a 0 or a 100% answer
  107. # [11:13] <@tbsaunde> Mark_Capella: why don't you wnat to add back NS_ENSURE_SUCCESS(rv, ); ?
  108. # [11:13] <Mark_Capella> it wont build returns a value
  109. # [11:13] <Mark_Capella> from a void function
  110. # [11:14] <@tbsaunde> your not returning a vlaue
  111. # [11:14] <@tbsaunde> we use NS_ENSURE_SUCCESS(rv, ); in other void functions
  112. # [11:14] <Mark_Capella> your returning a 0 or 1 100% as the the Value()
  113. # [11:15] <@tbsaunde> Mark_Capella: no, you aren't
  114. # [11:15] <Mark_Capella> i only changed it because the build failed originally, research showed the macro tries to retrun a value ... i can double check for you
  115. # [11:16] <@tbsaunde> Mark_Capella: the macro returns its second argument, if the second argument is empty it ends up being return ;
  116. # [11:16] <@tbsaunde> which is fine
  117. # [11:17] <@tbsaunde> Mark_Capella: for an example of the craziness you have to check and be sure is never introduced if you take your path consider nsForControlAccessible.cpp:152
  118. # [11:23] <Mark_Capella> tbsaunde: thats just ugly code!
  119. # [11:24] <@tbsaunde> Mark_Capella: what is?
  120. # [11:25] <Mark_Capella> :152 in ProgressMeter :)
  121. # [11:25] <@surkov> Mark_Capella: ok, joined
  122. # [11:26] <@tbsaunde> Mark_Capella: sort of maybe, but tell me what the value will be after the call to GetMaximumValue() that takes the failure path there is
  123. # [11:27] <Mark_Capella> in that case you get what you started with for the aMaximumValue ... is that what you ask?
  124. # [11:29] <@tbsaunde> Mark_Capella: are you sure that's true? you get whatever nsString::ToDouble() returns if it fails to parse its input as a string
  125. # [11:29] <@surkov> tbsaunde: ok, also it makes add 0 check for maxvalue
  126. # [11:29] <@tbsaunde> surkov: I don't understand you
  127. # [11:30] <@surkov> tbsaunde: otherwise we can get 0 / 0 which is a crash
  128. # [11:31] <Mark_Capella> right surkov: I ran it through my tine c compiler
  129. # [11:31] <Mark_Capella> failssafe
  130. # [11:31] <@tbsaunde> surkov: well, NAN would be bad to
  131. # [11:32] <@surkov> tbsaunde: here your suggestion will work
  132. # [11:33] <@tbsaunde> surkov: so, to be clear you agree with NS_ENSURE_SUCCESS(rv, ); after GetMaxValue();?
  133. # [11:34] <@surkov> tbsaunde: perhaps if (NS_FAILED(rv)) return;
  134. # [11:34] <@surkov> we don't need a warning if values were incorrect
  135. # [11:36] <@tbsaunde> surkov: I can imageine cases where the warning would be nice, but I can also see cases where verything is fine but the html is busted
  136. # [11:36] <@surkov> yes
  137. # [11:37] <Mark_Capella> gigo
  138. # [11:37] <@surkov> we don't control HTML on the web page so warning is useless
  139. # [11:38] <@tbsaunde> surkov: sure, but in other cases something could actually be broken, in which the warning is useful
  140. # [11:39] <@tbsaunde> but if (NS_FAILED(rv)) return; is at least not broken so if you really prefer it sure
  141. # [11:39] <@tbsaunde> I'd just go for NS_ENSURE_SUCCESS since its the same behavior as before
  142. # [11:41] <@surkov> tbsaunde: ok, if you like I don't mind
  143. # [11:41] <Mark_Capella> build failed for me that way - only reason i changed it - maybe itss a WIN7 thing
  144. # [11:43] <@surkov> Mark_Capella: build failed for NS_ENSURE_SUCCESS(rv, );?
  145. # [11:44] <Mark_Capella> yes
  146. # [11:44] <Mark_Capella> hey i make mistakes want me to try again?
  147. # [11:45] <@surkov> nope
  148. # [11:45] <@surkov> we use this pattern in code already
  149. # [11:45] <@surkov> it should be ok
  150. # [11:46] <Mark_Capella> is that what is was to begin with? before we changed the functino to return a void?
  151. # [11:46] <Mark_Capella> damn let me chec my patch
  152. # [11:48] <Mark_Capella> nah ... it was NS_ENSURE_SUCCESS(rv, rv) thats not the same as what you're talking about
  153. # [11:50] <@firebot> trev.saunders@gmail.com granted review for attachment 613222 on bug 740758.
  154. # [11:50] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740758 nor, --, ---, markcapella, ASSI, dexpcom nsAccessible::GetValue
  155. # [11:51] <@tbsaunde> surkov: up to you I think
  156. # [11:52] <@surkov> ok
  157. # [11:58] <@firebot> surkov.alexander@gmail.com changed the Target Milestone on bug 740758 from --- to mozilla14.
  158. # [12:15] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  159. # [12:16] * Joins: fxa90id (fxa90id@moz-883BBD1B.neoplus.adsl.tpnet.pl)
  160. # [12:16] <fxa90id> hi
  161. # [12:17] <@tbsaunde> hi
  162. # [12:23] * Joins: sgimeno (santiago.g@A922400D.48362F18.BCC03455.IP)
  163. # [12:25] * Quits: icaaq1 (Adium@moz-7D09279.cust.bredbandsbolaget.se) (Ping timeout)
  164. # [12:27] <@tbsaunde> surkov: in theory you should probably null check the result of those do_QueryObject()s but I can't see how they'd fail and they should go away soon so I'm not sure it matters...
  165. # [12:27] * Joins: icaaq (Adium@moz-7D09279.cust.bredbandsbolaget.se)
  166. # [12:28] <fxa90id> surkov: hi
  167. # [12:28] <@surkov> tbsaunde: yes, and especially those calls gets away
  168. # [12:28] <@surkov> hi, fxa90id
  169. # [12:28] <@firebot> surkov.alexander@gmail.com changed the Target Milestone on bug 742695 from --- to mozilla14.
  170. # [12:28] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=742695 nor, --, mozilla14, markcapella, ASSI, densify nsARIAGrid
  171. # [12:28] <fxa90id> but what should I do with this createHTMLTableRowAccessible ?
  172. # [12:29] <fxa90id> in 727722 if I good remember number
  173. # [12:29] <@tbsaunde> surkov: I assume you meant if (!IsLink()) return s-FALSE; ?
  174. # [12:29] <@surkov> tbsaunde: yes
  175. # [12:29] <fxa90id> anyway pm me later ;p
  176. # [12:30] <fxa90id> brb
  177. # [12:30] * Quits: fxa90id (fxa90id@moz-883BBD1B.neoplus.adsl.tpnet.pl) (Quit: leaving)
  178. # [12:31] <@tbsaunde> surkov: ok, since its incorrect and we don't test that I feel like somebody should probably look it over just to be careful, but I really don't have a lot of interest in reading that patch twice
  179. # [12:33] <@surkov> tbsaunde: what does "look it over" mean?
  180. # [12:34] <@surkov> and what's incorrect?
  181. # [12:35] <@tbsaunde> surkov: only thing I see so far that's incorrect is if (IsLink()
  182. # [12:36] <@surkov> tbsaunde: I said that I fixed it locally?
  183. # [12:37] <@tbsaunde> surkov: yes, but I mean to double check you got them all
  184. # [12:37] <@surkov> tbsaunde: you can trust me
  185. # [12:37] <@surkov> :)
  186. # [12:37] <@tbsaunde> surkov: well, if you say so
  187. # [12:37] <@surkov> :)
  188. # [12:37] <@tbsaunde> I trust you, but I also think your a human :)
  189. # [12:40] <@surkov> that's right, but you don't want to read it twice, I'd like to not upload the patch twice if no serious changes
  190. # [12:41] <@surkov> actually up to you since you you are reviewer
  191. # [12:42] <@tbsaunde> surkov: sure, if you've checked I think I'm fine with it once I finish reading
  192. # [12:43] <@surkov> ok
  193. # [12:45] <@tbsaunde> surkov: sort of funny part is we can go back to null checks when msaa objects are seperate
  194. # [12:45] <@surkov> tbsaunde: yep
  195. # [12:46] <@surkov> but we need those defunct and it'd be nice to follow spec
  196. # [12:46] <Mark_Capella> omigod the sky is falling
  197. # [12:46] <@surkov> Mark_Capella: why?
  198. # [12:46] <Mark_Capella> laughs
  199. # [12:46] <Mark_Capella> [rivate joke ill share it later :)
  200. # [12:47] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  201. # [12:47] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  202. # [12:48] <@firebot> surkov.alexander@gmail.com changed the Target Milestone on bug 741701 from --- to mozilla14.
  203. # [12:48] * Joins: fxa90id (fxa90id@moz-883BBD1B.neoplus.adsl.tpnet.pl)
  204. # [12:48] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741701 nor, --, mozilla14, markcapella, ASSI, CAccessibleHyperlink::get_anchor shouldn't use nsIWinAccessNode
  205. # [12:51] <@tbsaunde> surkov: should we check that xpAccessible isn't defunct in nsAccessibleWrap methods like get_accValue()?
  206. # [12:52] <@surkov> tbsaunde: mm, yeah we should :)
  207. # [12:54] <@surkov> tbsaunde: not sure about CO_E_OBJNOTCONNECTED though in this case
  208. # [12:55] * Quits: icaaq (Adium@moz-7D09279.cust.bredbandsbolaget.se) (Ping timeout)
  209. # [12:55] <@tbsaunde> surkov: I'm not sure without reminding myself of the spec
  210. # [12:56] <@tbsaunde> but my feeling is ifthe accessible they ask about is not "connected" meaning its dead then we should return CO_E_OBJECTNOTCONNECTED
  211. # [12:56] <@surkov> yep
  212. # [12:57] <@surkov> I don't see anything in the spec
  213. # [12:58] <@tbsaunde> surkov: ok, have a reason not to use CO_E_OBJECTNOTCONNECTED?
  214. # [12:58] <@surkov> no reason
  215. # [13:04] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  216. # [13:05] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  217. # [13:07] * Joins: icaaq (Adium@moz-7D09279.cust.bredbandsbolaget.se)
  218. # [13:09] <@firebot> trev.saunders@gmail.com granted review for attachment 613249 on bug 539683.
  219. # [13:09] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=539683 nor, --, ---, surkov.alexander, ASSI, defunct object methods should return CO_E_OBJECTNOTCONNECTED
  220. # [13:10] <@surkov> tbsaunde: can you turn off caps lock please
  221. # [13:10] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  222. # [13:10] <@tbsaunde> surkov: yeah, can try, oops
  223. # [13:10] <@surkov> thx!
  224. # [13:14] <Mark_Capella> woo hoo! all sorts of stuff moving this morning :)
  225. # [13:16] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  226. # [13:18] <@firebot> trev.saunders@gmail.com granted feedback for attachment 613237 on bug 741703.
  227. # [13:18] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741703 nor, --, ---, markcapella, ASSI, stop using QueryInterface in CAccessibleHypertext
  228. # [13:21] * Quits: fxa90id (fxa90id@moz-883BBD1B.neoplus.adsl.tpnet.pl) (Ping timeout)
  229. # [13:24] <@firebot> surkov.alexander@gmail.com granted review for attachment 613237 on bug 741703.
  230. # [13:24] <@surkov> Mark_Capella: yep, cool work
  231. # [13:28] <@firebot> surkov.alexander@gmail.com cancelled review+ for attachment 613237 on bug 741703.
  232. # [13:36] * Quits: victorporof (victorporo@BD5B5B26.602BD6C.79933D60.IP) (Quit: Linkinus - http://linkinus.com)
  233. # [13:51] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  234. # [14:02] * Mark_Capella is now known as Mark_Capella|nearby
  235. # [14:04] <@tbsaunde> surkov: so, you only want to expose IAccessibleHypertext when we expose nsIAccessibleHypertext? iirc you said before AsHypertext() was good enough to replace QI to nsIAccessibleText / EditableText
  236. # [14:08] <@tbsaunde> surkov: I don't see why we bother check for roles::{GRAPHIC,IMAGEMAP} since it doesn't seem like aria provides way to make something have that role
  237. # [14:09] <@tbsaunde> for the others why do we create HypertextAccessible for them instead a better type of object?
  238. # [14:35] <icaaq> Hi, a question about the video/audio elements, why is javascript necessary for keyboardnavigation?
  239. # [14:37] <@surkov> tbsaunde: ARIA removes native semantics
  240. # [14:38] <@surkov> icaaq: perhaps it's better to ask on #developers
  241. # [14:38] <icaaq> ok, thanks
  242. # [14:38] <@surkov> tbsaunde: it sounds the best way to not create nsHyperTextAccessible for those roles at all
  243. # [14:39] <@surkov> at least in case of plain hypertext
  244. # [14:41] <@tbsaunde> surkov: I'd tend to agree with not creating plain hypertext for them
  245. # [14:42] <@tbsaunde> surkov: I meant while aria may remove native sumantics we get to choose how we implement that
  246. # [14:42] <@surkov> tbsaunde: but that should be tricky because ARIA doesn't always remove native semantics :)
  247. # [14:42] <@tbsaunde> surkov: true
  248. # [14:43] <@tbsaunde> surkov: so, I gues what we need to figure out is how to handle things like <div role="progressbar"><p>some text</p></div>
  249. # [14:43] <@tbsaunde> how does that show on screen btw?
  250. # [14:44] <@surkov> tbsaunde: yep, but I have a feeling that what ARIA requires is not always sane
  251. # [14:44] <@tbsaunde> surkov: ok, I haven't though about this much, but probably should
  252. # [14:44] <@surkov> so for this bug I'd go with having a function like AreTextInterfacesImplemented
  253. # [14:45] <@tbsaunde> surkov: yeah, that seems fine call it IsText()?
  254. # [14:45] <@tbsaunde> or maybe HasText()
  255. # [14:46] <@surkov> iirc we have IsText in nsAccUtils with different meaning
  256. # [14:46] <@surkov> HasText is too close to it
  257. # [14:46] <@surkov> maybe IsTextRole()
  258. # [14:46] * Joins: peteb-away (ptbrunet@moz-E9B02845.austin.res.rr.com)
  259. # [14:47] <@tbsaunde> surkov: kind of funny, but ok
  260. # [14:47] <@surkov> :)
  261. # [14:50] <@tbsaunde> surkov: well, I wonder if we could change nsAccUtils::IsText() to do what we want?
  262. # [14:50] <@surkov> without looking into details it'd be dangerous change
  263. # [14:51] <@tbsaunde> surkov: sure
  264. # [14:51] <@tbsaunde> surkov: but maybe nice thing to consider for future cleanup
  265. # [14:52] <@surkov> tbsaunde: as you can see nsAccUtils::IsText is about text nodes accessible not about hypertext accessible, so it should be tricky
  266. # [14:52] <@tbsaunde> surkov: yeah, true
  267. # [14:57] * Joins: nickc (Mibbit@9D658438.D0521ABA.428E23B0.IP)
  268. # [14:57] <nickc> Anyone here to help?
  269. # [14:57] <nickc> I wanna get some instructions on bug 740764
  270. # [14:57] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=740764 nor, --, ---, nickyekaiqi, NEW, merge GetAttributes and GetAttributesInternal
  271. # [14:57] <nickc> firebot is good!
  272. # [14:58] <@firebot> nickc: Sorry, I've no idea what 'is good' might be.
  273. # [14:59] <nickc> Why I cannot find generic folder which is the match in mxr?
  274. # [14:59] <nickc> http://mxr.mozilla.org/mozilla-central/ident?i=GetAttributesInternal&;filter=
  275. # [15:00] <@tbsaunde> nickc: not sure I understand the question
  276. # [15:02] * tbsaunde is now known as tbsaunde|afk
  277. # [15:06] <nickc> In the first match in that mxr url, which is /accessible/src/generic/OuterDocAccessible.h (View Hg log or Hg annotations) 20:57 nickc I cannot find generic folder in my repo
  278. # [15:12] * Joins: davidb (davidb@F2D29657.F60B0462.67AC9B1.IP)
  279. # [15:12] * ChanServ sets mode: +qo davidb davidb
  280. # [15:15] <@firebot> New Core - Disability Access APIs bug 743654 filed by surkov.alexander@gmail.com.
  281. # [15:15] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=743654 nor, --, ---, nobody, NEW, replace nsIWinAccessNode on static_cast<nsAccessibleWrap*>
  282. # [15:15] <@surkov> nickc: do you have updated repo? (that was recent changes)
  283. # [15:16] <nickc> Yes. I just updated it.
  284. # [15:16] <nickc> After I updated it, it seems that it is not changed.
  285. # [15:16] <@surkov> I wonder if yo had any local changes that prevented your from update
  286. # [15:16] <nickc> So I delete the obj and build everything
  287. # [15:17] <nickc> Correct local change is also allowable?
  288. # [15:18] <nickc> Since I fixed some bug and changed some files.
  289. # [15:18] <nickc> But they are actually correct.
  290. # [15:18] <nickc> And the build do not have any error
  291. # [15:21] <@surkov> nickc: if you fixed bugs (the patches were landed) then you can revert changes and then update your tree
  292. # [15:21] * Joins: victorporof (victorporo@BD5B5B26.602BD6C.79933D60.IP)
  293. # [15:22] <@surkov> if some patches weren't landed or you're going to hack on some of them then it's best way to import them into queue
  294. # [15:22] <@surkov> https://developer.mozilla.org/en/Mercurial_Queues
  295. # [15:23] <nickc> Yes I used MQ
  296. # [15:24] <@surkov> so you need to do qpop -a
  297. # [15:24] <@surkov> and if you don't have uncommitted local changes
  298. # [15:24] <@surkov> then your queue is clean
  299. # [15:24] <@surkov> then you can do hg update
  300. # [15:25] <@surkov> and then to make sure you're on trunk hg pull -u again
  301. # [15:25] <@surkov> if it doesn't pop you changes then you should do hg qref to commit changes into current patch
  302. # [15:25] <@surkov> if that's unwanted then you can create new patch
  303. # [15:25] <@surkov> and commit changes there
  304. # [15:26] <nickc> I will try, but actually I have rebuilt everything after deleting obj directory. So still need to update?
  305. # [15:26] <nickc> I did the building just now
  306. # [15:27] <@surkov> nickc: build and source update are different things
  307. # [15:27] <@surkov> the problem you didn't update your sources if I understand correct
  308. # [15:29] <nickc> I understood it now. Thank you!
  309. # [15:29] <@surkov> yw!
  310. # [15:35] * Joins: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
  311. # [15:36] * Joins: jimm (jmathies@moz-7F164CA1.pn.at.cox.net)
  312. # [15:44] * Quits: nickc (Mibbit@9D658438.D0521ABA.428E23B0.IP) (Quit: http://www.mibbit.com ajax IRC Client)
  313. # [15:49] * Joins: ehsan (ehsan@F2D29657.F60B0462.67AC9B1.IP)
  314. # [16:04] * Quits: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp) (Quit: mike5w3c)
  315. # [16:08] * clown is now known as clown_mtg
  316. # [16:09] * Quits: victorporof (victorporo@BD5B5B26.602BD6C.79933D60.IP) (Connection reset by peer)
  317. # [16:19] * Joins: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp)
  318. # [16:21] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  319. # [16:31] * Joins: victorporof (victorporo@BD5B5B26.602BD6C.79933D60.IP)
  320. # [16:40] <@tbsaunde|afk> davidb: have a chance to look at that talos try push I did?
  321. # [16:41] * tbsaunde|afk is now known as tbsaunde
  322. # [16:42] <@tbsaunde> jimm: I saw you had some questions with nsIAccessibleEvent for some reason friday? resolve them?
  323. # [16:42] <@davidb> tbsaunde: do you have the link handy?
  324. # [16:43] <jimm> tbsaunde: I was looking for a way to recognize the accessible associated with a particular event
  325. # [16:43] <@tbsaunde> davidb: one sec
  326. # [16:43] <@tbsaunde> jimm: event should have a property that was the target
  327. # [16:44] <@davidb> surkov: do you know who is going to uplift bug 729831? (to ff 13)
  328. # [16:44] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=729831 cri, --, mozilla13, surkov.alexander, RESO FIXED, Crash @ mozilla::a11y::StyleInfo::Margin
  329. # [16:44] <@tbsaunde> jimm: also fyi eventually we should end up using AccEvent since it should be faster
  330. # [16:44] <@tbsaunde> probably by doing what you need to do in nsAccessibleWrap::HandleAccEvent()
  331. # [16:44] <@surkov> davidb: usually MarcoZ do that
  332. # [16:44] <@davidb> yeah
  333. # [16:44] <@davidb> surkov: I'll forward the email to him
  334. # [16:45] <@surkov> ok, cool
  335. # [16:45] <jimm> tbsaunde: I was looking for some sort of unique id on the nsIAccessible. so for example, say I get a focus event on a text input element, then I get a focus lost event. internally in my code I needed a way to identify these two events as being associated with the same element.
  336. # [16:46] <@tbsaunde> davidb: it was rev 00d90cc9da61 so tbpl.mozilla.org/?tree=try&rev=00d90cc9da61 probably
  337. # [16:46] <jimm> I ended up using nsIPersistentProperties attributes
  338. # [16:46] <@davidb> tbsaunde: ok thanks
  339. # [16:46] <@tbsaunde> jimm: ok, I see
  340. # [16:49] <@tbsaunde> surkov: isn't it a little anoying how much static_cast<nsAccessibleWrap*>(foo) we're starting to have? ideas?
  341. # [16:49] <@surkov> tbsaunde: yeah but can't see a better way now
  342. # [16:49] <@tbsaunde> ok
  343. # [16:50] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Ping timeout)
  344. # [16:51] <@davidb> tbsaunde: 00d90cc9da61 doesn't seem to work
  345. # [16:52] <@davidb> tbsaunde: Neil might have ideas
  346. # [16:52] * Joins: hub (hub@moz-E2FCA694.figuiere.net)
  347. # [16:52] * ChanServ sets mode: +o hub
  348. # [16:53] <@tbsaunde> davidb: hm, interesting
  349. # [16:54] <@firebot> surkov.alexander@gmail.com changed the Target Milestone on bug 539683 from --- to mozilla14.
  350. # [16:54] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=539683 nor, --, mozilla14, surkov.alexander, ASSI, defunct object methods should return CO_E_OBJECTNOTCONNECTED
  351. # [16:55] <@tbsaunde> davidb: what about searching for me as the pusher? I don't remember the exact syntax, but think its pusher=email
  352. # [16:56] <@davidb> tbsaunde: yeah i'll do that
  353. # [16:56] <@davidb> tbsaunde: was it your most recent push?
  354. # [16:57] <@davidb> nope
  355. # [16:57] <@davidb> oh
  356. # [16:59] * Joins: jprmc (jprmc@F2D29657.F60B0462.67AC9B1.IP)
  357. # [16:59] * ChanServ sets mode: +o jprmc
  358. # [17:00] <@davidb> tbsaunde: the search is all out of order. do you recall any of the summary?
  359. # [17:00] <@tbsaunde> davidb: yeah, most recent
  360. # [17:01] <@tbsaunde> davidb: probably just try: -b o -t all or something
  361. # [17:02] <@davidb> wow this Mark_Capella guy does a lot of try pushing :)
  362. # [17:03] <@firebot> New Core - Disability Access APIs bug 743676 filed by surkov.alexander@gmail.com.
  363. # [17:03] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=743676 nor, --, ---, nobody, NEW, densify base/nsFormControlAccessible
  364. # [17:04] <@surkov> davidb: because he's producing big patches
  365. # [17:04] <@davidb> surkov: indeed
  366. # [17:05] * Joins: arky (arky@558EFADF.6A6C9B09.FA662B63.IP)
  367. # [17:06] * Quits: a-865 (fmcz@moz-A5D13CA.cable.mindspring.com) (Quit: ChatZilla 0.9.88.1 [SeaMonkey 2.9/20120329002549])
  368. # [17:07] <@surkov> Mark_Capella|nearby: btw, you should cancel try server build if you aren't interested in them
  369. # [17:07] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  370. # [17:07] <Mark_Capella|nearby> hmmm ... ok lemme look
  371. # [17:07] * Mark_Capella|nearby is now known as Mark_Capella
  372. # [17:19] * khuey|away is now known as khuey
  373. # [17:21] <Mark_Capella> I do a lot? I did did only two over the weekend
  374. # [17:21] <@firebot> New Core - Disability Access APIs bug 743680 filed by surkov.alexander@gmail.com.
  375. # [17:21] <@firebot> surkov.alexander@gmail.com requested review from trev.saunders@gmail .com for attachment 613293 on bug 743680.
  376. # [17:22] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=743680 nor, --, ---, surkov.alexander, ASSI, don't export nsARIAMap.h
  377. # [17:22] <@surkov> Mark_Capella: yeah I landed that GetValue patch but you still have try server builds on it
  378. # [17:22] <Mark_Capella> hows that? I only pushed it once
  379. # [17:23] <Mark_Capella> and I just killed the only one i know about
  380. # [17:24] <Mark_Capella> surkov: TRY server was >mucked< up this weekend ... if theres something wrong that needs
  381. # [17:24] <Mark_Capella> cleanup still let me know
  382. # [17:25] <@tbsaunde> davidb: if you can't find that push I can write the patch again and push, the machine I wrote it on is hopefully breifly dead, so I'd need to write it again, but that should be easy
  383. # [17:25] <@surkov> Mark_Capella: it appears this one is still running https://tbpl.mozilla.org/?tree=Try&rev=2a51ac9edfb2
  384. # [17:25] * Quits: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp) (Quit: mike5w3c)
  385. # [17:26] <@surkov> oh but that one is different
  386. # [17:26] <@davidb> tbsaunde: ok yeah, sorry I can't find anything for https://tbpl.mozilla.org/?tree=Try&pusher=trev.saunders@gmail.com
  387. # [17:26] <@surkov> anyway it failed
  388. # [17:26] <Mark_Capella> yah different patch for waldo ... wonder why its dying
  389. # [17:26] <@surkov> but is still continue to build
  390. # [17:27] <@tbsaunde> davidb: ok
  391. # [17:28] * Joins: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp)
  392. # [17:32] <@tbsaunde> surkov: I'm curious any reason not to deal with the rest of the stuff we don't need to export?
  393. # [17:32] * Joins: nhirata (nhirata.bu@moz-C03D0C61.vlan426.asr1.sfo1.gblx.net)
  394. # [17:32] <Mark_Capella> (waldo can figure that one out later)
  395. # [17:32] <@surkov> tbsaunde: like what?
  396. # [17:33] <@surkov> we have a lot of stuff we shouldn't export I think
  397. # [17:35] <@tbsaunde> surkov: nsXULTreeGridAccessible{,Wrap}.h
  398. # [17:35] <@tbsaunde> nsHTMLTable{,Wrap}.h
  399. # [17:35] <@surkov> yeah, you wanted to see that in one patch?
  400. # [17:36] <@tbsaunde> surkov: not necessary, but curious
  401. # [17:36] <@surkov> actually that nsARIAMap.h was interested for me because of new patch for inline functions, that's new for our module I wanted to try it
  402. # [17:36] <@surkov> to see how it goes
  403. # [17:36] <@surkov> we need to stop other exports
  404. # [17:37] <@surkov> we can do that as good first bugs I guess
  405. # [17:37] <@surkov> but I'm not sure what is priority is
  406. # [17:37] <@tbsaunde> surkov: ok, I haven't actually looked at the patch yet
  407. # [17:37] <@surkov> k
  408. # [17:39] <@tbsaunde> surkov: I tend to think its not a huge priority, just makes things a little nicer if nobody outside of us could wnat to use it
  409. # [17:39] <@surkov> right
  410. # [17:40] <@surkov> so the first thing is dexpcomination and then this one :)
  411. # [17:40] * @davidb hears the word priority :)
  412. # [17:40] <@davidb> https://wiki.mozilla.org/Platform/2012-Q2-Goals#Accessibility
  413. # [17:41] <@davidb> I do like all the clean up - especially as it helps goals.
  414. # [17:42] <@surkov> yeah but it would be mistake if we stop any activities for anything than isn't mapped into goals
  415. # [17:42] <@davidb> agreed
  416. # [17:42] <@davidb> we can't only do goals.
  417. # [17:43] <@davidb> surkov: I recall wondering why we exported so much. I guess this was the gecko pattern back in the day?
  418. # [17:43] <@davidb> "back in the day" means a long time ago
  419. # [17:43] <@surkov> davidb: perhaps we just didn't care
  420. # [17:43] <@surkov> we didn't care about many things in the past
  421. # [17:43] <@davidb> hrm
  422. # [17:44] <@davidb> we can care more now.
  423. # [17:44] <Mark_Capella> davidb: do you have tracking bugs for each priority to refer work back to?
  424. # [17:44] <@surkov> we do
  425. # [17:44] <@davidb> Mark_Capella: not for all of them
  426. # [17:44] <@davidb> I was discussing this with tbsaunde earlier...
  427. # [17:44] <@davidb> we can use whiteboards or tracker bugs
  428. # [17:44] <Mark_Capella> yah like that !
  429. # [17:45] <@davidb> the trickiest ones are the perf goal and the mac goal
  430. # [17:45] <@davidb> tbsaunde is trying to get us a baseline for the perf goal. but it we also need a specific metric for speech rec usage
  431. # [17:46] <@davidb> the mac goal is hard since it will be a judgement call.
  432. # [17:46] <@davidb> we can measure accuracy, but the perf is dependant on VoiceOver
  433. # [17:46] <@tbsaunde> surkov: so, I that patch seems fine and nice, but I don't understand why you need to do that reorg
  434. # [17:47] <@surkov> tbsaunde: which reorg?
  435. # [17:47] <@surkov> putting inlines into separate file?
  436. # [17:47] <Mark_Capella> davidb: are there current whiteboard tags? Can I scan for <foo priority #> to see all related work for example?
  437. # [17:48] <@davidb> Mark_Capella: not yet.
  438. # [17:48] <@surkov> if yes then it's nice mechanism to stop you saying well maybe we won't do that as inline because I don't want to include that header, it shouldn't be big perf hit after all :)
  439. # [17:49] <@tbsaunde> surkov: yeah, I think its a good idea, just confused why its on a bug to stop exporting some headers
  440. # [17:50] <@firebot> hub@mozilla.com denied review for attachment 613074 on bug 741681.
  441. # [17:50] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=741681 nor, --, ---, orangedaylily, NEW, replace nsAccessNode::GetPresContext by nsDocAccessible::PresContext
  442. # [17:50] <@surkov> tbsaunde: perhaps it's not a perfect name for the bug
  443. # [17:50] <@surkov> but that's what I did finally
  444. # [17:50] * Joins: a-865 (fmcz@moz-A5D13CA.cable.mindspring.com)
  445. # [17:50] <@davidb> Mark_Capella: we'll have something in a couple of days
  446. # [17:50] <@tbsaunde> surkov: sure
  447. # [17:50] <@davidb> Mark_Capella: goals get locked in Friday-ish.
  448. # [17:51] * @davidb steps out
  449. # [17:51] <@tbsaunde> surkov: btw we still export the nsARIAGridAccessibleWrap.h's but they can't be used since we don't export nsARIAGridAccessible.h
  450. # [17:51] <@surkov> nice
  451. # [17:51] <@surkov> we need to fix that
  452. # [17:52] <@tbsaunde> surkov: I mena that's what the state is after your patch
  453. # [17:52] <@tbsaunde> can you stop exporting them as well as ARIAGridAccessible.h ?
  454. # [17:53] <@surkov> ok
  455. # [17:53] <@surkov> tbsaunde: just to make sure you talk about bug 742695 now?
  456. # [17:53] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=742695 nor, --, mozilla14, markcapella, ASSI, densify nsARIAGridAccessible
  457. # [17:54] <@tbsaunde> surkov: no, bug 743680 the one you wrote a patch for
  458. # [17:54] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=743680 nor, --, ---, surkov.alexander, ASSI, don't export nsARIAMap.h
  459. # [17:55] <@surkov> tbsaunde: I don't follow how it's related with nsARIAGridAccessible.h exports
  460. # [17:56] <@tbsaunde> surkov: in your patch you stopped exporting nsARIAGridAcccessible.h
  461. # [17:56] <@tbsaunde> but you didn't stop exporting nsARIAGridAccessibleWrap.h
  462. # [17:56] <@tbsaunde> which we still export after the patch but is completely useless
  463. # [17:56] <@surkov> really?
  464. # [17:56] <@surkov> I don't see related changes in Makefile.in
  465. # [17:57] <@tbsaunde> surkov: the point is there isn't changes
  466. # [17:57] <@surkov> sorry but I don't follow you
  467. # [17:57] <@tbsaunde> you should have removed nsAccessibleWrap.h from {atk,msaa,mac,other}/Makefile.in
  468. # [17:57] * Quits: arky (arky@558EFADF.6A6C9B09.FA662B63.IP) (Quit: Leaving)
  469. # [17:58] <@surkov> why?
  470. # [17:58] <@tbsaunde> err, nsARIAGridAccessibleWrap.h
  471. # [17:58] <@surkov> why?
  472. # [17:58] <@tbsaunde> well, nobody external can include it now
  473. # [17:58] <@tbsaunde> so why not?
  474. # [17:59] <Mark_Capella> must ..... focus ....
  475. # [17:59] <Mark_Capella> and my coffees all gone :(
  476. # [17:59] <@surkov> I don't understand why do you talk about nsARIAGridAccessible stuff since I don't understand how it's related with my patch at all
  477. # [17:59] <@surkov> if we still talk about bug 743680
  478. # [18:00] <@surkov> tbsaunde: ^
  479. # [18:01] <@tbsaunde> surkov: never mind I think I'm being a complete clown
  480. # [18:01] <@surkov> fuf, I thought I've lost my mind :)
  481. # [18:01] <clown_mtg> tbsaunde: masquarding as me?
  482. # [18:01] * clown_mtg is now known as clown
  483. # [18:03] <@tbsaunde> surkov: heh
  484. # [18:09] <@tbsaunde> surkov: that patch also makes a lot more sense to me now that I have the file name right
  485. # [18:09] <@surkov> ok, cool
  486. # [18:12] * Quits: @surkov (surkov@B9538DF7.B7015738.33A1AC3C.IP) (Quit: surkov)
  487. # [18:24] * jimm is now known as jimm-lunch
  488. # [18:25] * ctalbert|afk is now known as ctalbert
  489. # [18:53] * Joins: askalski (akuda@moz-6A36EC49.ip.abpl.pl)
  490. # [18:53] * ChanServ sets mode: +o askalski
  491. # [18:53] <@askalski> hi everyone
  492. # [18:53] * khuey is now known as specquoter
  493. # [18:54] <@firebot> eitan@monotonous.org requested review from surkov.alexander@gm ail.com for attachment 613325 on bug 739498.
  494. # [18:54] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=739498 nor, --, ---, eitan, NEW, Add Javascript layer for mobile accessibility
  495. # [18:55] * specquoter is now known as khuey
  496. # [18:55] * Quits: @hub (hub@moz-E2FCA694.figuiere.net) (Ping timeout)
  497. # [19:01] * Joins: icaaq1 (Adium@moz-7D09279.cust.bredbandsbolaget.se)
  498. # [19:01] * Quits: icaaq (Adium@moz-7D09279.cust.bredbandsbolaget.se) (Ping timeout)
  499. # [19:04] * jimm-lunch is now known as jimm
  500. # [19:10] <@davidb> hmmm do I keep hitting bug 729752 and has the workaround changed?
  501. # [19:10] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=729752 nor, --, ---, nobody, REOP, Old .pyc files in srcdir shouldn't break the build
  502. # [19:11] <Mark_Capella> eeejay: was peeking at your change ---^ how recent is your makefile? it loses generic library I added recently
  503. # [19:11] <@eeejay> Mark_Capella, dang. i should rebase it
  504. # [19:11] <Mark_Capella> i would have turned up there :)
  505. # [19:11] <Mark_Capella> it
  506. # [19:11] <@eeejay> Mark_Capella, i'll rebase and resubmit
  507. # [19:12] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  508. # [19:13] <Mark_Capella> cool B) And im learning a lot from your api ...
  509. # [19:14] <@eeejay> Mark_Capella, awesome :)
  510. # [19:14] * @eeejay learned a lot from it too
  511. # [19:18] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  512. # [19:22] <@firebot> eitan@monotonous.org cancelled review?(surkov.alexander@gm ail.com) for attachment 613325 on bug 739498.
  513. # [19:23] <@firebot> eitan@monotonous.org requested review from surkov.alexander@gm ail.com for attachment 613332 on bug 739498.
  514. # [19:23] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=739498 nor, --, ---, eitan, NEW, Add Javascript layer for mobile accessibility
  515. # [19:23] <@eeejay> i think i am going to clear ccache and do a long build before heading to out...
  516. # [19:25] * Quits: @askalski (akuda@moz-6A36EC49.ip.abpl.pl) (Quit: Wychodzi)
  517. # [19:29] * Joins: hub (hub@21B7B9F2.B87E9213.6E712CE2.IP)
  518. # [19:29] * ChanServ sets mode: +o hub
  519. # [19:39] <@hub> davidb: vidyo? or are you busy?
  520. # [19:40] <@davidb> hub: i'm already on :)
  521. # [19:43] * Quits: sgimeno (santiago.g@A922400D.48362F18.BCC03455.IP) (Quit: KVIrc 4.0.2 Insomnia http://www.kvirc.net/)
  522. # [19:54] <@firebot> jonas@sicking.cc granted review for attachment 612470 on bug 652635.
  523. # [19:54] <@firebot> Bug https://bugzilla.mozilla.org/show_bug.cgi?id=652635 nor, --, ---, markcapella, ASSI, Fallback missing @longdesc to aria-describedby pointing to <a href>
  524. # [20:00] * Joins: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP)
  525. # [20:00] * Quits: clown (clown@67828CC7.C1A51174.9D42CF23.IP) (Quit: Leaving.)
  526. # [20:04] * khuey is now known as khuey|away
  527. # [20:09] <@eeejay> did i get time zones mixed up? is the project meeting now?
  528. # [20:09] <@eeejay> nope, i see davidb :)
  529. # [20:09] <@davidb> my apologies
  530. # [20:10] <@eeejay> :)
  531. # [20:11] <@hub> it is now
  532. # [20:11] <@hub> eeejay: always 11 pacific time :-)
  533. # [20:12] <@eeejay> hub, i am in tel-aviv time (until wednesday)
  534. # [20:15] <@hub> eeejay: oh. ok.
  535. # [20:16] <@tbsaunde> davidb: repeatedly hitting that error on the same tree sounds very strange
  536. # [20:16] <@tbsaunde> unless you build very old stuff in that tree a lot
  537. # [20:16] <@davidb> naw
  538. # [20:16] <@davidb> but yes, strange.
  539. # [20:30] * Quits: mike5w3c (MikeS@moz-DAFE1A45.tokyo.ocn.ne.jp) (Quit: mike5w3c)
  540. # [20:32] <@davidb> eeejay: how would you describe "_pendingDocuments"?
  541. # [20:33] <@eeejay> davidb, documents that have been created but don't have accessible trees yet
  542. # [20:33] <@davidb> eeejay: thanks
  543. # [20:34] <@davidb> eeejay: does browserApp.selectedBrowser refer to a single tab document?
  544. # [20:34] <@eeejay> davidb, yeah
  545. # [20:34] <@davidb> ty
  546. # [20:34] <@eeejay> davidb, browser has like 3 different meetings in the chrome
  547. # [20:35] <@eeejay> meanings
  548. # [20:35] <@davidb> :/
  549. # [20:35] <@davidb> yeah
  550. # [20:37] <@davidb> eeejay: i like that you've forecast multiple presenters
  551. # [20:37] <@davidb> do we have only one currently?
  552. # [20:37] <@eeejay> yeah, it'll happen
  553. # [20:37] * @davidb should read comments
  554. # [20:37] <@davidb> eeejay: yeah
  555. # [20:37] <@eeejay> davidb, no, the visual ring is one. and android a11y events is another
  556. # [20:37] * @davidb nods
  557. # [20:44] <@eeejay> .. and then there is the android dummy one that is useful for desktop.
  558. # [20:47] * khuey|away is now known as khuey
  559. # [21:09] * Joins: clown (clown@67828CC7.C1A51174.9D42CF23.IP)
  560. # [21:39] * Joins: fxa90id (fxa90id@moz-AA497BE3.neoplus.adsl.tpnet.pl)
  561. # [21:48] * Quits: @davidb (davidb@F2D29657.F60B0462.67AC9B1.IP) (Quit: blast off!)
  562. # [22:07] * Joins: davidb (davidb@F2D29657.F60B0462.67AC9B1.IP)
  563. # [22:07] * ChanServ sets mode: +qo davidb davidb
  564. # [22:07] * Quits: nhirata (nhirata.bu@moz-C03D0C61.vlan426.asr1.sfo1.gblx.net) (Quit: nhirata)
  565. # [22:11] * Joins: nhirata (nhirata.bu@moz-C03D0C61.vlan426.asr1.sfo1.gblx.net)
  566. # [22:20] * Quits: @jprmc (jprmc@F2D29657.F60B0462.67AC9B1.IP) (Ping timeout)
  567. # [22:24] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Quit: )
  568. # [22:25] * Joins: jprmc (jprmc@F2D29657.F60B0462.67AC9B1.IP)
  569. # [22:25] * ChanServ sets mode: +o jprmc
  570. # [22:32] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  571. # [22:32] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  572. # [22:32] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  573. # [22:32] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  574. # [22:32] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  575. # [22:32] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  576. # [22:34] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  577. # [22:34] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  578. # [22:34] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  579. # [22:34] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  580. # [22:34] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  581. # [22:34] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  582. # [22:36] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  583. # [22:36] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  584. # [22:36] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  585. # [22:36] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  586. # [22:36] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  587. # [22:36] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  588. # [22:37] <@davidb> this day… got in my way
  589. # [22:38] * Quits: @davidb (davidb@F2D29657.F60B0462.67AC9B1.IP) (Quit: davidb)
  590. # [22:39] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  591. # [22:39] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  592. # [22:39] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  593. # [22:39] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  594. # [22:39] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  595. # [22:39] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  596. # [22:40] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  597. # [22:40] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  598. # [22:40] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  599. # [22:40] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  600. # [22:41] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  601. # [22:41] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  602. # [22:41] * khuey is now known as khuey|away
  603. # [22:41] <fxa90id> :<
  604. # [22:45] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  605. # [22:45] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  606. # [22:45] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  607. # [22:45] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  608. # [22:45] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  609. # [22:45] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  610. # [22:46] * Quits: fxa90id (fxa90id@moz-AA497BE3.neoplus.adsl.tpnet.pl) (Quit: Leaving)
  611. # [22:47] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  612. # [22:47] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  613. # [22:47] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  614. # [22:47] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  615. # [22:48] * Joins: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP)
  616. # [22:48] * Quits: Radium (carbon@82CA301.D4EC6E64.E0A835D5.IP) (Max SendQ exceeded)
  617. # [22:48] * Quits: nhirata (nhirata.bu@moz-C03D0C61.vlan426.asr1.sfo1.gblx.net) (Quit: nhirata)
  618. # [22:49] * Joins: nhirata (nhirata.bu@moz-C03D0C61.vlan426.asr1.sfo1.gblx.net)
  619. # [22:53] * khuey|away is now known as khuey
  620. # [22:57] * Quits: jhk (jiggy@8E6C34C1.A3F9767A.1C37C358.IP) (Connection reset by peer)
  621. # [23:04] * Joins: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP)
  622. # [23:04] * Quits: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP) (Max SendQ exceeded)
  623. # [23:04] * Joins: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP)
  624. # [23:04] * Quits: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP) (Max SendQ exceeded)
  625. # [23:05] * Joins: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP)
  626. # [23:05] * Quits: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP) (Max SendQ exceeded)
  627. # [23:06] * Joins: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP)
  628. # [23:06] * Quits: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP) (Max SendQ exceeded)
  629. # [23:08] * Joins: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP)
  630. # [23:08] * Quits: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP) (Max SendQ exceeded)
  631. # [23:08] * Joins: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP)
  632. # [23:08] * Quits: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP) (Max SendQ exceeded)
  633. # [23:08] * Joins: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP)
  634. # [23:08] * Quits: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP) (Max SendQ exceeded)
  635. # [23:09] * Joins: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP)
  636. # [23:09] * Quits: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP) (Max SendQ exceeded)
  637. # [23:10] * Joins: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP)
  638. # [23:10] * Quits: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP) (Max SendQ exceeded)
  639. # [23:10] * Quits: clown (clown@67828CC7.C1A51174.9D42CF23.IP) (Quit: Leaving.)
  640. # [23:11] * Joins: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP)
  641. # [23:11] * Quits: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP) (Max SendQ exceeded)
  642. # [23:11] * Joins: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP)
  643. # [23:11] * Quits: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP) (Max SendQ exceeded)
  644. # [23:12] * Joins: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP)
  645. # [23:12] * Quits: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP) (Max SendQ exceeded)
  646. # [23:12] * Joins: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP)
  647. # [23:12] * Quits: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP) (Max SendQ exceeded)
  648. # [23:13] * Joins: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP)
  649. # [23:13] * Quits: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP) (Max SendQ exceeded)
  650. # [23:14] * Joins: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP)
  651. # [23:14] * Quits: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP) (Max SendQ exceeded)
  652. # [23:14] * Joins: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP)
  653. # [23:14] * Quits: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP) (Max SendQ exceeded)
  654. # [23:15] * Joins: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP)
  655. # [23:15] * Quits: Radium (carbon@A238DEBD.D6671F7E.E0A835D5.IP) (Max SendQ exceeded)
  656. # [23:35] * khuey is now known as khuey|away
  657. # [23:36] * khuey|away is now known as khuey
  658. # Session Close: Tue Apr 10 00:00:00 2012

The end :)