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

Options:

Previous day, Next day

  1. # Session Start: Wed Jan 21 00:00:00 2015
  2. # Session Ident: #testing
  3. # [00:13] * Quits: plh (plehegar@public.cloak) ("Leaving")
  4. # [00:42] <plinss> jgraham: so when we have multiple references we have two cases (and they may overlap)
  5. # [00:43] <plinss> the test links to a ref, which may link to other refs (or tests), which means the test must [mis]match all references in the chain
  6. # [00:43] <jgraham> plinss: Is that actually how it works?
  7. # [00:43] <plinss> or: the test has multiple rel=‘[mis]match’, which means the test must match one of the refs (and any refs chained from that one)
  8. # [00:43] <plinss> yes
  9. # [00:44] <jgraham> It sounds like you are saying A != B != C generates the comparisons A != B and A != C whereas I assume it is supposed to be B != C
  10. # [00:44] <plinss> so reference files may have rel=‘[mis]match’ but are not tests themselves
  11. # [00:44] <jgraham> plinss: Right, that part I get
  12. # [00:45] <plinss> when mismatch references are involved, they don’t (necessarily) create other relationships
  13. # [00:46] <plinss> so if test A has rel=mismatch href=B, and B has rel=mismatch href=C, then it does not follow that A != C
  14. # [00:46] <plinss> only that A must not match B and B must not match A
  15. # [00:46] <jgraham> plinss: Right, but you said something different
  16. # [00:46] <plinss> sorry, b must not match C
  17. # [00:46] <jgraham> "the *test* must [mis]match all references in the chain"
  18. # [00:47] <plinss> it depends on the chain
  19. # [00:47] <plinss> if test A mismatch B and B match C, then A must not match C
  20. # [00:47] <plinss> but if A mismatch B and B mismatch C, nothing is said about the realtionship between A and C
  21. # [00:47] <jgraham> Right, but you can just evaluate the chain left to right
  22. # [00:48] <plinss> actaully the chain is a graph
  23. # [00:48] <jgraham> Sure
  24. # [00:50] <jgraham> But it's a directed graph that can be written as an ordered list of comparison pairs (at least in the absence of the OR case, which I don't think actually makes things harder)
  25. # [00:50] <plinss> right, once you break cycles
  26. # [00:50] <jgraham> Yeah. And you evaluate that list one at a time from left to right
  27. # [00:50] <plinss> sure
  28. # [00:51] <jgraham> Good, so we at least agree on the semantics
  29. # [00:52] <plinss> I believe so, the only difference I see is that if you see a rel=‘[mis]match’ you presume the file is a test, which we do not do
  30. # [00:52] <jgraham> Well currently, yes
  31. # [00:53] <jgraham> But the plan going forward is that it forms a test unless it is identified as a ref
  32. # [00:53] <jgraham> Given the precedent the most reasonable way of doing the ref identification seems to be having a filename like -ref.ext
  33. # [00:54] <plinss> which sounds (mostly) compatible with us, we identify it as a ref based on filename or directory
  34. # [00:55] <plinss> but to be a test, we need it to not be a reference and to have some link to a spec
  35. # [00:55] <jgraham> Right. My original proposal was to have a system where any file pointed to by a rel=[mis]match was a ref. That also turns out to be surprisingly compatible with CSS
  36. # [00:55] <jgraham> But perhaps more magic
  37. # [00:55] <plinss> that would not be compatible
  38. # [00:56] <jgraham> You say that
  39. # [00:56] <plinss> we do have a few tests that are being uses as refs
  40. # [00:56] <jgraham> But I checked
  41. # [00:56] <jgraham> There are 2
  42. # [00:56] <plinss> not many though
  43. # [00:56] <plinss> could be at this point, it has changed over time
  44. # [00:56] <jgraham> And some that form weird cycles like A == B == A (or just A == A)
  45. # [00:56] <jgraham> Which seem to me to be bugs
  46. # [00:57] <plinss> we encourage A==B==A within references in case other tests just link to B
  47. # [00:57] <plinss> I’m not aware of any A==A cases, that would be silly
  48. # [00:57] <jgraham> Right, these are cases where nothing other than B links to A
  49. # [00:57] <plinss> sure
  50. # [00:57] <plinss> the cycle is just there as insurance
  51. # [00:57] <plinss> as you know the quality of tests submitted varies...
  52. # [00:58] <jgraham> Yeah
  53. # [00:58] <jgraham> The A == A one I looked at (there wre more) just seemed to have href="" in the <link> element
  54. # [00:58] <jgraham> So I guess it wasn't finished
  55. # [00:58] <plinss> so the safer we can keep things that (should) be resues often, like references, the better
  56. # [00:58] <plinss> ah, sure, most likely someone copied a template and never filled in that part
  57. # [00:58] <plinss> we ignore those
  58. # [00:59] <plinss> (ie. link rel without an href is an error)
  59. # [01:00] <jgraham> Anyway, I think the essential point is that a system which works like I described above seems to be something that could be adopted by both CSS and WPT with only minor changes to CSS (e.g. fixing reference files without -ref in the name to have -ref in the name)
  60. # [01:02] <plinss> as I said before, I don’t care about the specifics of how it works, so if we want to make the rule that all reference files must have a specific file name convention, that’s fine
  61. # [01:02] <plinss> but someone needs to update all the <links> if files get renamed
  62. # [01:03] <plinss> my hope (and plan) was that all the rules like this could be embodied in code and we just share a common library
  63. # [01:03] <plinss> which was the point of w3ctestlib
  64. # [01:03] <plinss> as i said before I’m happy to clean that code up and/or refactor for suitability
  65. # [01:03] <plinss> (it’s a bit ild and crufty, could use some cleanup anyway)
  66. # [01:04] <jgraham> Yeah, that could be possible
  67. # [01:05] <jgraham> Although there are infra issues ofc (hg vs git when code is being used as a submodule)
  68. # [01:05] <plinss> specifically we should both be using the SourceTree class (or equivalent) to determine what’s what in the repos
  69. # [01:06] <plinss> sure, but I can eaily maintain a bi-directional hg<->git mirror, I already have several
  70. # [01:06] <jgraham> and the wpt manifest generation code has specific requirements that I don't think the w3ctestlib is designed for (local, uncommitted, changes are stored seperately and the update is incremental)
  71. # [01:07] <plinss> I don’t mind making a superset of functionality
  72. # [01:08] <jgraham> Right, this seems like a reasonable phase 2 goal
  73. # [01:08] <plinss> fwiw, the bulk of the w3ctestlib code doesn’t care about the repo
  74. # [01:08] <jgraham> (phase 1 is "I implement this directly in wpt's manifest.py")
  75. # [01:08] <plinss> there was some hg code in there but I think I removed it (or at least most of it)
  76. # [01:11] <plinss> fine, but don’t be afraid to copy code from w3ctestlib as appropriate
  77. # [01:12] <jgraham> OK, thanks for your help
  78. # [01:12] <plinss> np
  79. # [01:13] <plinss> at some point, I’d really like for us to have a common library of code to do low level test & metadata work
  80. # [01:13] <plinss> I don’t care if it’s w3ctestlib or something new
  81. # [01:13] <plinss> I’m happy to refactor/rework all my other systems that currently use w3ctestlib to something else, so long as it covers the functionality we all need
  82. # [01:14] <plinss> and I’m happy to spend cycles writing something
  83. # [02:00] * Joins: tripu (antonio@public.cloak)
  84. # [02:05] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Client closed connection)
  85. # [02:11] * terri is now known as terri_offline
  86. # [02:14] * Joins: zcorpan_ (~zcorpan@public.cloak)
  87. # [02:14] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  88. # [02:15] * heycam is now known as heycam|away
  89. # [02:19] * Quits: zcorpan_ (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  90. # [03:14] * heycam|away is now known as heycam
  91. # [04:04] * Quits: artb (~ArtB@public.cloak) ("Leaving.")
  92. # [04:45] * Joins: bhill2_ (~bhill2@public.cloak)
  93. # [04:50] * Quits: krit (~sid15081@public.cloak) (Client closed connection)
  94. # [04:51] * Quits: bhill2 (~bhill2@public.cloak) (Ping timeout: 180 seconds)
  95. # [04:52] * Quits: bhill2_ (~bhill2@public.cloak) (Ping timeout: 180 seconds)
  96. # [05:07] * Joins: krit_ (~sid15081@public.cloak)
  97. # [05:21] * heycam is now known as heycam|away
  98. # [05:21] * heycam|away is now known as heycam
  99. # [06:42] * Joins: mehdi (~mehdi@public.cloak)
  100. # [06:43] * Quits: mehdi (~mehdi@public.cloak) ("Page closed")
  101. # [08:19] * heycam is now known as heycam|away
  102. # [08:24] * Joins: dom (dom@public.cloak)
  103. # [09:24] * Joins: zcorpan (~zcorpan@public.cloak)
  104. # [09:26] * Joins: Ms2ger (~Ms2ger@public.cloak)
  105. # [09:27] * Joins: Mayur (~Mayur@public.cloak)
  106. # [09:32] * Parts: Mayur (~Mayur@public.cloak)
  107. # [09:48] * Joins: darobin (rberjon@public.cloak)
  108. # [10:01] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  109. # [10:21] * Joins: zcorpan (~zcorpan@public.cloak)
  110. # [11:01] * Joins: anssik (~uid10742@public.cloak)
  111. # [11:54] * Joins: artb (~ArtB@public.cloak)
  112. # [12:03] <jgraham> Are we sold on requiring a <link> element in reftests?
  113. # [12:04] <Ms2ger> I can live with it
  114. # [12:04] <jgraham> I want to check before I start converting 350 files in wpt that currently rely on naming conventions
  115. # [12:04] <Ms2ger> Do you support it in SVG files?
  116. # [12:04] <jgraham> zcorpan: ^
  117. # [12:05] <jgraham> I think in SVG files it would have to be a html:link element, but I don't think that's a problem?
  118. # [12:06] <jgraham> In fact it looks like you already wrote a SVG reftest with a whole load of useless <html:link> elements, but not a rel=match
  119. # [12:07] <jgraham> On an entirely different subject I notice that acollwell is GONE FROM CHROMIUM
  120. # [12:07] <jgraham> So I guess he isn't going to help with https://github.com/w3c/web-platform-tests/pull/1238
  121. # [12:08] * Ms2ger wonders why the shouting
  122. # [12:08] <Ms2ger> jgraham, hah
  123. # [12:10] <jgraham> The shouting is [sic]
  124. # [12:10] <Ms2ger> Heh
  125. # [12:11] <Ms2ger> Anyway, I guess we don't need him
  126. # [12:24] <jgraham> I am in favour of just landing the tests
  127. # [12:24] <jgraham> They seem to work enough that they are better running than not, and they were reviewed upstream
  128. # [12:26] * Joins: gitbot (~gitbot@public.cloak)
  129. # [12:26] -gitbot:#testing- [web-platform-tests] jgraham closed pull request #1238: Update MediaSource tests with new cases from Blink. (master...acolwell/update-media-source-tests) https://github.com/w3c/web-platform-tests/pull/1238
  130. # [12:26] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  131. # [12:26] <Ms2ger> wfm
  132. # [12:26] <jgraham> Just as well ;)
  133. # [12:28] <Ms2ger> I wish we had someone reviewing web components tests
  134. # [12:28] <Ms2ger> Isn't that what anne is working on now? :)
  135. # [12:29] * jgraham wishes someone would volunteer to finish the conversion of blink service worker tests
  136. # [12:48] <MikeSmith> jgraham: who says Aaron's gone from Chromium?
  137. # [12:48] <zcorpan> jgraham: what's the question?
  138. # [12:52] <jgraham> MikeSmith: https://codereview.chromium.org/245893002 does
  139. # [12:53] <jgraham> zcorpan: Is it acceptable to require <link href="something" rel="[mis]match"> in reftests (like CSS does) rather than also supporting an implicit match relationship between foo.html and foo-ref.html?
  140. # [12:54] * Joins: zcorpan_ (~zcorpan@public.cloak)
  141. # [12:54] <zcorpan_> jgraham: yes, i think naming convention is counter-productive
  142. # [12:55] <zcorpan_> because it makes duplicating refs more obvious compared to changing an existing test to use <link> and reusing the ref
  143. # [12:56] <jgraham> zcorpan_: OK good
  144. # [12:56] <jgraham> zcorpan_: Pretty sure you are responsible for most of the use of the convention in the repo at the moment ;)
  145. # [12:56] <zcorpan_> jgraham: quite possible
  146. # [12:57] <zcorpan_> do you want me to fix them?
  147. # [12:58] <jgraham> zcorpan_: I am *so* not going to refuse that offer :)
  148. # [12:59] <zcorpan_> doh!
  149. # [12:59] <zcorpan_> ok :-)
  150. # [12:59] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  151. # [13:02] <MikeSmith> jgraham: as far as I know, Aaron still works for Google. Maybe that note just means he's no longer a Chromium comitter. I know at least he's still committing changes to the Media Source spec https://github.com/w3c/media-source/commits/gh-pages
  152. # [13:03] <Ms2ger> zcorpan_, rookie mistake :)
  153. # [13:03] <MikeSmith> jgraham: So I think we could still ask him to follow up on that PR
  154. # [13:03] <jgraham> MikeSmith: OK. I merged that PR, but there might well be followup to do
  155. # [13:03] <MikeSmith> ok
  156. # [13:04] <MikeSmith> ah yeah I see the merge in scrollback now
  157. # [13:06] <Ms2ger> MikeSmith, hey, do you happen to be interested in idlharness reviews?
  158. # [13:07] <MikeSmith> what's idlharness?
  159. # [13:07] <MikeSmith> what's "review"?
  160. # [13:08] * Ms2ger gives MikeSmith an "r+" stamp and https://critic.hoppipolla.co.uk/r/3722 and https://critic.hoppipolla.co.uk/r/3797
  161. # [13:08] * Quits: zcorpan_ (~zcorpan@public.cloak) (Client closed connection)
  162. # [13:08] <Ms2ger> See? You can put the stamp here
  163. # [13:08] <MikeSmith> heh
  164. # [13:09] <MikeSmith> I have nothing against idlharness reviews, it's that hatefule "ES6" word in there that's making me hesitate
  165. # [13:10] <MikeSmith> oh god the other one looks like actual work too
  166. # [13:10] <Ms2ger> custom-elements/testcommon.js is weird
  167. # [13:10] <Ms2ger> MikeSmith, oh, the ES6 one doesn't actually require touching ES6, IDL specs it
  168. # [13:11] <MikeSmith> yeah I'm just kvetching. Neither of these look very challenging now that I look at them
  169. # [13:11] <MikeSmith> anyway, looking now
  170. # [13:18] * Quits: anssik (~uid10742@public.cloak) ("Connection closed for inactivity")
  171. # [13:21] <Ms2ger> Thank
  172. # [13:23] <jgraham> who?
  173. # [13:26] <Ms2ger> Thank MikeSmith, mommy
  174. # [13:27] * Quits: terri_offline (~terri@public.cloak) (Ping timeout: 180 seconds)
  175. # [13:48] * Joins: gitbot (~gitbot@public.cloak)
  176. # [13:48] -gitbot:#testing- [web-platform-tests] jgraham closed pull request #879: Add test case that detached callback is called when element detached (master...detached-callback-test) https://github.com/w3c/web-platform-tests/pull/879
  177. # [13:48] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  178. # [13:48] * Joins: gitbot (~gitbot@public.cloak)
  179. # [13:48] -gitbot:#testing- [web-platform-tests] jgraham closed pull request #863: Add test cast for instantiation using type extensions (master...type-extensions-instantiation) https://github.com/w3c/web-platform-tests/pull/863
  180. # [13:48] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  181. # [13:48] * Joins: gitbot (~gitbot@public.cloak)
  182. # [13:48] -gitbot:#testing- [web-platform-tests] jgraham reopened pull request #863: Add test cast for instantiation using type extensions (master...type-extensions-instantiation) https://github.com/w3c/web-platform-tests/pull/863
  183. # [13:48] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  184. # [13:48] * Joins: gitbot (~gitbot@public.cloak)
  185. # [13:48] -gitbot:#testing- [web-platform-tests] jgraham closed pull request #860: Add test case for instantiation using local name (master...custom-tag-instantiation) https://github.com/w3c/web-platform-tests/pull/860
  186. # [13:48] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  187. # [13:50] * Joins: gitbot (~gitbot@public.cloak)
  188. # [13:50] -gitbot:#testing- [web-platform-tests] jgraham closed pull request #1524: event.path should include Window object as per W3C bug 21066 (master...event.path-21066) https://github.com/w3c/web-platform-tests/pull/1524
  189. # [13:50] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  190. # [13:50] * Joins: gitbot (~gitbot@public.cloak)
  191. # [13:50] -gitbot:#testing- [web-platform-tests] jgraham pushed 3 new commits to master: https://github.com/w3c/web-platform-tests/compare/412773b71c81...8e6faf6ee47d
  192. # [13:50] -gitbot:#testing- web-platform-tests/master 1181b84 Koji Ishii: event.path should include Window object as per W3C bug 21066
  193. # [13:50] -gitbot:#testing- web-platform-tests/master e958573 Koji Ishii: event.path test had a incorrect expected window object
  194. # [13:50] -gitbot:#testing- web-platform-tests/master 8e6faf6 jgraham: Merge pull request #1524 from kojiishi/event.path-21066...
  195. # [13:50] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  196. # [13:51] * Joins: gitbot (~gitbot@public.cloak)
  197. # [13:51] -gitbot:#testing- [web-platform-tests] Ms2ger created pr-852 (+1 new commit): https://github.com/w3c/web-platform-tests/commit/0ac8a0d0e61b
  198. # [13:51] -gitbot:#testing- web-platform-tests/pr-852 0ac8a0d bylee: Add test-case that changing the value of the is attribute must not affect this element's custom element type after a custom element is instantiated.
  199. # [13:51] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  200. # [13:51] * Joins: terri_offline (~terri@public.cloak)
  201. # [13:51] * terri_offline is now known as terri
  202. # [13:52] * Joins: gitbot (~gitbot@public.cloak)
  203. # [13:52] -gitbot:#testing- [web-platform-tests] Ms2ger force-pushed pr-852 from 0ac8a0d to b10e0c1: https://github.com/w3c/web-platform-tests/commits/pr-852
  204. # [13:52] -gitbot:#testing- web-platform-tests/pr-852 b10e0c1 bylee: Test that changing the value of the is attribute does not affect this element's custom element type after a custom element is instantiated.
  205. # [13:52] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  206. # [13:53] * Joins: gitbot (~gitbot@public.cloak)
  207. # [13:53] -gitbot:#testing- [web-platform-tests] Ms2ger force-pushed pr-852 from b10e0c1 to 0a4a88a: https://github.com/w3c/web-platform-tests/commits/pr-852
  208. # [13:53] -gitbot:#testing- web-platform-tests/pr-852 0a4a88a bylee: Test that changing the value of the is attribute does not affect this element's custom element type after a custom element is instantiated.
  209. # [13:53] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  210. # [13:53] * Joins: gitbot (~gitbot@public.cloak)
  211. # [13:53] -gitbot:#testing- [web-platform-tests] Ms2ger opened pull request #1569: Test that changing the value of the is attribute does not affect this el... (master...pr-852) https://github.com/w3c/web-platform-tests/pull/1569
  212. # [13:53] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  213. # [13:53] * Joins: gitbot (~gitbot@public.cloak)
  214. # [13:53] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #852: Add test-case that changing the value of the is attribute must not affec... (master...master) https://github.com/w3c/web-platform-tests/pull/852
  215. # [13:53] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  216. # [13:54] * Ms2ger pokes critic
  217. # [13:54] * Joins: gitbot (~gitbot@public.cloak)
  218. # [13:54] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #1569: Test that changing the value of the is attribute does not affect this el... (master...pr-852) https://github.com/w3c/web-platform-tests/pull/1569
  219. # [13:54] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  220. # [13:55] * Joins: gitbot (~gitbot@public.cloak)
  221. # [13:55] -gitbot:#testing- [web-platform-tests] Ms2ger reopened pull request #1569: Test that changing the value of the is attribute does not affect this el... (master...pr-852) https://github.com/w3c/web-platform-tests/pull/1569
  222. # [13:55] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  223. # [13:57] * Joins: gitbot (~gitbot@public.cloak)
  224. # [13:57] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #1569: Test that changing the value of the is attribute does not affect this el... (master...pr-852) https://github.com/w3c/web-platform-tests/pull/1569
  225. # [13:57] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  226. # [13:57] * Joins: gitbot (~gitbot@public.cloak)
  227. # [13:57] -gitbot:#testing- [web-platform-tests] Ms2ger deleted pr-852 at 0a4a88a: https://github.com/w3c/web-platform-tests/commit/0a4a88a
  228. # [13:57] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  229. # [14:01] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  230. # [14:02] * Joins: plh (plehegar@public.cloak)
  231. # [14:03] * Quits: terri (~terri@public.cloak) (Ping timeout: 180 seconds)
  232. # [14:04] <jgraham> parent.webkitCreateShadowRoot();
  233. # [14:04] <jgraham> really?
  234. # [14:09] * Joins: gitbot (~gitbot@public.cloak)
  235. # [14:09] -gitbot:#testing- [web-platform-tests] jgraham created jgraham/reftest_match (+1 new commit): https://github.com/w3c/web-platform-tests/commit/4b92f1e66bfd
  236. # [14:09] -gitbot:#testing- web-platform-tests/jgraham/reftest_match 4b92f1e James Graham: Fix reftests missing rel=match excluding webvtt tests.
  237. # [14:09] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  238. # [14:12] * Joins: zcorpan (~zcorpan@public.cloak)
  239. # [14:20] * Joins: gitbot (~gitbot@public.cloak)
  240. # [14:20] -gitbot:#testing- [web-platform-tests] jgraham pushed 1 new commit to jgraham/reftest_match: https://github.com/w3c/web-platform-tests/commit/cf7a0fb0cfc223cd4378a144fa7093700acb7e9e
  241. # [14:20] -gitbot:#testing- web-platform-tests/jgraham/reftest_match cf7a0fb James Graham: fixup! Fix reftests missing rel=match excluding webvtt tests.
  242. # [14:20] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  243. # [14:22] * Joins: terri_offline (~terri@public.cloak)
  244. # [14:22] * terri_offline is now known as terri
  245. # [14:30] * Joins: gitbot (~gitbot@public.cloak)
  246. # [14:30] -gitbot:#testing- [web-platform-tests] Ms2ger created pr-854-a (+1 new commit): https://github.com/w3c/web-platform-tests/commit/0ebbaaad184b
  247. # [14:30] -gitbot:#testing- web-platform-tests/pr-854-a 0ebbaaa bylee: Add test-case that if both types of custom element types are provided at the time of element's instantiation, the custom tag must win over the type extension.
  248. # [14:30] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  249. # [14:35] * Joins: gitbot (~gitbot@public.cloak)
  250. # [14:35] -gitbot:#testing- [web-platform-tests] Ms2ger created pr-854-b (+1 new commit): https://github.com/w3c/web-platform-tests/commit/2c82c7d7895b
  251. # [14:35] -gitbot:#testing- web-platform-tests/pr-854-b 2c82c7d Bon-Yong Lee: Test that the attached callback is only called in a browsing context.
  252. # [14:35] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  253. # [14:36] * Joins: gitbot (~gitbot@public.cloak)
  254. # [14:36] -gitbot:#testing- [web-platform-tests] Ms2ger force-pushed pr-854-b from 2c82c7d to 899931d: https://github.com/w3c/web-platform-tests/commits/pr-854-b
  255. # [14:36] -gitbot:#testing- web-platform-tests/pr-854-b 899931d Bon-Yong Lee: Test that the attached callback is only called in a browsing context.
  256. # [14:36] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  257. # [14:36] * Joins: gitbot (~gitbot@public.cloak)
  258. # [14:36] -gitbot:#testing- [web-platform-tests] Ms2ger opened pull request #1570: Add test-case that if both types of custom element types are provided at... (master...pr-854-a) https://github.com/w3c/web-platform-tests/pull/1570
  259. # [14:36] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  260. # [14:36] * Joins: gitbot (~gitbot@public.cloak)
  261. # [14:36] -gitbot:#testing- [web-platform-tests] Ms2ger opened pull request #1571: Test that the attached callback is only called in a browsing context. (master...pr-854-b) https://github.com/w3c/web-platform-tests/pull/1571
  262. # [14:36] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  263. # [14:37] * Joins: gitbot (~gitbot@public.cloak)
  264. # [14:37] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #854: Add test-case that if both types of custom element types are provided at... (master...custom-tags-wins-type-extensions) https://github.com/w3c/web-platform-tests/pull/854
  265. # [14:37] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  266. # [15:02] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  267. # [15:07] * Joins: gitbot (~gitbot@public.cloak)
  268. # [15:07] -gitbot:#testing- [web-platform-tests] jgraham pushed 1 new commit to jgraham/reftest_match: https://github.com/w3c/web-platform-tests/commit/03466930f5c174f194d7c831ad08d29fff4cc289
  269. # [15:07] -gitbot:#testing- web-platform-tests/jgraham/reftest_match 0346693 James Graham: fixup! Fix reftests missing rel=match excluding webvtt tests....
  270. # [15:07] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  271. # [15:07] * Joins: zcorpan (~zcorpan@public.cloak)
  272. # [15:09] * Joins: gitbot (~gitbot@public.cloak)
  273. # [15:09] -gitbot:#testing- [web-platform-tests] jgraham opened pull request #1572: Fix reftests missing rel=match (master...jgraham/reftest_match) https://github.com/w3c/web-platform-tests/pull/1572
  274. # [15:09] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  275. # [15:09] <jgraham> Ms2ger, zcorpan: r? https://critic.hoppipolla.co.uk/r/3808
  276. # [15:10] * zcorpan looks
  277. # [15:23] <zcorpan> jgraham: the change from shadow-root to text-indent was a bug fix, yes?
  278. # [15:23] <zcorpan> or text-decoration or what it was
  279. # [15:31] <jgraham> zcorpan: I assume so. Or at least it was fixing it to be the same as the previous semantics
  280. # [15:31] <jgraham> (a -ref file was used in preference to looking for <link> elements before)
  281. # [15:32] <jgraham> Speaking of which
  282. # [15:32] <jgraham> html/dom/elements/global-attributes/dir_auto-N-EN.html is still broken
  283. # [15:32] <jgraham> Any idea what the right ref there is
  284. # [15:32] <jgraham> ?
  285. # [15:33] <jgraham> I think the rel=reference is wrong
  286. # [15:37] * Joins: gitbot (~gitbot@public.cloak)
  287. # [15:37] -gitbot:#testing- [web-platform-tests] jgraham pushed 1 new commit to jgraham/reftest_match: https://github.com/w3c/web-platform-tests/commit/ad4b61caa6bd80ecffbd88c77ac840d48a4056a9
  288. # [15:37] -gitbot:#testing- web-platform-tests/jgraham/reftest_match ad4b61c James Graham: fixup! Fix reftests missing rel=match excluding webvtt tests.
  289. # [15:37] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  290. # [15:38] <jgraham> Sorry, one more line to r?
  291. # [15:39] <zcorpan> r+
  292. # [15:41] <jgraham> Thanks++
  293. # [15:42] <zcorpan> thank you :-)
  294. # [15:42] * Joins: gitbot (~gitbot@public.cloak)
  295. # [15:42] -gitbot:#testing- [web-platform-tests] jgraham closed pull request #1572: Fix reftests missing rel=match (master...jgraham/reftest_match) https://github.com/w3c/web-platform-tests/pull/1572
  296. # [15:42] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  297. # [15:53] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  298. # [16:03] * Joins: zcorpan (~zcorpan@public.cloak)
  299. # [16:31] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  300. # [16:34] * Joins: zcorpan (~zcorpan@public.cloak)
  301. # [16:37] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  302. # [16:41] * Joins: zcorpan (~zcorpan@public.cloak)
  303. # [16:50] <MikeSmith> Ms2ger: I still plan on finishing review of those 2 idlharness PRs but got delayed by getting stuck on a 2 hour and 45 minute telcon, and just got free now
  304. # [16:50] <Ms2ger> I'm so sorry
  305. # [16:50] <MikeSmith> so that I'm free?
  306. # [16:50] <MikeSmith> free to wreak havoc once again
  307. # [16:50] <Ms2ger> I wouldn't wish such a telcon on anyway
  308. # [16:50] <Ms2ger> anyone
  309. # [16:51] * Ms2ger can't type today
  310. # [16:51] <MikeSmith> well normally it's just a 2-hour telcon!
  311. # [16:51] <MikeSmith> every week
  312. # [16:52] <Ms2ger> I'm pretty sure that's cruel and unusual punishment
  313. # [16:52] <plh> he deserved it
  314. # [16:52] <MikeSmith> heh
  315. # [16:53] <plh> :)
  316. # [16:53] <MikeSmith> I earned it!
  317. # [16:53] <MikeSmith> I volunteered for it
  318. # [16:53] <MikeSmith> sort of
  319. # [16:53] <MikeSmith> rookie mistake
  320. # [16:53] <Ms2ger> plh, and you let him do that? Clearly he's not in his right mind
  321. # [16:53] <Ms2ger> If he has one
  322. # [16:55] <MikeSmith> heh we're way past "right mind"
  323. # [16:59] <jgraham> Did we wrap over into left mind yet?
  324. # [16:59] <Ms2ger> Something something left field?
  325. # [16:59] <MikeSmith> still trying to get all my minds working together
  326. # [17:00] <MikeSmith> anyway when I'm on calls and I queue up to speak I can often feel a kind of electro-spiritual uneasiness coming through from others on the call as they hope I'm not drunk/unhinged at the moment and about to say something nonsensical or whatever
  327. # [17:01] <MikeSmith> or actually most of the time it's not nearly that interesting -- instead I start to say something and then somebody points out that was already discussed 30 minutes ago in the part of the call I wasn't listening to because I'm instead been trying to review PRs from Ms2ger
  328. # [17:26] * Quits: tripu (antonio@public.cloak) ("Leaving")
  329. # [17:40] * Joins: ptressel (~chatzilla@public.cloak)
  330. # [17:41] * Quits: ptressel (~chatzilla@public.cloak) ("zzz")
  331. # [17:42] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  332. # [17:47] * Joins: gitbot (~gitbot@public.cloak)
  333. # [17:47] -gitbot:#testing- [web-platform-tests] jgraham created jgraham/reftest_multiple_refs_fix (+1 new commit): https://github.com/w3c/web-platform-tests/commit/8caece65200e
  334. # [17:47] -gitbot:#testing- web-platform-tests/jgraham/reftest_multiple_refs_fix 8caece6 James Graham: Change multiple refs in a file into a chain of references.
  335. # [17:47] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  336. # [17:49] * Joins: gitbot (~gitbot@public.cloak)
  337. # [17:49] -gitbot:#testing- [web-platform-tests] jgraham opened pull request #1573: Change multiple refs in a file into a chain of references. (master...jgraham/reftest_multiple_refs_fix) https://github.com/w3c/web-platform-tests/pull/1573
  338. # [17:49] * Parts: gitbot (~gitbot@public.cloak) (gitbot)
  339. # [17:59] * Joins: zcorpan (~zcorpan@public.cloak)
  340. # [18:18] <plinss> jgraham, Ms2ger: SVG tests use a similar metadata format: http://www.w3.org/Graphics/SVG/WG/wiki/Main_Page#Structure_of_SVG_tests
  341. # [18:24] * Quits: AutomatedTester (~AutomatedTester@public.cloak) (Client closed connection)
  342. # [18:28] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
  343. # [18:36] * Quits: darobin (rberjon@public.cloak) (Client closed connection)
  344. # [18:36] * Joins: darobin (rberjon@public.cloak)
  345. # [18:44] * Quits: darobin (rberjon@public.cloak) (Ping timeout: 180 seconds)
  346. # [18:44] * Joins: bhill2 (~bhill2@public.cloak)
  347. # [18:51] * Quits: dom (dom@public.cloak) ("")
  348. # [19:25] * Joins: darobin (rberjon@public.cloak)
  349. # [19:33] * terri is now known as terri_offline
  350. # [19:34] * Quits: darobin (rberjon@public.cloak) (Client closed connection)
  351. # [19:38] * Joins: zcorpan (~zcorpan@public.cloak)
  352. # [19:46] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
  353. # [19:51] * Joins: darobin (rberjon@public.cloak)
  354. # [19:54] * terri_offline is now known as terri
  355. # [19:59] * Quits: darobin (rberjon@public.cloak) (Client closed connection)
  356. # [20:00] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
  357. # [20:48] * Joins: AutomatedTester (~AutomatedTester@public.cloak)
  358. # [21:52] * heycam|away is now known as heycam
  359. # [22:18] * Joins: darobin (rberjon@public.cloak)
  360. # [22:48] * Quits: darobin (rberjon@public.cloak) (Client closed connection)
  361. # [22:57] * Quits: Ms2ger (~Ms2ger@public.cloak) ("nn")
  362. # [23:05] * Quits: plh (plehegar@public.cloak) ("Leaving")
  363. # [23:11] * Joins: bhill2 (~bhill2@public.cloak)
  364. # [23:16] * Joins: bhill2_ (~bhill2@public.cloak)
  365. # [23:16] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
  366. # [23:20] * Joins: bhill2 (~bhill2@public.cloak)
  367. # [23:20] * Quits: bhill2_ (~bhill2@public.cloak) (Client closed connection)
  368. # [23:26] * Joins: bhill2_ (~bhill2@public.cloak)
  369. # [23:26] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
  370. # [23:44] * Joins: bhill2 (~bhill2@public.cloak)
  371. # [23:44] * Quits: bhill2_ (~bhill2@public.cloak) (Client closed connection)
  372. # Session Close: Thu Jan 22 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