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

Options:

  1. # Session Start: Tue May 27 00:00:00 2014
  2. # Session Ident: #testing
  3. # [00:06] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  4. # [00:06] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  5. # [00:13] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Ping timeout: 180 seconds)
  6. # [00:18] * Joins: ptressel (~chatzilla@public.cloak)
  7. # [00:21] * Joins: Lachy (~Lachy@public.cloak)
  8. # [00:23] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  9. # [00:37] * Joins: Lachy (~Lachy@public.cloak)
  10. # [01:06] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  11. # [01:11] * Joins: Lachy (~Lachy@public.cloak)
  12. # [01:27] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  13. # [01:31] * Joins: Lachy (~Lachy@public.cloak)
  14. # [01:33] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  15. # [01:36] * Joins: Lachy (~Lachy@public.cloak)
  16. # [01:38] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  17. # [01:40] * Joins: Lachy (~Lachy@public.cloak)
  18. # [01:43] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  19. # [01:44] * Joins: Lachy (~Lachy@public.cloak)
  20. # [01:59] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  21. # [02:06] * Joins: hank (~hank@public.cloak)
  22. # [02:32] * Joins: Lachy (~Lachy@public.cloak)
  23. # [02:42] * Quits: hank (~hank@public.cloak) (Ping timeout: 180 seconds)
  24. # [02:47] * Quits: Lachy (~Lachy@public.cloak) ("Textual IRC Client: www.textualapp.com")
  25. # [02:48] * Joins: Lachy (~Lachy@public.cloak)
  26. # [03:42] * Joins: hank (~hank@public.cloak)
  27. # [04:24] * Joins: gitbot (~gitbot@public.cloak)
  28. # [04:24] -gitbot:#testing- [web-platform-tests] zqzhang pushed 1 new commit to master: https://github.com/w3c/web-platform-tests/commit/71b37fb1b1dd227268e389cb358851ae88f98ff9
  29. # [04:24] -gitbot:#testing- web-platform-tests/master 71b37fb Zhiqiang Zhang: Merge pull request #1015 from w3c/submission/sideshowbarker/move-eventsource-idl...
  30. # [04:24] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  31. # [04:24] * Joins: gitbot (~gitbot@public.cloak)
  32. # [04:24] -gitbot:#testing- [web-platform-tests] zqzhang deleted submission/sideshowbarker/move-eventsource-idl at 1f66bf6: https://github.com/w3c/web-platform-tests/commit/1f66bf6
  33. # [04:24] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  34. # [04:25] * Quits: hank (~hank@public.cloak) (Ping timeout: 180 seconds)
  35. # [04:27] * Joins: gitbot (~gitbot@public.cloak)
  36. # [04:27] -gitbot:#testing- [web-platform-tests] zqzhang pushed 1 new commit to master: https://github.com/w3c/web-platform-tests/commit/c6bb8fca02647bfacd30715b2026db219381b0e6
  37. # [04:27] -gitbot:#testing- web-platform-tests/master c6bb8fc Zhiqiang Zhang: Merge pull request #1016 from w3c/submission/sideshowbarker/remove-duplicated-webstorage-idl...
  38. # [04:27] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  39. # [04:28] * Joins: gitbot (~gitbot@public.cloak)
  40. # [04:28] -gitbot:#testing- [web-platform-tests] zqzhang deleted submission/sideshowbarker/remove-duplicated-webstorage-idl at 583fd15: https://github.com/w3c/web-platform-tests/commit/583fd15
  41. # [04:28] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  42. # [04:34] * Joins: gitbot (~gitbot@public.cloak)
  43. # [04:34] -gitbot:#testing- [web-platform-tests] yuanyuanzou opened pull request #1017: fixed bug,the test will continue after stopped a few minutes (master...fixedbugcanot_stop_test) https://github.com/w3c/web-platform-tests/pull/1017
  44. # [04:34] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  45. # [04:47] * Joins: hank (~hank@public.cloak)
  46. # [05:21] * Joins: gitbot (~gitbot@public.cloak)
  47. # [05:21] -gitbot:#testing- [web-platform-tests] yuanyuanzou opened pull request #1018: Need to clear the progressbar to 0% when start the test (master...fixed_progressbar_issue) https://github.com/w3c/web-platform-tests/pull/1018
  48. # [05:21] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  49. # [05:29] * Quits: hank (~hank@public.cloak) (Ping timeout: 180 seconds)
  50. # [05:42] * Joins: gitbot (~gitbot@public.cloak)
  51. # [05:42] -gitbot:#testing- [web-platform-tests] yuanyuanzou opened pull request #1019: Show case path at the behind of progress bar (master...add_case_path) https://github.com/w3c/web-platform-tests/pull/1019
  52. # [05:42] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  53. # [06:02] * Joins: zcorpan (~zcorpan@public.cloak)
  54. # [07:35] * Joins: gitbot (~gitbot@public.cloak)
  55. # [07:35] -gitbot:#testing- [web-platform-tests] sideshowbarker pushed 1 new commit to submission/sideshowbarker/move-workers-idl: https://github.com/w3c/web-platform-tests/commit/53168510e48239ff4f78e8b4129a970cd26b20f2
  56. # [07:35] -gitbot:#testing- web-platform-tests/submission/sideshowbarker/move-workers-idl 5316851 Michael[tm] Smith: Removed workers/interfaces.html....
  57. # [07:35] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  58. # [07:37] * Disconnected
  59. # [07:40] * Attempting to rejoin channel #testing
  60. # [07:40] * Rejoined channel #testing
  61. # [07:40] * Topic is '"also, I hope no one actually reads this, but I have to admit that I've kind of gotten used to Critic" --darobin'
  62. # [07:40] * Set by plh on Mon Mar 31 21:50:34
  63. # [07:45] * Joins: gitbot (~gitbot@public.cloak)
  64. # [07:45] -gitbot:#testing- [web-platform-tests] sideshowbarker closed pull request #1018: Need to clear the progressbar to 0% when start the test (master...fixed_progressbar_issue) https://github.com/w3c/web-platform-tests/pull/1018
  65. # [07:45] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  66. # [08:05] * Joins: hank (~hank@public.cloak)
  67. # [08:34] * Joins: gitbot (~gitbot@public.cloak)
  68. # [08:34] -gitbot:#testing- [web-platform-tests] sideshowbarker closed pull request #1017: fixed bug,the test will continue after stopped a few minutes (master...fixedbugcanot_stop_test) https://github.com/w3c/web-platform-tests/pull/1017
  69. # [08:34] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  70. # [08:38] * Joins: gitbot (~gitbot@public.cloak)
  71. # [08:38] -gitbot:#testing- [web-platform-tests] sideshowbarker created submission/sideshowbarker/runner-stopped (+1 new commit): https://github.com/w3c/web-platform-tests/commit/a4b2fc6bfb1c
  72. # [08:38] -gitbot:#testing- web-platform-tests/submission/sideshowbarker/runner-stopped a4b2fc6 Michael[tm] Smith: Make stopped runner progressbar say Stopped.
  73. # [08:38] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  74. # [08:38] * Joins: gitbot (~gitbot@public.cloak)
  75. # [08:38] -gitbot:#testing- [web-platform-tests] sideshowbarker opened pull request #1021: Make stopped runner progressbar say Stopped. (master...submission/sideshowbarker/runner-stopped) https://github.com/w3c/web-platform-tests/pull/1021
  76. # [08:38] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  77. # [08:46] * Joins: gitbot (~gitbot@public.cloak)
  78. # [08:46] -gitbot:#testing- [web-platform-tests] sideshowbarker pushed 2 new commits to master: https://github.com/w3c/web-platform-tests/compare/0487db2b6705...abd81dcac998
  79. # [08:46] -gitbot:#testing- web-platform-tests/master 7a69236 yuanyuan.zou: Show case path at the behind of progress bar
  80. # [08:46] -gitbot:#testing- web-platform-tests/master abd81dc Michael[tm] Smith: Merge pull request #1019 from yuanyuanzou/add_case_path...
  81. # [08:46] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  82. # [08:46] * Joins: gitbot (~gitbot@public.cloak)
  83. # [08:46] -gitbot:#testing- [web-platform-tests] sideshowbarker closed pull request #1019: Show case path at the behind of progress bar (master...add_case_path) https://github.com/w3c/web-platform-tests/pull/1019
  84. # [08:46] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  85. # [08:51] * Joins: gitbot (~gitbot@public.cloak)
  86. # [08:51] -gitbot:#testing- [web-platform-tests] sideshowbarker closed pull request #1012: Should disable the manual/refer harness after click pause button (master...fixedpausedbug) https://github.com/w3c/web-platform-tests/pull/1012
  87. # [08:51] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  88. # [08:51] * Joins: gitbot (~gitbot@public.cloak)
  89. # [08:51] -gitbot:#testing- [web-platform-tests] sideshowbarker pushed 2 new commits to master: https://github.com/w3c/web-platform-tests/compare/abd81dcac998...41f230d5c068
  90. # [08:51] -gitbot:#testing- web-platform-tests/master 05b7ffc yuanyuan.zou: fixed bug of the test can be set passed or failed when paused the test
  91. # [08:51] -gitbot:#testing- web-platform-tests/master 41f230d Michael[tm] Smith: Merge pull request #1012 from yuanyuanzou/fixedpausedbug...
  92. # [08:51] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  93. # [08:51] * Joins: anssik (~uid10742@public.cloak)
  94. # [08:55] * Joins: Ms2ger (~Ms2ger@public.cloak)
  95. # [09:24] * Joins: darobin (rberjon@public.cloak)
  96. # [09:42] * Quits: hank (~hank@public.cloak) (Ping timeout: 180 seconds)
  97. # [10:06] * Quits: ptressel (~chatzilla@public.cloak) ("bbs")
  98. # [10:14] * Joins: ptressel (~chatzilla@public.cloak)
  99. # [10:48] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  100. # [10:50] * Joins: gitbot (~gitbot@public.cloak)
  101. # [10:50] -gitbot:#testing- [web-platform-tests] zqzhang pushed 1 new commit to master: https://github.com/w3c/web-platform-tests/commit/5d797b83473e77859658cc623fefbb13c0bb7e74
  102. # [10:50] -gitbot:#testing- web-platform-tests/master 5d797b8 Zhiqiang Zhang: Create tests for http://www.w3.org/TR/html-media-capture/
  103. # [10:50] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  104. # [10:51] * Joins: gitbot (~gitbot@public.cloak)
  105. # [10:51] -gitbot:#testing- [web-platform-tests] zqzhang closed pull request #306: Created tests for http://www.w3.org/TR/html-media-capture/ (master...submission/zqzhang/html-media-capture) https://github.com/w3c/web-platform-tests/pull/306
  106. # [10:51] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  107. # [11:04] * Joins: hank (~hank@public.cloak)
  108. # [11:08] * Joins: Lachy (~Lachy@public.cloak)
  109. # [11:09] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  110. # [11:16] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  111. # [11:31] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  112. # [11:33] * Joins: Lachy (~Lachy@public.cloak)
  113. # [11:46] * Quits: anssik (~uid10742@public.cloak) ("Connection closed for inactivity")
  114. # [11:49] * Quits: hank (~hank@public.cloak) (Ping timeout: 180 seconds)
  115. # [12:08] * Joins: MK (kawamori@public.cloak)
  116. # [12:17] * Quits: Lachy (~Lachy@public.cloak) (Client closed connection)
  117. # [12:18] * Joins: Lachy (~Lachy@public.cloak)
  118. # [12:21] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Client closed connection)
  119. # [12:22] <MikeSmith> darobin: thanks for the review of the Stopped thing
  120. # [12:22] <darobin> np
  121. # [12:23] <MikeSmith> yuanyuanzou doing some nice refining and bug-fixing of the Web runner UI
  122. # [12:25] <darobin> yes
  123. # [12:25] <darobin> it's cool to see new contributors
  124. # [12:26] <darobin> so, WebKit throttles tests that aren't running in the foreground: is there a way to disable that feature for test running?
  125. # [12:26] * Joins: gitbot (~gitbot@public.cloak)
  126. # [12:26] -gitbot:#testing- [web-platform-tests] sideshowbarker closed pull request #1021: Make stopped runner progressbar say Stopped. (master...submission/sideshowbarker/runner-stopped) https://github.com/w3c/web-platform-tests/pull/1021
  127. # [12:26] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  128. # [12:27] * Joins: gitbot (~gitbot@public.cloak)
  129. # [12:27] -gitbot:#testing- [web-platform-tests] sideshowbarker deleted submission/sideshowbarker/runner-stopped at a4b2fc6: https://github.com/w3c/web-platform-tests/commit/a4b2fc6
  130. # [12:27] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  131. # [12:28] <darobin> I can't even seem to find a list of command line args for Safari
  132. # [12:30] <MikeSmith> darobin: oh, didn't know it was doing that. But fwiw I've found that I don't always get consistent results in any browser if I run some tests in the background
  133. # [12:30] <MikeSmith> darobin: e.g., the 2dcontext tests
  134. # [12:31] <MikeSmith> darobin: anyway I think #webkit would be the quickest route to an answer, about the command-line args
  135. # [12:31] <darobin> MikeSmith: yeah, there could be some painting issues too I guess
  136. # [12:31] <darobin> good point
  137. # [12:32] <darobin> MikeSmith: hober pointed out the background throttling thing (which is a smart move otherwise) when I noted that Safari was passing something like 5% of the tests
  138. # [12:32] <darobin> and indeed, running it in the foreground immediately solved the problem
  139. # [12:32] <MikeSmith> oh wow
  140. # [13:07] * Joins: Lachy_ (~Lachy@public.cloak)
  141. # [13:09] * Quits: Lachy (~Lachy@public.cloak) (Ping timeout: 180 seconds)
  142. # [13:25] * Joins: dom (dom@public.cloak)
  143. # [13:37] <darobin> ah, trying to coerce test results out of browsers. wow. much fun. such pleasant.
  144. # [13:42] * Joins: Lachy (~Lachy@public.cloak)
  145. # [13:43] <jgraham> If it was easy just imagine how much time we would have to <del>sip cocktails on the beach</del><ins>hang around the job center</ins>
  146. # [13:46] * Quits: Lachy_ (~Lachy@public.cloak) (Ping timeout: 180 seconds)
  147. # [13:52] * Quits: wilhelm (~wilhelm@public.cloak) (Client closed connection)
  148. # [13:54] <darobin> yeah, you don't say
  149. # [13:54] <Ms2ger> jgraham, I'm sure I could still find work for you ;)
  150. # [13:54] <Ms2ger> darobin, otoh...
  151. # [13:55] <darobin> I particularly love WebKit Nightly: run tests, have to keep at front doing nothing else with your laptop otherwise it gets throttling, click save, pick location, quit... realise it has a bug with <a href=blob:> that makes it save nothing even though it pretends to
  152. # [13:57] <Ms2ger> "Moral of the story: always do what @zcorpan says"
  153. # [13:57] <darobin> Ms2ger: if there are cocktails to be made, there'll be work for me :)
  154. # [13:57] * Ms2ger makes his own cocktails
  155. # [13:57] <Ms2ger> Now what? :)
  156. # [13:57] <darobin> Ms2ger: you making them for everyone?
  157. # [13:58] <Ms2ger> Nah, just for me
  158. # [13:58] * Joins: zcorpan (~zcorpan@public.cloak)
  159. # [13:59] <darobin> well then
  160. # [13:59] <jgraham> darobin: With something that isn't safari you could try Xvfb, but I suppose that there isn't a way to run OSX apps headless like that
  161. # [13:59] <jgraham> And it's Apple so VMs are hard
  162. # [14:00] <darobin> jgraham: yeah, I don't know of OSX options
  163. # [14:00] <darobin> a VM for WebKit might be an option actually
  164. # [14:00] <jgraham> Conclusion: get Apple to provide their own test results :)
  165. # [14:00] <darobin> alternatively, I could build PhantomJS against the latest WebKit and run the whole thing headless, but I strongly suspect I'd get artefacts
  166. # [14:01] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  167. # [14:01] * Joins: zcorpan (~zcorpan@public.cloak)
  168. # [14:01] <jgraham> Yeah, that sounds unlikely to work
  169. # [14:01] <Ms2ger> Run Safari on Windows in a VM?
  170. # [14:02] <darobin> Ms2ger: yeah, I might try that actually
  171. # [14:02] <jgraham> Safari for Windows doesn't exist, right?
  172. # [14:02] * darobin wonders if the VM can make something believe it's at the front when it's not
  173. # [14:02] <jgraham> And you aren't supposed to run OSX in VMs
  174. # [14:02] <darobin> mmmmm
  175. # [14:02] <jgraham> (except with an OSX host)
  176. # [14:02] <darobin> actually there's this old mac laptop I could commandeer
  177. # [14:02] <jgraham> (I think)
  178. # [14:02] <darobin> just drop it in a corner, use it to run Safari tests
  179. # [14:02] <Ms2ger> jgraham, doesn't?
  180. # [14:02] <darobin> like I do with the Surface
  181. # [14:03] <jgraham> Ms2ger: I thought they stopped making it
  182. # [14:03] <darobin> jgraham: what I'm supposed to do? I don't care about that :)
  183. # [14:03] <Ms2ger> Oh, that's possible
  184. # [14:03] <darobin> well I just need a WebKit Nightly that runs
  185. # [14:03] <darobin> but anyway, I reckon commandeering an old Mac laptop will do the trick just fine
  186. # [14:03] <Ms2ger> I guess then we're back to my solution of not caring
  187. # [14:04] <darobin> that works too
  188. # [14:08] * Joins: wilhelm (~wilhelm@public.cloak)
  189. # [14:08] * Quits: wilhelm (~wilhelm@public.cloak) (Client closed connection)
  190. # [14:08] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  191. # [14:13] * Joins: wilhelm (~wilhelm@public.cloak)
  192. # [14:56] <tobie_> MikeSmith: What's the story around running IDL tests in workers?
  193. # [14:56] <tobie_> Does it even work at all?
  194. # [14:56] <Ms2ger> We talked about that yesterday
  195. # [14:57] <Ms2ger> Status is no, they don't
  196. # [14:59] <jgraham> Yeah, someone needs to understand idlharness.js
  197. # [15:00] <jgraham> Then update it to not assume a Window
  198. # [15:05] <Ms2ger> Well, we should support th.js in workers first
  199. # [15:05] <Ms2ger> I need to figure out what you thought about my attempt at that
  200. # [15:05] <Ms2ger> And then find time to finish it
  201. # [15:06] <tobie_> Ms2ger: right, I saw the thread.
  202. # [15:06] <tobie_> Ms2ger: is that code somewhere?>
  203. # [15:06] <Ms2ger> Yes
  204. # [15:07] <tobie_> would you have a URL you could share?
  205. # [15:07] <tobie_> Or a patch or something?
  206. # [15:07] <Ms2ger> https://github.com/Ms2ger/testharness.js/compare/worker-support
  207. # [15:08] <Ms2ger> And https://github.com/Ms2ger/web-platform-tests/compare/auto-workers
  208. # [15:09] <Ms2ger> tobie_, ^
  209. # [15:13] <tobie_> ty
  210. # [15:13] <tobie_> How ready is it?
  211. # [15:13] <tobie_> Seems rather complete to me.
  212. # [15:14] <Ms2ger> I think I asked jgraham to look at it, I don't remember what he thought
  213. # [15:16] * jgraham neither
  214. # [15:16] <jgraham> Did you open a review?
  215. # [15:17] <Ms2ger> Doesn't look like it
  216. # [15:18] <tobie_> Ms2ger: in retrospect, I think we'd have been better off not to rely on the presence of a DOM, etc. then to special case for workers. But that's always easy to say after the fact. :)
  217. # [15:18] <tobie_> s/then/than/
  218. # [15:21] <jgraham> Not to rely where? In testharness.js? There isn't *that* much that assumes a DOM
  219. # [15:21] <Ms2ger> Yeah, some for reporting
  220. # [15:21] <jgraham> the postMessage stuff, mostly
  221. # [15:21] <tobie_> (which I'm to blame for)
  222. # [15:21] <Ms2ger> And the meta timeout
  223. # [15:21] <jgraham> We could try and get rid of that, although some things depend on it now
  224. # [15:22] <jgraham> Yeah
  225. # [15:22] <tobie_> re timeout, the worker should somehow receive that as an option
  226. # [15:22] <tobie_> (or as a message?)
  227. # [15:23] <jgraham> If js had some sort of modules, we could split things into the part that has to be loaded in a browsing context and the part that can be loaded in a worker
  228. # [15:23] <jgraham> perhaps
  229. # [15:24] <Ms2ger> I guess we could move the div id=log handling into threport.js
  230. # [15:25] <Ms2ger> I don't know if it'd be a big win
  231. # [15:25] <tobie_> or we could just have the main thread terminate the worker after timeout.
  232. # [15:26] <jgraham> Moving stuff into thr.js sounds like a bad idea; the idea is that you should be able to blow that away without affecting functionality
  233. # [15:28] * Joins: gitbot (~gitbot@public.cloak)
  234. # [15:28] -gitbot:#testing- [web-platform-tests] jgraham created jgraham/servers_alive (+1 new commit): https://github.com/w3c/web-platform-tests/commit/d3317948c9ce
  235. # [15:28] -gitbot:#testing- web-platform-tests/jgraham/servers_alive d331794 James Graham: Add method to check if server process is running.
  236. # [15:28] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  237. # [15:28] * Joins: gitbot (~gitbot@public.cloak)
  238. # [15:28] -gitbot:#testing- [web-platform-tests] jgraham opened pull request #1023: Add method to check if server process is running. (master...jgraham/servers_alive) https://github.com/w3c/web-platform-tests/pull/1023
  239. # [15:28] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  240. # [15:28] <jgraham> https://critic.hoppipolla.co.uk/r/1654 r?
  241. # [15:30] <tobie_> jgraham: agreed.
  242. # [15:33] <tobie_> What you might want to have however is a js page that contains all of the basic (dom-less) code, which could be used directly from within workers, and which would get pulled by testharness.js [pretty much like the CSS file gets pulled.
  243. # [15:35] <tobie_> idlharness should work out of the box, imho.
  244. # [15:38] <jgraham> Can't manipulate the DOM before the tests have run (the CSS file only gets added afterwards)
  245. # [15:40] <tobie_> you'd need a server-side build step, then. :(
  246. # [15:41] <tobie_> Which you'd understandably would want to avoid.
  247. # [15:42] <jgraham> Yeah, like I said, it feels like this will be easier once es modules work
  248. # [15:42] <jgraham> But that could be years away
  249. # [15:42] <tobie_> yeah...
  250. # [15:44] <tobie_> we could change the test to all require an extra JS file on top of testharness.js
  251. # [15:44] <Ms2ger> Please no
  252. # [15:45] <jgraham> tobie_: I'll let you try and sell that to bz…
  253. # [15:45] <jgraham> Ms2ger's reaction was relatively mild
  254. # [15:46] <tobie_> Well, I'm not sure I measure what that implies. Fixing that for the whole repo seems painful bu relatively doable.
  255. # [15:47] <Ms2ger> New tests
  256. # [15:47] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  257. # [15:47] <tobie_> I haven't thought too much about collateral damage.
  258. # [15:47] <tobie_> But agree that kind of sucks.
  259. # [15:47] <tobie_> Other option would be a built step for testharness.js
  260. # [15:48] <tobie_> But maybe it's easiest to just wrap the DOM-dependent code together the way it's been done so far.
  261. # [15:51] <jgraham> Yeah, so it seems like the best option might be to investigate if there's some internal restructuring that makes the boundaries clearer
  262. # [15:52] <jgraham> (anyone want to look at that review btw? It's only really two lines but some upstream changes need it)
  263. # [15:54] * Joins: Automate_ (~AutomatedTester@public.cloak)
  264. # [15:54] <jgraham> s/up/down/
  265. # [15:54] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Ping timeout: 180 seconds)
  266. # [15:58] * jgraham thinks Simon typoed "Euston is a horrible dive that would be well served by demolition"
  267. # [16:06] <darobin> assert_equals: expected function "function addEventListener() { [native code] }" but got function "function addEventListener() { [native code] }"
  268. # [16:06] <darobin> why, but thank you test harness!
  269. # [16:07] <jgraham> darobin: Patches welcome :p
  270. # [16:08] <darobin> jgraham: I have to wrap other stuff up first, but I'm tucking that one away
  271. # [16:08] <darobin> mmm, actually I'll tuck it into a bug
  272. # [16:09] <Ms2ger> darobin, where's that?
  273. # [16:10] <darobin> Ms2ger: first failure for Fx at http://www.w3c-test.org/html/browsers/the-window-object/window-properties.html
  274. # [16:11] <tobie_> darobin: why is this stuff not idlharness?
  275. # [16:11] <darobin> tobie_: patches welcome!
  276. # [16:12] <darobin> srsly tobie_, no idea, but the test is signed Ms2ger
  277. # [16:12] <tobie_> That a good enough answer.
  278. # [16:12] <darobin> so he may have a reason, or failing that a fix
  279. # [16:12] <tobie_> That was in response to your first comment
  280. # [16:13] <darobin> heh
  281. # [16:13] <jgraham> Did I not paste https://pastebin.mozilla.org/5258980 in here? Latest unstable tests
  282. # [16:13] <Ms2ger> Expected, the patch for Window is bouncing
  283. # [16:13] <Ms2ger> tobie_, too much Window-specific stuff
  284. # [16:14] <tobie_> Ms2ger: not sure what you mean/
  285. # [16:14] <jgraham> idlharness.js probably doesn't cover all the things that only apply to Window objects
  286. # [16:15] <jgraham> either because they are described in prose or because they are only used for Window
  287. # [16:15] <Ms2ger> Thank you for the translation, jgraham :)
  288. # [16:15] <darobin> put differently, Window is weird
  289. # [16:15] <darobin> it's possible that idlharness handles more than it used to
  290. # [16:15] <darobin> but probably not all the way
  291. # [16:15] <jgraham> And the patch bouncing bit is about the gecko patch to convert Window to use WebIDL
  292. # [16:16] <darobin> ah, gotcha
  293. # [16:16] <jgraham> Which I assume from Ms2ger's comment, didn't stick
  294. # [16:23] * Quits: ptressel (~chatzilla@public.cloak) ("ZZZzzz...")
  295. # [16:24] * Quits: Ms2ger (~Ms2ger@public.cloak) (Ping timeout: 180 seconds)
  296. # [16:37] * Joins: Ms2ger (~Ms2ger@public.cloak)
  297. # [17:22] * Quits: darobin (rberjon@public.cloak) (Client closed connection)
  298. # [17:22] * Joins: darobin (rberjon@public.cloak)
  299. # [17:29] * Quits: darobin (rberjon@public.cloak) (Ping timeout: 180 seconds)
  300. # [17:48] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  301. # [18:01] * Joins: zcorpan (~zcorpan@public.cloak)
  302. # [18:05] * Joins: lmclister (~lmclister@public.cloak)
  303. # [18:06] * jgraham looks confused
  304. # [18:07] <jgraham> zcorpan: You might know
  305. # [18:07] <jgraham> But anyone else who is interested
  306. # [18:07] <jgraham> http://w3c-test.org/2dcontext/fill-and-stroke-styles/2d.pattern.image.broken.html seems to fail in Gecko + Chrome
  307. # [18:07] <jgraham> It also seems to be racy
  308. # [18:08] <jgraham> When I change it to create the broken <img> in script and do the test on onerror, it passes
  309. # [18:08] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  310. # [18:08] <jgraham> But the test there should only run on load, so I can't see ehy it is racy or why the two situations are different
  311. # [18:09] <jgraham> Apart from the fact that the image isn't in the document when it passes
  312. # [18:11] <jgraham> Ah!
  313. # [18:11] <jgraham> Now it fails!
  314. # [18:11] <jgraham> But I don't know if this fixed the possible race :|
  315. # [18:12] <jgraham> OK, so it seems to depend on just how broken the image is whether the test fails
  316. # [18:12] * Joins: rhauck (~Adium@public.cloak)
  317. # [18:24] <Ms2ger> That seems plausible
  318. # [18:28] * Quits: dom (dom@public.cloak) ("")
  319. # [18:32] <jgraham> Yeah, so it seems like the test fails if the PNG has a valid header, but passes if it doesn't, or something
  320. # [18:32] <jgraham> Which is a bug in implementations, I think
  321. # [18:32] <jgraham> But doesn't explain the randomness
  322. # [18:35] <jgraham> And it doesn't seem to be random when loaded over w3c-test.org, which suggests it's a race on network timing
  323. # [18:36] <jgraham> Or in Chrome, which suggests a gecko bug :|
  324. # [18:36] <jgraham> (although not conclusively because Chrome could just be more brokwn than Gecko)
  325. # [18:44] <jgraham> Ms2ger: Third time lucky? https://critic.hoppipolla.co.uk/d3317948?review=1654
  326. # [18:46] * Ms2ger passes
  327. # [18:47] * jgraham cries
  328. # [18:47] <jgraham> People, the entire patch is
  329. # [18:47] <jgraham> def is_alive(self):
  330. # [18:47] <Ms2ger> Perhaps if you point me to the docs of self.proc
  331. # [18:47] <jgraham> return self.proc.is_alive()
  332. # [18:48] <jgraham> self.proc is a multiprocessing.Process
  333. # [18:48] <Ms2ger> jgraham, lies
  334. # [18:48] <Ms2ger> It's return self.proc.is_alive
  335. # [18:48] <jgraham> https://docs.python.org/2/library/multiprocessing.html#multiprocessing.Process.is_alive
  336. # [18:48] <jgraham> Probably needs some () then
  337. # [18:49] <jgraham> Aren't you glad you reviewed it now
  338. # [18:49] <Ms2ger> That's why I didn't want to sign it off ;)
  339. # [18:52] * Joins: gitbot (~gitbot@public.cloak)
  340. # [18:52] -gitbot:#testing- [web-platform-tests] jgraham pushed 1 new commit to jgraham/servers_alive: https://github.com/w3c/web-platform-tests/commit/96e081b26f15755189cb9c7a3734d848994bfa33
  341. # [18:52] -gitbot:#testing- web-platform-tests/jgraham/servers_alive 96e081b James Graham: fixup! Add method to check if server process is running.
  342. # [18:52] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  343. # [18:53] * Quits: scottgonzalez (~scott_gonzalez@public.cloak) (scottgonzalez)
  344. # [18:57] <jgraham> So do you want to sign off on the fixed version?
  345. # [18:58] <jgraham> Ms2ger: Thanks
  346. # [18:59] * Joins: gitbot (~gitbot@public.cloak)
  347. # [18:59] -gitbot:#testing- [web-platform-tests] jgraham pushed 1 new commit to master: https://github.com/w3c/web-platform-tests/commit/77f8216282cbd1113005a94da02267092d8f326b
  348. # [18:59] -gitbot:#testing- web-platform-tests/master 77f8216 jgraham: Merge pull request #1023 from w3c/jgraham/servers_alive...
  349. # [18:59] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  350. # [19:17] * Quits: MK (kawamori@public.cloak) (Ping timeout: 180 seconds)
  351. # [19:20] * Joins: MK (kawamori@public.cloak)
  352. # [19:30] * Joins: scott_gonzalez (~scott_gonzalez@public.cloak)
  353. # [19:43] * Quits: MK (kawamori@public.cloak) (Ping timeout: 180 seconds)
  354. # [20:21] * Quits: scott_gonzalez (~scott_gonzalez@public.cloak) (scott_gonzalez)
  355. # [20:34] * Joins: scott_gonzalez (~scott_gonzalez@public.cloak)
  356. # [21:07] * Quits: Ms2ger (~Ms2ger@public.cloak) ("nn")
  357. # [21:42] * Joins: zcorpan (~zcorpan@public.cloak)
  358. # [21:45] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  359. # [21:54] * Joins: Lachy (~Lachy@public.cloak)
  360. # [23:56] * Quits: rhauck (~Adium@public.cloak) ("Leaving.")
  361. # Session Close: Wed May 28 00:00:01 2014

The end :)