/irc-logs / w3c / #testing / 2015-02-26 / end

Options:

Previous day, Next day

  1. # Session Start: Thu Feb 26 00:00:00 2015
  2. # Session Ident: #testing
  3. # [00:03] * Quits: scott_gonzalez (~scott_gonzalez@public.cloak) (scott_gonzalez)
  4. # [00:03] * terri is now known as terri_offline
  5. # [00:09] * Quits: plh (plehegar@public.cloak) ("Leaving")
  6. # [00:26] * terri_offline is now known as terri
  7. # [00:43] * Joins: zcorpan_ (~zcorpan@public.cloak)
  8. # [00:43] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  9. # [00:50] * Quits: zcorpan_ (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  10. # [01:10] * Joins: gitbot (~gitbot@public.cloak)
  11. # [01:10] -gitbot:#testing- [web-platform-tests] luniv opened pull request #1645: Add test for default prevented 'beforescriptexecute' event (master...script-before-event-prevent-default) https://github.com/w3c/web-platform-tests/pull/1645
  12. # [01:10] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  13. # [01:28] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Client closed connection)
  14. # [01:32] * Joins: scott_gonzalez (~scott_gonzalez@public.cloak)
  15. # [01:53] * Joins: gitbot (~gitbot@public.cloak)
  16. # [01:53] -gitbot:#testing- [web-platform-tests] zqzhang closed pull request #1623: Check getBattery() always return the same promise (master...battery-promise) https://github.com/w3c/web-platform-tests/pull/1623
  17. # [01:53] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  18. # [01:53] * Joins: gitbot (~gitbot@public.cloak)
  19. # [01:53] -gitbot:#testing- [web-platform-tests] zqzhang pushed 2 new commits to master: https://github.com/w3c/web-platform-tests/compare/9a2c04e06cb4...bfd2ab303456
  20. # [01:53] -gitbot:#testing- web-platform-tests/master 55c8997 Zhiqiang Zhang: Check getBattery() always return the same promise...
  21. # [01:53] -gitbot:#testing- web-platform-tests/master bfd2ab3 Zhiqiang Zhang: Merge pull request #1623 from zqzhang/battery-promise...
  22. # [01:53] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  23. # [01:54] * Joins: gitbot (~gitbot@public.cloak)
  24. # [01:54] -gitbot:#testing- [web-platform-tests] zqzhang closed pull request #1644: Update description of battery tests (master...BatteryManager) https://github.com/w3c/web-platform-tests/pull/1644
  25. # [01:54] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  26. # [02:11] * heycam is now known as heycam|away
  27. # [04:02] * Quits: bhill2 (~bhill2@public.cloak) (Ping timeout: 180 seconds)
  28. # [04:31] * denis is now known as Denis
  29. # [04:31] * Denis is now known as denis
  30. # [04:40] * heycam|away is now known as heycam
  31. # [04:46] * terri is now known as terri_offline
  32. # [05:39] * Joins: MK (kawamori@public.cloak)
  33. # [06:16] * Quits: scott_gonzalez (~scott_gonzalez@public.cloak) (Ping timeout: 180 seconds)
  34. # [06:39] * Joins: playjax (~chatzilla@public.cloak)
  35. # [08:25] * heycam is now known as heycam|away
  36. # [08:49] * Joins: Ms2ger (~Ms2ger@public.cloak)
  37. # [08:55] * Joins: gitbot (~gitbot@public.cloak)
  38. # [08:55] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #1645: Add test for default prevented 'beforescriptexecute' event (master...script-before-event-prevent-default) https://github.com/w3c/web-platform-tests/pull/1645
  39. # [08:55] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  40. # [08:55] * Joins: gitbot (~gitbot@public.cloak)
  41. # [08:55] -gitbot:#testing- [web-platform-tests] Ms2ger pushed 2 new commits to master: https://github.com/w3c/web-platform-tests/compare/10131c2d3700...79dfb1f28534
  42. # [08:55] -gitbot:#testing- web-platform-tests/master 2027376 James Gilbertson: Add test for default prevented 'beforescriptexecute' event
  43. # [08:55] -gitbot:#testing- web-platform-tests/master 79dfb1f Ms2ger: Merge pull request #1645 from luniv/script-before-event-prevent-default...
  44. # [08:55] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  45. # [08:55] <Ms2ger> Yay for Servo devs ^
  46. # [08:56] * Joins: zcorpan (~zcorpan@public.cloak)
  47. # [09:15] * Quits: MK (kawamori@public.cloak) (Ping timeout: 180 seconds)
  48. # [09:17] <Ms2ger> denis, r? https://critic.hoppipolla.co.uk/r/4088
  49. # [10:35] * Quits: playjax (~chatzilla@public.cloak) (Ping timeout: 180 seconds)
  50. # [10:40] <jgraham> Ms2ger: Nice!
  51. # [10:43] * Joins: Lachy (~Lachy@public.cloak)
  52. # [11:07] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  53. # [11:08] * Joins: gitbot (~gitbot@public.cloak)
  54. # [11:08] -gitbot:#testing- [web-platform-tests] Ms2ger opened pull request #1646: Rewrite the test for reentering event dispatching. (master...reenter) https://github.com/w3c/web-platform-tests/pull/1646
  55. # [11:08] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  56. # [11:28] * Joins: MK (kawamori@public.cloak)
  57. # [11:36] * Quits: MK (kawamori@public.cloak) (Ping timeout: 180 seconds)
  58. # [12:03] * Joins: MK (kawamori@public.cloak)
  59. # [12:53] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Client closed connection)
  60. # [12:54] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  61. # [13:01] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Ping timeout: 180 seconds)
  62. # [13:28] * Quits: terri_offline (~terri@public.cloak) (Ping timeout: 180 seconds)
  63. # [13:40] * Joins: playjax (~chatzilla@public.cloak)
  64. # [13:59] * Quits: playjax (~chatzilla@public.cloak) (Client closed connection)
  65. # [14:06] * Joins: scott_gonzalez (~scott_gonzalez@public.cloak)
  66. # [14:12] * Quits: MK (kawamori@public.cloak) (Ping timeout: 180 seconds)
  67. # [14:14] * Joins: MK (kawamori@public.cloak)
  68. # [14:16] * Joins: plh (plehegar@public.cloak)
  69. # [14:26] * Joins: terri_offline (~terri@public.cloak)
  70. # [14:27] * terri_offline is now known as terri
  71. # [14:32] * Joins: darobin (rberjon@public.cloak)
  72. # [14:35] * Joins: gitbot (~gitbot@public.cloak)
  73. # [14:35] -gitbot:#testing- [testharness.js] zcorpan closed pull request #105: Remove support for IDL exceptions. (master...rm-exn) https://github.com/w3c/testharness.js/pull/105
  74. # [14:35] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  75. # [14:35] * Joins: gitbot (~gitbot@public.cloak)
  76. # [14:35] -gitbot:#testing- [testharness.js] zcorpan pushed 3 new commits to master: https://github.com/w3c/testharness.js/compare/6b0d7bf306ad...db7189340cdb
  77. # [14:35] -gitbot:#testing- testharness.js/master 8230e56 Ms2ger: Remove IDLException....
  78. # [14:35] -gitbot:#testing- testharness.js/master 78eaec5 Ms2ger: Collapse IdlExceptionOrInterface and IdlInterface.
  79. # [14:35] -gitbot:#testing- testharness.js/master db71893 Simon Pieters: Merge pull request #105 from Ms2ger/rm-exn...
  80. # [14:35] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  81. # [14:40] * Quits: MK (kawamori@public.cloak) (Ping timeout: 180 seconds)
  82. # [14:40] * Joins: MK (kawamori@public.cloak)
  83. # [15:01] * Quits: MK (kawamori@public.cloak) (Ping timeout: 180 seconds)
  84. # [15:26] <darobin> jgraham: do you know why pages on http://testthewebforward.org/ try to load something from tracking.webplatform.org?
  85. # [15:29] <jgraham> darobin: 303 tobie
  86. # [15:29] <darobin> heh
  87. # [15:30] <darobin> actually, given there was a payload I'd suggest 307
  88. # [15:30] <darobin> </pedant>
  89. # [15:31] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  90. # [15:45] * Quits: Ms2ger (~Ms2ger@public.cloak) (Ping timeout: 180 seconds)
  91. # [16:37] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  92. # [17:11] * Quits: darobin (rberjon@public.cloak) (Client closed connection)
  93. # [17:38] * Joins: zcorpan (~zcorpan@public.cloak)
  94. # [18:03] * Joins: Ms2ger (~Ms2ger@public.cloak)
  95. # [18:04] * Joins: gitbot (~gitbot@public.cloak)
  96. # [18:04] -gitbot:#testing- [web-platform-tests] Ms2ger created scriptexecute (+2 new commits): https://github.com/w3c/web-platform-tests/compare/63b9a3874dde^...ae787c5d1ccc
  97. # [18:04] -gitbot:#testing- web-platform-tests/scriptexecute 63b9a38 Ms2ger: Submit Opera's before/afterscriptexecute events tests.
  98. # [18:04] -gitbot:#testing- web-platform-tests/scriptexecute ae787c5 Ms2ger: Fix typos.
  99. # [18:04] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  100. # [18:13] * Joins: gitbot (~gitbot@public.cloak)
  101. # [18:13] -gitbot:#testing- [web-platform-tests] Ms2ger opened pull request #1647: Submit Opera's before/afterscriptexecute events tests. (master...scriptexecute) https://github.com/w3c/web-platform-tests/pull/1647
  102. # [18:13] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  103. # [18:20] <Ms2ger> positional arguments:
  104. # [18:20] <Ms2ger> path an integer for the accumulator
  105. # [18:24] <jgraham> ?
  106. # [18:27] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Client closed connection)
  107. # [18:35] <Ms2ger> jgraham, stuctured-render
  108. # [18:38] <jgraham> Ms2ger: 303 Manishearth
  109. # [18:44] * Joins: r12a (rishida@public.cloak)
  110. # [18:44] <r12a> can someone help with a github submission question?
  111. # [18:45] <jgraham> Sure, why not
  112. # [18:45] <r12a> thanks james
  113. # [18:45] <r12a> i just committed some test updates
  114. # [18:45] * jgraham searches for a third channel in which to simultaneously answer git questions
  115. # [18:45] <r12a> i created a branch, following the 101 guidelines
  116. # [18:45] <Ms2ger> Ha
  117. # [18:45] <r12a> i ended up with a branch called counter-style-corrections
  118. # [18:46] <r12a> i think it should have been r12a/counter-style-corrections
  119. # [18:46] <Ms2ger> Nah
  120. # [18:46] <jgraham> r12a: In your local repository?
  121. # [18:46] <r12a> when i try to raise the pull request, i see a huge comparison list
  122. # [18:46] <r12a> erm
  123. # [18:46] <jgraham> Oh, if you already pushed, I'll look at the GH UI
  124. # [18:46] <Ms2ger> csswg-test?
  125. # [18:47] <r12a> yes please
  126. # [18:47] <r12a> correct
  127. # [18:47] <Ms2ger> Ah
  128. # [18:47] <Ms2ger> I see
  129. # [18:48] <Ms2ger> Github's fault
  130. # [18:48] * r12a thanks goodness for that
  131. # [18:48] <Ms2ger> https://github.com/w3c/csswg-test/compare/master...r12a:counter-style-corrections
  132. # [18:48] <Ms2ger> There's been a mishap that caused github to think that astearns's fork is the canonical repo
  133. # [18:49] <r12a> yes, i wondered why his name kept appearing in the url
  134. # [18:49] <r12a> what should i do to avoid this in future?
  135. # [18:49] <r12a> and should i name my branches r12a/.... ?
  136. # [18:49] <Ms2ger> jgraham, do you recall who dealt with the issue last time?
  137. # [18:50] <Ms2ger> r12a, no real need; it's already namespaced by being in your fork
  138. # [18:50] <r12a> ok
  139. # [18:51] <r12a> btw, i created the branch in the terminal, but committed and pushed using SmartGit - that shouldn't have made any difference, should it?
  140. # [18:51] <jgraham> Ms2ger: Someone email github support I think
  141. # [18:51] <Ms2ger> Everything appears to be in order
  142. # [18:51] <jgraham> MikeSmith, perhaps
  143. # [18:52] <r12a> so should i go ahead now and create the pull request from https://github.com/w3c/csswg-test/compare/master...r12a:counter-style-corrections ?
  144. # [18:53] <Ms2ger> Yep
  145. # [18:53] <r12a> ok
  146. # [18:54] <r12a> done, thanks !
  147. # [18:54] <Ms2ger> Np
  148. # [18:54] <Ms2ger> Let's see if we can have MikeSmith fix this up :)
  149. # [18:54] <r12a> so i assume that next time i try to do this, in about a week, the problem will be sorted ?
  150. # [18:55] <r12a> and i don't need to ask how you came up with that page ?
  151. # [18:56] * Joins: gitbot (~gitbot@public.cloak)
  152. # [18:56] -gitbot:#testing- [web-platform-tests] jgraham created jgraham/https (+1 new commit): https://github.com/w3c/web-platform-tests/commit/45ee93deb63c
  153. # [18:56] -gitbot:#testing- web-platform-tests/jgraham/https 45ee93d James Graham: Add support for top-level https tests
  154. # [18:56] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  155. # [18:56] * Joins: gitbot (~gitbot@public.cloak)
  156. # [18:56] -gitbot:#testing- [wptrunner] jgraham created jgraham/https (+1 new commit): https://github.com/w3c/wptrunner/commit/12caf6e58b7d
  157. # [18:56] -gitbot:#testing- wptrunner/jgraham/https 12caf6e James Graham: Add support for running top-level https tests
  158. # [18:56] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  159. # [18:57] * Joins: gitbot (~gitbot@public.cloak)
  160. # [18:57] -gitbot:#testing- [web-platform-tests] jgraham opened pull request #1648: Add support for top-level https tests (master...jgraham/https) https://github.com/w3c/web-platform-tests/pull/1648
  161. # [18:57] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  162. # [18:57] <Ms2ger> r12a, did you make it to https://github.com/astearns/csswg-test/compare/master...r12a:counter-style-corrections ?
  163. # [18:57] <r12a> yes, i think so - that looks familiar
  164. # [18:57] * Joins: gitbot (~gitbot@public.cloak)
  165. # [18:57] -gitbot:#testing- [wptrunner] jgraham opened pull request #77: Add support for running top-level https tests (jgraham/pause_after_test...jgraham/https) https://github.com/w3c/wptrunner/pull/77
  166. # [18:57] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  167. # [18:57] * r12a checks...
  168. # [18:58] <Ms2ger> You can click the "base fork" widget at the top and select w3c instead of astearns
  169. # [18:59] <r12a> this is where i kept ending up: https://github.com/astearns/csswg-test/compare/master...r12a:counter-style-corrections?expand=1
  170. # [19:01] <r12a> ah, that did it, thanks ms2ger
  171. # [19:01] <Ms2ger> Happy to help :)
  172. # [19:04] * Joins: gitbot (~gitbot@public.cloak)
  173. # [19:04] -gitbot:#testing- [wptrunner] jgraham pushed 1 new commit to jgraham/pause_after_test: https://github.com/w3c/wptrunner/commit/215a4d30d64a2a88ce24633ed963e8aed374c820
  174. # [19:04] -gitbot:#testing- wptrunner/jgraham/pause_after_test 215a4d3 James Graham: fixup! Add ability to pause after running tests.
  175. # [19:04] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  176. # [19:07] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  177. # [19:07] * Joins: zcorpan (~zcorpan@public.cloak)
  178. # [19:11] * Joins: MK (kawamori@public.cloak)
  179. # [19:13] <jgraham> Any more opinions on how to do query strings in test urls?
  180. # [19:13] <jgraham> zcorpan, Ms2ger, darobin^
  181. # [19:13] <Ms2ger> Not me
  182. # [19:15] <zcorpan> jgraham: no opinion
  183. # [19:16] <jgraham> You people are *useless* :p
  184. # [19:16] <jgraham> ;)
  185. # [19:17] <ato> Isn't this a dream situation where you just get to make something up?
  186. # [19:17] <jgraham> If we did <meta name=something> do you have any desire to bikeshed the value of 'something'?
  187. # [19:17] <jgraham> All I have is 'test' which is fairly terrible
  188. # [19:17] <Ms2ger> Fairly
  189. # [19:17] <Ms2ger> I'm not quite sure I ever saw the point of query strings in test urls
  190. # [19:18] <jgraham> ato: It's the beginning of a nightmare situation where I do something, it turns out to be awful, and no one else can be blames ;)
  191. # [19:18] <ato> Haha
  192. # [19:18] <zcorpan> jgraham: variant
  193. # [19:18] <Ms2ger> We blame you anyway
  194. # [19:18] <zcorpan> jgraham: subtest
  195. # [19:18] <zcorpan> jgraham: papayawhip
  196. # [19:19] <jgraham> zcorpan: So <meta name="variant" content=""> would give just the plain url (obviously you wouldn't include that if you didn't also have other variants)
  197. # [19:19] <jgraham> And <meta name="variant" content="?foo=1"> would give the foo=1 variant
  198. # [19:19] <jgraham> Seems OK
  199. # [19:20] <jgraham> Ms2ger: Well e.g. the html5lib tests have three (or four?) varaints that are all the same test data but run with slight variations on the harness
  200. # [19:21] <Ms2ger> Variants that you'd all want to run?
  201. # [19:21] <jgraham> Copying the tests for each varaint seems very wasteful
  202. # [19:21] <jgraham> Sure. Creating the dom via data urls, document.write and document.write one character at a time
  203. # [19:22] <jgraham> Plus something innerHTML, which I suppose must be a bit different
  204. # [19:22] <Ms2ger> I guess that somewhat useful
  205. # [19:22] * jgraham forgets the details
  206. # [19:22] <Ms2ger> And probably painful to just do in one test
  207. # [19:24] <jgraham> bholley also wants it for the media tests so that they can run with all codecs. This way you could also disable all the tests with some codec if you don't see the value in running them multiple times (which is foolip's position)
  208. # [19:24] <Ms2ger> Okay, I'm moderately convinced
  209. # [19:30] * Parts: r12a (rishida@public.cloak) (Leaving...)
  210. # [19:50] * plh cleans a bit email addresses on public-test-infra list
  211. # [19:54] <jgraham> So the problem with this <meta> proposal is that you can't have a different timeout or different references per variant. I guess references aren't too much of a bother, but should ref urls get the same query/fragment as the main test url?
  212. # [19:54] <jgraham> Not sure if there's anything to do about timeouts really
  213. # [19:56] <jgraham> Threading the variant through more than one reference url would be bothersome
  214. # [19:56] <jgraham> Maybe it should only work for testharness tests for now
  215. # [19:58] * Quits: Lachy (~Lachy@public.cloak) ("My MacBook Pro has gone to sleep. ZZZzzz…")
  216. # [20:05] <zcorpan> testharness-only i'd say, until someone really wants this for reftests (if ever)
  217. # [20:52] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  218. # [21:03] * Joins: zcorpan (~zcorpan@public.cloak)
  219. # [21:12] * Joins: gitbot (~gitbot@public.cloak)
  220. # [21:12] -gitbot:#testing- [web-platform-tests] Ms2ger opened pull request #1649: Fix the manifest.load call in the test runner. (master...fix-manifest) https://github.com/w3c/web-platform-tests/pull/1649
  221. # [21:12] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  222. # [21:23] * heycam|away is now known as heycam
  223. # [21:32] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  224. # [21:48] <Ms2ger> jgraham, r?
  225. # [22:00] * Joins: zcorpan (~zcorpan@public.cloak)
  226. # [22:03] * Joins: gitbot (~gitbot@public.cloak)
  227. # [22:03] -gitbot:#testing- [web-platform-tests] Ms2ger pushed 1 new commit to scriptexecute: https://github.com/w3c/web-platform-tests/commit/c3769dc9ada9a0d646bee22f5f4003b18f3a37fa
  228. # [22:03] -gitbot:#testing- web-platform-tests/scriptexecute c3769dc Ms2ger: Fix whitespace.
  229. # [22:03] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  230. # [22:07] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  231. # [22:08] <Ms2ger> Something you never want to see:
  232. # [22:08] <Ms2ger> There were 20283 errors (INDENT TABS: 20283)
  233. # [22:24] * Joins: gitbot (~gitbot@public.cloak)
  234. # [22:24] -gitbot:#testing- [web-platform-tests] Ms2ger opened pull request #1650: Lint more of html. (master...lint-html) https://github.com/w3c/web-platform-tests/pull/1650
  235. # [22:24] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  236. # [23:01] * Joins: zcorpan (~zcorpan@public.cloak)
  237. # [23:51] * Joins: gitbot (~gitbot@public.cloak)
  238. # [23:51] -gitbot:#testing- [web-platform-tests] jgraham closed pull request #1649: Fix the manifest.load call in the test runner. (master...fix-manifest) https://github.com/w3c/web-platform-tests/pull/1649
  239. # [23:51] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  240. # Session Close: Fri Feb 27 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