/irc-logs / w3c / #testing / 2015-12-01 / end

Options:

Previous day, Next day

  1. # Session Start: Tue Dec 01 00:00:00 2015
  2. # Session Ident: #testing
  3. # [00:17] * Joins: zcorpan (~zcorpan@public.cloak)
  4. # [00:26] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  5. # [00:43] * Quits: simonstewart (~textual@public.cloak) ("My Mac has gone to sleep. ZZZzzz…")
  6. # [00:44] * Joins: simonstewart (~textual@public.cloak)
  7. # [00:48] * Joins: hober (~ted@public.cloak)
  8. # [00:51] * Quits: simonstewart (~textual@public.cloak) (Ping timeout: 180 seconds)
  9. # [00:54] * Quits: bhill2 (~bhill2@public.cloak) ("Leaving...")
  10. # [01:19] * Joins: zcorpan (~zcorpan@public.cloak)
  11. # [01:27] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  12. # [01:38] * Quits: plh (plehegar@public.cloak) ("Leaving")
  13. # [01:41] * Joins: gitbot (~gitbot@public.cloak)
  14. # [01:41] -gitbot:#testing- [web-platform-tests] inexorabletash opened pull request #2390: Correct assertion when passing undefined as second argument to IDBFactory.open (master...indexeddb-open-undefined) https://github.com/w3c/web-platform-tests/pull/2390
  15. # [01:41] * Parts: gitbot (~gitbot@public.cloak)
  16. # [02:20] * Joins: zcorpan (~zcorpan@public.cloak)
  17. # [02:24] * terri is now known as terri_offline
  18. # [02:27] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  19. # [03:21] * Joins: zcorpan (~zcorpan@public.cloak)
  20. # [03:28] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  21. # [04:22] * Joins: zcorpan (~zcorpan@public.cloak)
  22. # [04:29] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  23. # [04:51] * Joins: fwtnb (~fwtnb@public.cloak)
  24. # [05:22] * Joins: zcorpan (~zcorpan@public.cloak)
  25. # [05:30] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  26. # [05:47] * Quits: fwtnb (~fwtnb@public.cloak) (Client closed connection)
  27. # [06:21] * Joins: zcorpan (~zcorpan@public.cloak)
  28. # [06:40] * Joins: gitbot (~gitbot@public.cloak)
  29. # [06:40] -gitbot:#testing- [web-platform-tests] zcorpan closed pull request #2377: Use actual files in meta refresh tests instead of 404s (master...meta-refresh-tweaks) https://github.com/w3c/web-platform-tests/pull/2377
  30. # [06:40] * Parts: gitbot (~gitbot@public.cloak)
  31. # [06:40] * Joins: gitbot (~gitbot@public.cloak)
  32. # [06:40] -gitbot:#testing- [web-platform-tests] zcorpan pushed 2 new commits to master: https://github.com/w3c/web-platform-tests/compare/7dda9a13574b...3df13e82fb39
  33. # [06:40] -gitbot:#testing- web-platform-tests/master 9755531 Simon Pieters: Use actual files in meta refresh tests instead of 404s...
  34. # [06:40] -gitbot:#testing- web-platform-tests/master 3df13e8 Simon Pieters: Merge pull request #2377 from zcorpan/meta-refresh-tweaks...
  35. # [06:40] * Parts: gitbot (~gitbot@public.cloak)
  36. # [06:42] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  37. # [07:02] * Quits: lmclister______ (~sid13822@public.cloak) (Client closed connection)
  38. # [07:03] * Quits: timeless (~sid4015@public.cloak) (Client closed connection)
  39. # [07:03] * Quits: koji (~sid53200@public.cloak) (Client closed connection)
  40. # [07:03] * Quits: sangwhan (~sid12645@public.cloak) (Client closed connection)
  41. # [07:03] * Quits: mkwst (~sid395@public.cloak) (Client closed connection)
  42. # [07:03] * Quits: tobie (~sid5692@public.cloak) (Client closed connection)
  43. # [07:03] * Quits: krit (~sid15081@public.cloak) (Client closed connection)
  44. # [07:03] * Quits: Martijnc (~sid38880@public.cloak) (Client closed connection)
  45. # [07:03] * Quits: majidvp (~sid96638@public.cloak) (Client closed connection)
  46. # [07:03] * Quits: frewsxcv (~sid86219@public.cloak) (Client closed connection)
  47. # [07:03] * Quits: birtles (~sid16523@public.cloak) (Client closed connection)
  48. # [07:03] * Quits: scheib (~sid4467@public.cloak) (Client closed connection)
  49. # [07:04] * Joins: koji (~sid53200@public.cloak)
  50. # [07:04] * Joins: Martijnc (~sid38880@public.cloak)
  51. # [07:04] * Joins: birtles (~sid16523@public.cloak)
  52. # [07:04] * Joins: sangwhan (~sid12645@public.cloak)
  53. # [07:04] * Joins: scheib (~sid4467@public.cloak)
  54. # [07:04] * Joins: mkwst (~sid395@public.cloak)
  55. # [07:04] * Joins: majidvp (~sid96638@public.cloak)
  56. # [07:04] * Joins: krit (~sid15081@public.cloak)
  57. # [07:04] * Joins: frewsxcv (~sid86219@public.cloak)
  58. # [07:04] * Joins: lmclister______ (~sid13822@public.cloak)
  59. # [07:04] * Joins: timeless (~sid4015@public.cloak)
  60. # [07:04] * Joins: tobie (~sid5692@public.cloak)
  61. # [07:42] * Joins: zcorpan (~zcorpan@public.cloak)
  62. # [07:50] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  63. # [08:34] * Joins: fwtnb (~fwtnb@public.cloak)
  64. # [08:34] * Quits: fwtnb (~fwtnb@public.cloak) (Client closed connection)
  65. # [08:35] * Joins: fwtnb (~fwtnb@public.cloak)
  66. # [08:39] * Joins: fwtnb_ (~fwtnb@public.cloak)
  67. # [08:40] * Quits: fwtnb_ (~fwtnb@public.cloak) (Client closed connection)
  68. # [08:40] * Joins: fwtnb_ (~fwtnb@public.cloak)
  69. # [08:43] * Joins: zcorpan (~zcorpan@public.cloak)
  70. # [08:45] * Quits: fwtnb (~fwtnb@public.cloak) (Ping timeout: 180 seconds)
  71. # [08:50] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  72. # [08:53] * Joins: dom (dom@public.cloak)
  73. # [08:53] * Joins: zcorpan (~zcorpan@public.cloak)
  74. # [08:56] * Joins: rego (~rego@public.cloak)
  75. # [09:19] * Joins: Ms2ger (~Ms2ger@public.cloak)
  76. # [09:58] * Quits: fwtnb_ (~fwtnb@public.cloak) (Client closed connection)
  77. # [11:09] * Joins: gitbot (~gitbot@public.cloak)
  78. # [11:09] -gitbot:#testing- [csswg-test] Ms2ger opened pull request #964: Various cssom improvements. (master...cssom) https://github.com/w3c/csswg-test/pull/964
  79. # [11:09] * Parts: gitbot (~gitbot@public.cloak)
  80. # [11:39] * Quits: MikeSmith (~MikeSmith@public.cloak) ("leaving")
  81. # [11:39] * Joins: MikeSmith (~MikeSmith@public.cloak)
  82. # [11:39] <Ms2ger> Oh, fremy is at MS now?
  83. # [11:45] <MikeSmith> yeah has been for a year or more now I think
  84. # [11:48] <Ms2ger> Shows what I know
  85. # [11:49] <MikeSmith> well I reckon I wouldn't have known either except he told me the one time I met him f2f
  86. # [11:54] * Quits: rego (~rego@public.cloak) (Ping timeout: 180 seconds)
  87. # [11:59] * Joins: simonstewart (~textual@public.cloak)
  88. # [12:11] * Joins: rego (~rego@public.cloak)
  89. # [12:42] * Joins: fwtnb (~fwtnb@public.cloak)
  90. # [12:51] * Quits: simonstewart (~textual@public.cloak) ("Textual IRC Client: www.textualapp.com")
  91. # [12:55] * Joins: simonstewart (~textual@public.cloak)
  92. # [13:09] * Quits: simonstewart (~textual@public.cloak) ("My Mac has gone to sleep. ZZZzzz…")
  93. # [14:23] * Joins: simonstewart (~textual@public.cloak)
  94. # [15:01] * Joins: gitbot (~gitbot@public.cloak)
  95. # [15:01] -gitbot:#testing- [web-platform-tests] Ms2ger opened pull request #2392: Rewrite importScripts tests. (master...importScripts) https://github.com/w3c/web-platform-tests/pull/2392
  96. # [15:01] * Parts: gitbot (~gitbot@public.cloak)
  97. # [15:08] <mkwst> jgraham: Could you explain the difference between `add_result_callback` and `add_cleanup`?
  98. # [15:09] * Joins: BuRn (~BuRn@public.cloak)
  99. # [15:09] <BuRn> oh crap... RUN!!!
  100. # [15:09] <BuRn> ^ÂÊÎÔÛâêîôûĈĉĜĝĤĥĴĵŜŝŴŵŶŷˆ̭̂᷍ḒḓḘḙḼḽṊṋṰṱṶṷẐẑẤấẦầẨẩẪẫẬậẾếỀềỂểỄễỆệỐốỒồỔổỖỗỘộ⨣⨶⩯ꞈ＾󠁞
  101. # [15:10] * Parts: BuRn (~BuRn@public.cloak)
  102. # [15:17] <jgraham> mkwst: Functionallu not much. Cleanup runs slightly later. The main difference is that cleanup is intended for running post-test cleanup actions (like "teardown" functions)
  103. # [15:20] <mkwst> jgraham: Ok. So in this serial-execution thing, you'd like `test.add_cleanup` to deal with the teardown step, and `add_result_callback` to deal with executing the next test?
  104. # [15:20] <jgraham> mkwst: Yeah, that seems more natural
  105. # [15:20] <mkwst> Ok. That makes sense.
  106. # [15:21] <jgraham> As long as you spin the event loop before starting the next test so that the cleanup actually runs in time
  107. # [15:21] <mkwst> I also didn't follow the comment about relying on the test being single-step. I don't think that's the case... Will upload a new patch in a moment with different variable names that might clear things up. Or maybe I'm just misinterpreting everything. :)
  108. # [15:21] <jgraham> mkwst: It looked to me like you created the actual test as part of your helper and then called t.step() on a provided function
  109. # [15:22] <jgraham> I guess that function could itself add more steps
  110. # [15:22] * Joins: gitbot (~gitbot@public.cloak)
  111. # [15:22] -gitbot:#testing- [web-platform-tests] mikewest pushed 1 new commit to mkwst/draft-west-leave-secure-cookies-alone: https://github.com/w3c/web-platform-tests/commit/3612491cfc9082671ea1940c20828a45e828893c
  112. # [15:22] -gitbot:#testing- web-platform-tests/mkwst/draft-west-leave-secure-cookies-alone 3612491 Mike West: fixup @jgraham's comments.
  113. # [15:22] * Parts: gitbot (~gitbot@public.cloak)
  114. # [15:22] <jgraham> I didn't mean that the callback setup forced it to be single-step
  115. # [15:22] <mkwst> I guess the idea is that the function provided would have been the body of a call to `async_test()`
  116. # [15:23] <jgraham> Right, my point point is that you don't already have the return value of that call
  117. # [15:23] <jgraham> But that's perhaps not a big limitation
  118. # [15:24] <mkwst> What limitation do you see there?
  119. # [15:25] <jgraham> Well I just mean normally you can do t = async_test(function() {}); onload=t.step_func(function() {}) [...]. In this case you can't because you don't have a handle to t
  120. # [15:28] <mkwst> I see. That might be a limitation, sure. I guess we'd have to know what other use cases folks have for serialized tests.
  121. # [15:29] * Joins: plh (plehegar@public.cloak)
  122. # [15:32] * Joins: gitbot (~gitbot@public.cloak)
  123. # [15:32] -gitbot:#testing- [web-platform-tests] mikewest pushed 1 new commit to mkwst/draft-west-leave-secure-cookies-alone: https://github.com/w3c/web-platform-tests/commit/ebba6c0296f690aad6df1464f72b508c76d2aaaf
  124. # [15:32] -gitbot:#testing- web-platform-tests/mkwst/draft-west-leave-secure-cookies-alone ebba6c0 Mike West: fixup slight reordering.
  125. # [15:32] * Parts: gitbot (~gitbot@public.cloak)
  126. # [15:50] * Joins: ShaneM (~ShaneM@public.cloak)
  127. # [16:34] <jgraham> Ms2ger: Did you have an opinion on https://github.com/w3c/web-platform-tests/pull/2349 ?
  128. # [16:48] <Ms2ger> jgraham, I agree about -svg rather than -1
  129. # [16:49] <Ms2ger> async_test(function(t) { onload=t.step_func(function() {}) });
  130. # [16:53] <jgraham> Right I agree that as long as you can do whatever in the first step everything's good
  131. # [16:54] <jgraham> But I don't know if that's always possible in general
  132. # [16:59] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  133. # [17:22] * Joins: gitbot (~gitbot@public.cloak)
  134. # [17:22] -gitbot:#testing- [web-platform-tests] choniong opened pull request #2393: Tests for ontouch* properties in GlobalEventHandlers (master...ontouch-globaleventhandler-test) https://github.com/w3c/web-platform-tests/pull/2393
  135. # [17:22] * Parts: gitbot (~gitbot@public.cloak)
  136. # [17:23] * Joins: gitbot (~gitbot@public.cloak)
  137. # [17:23] -gitbot:#testing- [csswg-test] Ms2ger opened pull request #965: Update CSS.escape test for null change. (master...escape) https://github.com/w3c/csswg-test/pull/965
  138. # [17:23] * Parts: gitbot (~gitbot@public.cloak)
  139. # [17:24] * Joins: gitbot (~gitbot@public.cloak)
  140. # [17:24] -gitbot:#testing- [web-platform-tests] domenic pushed 1 new commit to promise-rejection-tracking: https://github.com/w3c/web-platform-tests/commit/96ff2b3ff7917dd1fde78c7800168f4ec1ac9437
  141. # [17:24] -gitbot:#testing- web-platform-tests/promise-rejection-tracking 96ff2b3 Domenic Denicola: Fixups to onerror test based on review
  142. # [17:24] * Parts: gitbot (~gitbot@public.cloak)
  143. # [17:28] * Joins: gitbot (~gitbot@public.cloak)
  144. # [17:28] -gitbot:#testing- [web-platform-tests] domenic pushed 1 new commit to promise-rejection-tracking: https://github.com/w3c/web-platform-tests/commit/f9b2eca3ae1bba2a5bc307d061e450e6ac8a98e0
  145. # [17:28] -gitbot:#testing- web-platform-tests/promise-rejection-tracking f9b2eca Domenic Denicola: Fix up disallow-crossorigin tests based on issues in OP
  146. # [17:28] * Parts: gitbot (~gitbot@public.cloak)
  147. # [18:00] * Joins: zcorpan (~zcorpan@public.cloak)
  148. # [18:07] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  149. # [18:13] * Quits: simonstewart (~textual@public.cloak) ("Textual IRC Client: www.textualapp.com")
  150. # [18:15] * Joins: gitbot (~gitbot@public.cloak)
  151. # [18:15] -gitbot:#testing- [web-platform-tests] domenic closed pull request #2261: Add more isEqualNode tests (master...isEqualNode) https://github.com/w3c/web-platform-tests/pull/2261
  152. # [18:15] * Parts: gitbot (~gitbot@public.cloak)
  153. # [18:18] * Joins: gitbot (~gitbot@public.cloak)
  154. # [18:18] -gitbot:#testing- [web-platform-tests] domenic created is-equal-node (+1 new commit): https://github.com/w3c/web-platform-tests/commit/10543c5fdc77
  155. # [18:18] -gitbot:#testing- web-platform-tests/is-equal-node 10543c5 Domenic Denicola: Add more isEqualNode tests
  156. # [18:18] * Parts: gitbot (~gitbot@public.cloak)
  157. # [18:18] * Joins: gitbot (~gitbot@public.cloak)
  158. # [18:18] -gitbot:#testing- [web-platform-tests] domenic opened pull request #2394: Add more isEqualNode tests (master...is-equal-node) https://github.com/w3c/web-platform-tests/pull/2394
  159. # [18:18] * Parts: gitbot (~gitbot@public.cloak)
  160. # [18:24] * Quits: Ms2ger (~Ms2ger@public.cloak) ("nn")
  161. # [18:38] * Quits: ShaneM (~ShaneM@public.cloak) (Ping timeout: 180 seconds)
  162. # [19:01] * Joins: zcorpan (~zcorpan@public.cloak)
  163. # [19:01] * Quits: dom (dom@public.cloak) ("")
  164. # [19:03] * Joins: zcorpan_ (~zcorpan@public.cloak)
  165. # [19:03] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  166. # [19:04] * Quits: zcorpan_ (~zcorpan@public.cloak) (Client closed connection)
  167. # [19:16] * terri_offline is now known as terri
  168. # [20:05] * Joins: zcorpan (~zcorpan@public.cloak)
  169. # [20:08] * Joins: ShaneM (~ShaneM@public.cloak)
  170. # [20:12] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  171. # [20:42] * Quits: chongz (~chongz@public.cloak) (Ping timeout: 180 seconds)
  172. # [21:06] * Joins: zcorpan (~zcorpan@public.cloak)
  173. # [21:06] * Quits: rego (~rego@public.cloak) ("Leaving")
  174. # [21:11] * Quits: plh (plehegar@public.cloak) ("Leaving")
  175. # [21:13] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  176. # [21:26] * Joins: plh (plehegar@public.cloak)
  177. # [22:07] * Joins: zcorpan (~zcorpan@public.cloak)
  178. # [22:14] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  179. # [23:03] * Joins: rniwa (~textual@public.cloak)
  180. # [23:07] * Joins: zcorpan (~zcorpan@public.cloak)
  181. # [23:16] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  182. # [23:30] * Quits: ShaneM (~ShaneM@public.cloak) (Ping timeout: 180 seconds)
  183. # [23:37] * Quits: plh (plehegar@public.cloak) ("Leaving")
  184. # Session Close: Wed Dec 02 00:00:00 2015

Previous day, Next day

Think these logs are useful? Then please donate to show your gratitude (and keep them up, of course). Thanks! — Krijn