/irc-logs / w3c / #testing / 2014-05-12 / end

Options:

  1. # Session Start: Mon May 12 00:00:00 2014
  2. # Session Ident: #testing
  3. # [00:06] * Joins: zcorpan (~zcorpan@public.cloak)
  4. # [00:13] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  5. # [00:35] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  6. # [00:36] * Joins: Lachy (~Lachy@public.cloak)
  7. # [01:09] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  8. # [01:10] * Joins: Lachy (~Lachy@public.cloak)
  9. # [01:13] * Joins: zcorpan (~zcorpan@public.cloak)
  10. # [01:20] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  11. # [01:28] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  12. # [01:29] * Joins: Lachy (~Lachy@public.cloak)
  13. # [01:45] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  14. # [01:50] * Joins: Lachy (~Lachy@public.cloak)
  15. # [02:13] * Joins: zcorpan (~zcorpan@public.cloak)
  16. # [02:20] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  17. # [02:23] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  18. # [02:32] * Joins: Lachy (~Lachy@public.cloak)
  19. # [02:42] * Joins: lmclister (~lmclister@public.cloak)
  20. # [02:43] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  21. # [02:44] * Joins: Lachy (~Lachy@public.cloak)
  22. # [02:54] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  23. # [02:56] * Joins: Lachy (~Lachy@public.cloak)
  24. # [03:14] * Joins: zcorpan (~zcorpan@public.cloak)
  25. # [03:21] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  26. # [03:34] * Joins: koji (~koji@public.cloak)
  27. # [04:06] * Quits: koji (~koji@public.cloak) (Client closed connection)
  28. # [04:06] * Joins: koji (~koji@public.cloak)
  29. # [04:13] * Quits: koji (~koji@public.cloak) (Ping timeout: 180 seconds)
  30. # [04:15] * Joins: zcorpan (~zcorpan@public.cloak)
  31. # [04:22] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  32. # [04:58] * Quits: lmclister (~lmclister@public.cloak) ("")
  33. # [05:16] * Joins: zcorpan (~zcorpan@public.cloak)
  34. # [05:18] * Joins: gitbot (~gitbot@public.cloak)
  35. # [05:18] -gitbot:#testing- [web-platform-tests] yunxiaoxie opened pull request #986: Add more tests for IDBCursor delete (master...yunxiaoxie/idbcursor-delete) https://github.com/w3c/web-platform-tests/pull/986
  36. # [05:18] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  37. # [05:23] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  38. # [05:57] * Joins: lmclister (~lmclister@public.cloak)
  39. # [06:16] * Joins: zcorpan (~zcorpan@public.cloak)
  40. # [06:17] * Joins: hank (~hank@public.cloak)
  41. # [06:23] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  42. # [06:27] * Joins: lmcliste_ (~lmclister@public.cloak)
  43. # [06:27] * Quits: lmclister (~lmclister@public.cloak) (Client closed connection)
  44. # [06:31] * Quits: lmcliste_ (~lmclister@public.cloak) (Client closed connection)
  45. # [06:31] * Joins: lmclister (~lmclister@public.cloak)
  46. # [06:35] * Quits: lmclister (~lmclister@public.cloak) (Client closed connection)
  47. # [06:35] * Joins: lmcliste_ (~lmclister@public.cloak)
  48. # [06:44] * Quits: lmcliste_ (~lmclister@public.cloak) (Client closed connection)
  49. # [06:44] * Joins: lmclister (~lmclister@public.cloak)
  50. # [06:44] * Quits: lmclister (~lmclister@public.cloak) (lmclister)
  51. # [06:45] * Joins: lmclister (~lmclister@public.cloak)
  52. # [06:49] * Quits: lmclister (~lmclister@public.cloak) (Client closed connection)
  53. # [06:50] * Joins: lmclister (~lmclister@public.cloak)
  54. # [06:54] * Quits: lmclister (~lmclister@public.cloak) (Client closed connection)
  55. # [07:02] * Joins: lmclister (~lmclister@public.cloak)
  56. # [07:05] * Quits: hank (~hank@public.cloak) (Ping timeout: 180 seconds)
  57. # [07:07] * Joins: hank (~hank@public.cloak)
  58. # [07:17] * Joins: zcorpan (~zcorpan@public.cloak)
  59. # [07:24] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  60. # [07:28] * Joins: zcorpan (~zcorpan@public.cloak)
  61. # [07:30] * Joins: zcorpan_ (~zcorpan@public.cloak)
  62. # [07:35] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  63. # [07:52] * Joins: gitbot (~gitbot@public.cloak)
  64. # [07:52] -gitbot:#testing- [web-platform-tests] hallvors pushed 2 new commits to hallvors/progress-gzip: https://github.com/w3c/web-platform-tests/compare/0f9bb408f6bc...5a9dc3d8de20
  65. # [07:52] -gitbot:#testing- web-platform-tests/hallvors/progress-gzip d17140d Hallvord R. M. Steen: make gzip.py output gzip, add zlib.py
  66. # [07:52] -gitbot:#testing- web-platform-tests/hallvors/progress-gzip 5a9dc3d Hallvord R. M. Steen: adding test for zlib-compressed response
  67. # [07:52] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  68. # [08:07] * Quits: lmclister (~lmclister@public.cloak) (Client closed connection)
  69. # [08:07] * Joins: lmclister (~lmclister@public.cloak)
  70. # [08:20] * Quits: zcorpan_ (~zcorpan@public.cloak) (Client closed connection)
  71. # [08:21] * Joins: zcorpan (~zcorpan@public.cloak)
  72. # [08:29] * Quits: lmclister (~lmclister@public.cloak) ("")
  73. # [08:32] * Quits: ptressel (~chatzilla@public.cloak) ("busy")
  74. # [09:36] * Joins: Ms2ger (~Ms2ger@public.cloak)
  75. # [09:39] * Joins: gitbot (~gitbot@public.cloak)
  76. # [09:39] -gitbot:#testing- [web-platform-tests] zcorpan pushed 2 new commits to master: https://github.com/w3c/web-platform-tests/compare/3db70e2de63b...a061b0287072
  77. # [09:39] -gitbot:#testing- web-platform-tests/master 4fd7f27 Caitlin Potter: URLUtils.search returns empty string, not '?'...
  78. # [09:39] -gitbot:#testing- web-platform-tests/master a061b02 Simon Pieters: Merge pull request #984 from caitp/patch-3...
  79. # [09:39] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  80. # [09:39] * Joins: gitbot (~gitbot@public.cloak)
  81. # [09:39] -gitbot:#testing- [web-platform-tests] zcorpan closed pull request #984: URLUtils.search returns empty string, not '?' (master...patch-3) https://github.com/w3c/web-platform-tests/pull/984
  82. # [09:39] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  83. # [09:40] * Joins: gitbot (~gitbot@public.cloak)
  84. # [09:40] -gitbot:#testing- [web-platform-tests] zcorpan pushed 2 new commits to master: https://github.com/w3c/web-platform-tests/compare/a061b0287072...b0d1f10efb79
  85. # [09:40] -gitbot:#testing- web-platform-tests/master b4986ed Caitlin Potter: assert 4th argument is a non-specific number...
  86. # [09:40] -gitbot:#testing- web-platform-tests/master b0d1f10 Simon Pieters: Merge pull request #983 from caitp/patch-2...
  87. # [09:40] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  88. # [09:40] * Joins: gitbot (~gitbot@public.cloak)
  89. # [09:40] -gitbot:#testing- [web-platform-tests] zcorpan closed pull request #983: assert 4th argument is a non-specific number (master...patch-2) https://github.com/w3c/web-platform-tests/pull/983
  90. # [09:40] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  91. # [10:04] * Quits: Ms2ger (~Ms2ger@public.cloak) (Ping timeout: 180 seconds)
  92. # [10:13] * Joins: Ms2ger (~Ms2ger@public.cloak)
  93. # [10:22] * Joins: darobin (rberjon@public.cloak)
  94. # [10:32] * Joins: gitbot (~gitbot@public.cloak)
  95. # [10:32] -gitbot:#testing- [web-platform-tests] jgraham closed pull request #962: Add test for EventSource#url in url query encoding testsuite (master...eventsource-url-query-encoding) https://github.com/w3c/web-platform-tests/pull/962
  96. # [10:32] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  97. # [10:43] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  98. # [10:49] * Quits: Ms2ger (~Ms2ger@public.cloak) (Ping timeout: 180 seconds)
  99. # [10:59] * Joins: Ms2ger (~Ms2ger@public.cloak)
  100. # [11:01] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  101. # [11:18] * Joins: gitbot (~gitbot@public.cloak)
  102. # [11:18] -gitbot:#testing- [web-platform-tests] jgraham closed pull request #891: Fix output summary error of tools/runner/runner.js when the Status of la... (master...fix_output_summary) https://github.com/w3c/web-platform-tests/pull/891
  103. # [11:18] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  104. # [11:45] * Joins: abarsto (~abarsto@public.cloak)
  105. # [11:45] * abarsto is now known as ArtB
  106. # [11:54] * Joins: Lachy (~Lachy@public.cloak)
  107. # [12:13] * Joins: Lachy_ (~Lachy@public.cloak)
  108. # [12:17] * Joins: Lachy__ (~Lachy@public.cloak)
  109. # [12:18] * Quits: Lachy (~Lachy@public.cloak) (Ping timeout: 180 seconds)
  110. # [12:20] * Quits: Lachy_ (~Lachy@public.cloak) (Ping timeout: 180 seconds)
  111. # [12:37] * Quits: Ms2ger (~Ms2ger@public.cloak) (Ping timeout: 180 seconds)
  112. # [12:43] <AutomatedTester> darobin: I think I have asked this before, how can I get respec to link to keys in a dictionary?
  113. # [12:44] <darobin> AutomatedTester: I'm not sure that's possible
  114. # [12:44] <darobin> AutomatedTester: pointer?
  115. # [12:45] <AutomatedTester> pointer?
  116. # [12:45] <darobin> or maybe it is!
  117. # [12:45] * darobin checks
  118. # [12:45] <darobin> AutomatedTester: to the spec in which you're trying to do that
  119. # [12:45] <AutomatedTester> ahh right, the url to the item
  120. # [12:45] <AutomatedTester> ok
  121. # [12:46] <AutomatedTester> I thought it might be something like <a>Dictionary#key</a> or something like that
  122. # [12:46] <darobin> yes, a URL :)
  123. # [12:47] <AutomatedTester> as with all things on the web... overloaded terms are great
  124. # [12:48] <darobin> AutomatedTester: dictionary items get their own IDs, so you can just link to that
  125. # [12:48] <darobin> but there isn't a syntax to generate the link for you if that's what you mean
  126. # [12:49] <AutomatedTester> I did mean the later
  127. # [12:49] <AutomatedTester> latter*
  128. # [12:49] <darobin> just use the link, it'll be something straightforward
  129. # [12:49] * darobin forgets what at this moment
  130. # [12:49] <AutomatedTester> yea it will be, just means I need to load the page to get the link
  131. # [12:49] <AutomatedTester> making me work...
  132. # [12:49] <AutomatedTester> :P
  133. # [12:50] <darobin> if you'd given me a URL you'd have the answer already :)
  134. # [13:21] * Joins: Ms2ger (~Ms2ger@public.cloak)
  135. # [13:45] * Quits: Ms2ger (~Ms2ger@public.cloak) (Ping timeout: 180 seconds)
  136. # [14:11] * Joins: dom (dom@public.cloak)
  137. # [14:16] * Joins: plh (plehegar@public.cloak)
  138. # [15:16] * Joins: Ms2ger (~Ms2ger@public.cloak)
  139. # [15:27] * Quits: plh (plehegar@public.cloak) ("Leaving")
  140. # [15:27] * Joins: plh (plehegar@public.cloak)
  141. # [15:28] * Quits: plh (plehegar@public.cloak) ("Leaving")
  142. # [15:46] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  143. # [15:56] * Quits: shepazu (schepers@public.cloak) (Client closed connection)
  144. # [16:14] * Joins: shepazu (schepers@public.cloak)
  145. # [17:08] * Joins: koji (~koji@public.cloak)
  146. # [17:19] * Quits: koji (~koji@public.cloak) (Client closed connection)
  147. # [17:19] * Joins: koji (~koji@public.cloak)
  148. # [17:28] * Quits: koji (~koji@public.cloak) (Ping timeout: 180 seconds)
  149. # [17:28] <jgraham> If anyone is interested in commenting on the wptrunner review: https://critic.hoppipolla.co.uk/r/1431
  150. # [17:30] * Joins: lmclister (~lmclister@public.cloak)
  151. # [17:36] * Joins: rhauck (~Adium@public.cloak)
  152. # [17:40] * Joins: koji (~koji@public.cloak)
  153. # [17:43] * Quits: koji (~koji@public.cloak) (Client closed connection)
  154. # [17:44] * Joins: koji (~koji@public.cloak)
  155. # [17:48] <MikeSmith> jgraham: I'll take a look
  156. # [17:53] * Quits: koji (~koji@public.cloak) (Ping timeout: 180 seconds)
  157. # [17:54] * Quits: Lachy__ (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  158. # [18:00] <MikeSmith> jgraham: btw I been trying to figure out how to get the JS runner to run to completion in cases where it hangs now
  159. # [18:02] <jgraham> MikeSmith: depending on why it hangs that might not be possible
  160. # [18:03] * Joins: rhauck1 (~Adium@public.cloak)
  161. # [18:03] <jgraham> I mean if the test is actually hanging the browser there's not much we can do
  162. # [18:03] <jgraham> OTOH if it's a bug in the runner we can fix it
  163. # [18:04] <MikeSmith> jgraham: well, I mean not a test hang but instead, sayd, when you have all the test types checked (i.e., the default) but you give it path value for a subdirectory that only contains ref tests but doesn't contain any JavaScript (testharness) tests
  164. # [18:05] <MikeSmith> jgraham: e.g., give it "/html/rendering/replaced-elements/attributes-for-embedded-content-and-images/" for the path
  165. # [18:05] * Quits: rhauck (~Adium@public.cloak) (Ping timeout: 180 seconds)
  166. # [18:06] <MikeSmith> or alternately, the simple case of mistyping the path
  167. # [18:07] <MikeSmith> e.g., just give it /foo as the path
  168. # [18:07] <jgraham> Oh, that sounds like a bug indeed
  169. # [18:07] <MikeSmith> yeah
  170. # [18:08] <MikeSmith> I've tried stepping through the code in a dev tools debugger but have had a hard time figuring out where it's getting to and why it's not returning
  171. # [18:15] <jgraham> MikeSmith: Ah, the problem is it calls runner.start which doesn't find a test so goes through all the "I ran out of tests" steps. And then it calls set_stop() and set_pause() which set the controls to the running state
  172. # [18:15] * MikeSmith wonders if AutomatedTester might want to look at the wptrunner review https://critic.hoppipolla.co.uk/r/1431 if he hasn't already
  173. # [18:16] <jgraham> It would probably be nice to swap those two
  174. # [18:16] <jgraham> Line 350
  175. # [18:16] * MikeSmith looks
  176. # [18:17] * Joins: koji (~koji@public.cloak)
  177. # [18:17] * MikeSmith tweaks
  178. # [18:19] * Quits: dom (dom@public.cloak) ("")
  179. # [18:21] <MikeSmith> jgraham: yeah but won't that actually cause it to always stop completely after it fails to find any testharness tests?
  180. # [18:21] <MikeSmith> jgraham: In the case where there are ref tests in teh subdir but no testharness test, it'll just just without geting to the ref tests
  181. # [18:22] <MikeSmith> or maybe I'm not understanding
  182. # [18:45] <jgraham> MikeSmith: I only looked at the "path with no tests at all" case
  183. # [18:46] <MikeSmith> jgraham: ah
  184. # [18:47] * Quits: darobin (rberjon@public.cloak) (Client closed connection)
  185. # [18:54] * Quits: rhauck1 (~Adium@public.cloak) ("Leaving.")
  186. # [18:57] * Joins: rhauck (~Adium@public.cloak)
  187. # [19:02] * Joins: edoyle (~edoyle@public.cloak)
  188. # [19:17] <MikeSmith> jgraham: do you use a python linter?
  189. # [19:17] <MikeSmith> jgraham: pyflakes?
  190. # [19:18] <jgraham> MikeSmith: I ought to :)
  191. # [19:21] <MikeSmith> jgraham: for the wptrunner review I'm just wondering if I should go ahead and make review comments from linting errors
  192. # [19:21] <MikeSmith> jgraham: duplicate imports or whatnot
  193. # [19:22] <jgraham> MikeSmith: Sure, if you aren't bored by it. Otherwise I will probably clean them up anyway, but it will be harder to tell if I did it right
  194. # [19:23] <MikeSmith> jgraham: OK well I'll put comments for the ones I find then. It's productive procrastination for now :-)
  195. # [19:42] <MikeSmith> hmm dunno if I wonder if I should have made those notes instead of issues
  196. # [19:42] * MikeSmith doesn't know what the conventions are
  197. # [19:42] * Quits: rhauck (~Adium@public.cloak) ("Leaving.")
  198. # [19:44] * Joins: rhauck (~Adium@public.cloak)
  199. # [19:46] * Joins: zcorpan (~zcorpan@public.cloak)
  200. # [19:53] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  201. # [19:56] <jgraham> MikeSmith: Note == optional, Issue == blocks issue being accepted
  202. # [19:56] <jgraham> s/issue/review/
  203. # [19:58] <MikeSmith> jgraham: ok
  204. # [19:58] <MikeSmith> jgraham: but you can close issues without making changes, can't you?
  205. # [19:59] <jgraham> Yeah, if needed
  206. # [19:59] <MikeSmith> OK I guess it just makes more work potentially
  207. # [19:59] <jgraham> Using issues is fine
  208. # [19:59] <MikeSmith> ok
  209. # [20:00] <jgraham> It's easier to track which ones you looked at so I prefer them for things that aren't really informative
  210. # [20:00] <MikeSmith> it seems like some of Andrew's issues there at least are actually questions, not necessarily blockers
  211. # [20:00] <MikeSmith> ok
  212. # [20:00] <jgraham> This is ahal's first time with critic, so I'm not sure his usage is quite right
  213. # [20:00] <MikeSmith> ah ok
  214. # [20:58] * Quits: koji (~koji@public.cloak) (Ping timeout: 180 seconds)
  215. # [21:14] <AutomatedTester> MikeSmith: I am happy to look at that
  216. # [21:15] <AutomatedTester> but I see Andrew Halberstadt is a reviewer. He is good enough, unless I am needed
  217. # [21:21] * Joins: koji (~koji@public.cloak)
  218. # [21:59] * Joins: rhauck1 (~Adium@public.cloak)
  219. # [22:05] * Quits: rhauck (~Adium@public.cloak) (Ping timeout: 180 seconds)
  220. # [22:12] * Joins: zcorpan (~zcorpan@public.cloak)
  221. # [22:15] * Quits: scott_gonzalez (~scott_gonzalez@public.cloak) (scott_gonzalez)
  222. # [22:19] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  223. # [22:31] * Joins: scott_gonzalez (~scott_gonzalez@public.cloak)
  224. # [22:37] <AutomatedTester> MikeSmith: yt?
  225. # [22:38] <AutomatedTester> Ms2ger: actually you can answer this, do I need to have links to webidl types in the spec that are used
  226. # [22:38] <AutomatedTester> Ms2ger: see ttps://www.w3.org/Bugs/Public/show_bug.cgi?id=25664
  227. # [22:38] <Ms2ger> ttps?
  228. # [22:38] <AutomatedTester> https://www.w3.org/Bugs/Public/show_bug.cgi?id=25664
  229. # [22:38] <AutomatedTester> the h is silent in some countries
  230. # [22:39] <Ms2ger> Definitely don't link there
  231. # [22:39] <AutomatedTester> I wont
  232. # [22:39] <Ms2ger> But DOMString is generally not linked
  233. # [22:39] <Ms2ger> Neither is "unsigned long"
  234. # [22:39] <AutomatedTester> should I add a thing at the top saying types are defined at http://www.w3.org/TR/WebIDL
  235. # [22:41] <jgraham> You have to say that the IDl fragments use [WebIDL]
  236. # [22:41] <jgraham> The IDL fragments in this specification must be interpreted as required for conforming IDL fragments, as described in the Web IDL specification. [WEBIDL]
  237. # [22:41] <jgraham> (that's from HTML)
  238. # [22:41] <AutomatedTester> aha
  239. # [22:44] <AutomatedTester> jgraham: thanks!
  240. # [22:45] <Ms2ger> I assumed you had that
  241. # [22:45] <Ms2ger> I should review your spec
  242. # [22:48] <AutomatedTester> Ms2ger: I do have that
  243. # [22:48] <Ms2ger> People don't read
  244. # [22:48] <Ms2ger> News at 11, I guess :)
  245. # [22:50] <Ms2ger> Speaking of which, hi 11pm
  246. # [22:50] <AutomatedTester> in 10 mins
  247. # [23:01] * Quits: Ms2ger (~Ms2ger@public.cloak) ("nn")
  248. # [23:02] * Quits: scott_gonzalez (~scott_gonzalez@public.cloak) (scott_gonzalez)
  249. # [23:03] * Joins: scott_gonzalez (~scott_gonzalez@public.cloak)
  250. # [23:10] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Client closed connection)
  251. # [23:11] * Quits: scott_gonzalez (~scott_gonzalez@public.cloak) (scott_gonzalez)
  252. # [23:44] * Joins: koji_ (~koji@public.cloak)
  253. # [23:44] * Quits: koji (~koji@public.cloak) (Client closed connection)
  254. # [23:51] * Joins: scott_gonzalez (~scott_gonzalez@public.cloak)
  255. # Session Close: Tue May 13 00:00:00 2014

The end :)