/irc-logs / w3c / #testing / 2015-11-17 / end

Options:

Previous day, Next day

1. # Session Start: Tue Nov 17 00:00:00 2015
2. # Session Ident: #testing
3. # [00:12] * Quits: rniwa (~textual@public.cloak) ("My Mac has gone to sleep. ZZZzzz…")
4. # [00:19] * Joins: rniwa (~textual@public.cloak)
5. # [00:34] * Quits: rniwa (~textual@public.cloak) ("My Mac has gone to sleep. ZZZzzz…")
6. # [00:49] * Joins: rniwa (~textual@public.cloak)
7. # [01:02] * Quits: rniwa (~textual@public.cloak) ("My Mac has gone to sleep. ZZZzzz…")
8. # [01:49] * Joins: rniwa (~textual@public.cloak)
9. # [02:45] * Joins: fwtnb (~fwtnb@public.cloak)
10. # [04:35] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
11. # [04:59] * Joins: fwtnb_ (~fwtnb@public.cloak)
12. # [04:59] * Quits: fwtnb (~fwtnb@public.cloak) (Client closed connection)
13. # [05:39] * Joins: bhill2 (~bhill2@public.cloak)
14. # [06:10] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
15. # [06:10] * Joins: zcorpan (~zcorpan@public.cloak)
16. # [07:17] * Joins: bhill2 (~bhill2@public.cloak)
17. # [07:17] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
18. # [07:32] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
19. # [07:37] * Joins: zcorpan (~zcorpan@public.cloak)
20. # [08:33] * Joins: Ms2ger (~Ms2ger@public.cloak)
21. # [08:42] * Quits: rniwa (~textual@public.cloak) ("My Mac has gone to sleep. ZZZzzz…")
22. # [09:15] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
23. # [09:43] * Joins: zcorpan (~zcorpan@public.cloak)
24. # [09:54] * Joins: gitbot (~gitbot@public.cloak)
25. # [09:54] -gitbot:#testing- [web-platform-tests] Ms2ger opened pull request #2335: Implement 'labels' attribute on 'labelable elements' (master...sync_c2bf48e0d59a2a9e0d404f0530218baaa3b3c122) https://github.com/w3c/web-platform-tests/pull/2335
26. # [09:54] * Parts: gitbot (~gitbot@public.cloak)
27. # [09:54] * Joins: gitbot (~gitbot@public.cloak)
28. # [09:54] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #2335: Implement 'labels' attribute on 'labelable elements' (master...sync_c2bf48e0d59a2a9e0d404f0530218baaa3b3c122) https://github.com/w3c/web-platform-tests/pull/2335
29. # [09:54] * Parts: gitbot (~gitbot@public.cloak)
30. # [09:54] * Joins: gitbot (~gitbot@public.cloak)
31. # [09:54] -gitbot:#testing- [web-platform-tests] Ms2ger deleted sync_c2bf48e0d59a2a9e0d404f0530218baaa3b3c122 at c2bf48e: https://github.com/w3c/web-platform-tests/commit/c2bf48e
32. # [09:54] * Parts: gitbot (~gitbot@public.cloak)
33. # [09:54] * Joins: gitbot (~gitbot@public.cloak)
34. # [09:54] -gitbot:#testing- [web-platform-tests] Ms2ger created sync_7d4d65a243623b6b0b6da4a4d90ea2a0ae43a1f9 (+1 new commit): https://github.com/w3c/web-platform-tests/commit/7d4d65a24362
35. # [09:54] -gitbot:#testing- web-platform-tests/sync_7d4d65a243623b6b0b6da4a4d90ea2a0ae43a1f9 7d4d65a Eli Friedman: Use attribute getter/setter macros for misc DOM attributes....
36. # [09:54] * Parts: gitbot (~gitbot@public.cloak)
37. # [09:54] * Joins: gitbot (~gitbot@public.cloak)
38. # [09:54] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #2336: Use attribute getter/setter macros for misc DOM attributes. (master...sync_7d4d65a243623b6b0b6da4a4d90ea2a0ae43a1f9) https://github.com/w3c/web-platform-tests/pull/2336
39. # [09:54] * Parts: gitbot (~gitbot@public.cloak)
40. # [09:54] * Joins: gitbot (~gitbot@public.cloak)
41. # [09:54] -gitbot:#testing- [web-platform-tests] Ms2ger deleted sync_7d4d65a243623b6b0b6da4a4d90ea2a0ae43a1f9 at 7d4d65a: https://github.com/w3c/web-platform-tests/commit/7d4d65a
42. # [09:54] * Parts: gitbot (~gitbot@public.cloak)
43. # [09:54] * Joins: gitbot (~gitbot@public.cloak)
44. # [09:54] -gitbot:#testing- [web-platform-tests] Ms2ger created sync_71075ab706f68bc24ea805ccd2de4fcafded597d (+1 new commit): https://github.com/w3c/web-platform-tests/commit/71075ab706f6
45. # [09:54] -gitbot:#testing- web-platform-tests/sync_71075ab706f68bc24ea805ccd2de4fcafded597d 71075ab Nikki: Fixes #8213: Implement Blob variant of WebSocket.send()
46. # [09:54] * Parts: gitbot (~gitbot@public.cloak)
47. # [09:55] * Joins: gitbot (~gitbot@public.cloak)
48. # [09:55] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #2337: Fixes #8213: Implement Blob variant of WebSocket.send() (master...sync_71075ab706f68bc24ea805ccd2de4fcafded597d) https://github.com/w3c/web-platform-tests/pull/2337
49. # [09:55] * Parts: gitbot (~gitbot@public.cloak)
50. # [09:55] * Joins: gitbot (~gitbot@public.cloak)
51. # [09:55] -gitbot:#testing- [web-platform-tests] Ms2ger opened pull request #2338: wpt for javascript urls with queries and fragments (master...sync_a08425822b21ff2a48c0752c0f839b347cc5eb43) https://github.com/w3c/web-platform-tests/pull/2338
52. # [09:55] * Parts: gitbot (~gitbot@public.cloak)
53. # [09:55] * Joins: gitbot (~gitbot@public.cloak)
54. # [09:55] -gitbot:#testing- [web-platform-tests] Ms2ger created sync_a08425822b21ff2a48c0752c0f839b347cc5eb43 at a084258 (+0 new commits): https://github.com/w3c/web-platform-tests/commits/sync_a08425822b21ff2a48c0752c0f839b347cc5eb43
55. # [09:55] * Parts: gitbot (~gitbot@public.cloak)
56. # [09:55] * Joins: gitbot (~gitbot@public.cloak)
57. # [09:55] -gitbot:#testing- [web-platform-tests] Ms2ger created sync_5b042b7d1bf6df5695ffa322a87ceb7f4120fb23 (+1 new commit): https://github.com/w3c/web-platform-tests/commit/5b042b7d1bf6
58. # [09:55] -gitbot:#testing- web-platform-tests/sync_5b042b7d1bf6df5695ffa322a87ceb7f4120fb23 5b042b7 Corey Farwell: Implement <hr> 'color' attribute
59. # [09:55] * Parts: gitbot (~gitbot@public.cloak)
60. # [09:55] * Joins: gitbot (~gitbot@public.cloak)
61. # [09:55] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #2339: Implement <hr> 'color' attribute (master...sync_5b042b7d1bf6df5695ffa322a87ceb7f4120fb23) https://github.com/w3c/web-platform-tests/pull/2339
62. # [09:55] * Parts: gitbot (~gitbot@public.cloak)
63. # [09:55] * Joins: gitbot (~gitbot@public.cloak)
64. # [09:55] -gitbot:#testing- [web-platform-tests] Ms2ger deleted sync_5b042b7d1bf6df5695ffa322a87ceb7f4120fb23 at 5b042b7: https://github.com/w3c/web-platform-tests/commit/5b042b7
65. # [09:55] * Parts: gitbot (~gitbot@public.cloak)
66. # [09:55] * Joins: gitbot (~gitbot@public.cloak)
67. # [09:55] -gitbot:#testing- [web-platform-tests] Ms2ger created sync_8ee5fef1ec2ca982ce80cd66eedc523a0d0a77d2 (+1 new commit): https://github.com/w3c/web-platform-tests/commit/8ee5fef1ec2c
68. # [09:55] -gitbot:#testing- web-platform-tests/sync_8ee5fef1ec2ca982ce80cd66eedc523a0d0a77d2 8ee5fef Josh Matthews: Add test for OnErrorEventHandler special case.
69. # [09:55] * Parts: gitbot (~gitbot@public.cloak)
70. # [09:55] * Joins: gitbot (~gitbot@public.cloak)
71. # [09:55] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #2340: Add test for OnErrorEventHandler special case. (master...sync_8ee5fef1ec2ca982ce80cd66eedc523a0d0a77d2) https://github.com/w3c/web-platform-tests/pull/2340
72. # [09:55] * Parts: gitbot (~gitbot@public.cloak)
73. # [09:55] * Joins: gitbot (~gitbot@public.cloak)
74. # [09:55] -gitbot:#testing- [web-platform-tests] Ms2ger pushed 1 new commit to master: https://github.com/w3c/web-platform-tests/commit/af9d938ba720259adbbaf3a1689614eb9c814889
75. # [09:55] -gitbot:#testing- web-platform-tests/master af9d938 Ms2ger: Merge pull request #2340 from w3c/sync_8ee5fef1ec2ca982ce80cd66eedc523a0d0a77d2...
76. # [09:55] * Parts: gitbot (~gitbot@public.cloak)
77. # [09:55] * Joins: gitbot (~gitbot@public.cloak)
78. # [09:55] -gitbot:#testing- [web-platform-tests] Ms2ger created sync_c878c22218b9ab30b828b1dde7c380df8a6f115e (+1 new commit): https://github.com/w3c/web-platform-tests/commit/c878c22218b9
79. # [09:55] -gitbot:#testing- web-platform-tests/sync_c878c22218b9ab30b828b1dde7c380df8a6f115e c878c22 Manish Goregaokar: Add tests for get/set/removeAttributeNode()...
80. # [09:55] * Parts: gitbot (~gitbot@public.cloak)
81. # [09:55] * Joins: gitbot (~gitbot@public.cloak)
82. # [09:55] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #2341: Add tests for get/set/removeAttributeNode() (master...sync_c878c22218b9ab30b828b1dde7c380df8a6f115e) https://github.com/w3c/web-platform-tests/pull/2341
83. # [09:55] * Parts: gitbot (~gitbot@public.cloak)
84. # [09:55] * Joins: gitbot (~gitbot@public.cloak)
85. # [09:55] -gitbot:#testing- [web-platform-tests] Ms2ger deleted sync_c878c22218b9ab30b828b1dde7c380df8a6f115e at c878c22: https://github.com/w3c/web-platform-tests/commit/c878c22
86. # [09:55] * Parts: gitbot (~gitbot@public.cloak)
87. # [09:55] * Joins: gitbot (~gitbot@public.cloak)
88. # [09:55] -gitbot:#testing- [web-platform-tests] Ms2ger created sync_09171b14cb0e5089f9d80640194ccc9a37be684a (+1 new commit): https://github.com/w3c/web-platform-tests/commit/09171b14cb0e
89. # [09:55] -gitbot:#testing- web-platform-tests/sync_09171b14cb0e5089f9d80640194ccc9a37be684a 09171b1 Corey Farwell: Implement <hr> element 'width' attribute
90. # [09:55] * Parts: gitbot (~gitbot@public.cloak)
91. # [09:55] * Joins: gitbot (~gitbot@public.cloak)
92. # [09:55] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #2342: Implement <hr> element 'width' attribute (master...sync_09171b14cb0e5089f9d80640194ccc9a37be684a) https://github.com/w3c/web-platform-tests/pull/2342
93. # [09:55] * Parts: gitbot (~gitbot@public.cloak)
94. # [09:56] * Joins: gitbot (~gitbot@public.cloak)
95. # [09:56] -gitbot:#testing- [web-platform-tests] Ms2ger deleted sync_09171b14cb0e5089f9d80640194ccc9a37be684a at 09171b1: https://github.com/w3c/web-platform-tests/commit/09171b1
96. # [09:56] * Parts: gitbot (~gitbot@public.cloak)
97. # [09:56] * Joins: gitbot (~gitbot@public.cloak)
98. # [09:56] -gitbot:#testing- [web-platform-tests] Ms2ger created sync_647963430dac82928c49ce9853e1bf1e26ddfa1c (+1 new commit): https://github.com/w3c/web-platform-tests/commit/647963430dac
99. # [09:56] -gitbot:#testing- web-platform-tests/sync_647963430dac82928c49ce9853e1bf1e26ddfa1c 6479634 Corey Farwell: Make util::str::parse_length work with floats
100. # [09:56] * Parts: gitbot (~gitbot@public.cloak)
101. # [09:56] * Joins: gitbot (~gitbot@public.cloak)
102. # [09:56] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #2343: Make util::str::parse_length work with floats (master...sync_647963430dac82928c49ce9853e1bf1e26ddfa1c) https://github.com/w3c/web-platform-tests/pull/2343
103. # [09:56] * Parts: gitbot (~gitbot@public.cloak)
104. # [09:56] * Joins: gitbot (~gitbot@public.cloak)
105. # [09:56] -gitbot:#testing- [web-platform-tests] Ms2ger deleted sync_647963430dac82928c49ce9853e1bf1e26ddfa1c at 6479634: https://github.com/w3c/web-platform-tests/commit/6479634
106. # [09:56] * Parts: gitbot (~gitbot@public.cloak)
107. # [10:05] * Joins: gitbot (~gitbot@public.cloak)
108. # [10:05] -gitbot:#testing- [web-platform-tests] annevk deleted urlsearchparams at da89213: https://github.com/w3c/web-platform-tests/commit/da89213
109. # [10:05] * Parts: gitbot (~gitbot@public.cloak)
110. # [10:11] * Joins: dom (dom@public.cloak)
111. # [11:27] * Joins: gitbot (~gitbot@public.cloak)
112. # [11:27] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #1278: Make new Blob throw (master...non-new-Blob) https://github.com/w3c/web-platform-tests/pull/1278
113. # [11:27] * Parts: gitbot (~gitbot@public.cloak)
114. # [11:29] * Joins: gitbot (~gitbot@public.cloak)
115. # [11:29] -gitbot:#testing- [web-platform-tests] Ms2ger closed pull request #2334: localStorage and document.domain test (master...storage-document-domain) https://github.com/w3c/web-platform-tests/pull/2334
116. # [11:29] * Parts: gitbot (~gitbot@public.cloak)
117. # [11:29] * Joins: gitbot (~gitbot@public.cloak)
118. # [11:29] -gitbot:#testing- [web-platform-tests] Ms2ger pushed 1 new commit to master: https://github.com/w3c/web-platform-tests/commit/ca1843553b87a2198df35a7ebe3371284ed2ecf5
119. # [11:29] -gitbot:#testing- web-platform-tests/master ca18435 Ms2ger: Merge pull request #2334 from w3c/storage-document-domain...
120. # [11:29] * Parts: gitbot (~gitbot@public.cloak)
121. # [11:29] * Joins: gitbot (~gitbot@public.cloak)
122. # [11:29] -gitbot:#testing- [web-platform-tests] Ms2ger deleted storage-document-domain at ac4fb38: https://github.com/w3c/web-platform-tests/commit/ac4fb38
123. # [11:29] * Parts: gitbot (~gitbot@public.cloak)
124. # [11:30] * Joins: gitbot (~gitbot@public.cloak)
125. # [11:30] -gitbot:#testing- [web-platform-tests] annevk deleted non-new-Blob at d8d1e56: https://github.com/w3c/web-platform-tests/commit/d8d1e56
126. # [11:30] * Parts: gitbot (~gitbot@public.cloak)
127. # [12:01] * Joins: gitbot (~gitbot@public.cloak)
128. # [12:01] -gitbot:#testing- [web-platform-tests] annevk created noreferrer-name-reuse (+1 new commit): https://github.com/w3c/web-platform-tests/commit/8ea6709d2042
129. # [12:01] -gitbot:#testing- web-platform-tests/noreferrer-name-reuse 8ea6709 Anne van Kesteren: Reusing a window name for a same-origin rel=noreferrer hyperlink
130. # [12:02] * Parts: gitbot (~gitbot@public.cloak)
131. # [12:02] * Joins: gitbot (~gitbot@public.cloak)
132. # [12:02] -gitbot:#testing- [web-platform-tests] annevk opened pull request #2344: Reusing a window name for a same-origin rel=noreferrer hyperlink (master...noreferrer-name-reuse) https://github.com/w3c/web-platform-tests/pull/2344
133. # [12:02] * Parts: gitbot (~gitbot@public.cloak)
134. # [12:09] * Joins: gitbot (~gitbot@public.cloak)
135. # [12:09] -gitbot:#testing- [web-platform-tests] Ms2ger opened pull request #2345: Update some spec links in html/. (master...spec-html) https://github.com/w3c/web-platform-tests/pull/2345
136. # [12:09] * Parts: gitbot (~gitbot@public.cloak)
137. # [12:28] <AutomatedTester> zcorpan: hey, I think I finished those elementFromPoint tests
138. # [12:28] <AutomatedTester> whenever you get time to review
139. # [12:28] <AutomatedTester> even thought I found a bug between browsers but turned out it was a flaky test
140. # [12:30] <Ms2ger> Aw
141. # [12:31] <zcorpan> AutomatedTester: sweet. got a link?
142. # [12:31] <AutomatedTester> zcorpan: https://github.com/w3c/web-platform-tests/pull/2309
143. # [12:32] <AutomatedTester> Ms2ger: there might be an inconsistency somewhere but it's not in elementFromPoint
144. # [12:32] <AutomatedTester> might be in getBoundingClientRect()
145. # [12:34] <zcorpan> lots of unrelated commits showing up :-(
146. # [12:35] <AutomatedTester> zcorpan: yea, I rebased off master and came up
147. # [12:35] <AutomatedTester> sorry
148. # [12:35] <AutomatedTester> I thought that critic would have noticed it as a rebase
149. # [12:36] <AutomatedTester> zcorpan: I can close that PR and recreate it if you would like
150. # [12:37] <zcorpan> AutomatedTester: yeah please do that
151. # [13:04] * Joins: gitbot (~gitbot@public.cloak)
152. # [13:04] -gitbot:#testing- [web-platform-tests] AutomatedTester closed pull request #2309: [cssom-view] Add Element From Point tests (master...elementFromPoint) https://github.com/w3c/web-platform-tests/pull/2309
153. # [13:04] * Parts: gitbot (~gitbot@public.cloak)
154. # [13:18] * Quits: fwtnb_ (~fwtnb@public.cloak) ("")
155. # [13:20] * Joins: fwtnb (~fwtnb@public.cloak)
156. # [13:21] * Quits: Ms2ger (~Ms2ger@public.cloak) (Ping timeout: 180 seconds)
157. # [13:31] * Joins: Ms2ger (~Ms2ger@public.cloak)
158. # [14:02] * Joins: plh (plehegar@public.cloak)
159. # [14:03] * Quits: plh (plehegar@public.cloak) (Client closed connection)
160. # [14:03] * Joins: plh (plehegar@public.cloak)
161. # [14:21] <AutomatedTester> ok... I have put my repo into some weird state
162. # [14:21] <AutomatedTester> it says my fork is 10 commits ahead of wpt/master
163. # [14:22] <Ms2ger> You did do something weird
164. # [14:23] * Joins: gitbot (~gitbot@public.cloak)
165. # [14:23] -gitbot:#testing- [web-platform-tests] AutomatedTester opened pull request #2348: element from point tests (master...dburns/elementFromPoint-tests) https://github.com/w3c/web-platform-tests/pull/2348
166. # [14:23] * Parts: gitbot (~gitbot@public.cloak)
167. # [14:23] <AutomatedTester> zcorpan: all that is of interest is 573fd57 from ^
168. # [14:24] <Ms2ger> AutomatedTester, what does git remote -v say?
169. # [14:24] <AutomatedTester> Ms2ger: the following
170. # [14:24] <AutomatedTester> origin git@github.com:AutomatedTester/web-platform-tests.git (fetch)
171. # [14:24] <AutomatedTester> origin git@github.com:AutomatedTester/web-platform-tests.git (push)
172. # [14:24] <AutomatedTester> w3c git@github.com:w3c/web-platform-tests.git (fetch)
173. # [14:24] <AutomatedTester> w3c git@github.com:w3c/web-platform-tests.git (push)
174. # [14:24] <Ms2ger> Okay
175. # [14:24] <Ms2ger> git fetch w3c
176. # [14:24] <Ms2ger> git checkout dburns/elementFromPoint-tests
177. # [14:25] <Ms2ger> git rebase -i w3c/master
178. # [14:25] <Ms2ger> Drop everything except your commit
179. # [14:25] <Ms2ger> Then git push -f origin dburns/elementFromPoint-tests
180. # [14:25] <AutomatedTester> Ms2ger: that will make critic sad
181. # [14:26] <Ms2ger> Critic already is sad :)
182. # [14:31] * Joins: gitbot (~gitbot@public.cloak)
183. # [14:31] -gitbot:#testing- [testharness.js] annevk pushed 1 new commit to master: https://github.com/w3c/testharness.js/commit/8658d6ca31d086e695b333aef55bb388091c7049
184. # [14:31] -gitbot:#testing- testharness.js/master 8658d6c Anne van Kesteren: Variable typo in `EventWachter` example
185. # [14:31] <zcorpan> i can review this as is, looking now
186. # [14:31] * Parts: gitbot (~gitbot@public.cloak)
187. # [14:32] <zcorpan> AutomatedTester: ^
188. # [14:32] <AutomatedTester> okey dokey
189. # [14:32] <AutomatedTester> thanks
190. # [14:36] <zcorpan> is `let` OK in wpt these days?
191. # [14:36] <MikeSmith> zcorpan: should be, but can't recall that it's come up til now
192. # [14:37] <Ms2ger> Prefer not to
193. # [14:37] <MikeSmith> oh
194. # [14:37] <MikeSmith> why not
195. # [14:37] <zcorpan> doesn't work in IE6
196. # [14:37] * zcorpan ducks
197. # [14:37] <Ms2ger> Or recent Firefox
198. # [14:37] <MikeSmith> wha?
199. # [14:37] <MikeSmith> backed it out?
200. # [14:37] <AutomatedTester> Ms2ger: oh... shu's change hasnt got to release yet?
201. # [14:37] <MikeSmith> it was there before though
202. # [14:38] <AutomatedTester> it's definitely there for nightly
203. # [14:38] * AutomatedTester remembers the pain...
204. # [14:39] <Ms2ger> Has 44 shipped yet?
205. # [14:39] <AutomatedTester> no
206. # [14:39] <AutomatedTester> that's Aurora
207. # [14:39] <Ms2ger> https://bugzilla.mozilla.org/show_bug.cgi?id=932517 claims let is on by default starting in 44
208. # [14:39] <Ms2ger> Is that wrong?
209. # [14:40] <AutomatedTester> that is right
210. # [14:40] <Ms2ger> I definitely think tests should work in release Fx, if possible
211. # [14:40] <Ms2ger> And preferably a few releases back
212. # [14:41] <jgraham> Also Safari, which is generally the blocker for fancy ES features
213. # [14:41] <AutomatedTester> oh yea... I forgot about Safari
214. # [14:41] <MikeSmith> hmm me too
215. # [14:41] <jgraham> Otherwise we could all write test(() => {})
216. # [14:41] <AutomatedTester> Ms2ger: why a few releases back?
217. # [14:41] <AutomatedTester> zcorpan: I will s/let/var/ then
218. # [14:42] <AutomatedTester> fat arrow tests would be nice
219. # [14:42] <jgraham> FWIW my unofficial policy has been "latest released version of major browsers"
220. # [14:42] <AutomatedTester> Ms2ger: unless you mean ESR
221. # [14:42] <Ms2ger> AutomatedTester, in case the test points out a regression, it's nice to be able to run it on older releases without touching the test
222. # [14:42] * jgraham doesn't care about Fx < stable
223. # [14:43] <jgraham> Ms2ger: For amny of these things, realistically, you will run it on nightly builds with more features preffed on
224. # [14:43] <zcorpan> AutomatedTester: reviewed
225. # [14:44] <Ms2ger> Which is why I don't feel too strongly about < stable
226. # [14:46] <Ms2ger> I think having tests work a few extra releases is more useful than using `let`, though
227. # [14:46] <Ms2ger> The tradeoff might be different for other features
228. # [14:47] <jgraham> Yeah, I agree let is rather low value
229. # [14:48] <jgraham> Fat arrow functions seem much more useful as soon as Apple deign to support them
230. # [15:00] <AutomatedTester> zcorpan: silly question (mostly because I am stupid), I don't know how to test those other times
231. # [15:03] <zcorpan> AutomatedTester: ok hold on
232. # [15:03] <AutomatedTester> items*
233. # [15:06] <zcorpan> http://software.hixie.ch/utilities/js/live-dom-viewer/saved/3751
234. # [15:09] <zcorpan> http://software.hixie.ch/utilities/js/live-dom-viewer/saved/3752
235. # [15:09] <AutomatedTester> ahh, now I get it, ok cool
236. # [15:09] <AutomatedTester> thanks zcorpan
237. # [15:09] <zcorpan> AutomatedTester: not silly and not stupid, i was just unclear :-)
238. # [15:24] * Joins: gitbot (~gitbot@public.cloak)
239. # [15:24] -gitbot:#testing- [wpt-tools] youennf opened pull request #40: Check clashing test filenames (master...clashing-names-lint) https://github.com/w3c/wpt-tools/pull/40
240. # [15:24] * Parts: gitbot (~gitbot@public.cloak)
241. # [15:25] <AutomatedTester> zcorpan: should createHTMLDocument() test case return null
242. # [15:25] * Joins: gitbot (~gitbot@public.cloak)
243. # [15:25] -gitbot:#testing- [web-platform-tests] youennf opened pull request #2349: Renaming test filenames that clash when removing extension (master...fix-clashing-test-filenames) https://github.com/w3c/web-platform-tests/pull/2349
244. # [15:25] * Parts: gitbot (~gitbot@public.cloak)
245. # [15:27] <AutomatedTester> both chrome and Firefox suggest yes
246. # [15:27] <AutomatedTester> and safari
247. # [15:29] <zcorpan> AutomatedTester: uh, yes, it should return null per spec also.
248. # [15:29] <zcorpan> (step 1)
249. # [15:29] <zcorpan> sorry about that
250. # [15:29] <AutomatedTester> zcorpan: your test above has doc.documentElement === doc.elementFromPoint
251. # [15:29] <AutomatedTester> that's why I was checking ;)
252. # [15:29] <zcorpan> yeah my bad :-)
253. # [15:30] <AutomatedTester> zcorpan: all done and pushed
254. # [15:30] <AutomatedTester> to my pr
255. # [15:31] * Quits: fwtnb (~fwtnb@public.cloak) (Client closed connection)
256. # [15:32] <zcorpan> AutomatedTester: new tests look good, but i noticed something else now. the arguments to assert_* are backwards, which will confuse the messages
257. # [15:32] <zcorpan> should be assert_equals(actual, expected, ...)
258. # [15:33] <AutomatedTester> hmmm... that is a bizarre order..
259. # [15:33] <AutomatedTester> I will reverse them all
260. # [15:34] <zcorpan> LGTM after that. (need to leave now)
261. # [15:34] <AutomatedTester> ok, zcorpan I will land then (after this change)
262. # [15:34] <zcorpan> thanks!
263. # [15:35] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
264. # [15:35] <jgraham> For every possible order of arguments to assert functions someone thinks the order is strange and has precedent to back them up
265. # [15:36] <AutomatedTester> jgraham: I have either worked with them not mattering or expected, actual
266. # [15:36] <AutomatedTester> but I agree with your statement
267. # [15:36] <jgraham> Well typically it doesn't matter much. It just means that the error message is wrong
268. # [15:36] <jgraham> (sometimes it matters more)
269. # [15:37] <AutomatedTester> jgraham: if I didnt manage the sheriffs I might not care
270. # [15:37] <AutomatedTester> about the messages
271. # [15:38] <AutomatedTester> but I know what will happen to me
272. # [15:48] <AutomatedTester> I do love git and it magically making my commits disappear
273. # [15:48] <AutomatedTester> </sarcasm>
274. # [16:03] * Quits: MikeSmith (~MikeSmith@public.cloak) ("Lost terminal")
275. # [16:04] * Joins: rego (~rego@public.cloak)
276. # [16:06] * Joins: MikeSmith (~MikeSmith@public.cloak)
277. # [16:07] * Quits: MikeSmith (~MikeSmith@public.cloak) ("Lost terminal")
278. # [16:08] * Joins: MikeSmith (~MikeSmith@public.cloak)
279. # [16:09] * Quits: MikeSmith (~MikeSmith@public.cloak) ("Lost terminal")
280. # [16:12] * Joins: gitbot (~gitbot@public.cloak)
281. # [16:12] -gitbot:#testing- [csswg-test] kojiishi pushed 1 new commit to master: https://github.com/w3c/csswg-test/commit/bd5659b77e265d8f4ea08b60db0e3207523461fd
282. # [16:12] -gitbot:#testing- csswg-test/master bd5659b Koji Ishii: [css-writing-modes-3] Add outline-inline-v(lr|rl)-006...
283. # [16:12] * Parts: gitbot (~gitbot@public.cloak)
284. # [16:14] * Joins: MikeSmith (~MikeSmith@public.cloak)
285. # [16:17] * Joins: gitbot (~gitbot@public.cloak)
286. # [16:17] -gitbot:#testing- [csswg-test] kojiishi pushed 1 new commit to master: https://github.com/w3c/csswg-test/commit/88f9e57ab438a3f907fe419b9e9b1592618f2047
287. # [16:17] -gitbot:#testing- csswg-test/master 88f9e57 Koji Ishii: [css-writing-modes-3] Fix outline-inline-vlr-006...
288. # [16:18] * Parts: gitbot (~gitbot@public.cloak)
289. # [16:18] * Joins: gitbot (~gitbot@public.cloak)
290. # [16:18] -gitbot:#testing- [web-platform-tests] youennf opened pull request #2350: Fixing link ref="match" in some reference files (master...fix-ref-info) https://github.com/w3c/web-platform-tests/pull/2350
291. # [16:18] * Parts: gitbot (~gitbot@public.cloak)
292. # [16:21] * Joins: gitbot (~gitbot@public.cloak)
293. # [16:21] -gitbot:#testing- [csswg-test] kojiishi pushed 1 new commit to master: https://github.com/w3c/csswg-test/commit/95a081859cc74460a19b194fb8757a53e0648a1b
294. # [16:21] -gitbot:#testing- csswg-test/master 95a0818 Koji Ishii: [css-writing-modes-3] Fixed ref links for outline-inline-v(lr|rl)-006
295. # [16:21] * Parts: gitbot (~gitbot@public.cloak)
296. # [16:22] * Quits: rego (~rego@public.cloak) (Ping timeout: 180 seconds)
297. # [16:33] * Joins: rego (~rego@public.cloak)
298. # [16:46] * Joins: ShaneM (~ShaneM@public.cloak)
299. # [17:03] * Joins: jyasskin_ (~textual@public.cloak)
300. # [17:03] * Quits: jyasskin_ (~textual@public.cloak) ("Textual IRC Client: www.textualapp.com")
301. # [17:42] * Quits: rego (~rego@public.cloak) ("Leaving")
302. # [18:09] * Joins: ShaneM_ (~ShaneM@public.cloak)
303. # [18:15] * Quits: ShaneM (~ShaneM@public.cloak) (Ping timeout: 180 seconds)
304. # [18:25] * Joins: zcorpan (~zcorpan@public.cloak)
305. # [18:29] * Quits: Ms2ger (~Ms2ger@public.cloak) ("nn")
306. # [18:36] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
307. # [19:04] * Joins: bhill2 (~bhill2@public.cloak)
308. # [19:06] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
309. # [19:06] * Joins: bhill2 (~bhill2@public.cloak)
310. # [19:10] * terri_offline is now known as terri
311. # [20:25] * terri is now known as terri_offline
312. # [20:31] * terri_offline is now known as terri
313. # [20:33] * Quits: ShaneM_ (~ShaneM@public.cloak) (Client closed connection)
314. # [20:34] * Quits: plh (plehegar@public.cloak) ("Leaving")
315. # [20:34] * Joins: plh (plehegar@public.cloak)
316. # [21:00] * Joins: zcorpan (~zcorpan@public.cloak)
317. # [21:04] * Joins: rniwa (~textual@public.cloak)
318. # [21:40] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
319. # [21:41] * Joins: zcorpan (~zcorpan@public.cloak)
320. # [21:48] * Quits: zcorpan (~zcorpan@public.cloak) (Ping timeout: 180 seconds)
321. # [21:48] * Joins: gitbot (~gitbot@public.cloak)
322. # [21:48] -gitbot:#testing- [web-platform-tests] AutomatedTester pushed 1 new commit to master: https://github.com/w3c/web-platform-tests/commit/846c2a11736fd0d9bc9dd09f60a38ab0b4b1dc56
323. # [21:48] -gitbot:#testing- web-platform-tests/master 846c2a1 AutomatedTester: [cssom-view] Add elementFromPoint tests...
324. # [21:49] * Parts: gitbot (~gitbot@public.cloak)
325. # [21:50] <ato> AutomatedTester: ++
326. # [21:50] * Joins: gitbot (~gitbot@public.cloak)
327. # [21:50] -gitbot:#testing- [web-platform-tests] AutomatedTester closed pull request #2348: element from point tests (master...dburns/elementFromPoint-tests) https://github.com/w3c/web-platform-tests/pull/2348
328. # [21:50] * Parts: gitbot (~gitbot@public.cloak)
329. # [21:51] <AutomatedTester> ato: I went to start work on elementsFromPoint and there are no tests for elementFromPoint in m-c
330. # [21:51] * Joins: gitbot (~gitbot@public.cloak)
331. # [21:51] -gitbot:#testing- [web-platform-tests] Wafflespeanut opened pull request #2352: More tests for crypto.getRandomValues() (master...webcrypto) https://github.com/w3c/web-platform-tests/pull/2352
332. # [21:51] * Parts: gitbot (~gitbot@public.cloak)
333. # [21:52] <ato> AutomatedTester: Test-driven development is so overrated, right?
334. # [21:52] <ato> s/Test-driven development/Tests/ I suppose.
335. # [21:53] <AutomatedTester> ato: are you suggesting people don't do TDD...
336. # [21:55] <AutomatedTester> now to do elementsFromPoint tests
337. # [22:00] * Joins: zcorpan (~zcorpan@public.cloak)
338. # [22:20] * Quits: dom (dom@public.cloak) ("")
339. # [22:20] <AutomatedTester> zcorpan: those tests have landed
340. # [22:20] <zcorpan> AutomatedTester: great, thanks!
341. # [22:20] <AutomatedTester> zcorpan: for elementsFromPoint tests, would you prefer a separate file?
342. # [22:21] <zcorpan> AutomatedTester: yeah
343. # [22:22] <AutomatedTester> zcorpan: since a lot of the tests are copies of the other file can I just CP and then make the changes I need?
344. # [22:26] <zcorpan> AutomatedTester: yes. but i can see that there are pros to using the same file, in particular for maintainability. or being DRY in some other way
345. # [22:26] <AutomatedTester> this is why I was asking :)
346. # [22:26] <zcorpan> yeah, it's not always clear for tests which approach is best
347. # [22:26] <AutomatedTester> let me do the tests and then I can come back to you
348. # [22:26] <AutomatedTester> since elementsFromPoint will have a more markup probably to get more layers of stuff
349. # [22:26] <zcorpan> making a file very busy is also bad because then it is difficult to debug what is going on
350. # [22:27] <AutomatedTester> yea, I will start separate and we can refactor afterwards
351. # [22:27] <AutomatedTester> let's get tests and then clean up later
352. # [22:28] <zcorpan> sounds good
353. # [22:28] <AutomatedTester> I dont want perfect to be the enemy of good
354. # [22:28] <zcorpan> sure
355. # [22:28] <zcorpan> i know i have copy-pasted tests :-)
356. # [22:33] <jgraham> DRY is less valuable for tests
357. # [22:34] <jgraham> You generally want them to be immediately understandable rather than composed of many layers of abstraction
358. # [22:35] * Joins: ShaneM (~ShaneM@public.cloak)
359. # [22:37] <AutomatedTester> I copied the tests over, tweaked and have 4 passing tests \o/
360. # [22:38] * AutomatedTester mumbles there are 5 failing tests
361. # [22:46] <AutomatedTester> all the tests pass
362. # [22:48] <AutomatedTester> jgraham: how do I disable these tests for m-c?
363. # [22:51] <jgraham> AutomatedTester: You don't
364. # [22:52] <AutomatedTester> awesome
365. # [22:52] <AutomatedTester> do I need to do anything to make the cssom tests now run?
366. # [22:52] <AutomatedTester> when you do your next merge
367. # [22:52] <AutomatedTester> trying to save you some work :)
368. # [22:54] <jgraham> No
369. # [22:54] <jgraham> Wait
370. # [22:54] <jgraham> Are these in wpt or css?
371. # [22:54] <AutomatedTester> wpt
372. # [22:55] <jgraham> Right, then no.
373. # [22:55] <jgraham> For css all bets are off; we will probably never run them
374. # [22:55] <AutomatedTester> this is why I put these into WPT
375. # [22:55] <jgraham> Or we have no current mechanism to
376. # [22:55] * Quits: plh (plehegar@public.cloak) ("Leaving")
377. # [22:55] <jgraham> Obviously I would like us to, and it's not technically impossible
378. # [22:58] <AutomatedTester> let
379. # [22:58] <AutomatedTester> let's leave that for servo :P
380. # [23:10] * Joins: gitbot (~gitbot@public.cloak)
381. # [23:10] -gitbot:#testing- [web-platform-tests] AutomatedTester opened pull request #2353: [cssom-view] Add elementsFromPoint tests (master...elementsFromPoint) https://github.com/w3c/web-platform-tests/pull/2353
382. # [23:10] * Parts: gitbot (~gitbot@public.cloak)
383. # [23:25] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
384. # [23:32] <gsnedders> I do wonder if adding CSS tests to wpt is just asking for a world of pain down the line when we have multiple testsuites for CSS testsuites…
385. # [23:51] * Joins: bhill2 (~bhill2@public.cloak)
386. # [23:51] * Quits: zcorpan (~zcorpan@public.cloak) (Client closed connection)
387. # [23:58] * Joins: bhill2_ (~bhill2@public.cloak)
388. # [23:58] * Quits: bhill2 (~bhill2@public.cloak) (Client closed connection)
389. # Session Close: Wed Nov 18 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