/irc-logs / w3c / #testing / 2015-10-08 / end

Options:

Previous day, Next day

  1. # Session Start: Thu Oct 08 00:00:00 2015
  2. # Session Ident: #testing
  3. # [00:07] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  4. # [00:15] * Joins: rego (~smuxi@public.cloak)
  5. # [00:17] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  6. # [01:17] * Joins: zcorpan (~zcorpan@public.cloak)
  7. # [01:25] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  8. # [02:22] * terri is now known as terri_offline
  9. # [03:48] * Quits: ArtB (~ArtB@public.cloak) ("Leaving.")
  10. # [04:17] * Joins: zcorpan (~zcorpan@public.cloak)
  11. # [04:25] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  12. # [05:19] * Joins: zcorpan (~zcorpan@public.cloak)
  13. # [05:26] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  14. # [06:47] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  15. # [06:56] * Joins: rego (~smuxi@public.cloak)
  16. # [06:57] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  17. # [07:05] * Joins: rego (~smuxi@public.cloak)
  18. # [07:07] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  19. # [07:16] * Joins: rego (~smuxi@public.cloak)
  20. # [07:24] * Joins: MK (kawamori@public.cloak)
  21. # [07:39] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  22. # [07:47] * Joins: rego (~smuxi@public.cloak)
  23. # [08:18] * Joins: zcorpan (~zcorpan@public.cloak)
  24. # [08:25] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  25. # [08:33] * Joins: zcorpan (~zcorpan@public.cloak)
  26. # [09:00] * Joins: simonstewart (~textual@public.cloak)
  27. # [09:00] * Quits: MK (kawamori@public.cloak) (Ping timeout: 180 seconds)
  28. # [09:32] * Joins: Ms2ger (~Ms2ger@public.cloak)
  29. # [09:37] * Quits: simonstewart (~textual@public.cloak) ("Textual IRC Client: www.textualapp.com")
  30. # [09:48] * Joins: MK (kawamori@public.cloak)
  31. # [10:11] * Joins: MK_ (kawamori@public.cloak)
  32. # [10:11] * Quits: MK (kawamori@public.cloak) (Ping timeout: 180 seconds)
  33. # [10:18] * Quits: MK_ (kawamori@public.cloak) (Ping timeout: 180 seconds)
  34. # [10:57] * Joins: MK (kawamori@public.cloak)
  35. # [11:14] * Quits: MK (kawamori@public.cloak) (Ping timeout: 180 seconds)
  36. # [11:52] * Joins: ArtB (~ArtB@public.cloak)
  37. # [13:39] * Joins: AryehGregor (~bc789e7b@public.cloak)
  38. # [13:40] <AryehGregor> Ms2ger: Could you look at <https://github.com/w3c/web-platform-tests/pull/2232>? It's six non-comment lines changed, all of which are basically the same change, and it fixes a test that's incorrect.
  39. # [13:40] <AryehGregor> (i.e., doesn't just improve test coverage)
  40. # [13:41] * Joins: gitbot (~gitbot@public.cloak)
  41. # [13:41] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #2232: Update Range tests for ProcessingInstruction (master...range-pi) https://github.com/w3c/web-platform-tests/pull/2232
  42. # [13:41] * Parts: gitbot (~gitbot@public.cloak)
  43. # [13:41] <Ms2ger> AryehGregor, yeah, thanks
  44. # [13:41] <AryehGregor> Ms2ger: :)!
  45. # [13:44] <AryehGregor> Ms2ger: https://github.com/w3c/web-platform-tests/pull/2231 also might fix incorrect tests (didn't check if the changes actually affect tests). https://github.com/w3c/web-platform-tests/pull/2233 adds a new test that's only a few lines. Both should be very easy to review.
  46. # [13:44] <AryehGregor> Just saying.
  47. # [13:46] * Joins: gitbot (~gitbot@public.cloak)
  48. # [13:46] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #2231: Range.detach() is now a no-op (master...range-detach) https://github.com/w3c/web-platform-tests/pull/2231
  49. # [13:46] * Parts: gitbot (~gitbot@public.cloak)
  50. # [13:47] * Joins: gitbot (~gitbot@public.cloak)
  51. # [13:47] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #2233: Test Range constructor (master...range-constructor) https://github.com/w3c/web-platform-tests/pull/2233
  52. # [13:47] * Parts: gitbot (~gitbot@public.cloak)
  53. # [14:05] <AryehGregor> Ms2ger: \o/
  54. # [14:07] * Joins: plh (plehegar@public.cloak)
  55. # [15:27] <AryehGregor> Ms2ger: Do you think it's reasonable that el.classList.remove() normalizes the class attribute instead of doing nothing or throwing? AFAICT that's what the spec implies.
  56. # [15:28] <AryehGregor> (that is, called with no arguments)
  57. # [15:29] <Ms2ger> No opinion
  58. # [16:05] <AryehGregor> Ms2ger: What is ":has"? Is it actually specced anywhere? I don't see it in the Selectors 4 WD. http://w3c-test.org/dom/nodes/Element-closest.html
  59. # [16:06] <Ms2ger> :contains?
  60. # [16:06] <Ms2ger> I have no clue
  61. # [16:06] <Ms2ger> Calling zcorpan
  62. # [16:07] <jgraham> Maybe SimonSapin would know?
  63. # [16:08] <AryehGregor> I should check the latest draft, but I'm having trouble accessing drafts.csswg.org.
  64. # [16:08] * AryehGregor curses WDs
  65. # [16:09] * Joins: simonstewart (~textual@public.cloak)
  66. # [16:11] <Ms2ger> There's always http://csswg.inkedblade.net/drafts/
  67. # [16:14] <AryehGregor> The last test here expects that 1) iframes work with data URLs, and 2) the parent of such an iframe has access to the child. That's not actually specced anywhere, is it? http://w3c-test.org/dom/nodes/Node-isEqualNode.xhtml
  68. # [16:14] <jgraham> 1) - yes
  69. # [16:14] <jgraham> 2) - it's contentious
  70. # [16:14] <jgraham> It works in gecko but blink won't implement it
  71. # [16:15] <AryehGregor> It's not good form to write tests that fail due to unrelated problems anyway, IMO.
  72. # [16:16] <jgraham> Well, it's not possible in general
  73. # [16:16] <jgraham> But yeah, relying on as little other surface area as practical is ofc good practice
  74. # [16:16] <AryehGregor> Obviously I mean where it's reasonable to avoid.
  75. # [16:18] <AryehGregor> Anything that doesn't include testharness is magically assumed to be a support file, right?
  76. # [16:18] <jgraham> Yeah
  77. # [16:18] <Ms2ger> Or a reftest
  78. # [16:18] <jgraham> It's unlikely that you'll acidentiallu make a reftest though
  79. # [16:24] * Joins: gitbot (~gitbot@public.cloak)
  80. # [16:24] -gitbot:#testing- [web-platform-tests] ayg opened pull request #2235: Don't depend on data URLs in iframes (master...Node-isEqualNode) https://github.com/w3c/web-platform-tests/pull/2235
  81. # [16:24] * Parts: gitbot (~gitbot@public.cloak)
  82. # [16:29] <AryehGregor> jgraham: Could you merge that pull request so that I can quickly test in IE whether it actually matches the spec? (My dev machine is Linux, so I can't test easily on the local copy, and AFAIK GitHub doesn't let me view raw copies of my repo's files.)
  83. # [16:30] * Joins: gitbot (~gitbot@public.cloak)
  84. # [16:30] -gitbot:#testing- [web-platform-tests] jgraham closed pull request #2235: Don't depend on data URLs in iframes (master...Node-isEqualNode) https://github.com/w3c/web-platform-tests/pull/2235
  85. # [16:30] * Parts: gitbot (~gitbot@public.cloak)
  86. # [16:30] * Joins: gitbot (~gitbot@public.cloak)
  87. # [16:30] -gitbot:#testing- [web-platform-tests] jgraham pushed 2 new commits to master: https://github.com/w3c/web-platform-tests/compare/ac90b3db87b8...eddfaddfff3b
  88. # [16:30] -gitbot:#testing- web-platform-tests/master 928fd22 Aryeh Gregor: Don't depend on data URLs in iframes
  89. # [16:30] -gitbot:#testing- web-platform-tests/master eddfadd jgraham: Merge pull request #2235 from ayg/Node-isEqualNode...
  90. # [16:30] * Parts: gitbot (~gitbot@public.cloak)
  91. # [16:32] <Ms2ger> AryehGregor, I'm expecting you to add a separate test for the data: case
  92. # [16:32] <AryehGregor> Ms2ger: Hmm, why?
  93. # [16:32] <AryehGregor> jgraham: Thanks!
  94. # [16:34] <AryehGregor> Wait, why does DOM ignore the internal subset for isEqualNode? The original definition in DOM 3 Core took internal subset into account, and IE and Gecko both still do.
  95. # [16:34] <AryehGregor> Although Chrome does not.
  96. # [16:51] * Joins: MK (kawamori@public.cloak)
  97. # [16:51] <zcorpan> Ms2ger: woot?
  98. # [16:52] <zcorpan> https://drafts.csswg.org/selectors-4/#relational
  99. # [16:53] <zcorpan> also https://drafts.csswg.org/selectors-4/#profiles
  100. # [16:55] <AryehGregor> Does anyone know where the HTML spec defines the microdata properties on HTMLElement (.itemRef, etc.)?
  101. # [16:55] <AryehGregor> I don't see it.
  102. # [16:55] <AryehGregor> I'm pretty sure this test is wrong, but I can't prove it without a spec.
  103. # [16:56] <Ms2ger> Today is your lucky day
  104. # [16:56] <Ms2ger> https://github.com/whatwg/html/pull/213
  105. # [16:58] <AryehGregor> . . .
  106. # [16:58] <AryehGregor> So Gecko should just drop support for those properties entirely?
  107. # [16:58] <AryehGregor> Fine by me.
  108. # [16:59] <Ms2ger> https://bugzilla.mozilla.org/show_bug.cgi?id=909633
  109. # [16:59] <Ms2ger> Patch welcome, I guess
  110. # [17:03] * Joins: gitbot (~gitbot@public.cloak)
  111. # [17:03] -gitbot:#testing- [web-platform-tests] ayg opened pull request #2236: Microdata DOM API was removed in whatwg/html@2aaecb89 (master...microdata) https://github.com/w3c/web-platform-tests/pull/2236
  112. # [17:03] * Parts: gitbot (~gitbot@public.cloak)
  113. # [17:05] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  114. # [17:07] <gsnedders> AryehGregor: IMO we shouldn't just remove the tests, but rather change them to test for microData's absence
  115. # [17:12] <AryehGregor> gsnedders: That's a philosophical issue. I don't particularly disagree, but those would be entirely new tests, not changed tests.
  116. # [17:12] <AryehGregor> They'd be a lot shorter. :)
  117. # [17:12] <AryehGregor> If everyone agrees we want to do that, I don't mind writing a test for it quickly. But in the past this has been disputed, IIRC.
  118. # [17:13] * Joins: rego (~smuxi@public.cloak)
  119. # [17:14] <zcorpan> there are historical.html files in wpt that test for things that should not be supported
  120. # [17:14] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  121. # [17:16] * Joins: gitbot (~gitbot@public.cloak)
  122. # [17:16] -gitbot:#testing- [web-platform-tests] jgraham pushed 2 new commits to master: https://github.com/w3c/web-platform-tests/compare/eddfaddfff3b...5601d854cd19
  123. # [17:16] -gitbot:#testing- web-platform-tests/master e955cd4 Aryeh Gregor: Microdata DOM API was removed in whatwg/html@2aaecb89
  124. # [17:16] -gitbot:#testing- web-platform-tests/master 5601d85 jgraham: Merge pull request #2236 from ayg/microdata...
  125. # [17:16] * Parts: gitbot (~gitbot@public.cloak)
  126. # [17:16] <jgraham> I agree a test for the absence of these properties would be god fwiw
  127. # [17:16] <jgraham> *good
  128. # [17:17] * Quits: simonstewart (~textual@public.cloak) ("My Mac has gone to sleep. ZZZzzz…")
  129. # [17:35] * Joins: r12a (rishida@public.cloak)
  130. # [17:35] <r12a> anyone able to troubleshoot a test for me?
  131. # [17:35] <r12a> http://www.w3.org/International/tests/repo/encoding/legacy-mb-tchinese/big5/big5-decode
  132. # [17:36] <r12a> seems to be getting stuck in the testharness code
  133. # [17:36] <r12a> the console.log produces the expected results
  134. # [17:42] * Quits: Ms2ger (~Ms2ger@public.cloak) (Ping timeout: 180 seconds)
  135. # [17:43] * Joins: gitbot (~gitbot@public.cloak)
  136. # [17:43] -gitbot:#testing- [web-platform-tests] ayg opened pull request #2237: Remove duplicate test names (master...lookupNamespaceURI) https://github.com/w3c/web-platform-tests/pull/2237
  137. # [17:43] * Parts: gitbot (~gitbot@public.cloak)
  138. # [17:46] <AryehGregor> r12a: Odd.
  139. # [17:46] * AryehGregor points r12a to jgraham
  140. # [17:46] <r12a> yeah :)
  141. # [17:46] <AryehGregor> I might have expected it not to work at all, but not to hang like that.
  142. # [17:47] <r12a> it was working ok at one point - i'm not sure what i did to change that
  143. # [17:47] <r12a> and i can't see why it wouldn't work given the code that's there
  144. # [17:48] * Joins: simonstewart (~textual@public.cloak)
  145. # [17:48] <AryehGregor> I don't think you're supposed to call test() asynchronously without some instruction to the framework to expect it.
  146. # [17:48] <AryehGregor> Otherwise it might just assume the test is finished before test() has even been called.
  147. # [17:49] <gsnedders> yeah, you'll need test_async
  148. # [17:49] <gsnedders> or similar
  149. # [17:49] <r12a> hmm, but then i have the problem that the iframe may not be loaded
  150. # [17:49] <gsnedders> there are various ways to do ths
  151. # [17:50] <r12a> gsnedders, could you give me some baby steps? i'm not a real programmer :)
  152. # [17:50] <AryehGregor> http://testthewebforward.org/docs/testharness-library.html
  153. # [17:51] <AryehGregor> It briefly describes how to use async_test there.
  154. # [17:51] <r12a> thanks, reading...
  155. # [17:54] <jgraham> This whole thing seems slightly weird. Can't you just use the load event to check if the iframe loaded?
  156. # [17:54] <r12a> i think that's what i'm doing, no?
  157. # [17:55] <jgraham> So why does it then double check that the iframe loaded? By the time the load fires on the parent the iframe must have loaded
  158. # [17:56] <jgraham> It seems like you know the number of tests synchronously
  159. # [17:56] <jgraham> Because the script loads synchronously?
  160. # [17:57] <jgraham> So I would create an array of async_tests and then once the page has loaded run a step() on each test and then call done on it
  161. # [17:58] <r12a> hmm, my apologies but i have no clue what all that means, let alone how to do it
  162. # [17:59] <r12a> are you saying in part that i can just use <body onload="showNodes();"> rather than <body onload="checkIframeLoaded();">, because when onload fires on the body the iframe will always have loaded?
  163. # [18:01] * Joins: kawai_ (~kawai@public.cloak)
  164. # [18:04] <jgraham> r12a: Yes
  165. # [18:04] <r12a> that's useful to know, thanks
  166. # [18:05] <jgraham> So I'm saying that in the script you can do something like var tests = []; for var (i=0;i<max) {tests.push(async_test("some title " + i))}
  167. # [18:08] * Joins: kawai (~kawai@public.cloak)
  168. # [18:09] <jgraham> Then in showNodes you do for (var i=0; i<tests.length; i++) {tests[i].step(function() {decode(some arguments)}} or something
  169. # [18:09] <jgraham> Uh
  170. # [18:09] <jgraham> add a tests[i].done();
  171. # [18:14] * Quits: kawai_ (~kawai@public.cloak) (Ping timeout: 180 seconds)
  172. # [18:16] <r12a> james, is "for var (i=0;i<max)" syntax i've not come across before, or should it be for(var i=0;i<max; i++) where max is the total number of tests?
  173. # [18:31] <jgraham> r12a: Can I handwaave and say "pseudocode"? ;) But yeah your one seems more likely to actually run
  174. # [18:32] <r12a> ok, thanks - i'm constantly discovering new syntax forms, so i wasn't sure ;)
  175. # [18:32] <jgraham> I'm constantly discovering new ways to mess up typing on irc :)
  176. # [18:59] * Joins: MK_ (kawamori@public.cloak)
  177. # [19:06] * Quits: MK (kawamori@public.cloak) (Ping timeout: 180 seconds)
  178. # [19:22] * Quits: kawai (~kawai@public.cloak) (Client closed connection)
  179. # [19:22] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  180. # [19:33] <r12a> jgraham, aryehgregor, gsnedders, seems to now work, thanks http://www.w3.org/International/tests/repo/encoding/legacy-mb-tchinese/big5/big5-decode.html
  181. # [19:33] <r12a> a bit slower
  182. # [19:34] <r12a> i hope it's manageable when i expose the full set of 14,000 subtests
  183. # [19:35] * Joins: rego (~smuxi@public.cloak)
  184. # [19:39] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  185. # [19:45] <jgraham> r12a: That was much faster for me, in that it worked ;)
  186. # [19:46] * Joins: rego (~smuxi@public.cloak)
  187. # [19:46] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  188. # [19:55] * Joins: rego (~smuxi@public.cloak)
  189. # [19:57] * Quits: simonstewart (~textual@public.cloak) ("My Mac has gone to sleep. ZZZzzz…")
  190. # [19:59] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  191. # [20:06] * Joins: rego (~smuxi@public.cloak)
  192. # [20:08] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  193. # [20:18] <AryehGregor> :)
  194. # [20:18] * Joins: rego (~smuxi@public.cloak)
  195. # [20:18] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  196. # [20:22] * Quits: MK_ (kawamori@public.cloak) (Ping timeout: 180 seconds)
  197. # [20:27] * Joins: rego (~smuxi@public.cloak)
  198. # [20:37] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  199. # [20:41] * Quits: AryehGregor (~bc789e7b@public.cloak) ("http://www.mibbit.com ajax IRC Client")
  200. # [20:46] * Joins: rego (~smuxi@public.cloak)
  201. # [20:48] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  202. # [20:58] * Joins: rego (~smuxi@public.cloak)
  203. # [20:59] * Joins: zcorpan (~zcorpan@public.cloak)
  204. # [21:02] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  205. # [21:08] * terri_offline is now known as terri
  206. # [21:09] * Joins: rego (~smuxi@public.cloak)
  207. # [21:13] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  208. # [21:20] * Joins: rego (~smuxi@public.cloak)
  209. # [21:22] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  210. # [21:29] * Joins: rego (~smuxi@public.cloak)
  211. # [21:31] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  212. # [21:38] * Joins: rego (~smuxi@public.cloak)
  213. # [21:40] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  214. # [21:48] * Joins: rego (~smuxi@public.cloak)
  215. # [21:49] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  216. # [21:57] * Joins: rego (~smuxi@public.cloak)
  217. # [22:02] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  218. # [22:08] * Joins: rego (~smuxi@public.cloak)
  219. # [22:08] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  220. # [22:17] * Joins: rego (~smuxi@public.cloak)
  221. # [22:18] * heycam|away is now known as heycam
  222. # [22:18] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  223. # [22:26] * Joins: rego (~smuxi@public.cloak)
  224. # [22:38] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  225. # [22:45] * Joins: rego (~smuxi@public.cloak)
  226. # [22:51] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  227. # [22:58] * Joins: rego (~smuxi@public.cloak)
  228. # [22:58] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  229. # [23:09] * Joins: rego (~smuxi@public.cloak)
  230. # [23:11] * Quits: rego (~smuxi@public.cloak) (Client closed connection)
  231. # [23:12] * Quits: r12a (rishida@public.cloak) (Client closed connection)
  232. # [23:18] * Joins: rego (~smuxi@public.cloak)
  233. # [23:40] * heycam is now known as heycam|away
  234. # [23:46] * Quits: plh (plehegar@public.cloak) ("Leaving")
  235. # Session Close: Fri Oct 09 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