Layout Test svg/custom/svg-font-support.html is flaky |
|||||||
Issue descriptionThe following layout test is flaky on WebKit Linux Leak svg/custom/svg-font-support.html Probable cause: No idea, it looks like the test has been flaky since at least March 10. Unclear who should be the owner since the file hasn't been touch in over a year and a half and that was just to disable SVG fonts.
,
Mar 14 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/665734410ded994bf435f1b9e6ed0c82ee29fea6 commit 665734410ded994bf435f1b9e6ed0c82ee29fea6 Author: csharp <csharp@chromium.org> Date: Mon Mar 14 16:53:48 2016 Mark svg/custom/svg-font-support.html as flaky on Linux TBR=trchen@chromium.org BUG= 594570 Review URL: https://codereview.chromium.org/1800593003 Cr-Commit-Position: refs/heads/master@{#380991} [modify] https://crrev.com/665734410ded994bf435f1b9e6ed0c82ee29fea6/third_party/WebKit/LayoutTests/TestExpectations
,
Mar 14 2016
+pdr to the loop. Do you have a quick guess why it could flake?
,
Mar 14 2016
We should just remove the test entirely. We no longer support the main font it is testing. I'll put up a cl.
,
Mar 15 2016
It'd be interesting to find out what causes the leak - after all, the code is gone, so leak would appear to be real...
,
Mar 16 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/802572da31184ac329d8935049e61e204fc74142 commit 802572da31184ac329d8935049e61e204fc74142 Author: pdr <pdr@chromium.org> Date: Wed Mar 16 00:33:27 2016 Remove unsupported font SVGFreeSans.svg and svg-font-support.html We have not supported svg fonts for a long time but left this test and svg font file. BUG= 594570 , 594589 Review URL: https://codereview.chromium.org/1799993002 Cr-Commit-Position: refs/heads/master@{#381366} [modify] https://crrev.com/802572da31184ac329d8935049e61e204fc74142/third_party/WebKit/LayoutTests/TestExpectations [delete] https://crrev.com/45d2caec616113d4685129d6ae167ff4491fcc8a/third_party/WebKit/LayoutTests/svg/custom/resources/SVGFreeSans.svg [delete] https://crrev.com/45d2caec616113d4685129d6ae167ff4491fcc8a/third_party/WebKit/LayoutTests/svg/custom/svg-font-support-expected.html [delete] https://crrev.com/45d2caec616113d4685129d6ae167ff4491fcc8a/third_party/WebKit/LayoutTests/svg/custom/svg-font-support.html
,
May 18 2016
,
May 18 2016
Deprecating component:Blink>LayoutTests, to use label Test=Layout instead. Merging these to component:Blink for the Blink rotation to pick up and re-triage as appropriate.
,
May 23 2016
So far as I can tell, since we deleted the test in comment 6, we can wontfix this bug. |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by csharp@chromium.org
, Mar 14 2016