[Webkit-unassigned] [Bug 185727] Remove cssom-remove-shorthand-property.html test

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu May 17 22:14:11 PDT 2018


https://bugs.webkit.org/show_bug.cgi?id=185727

Frédéric Wang (:fredw) <fred.wang at free.fr> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |177684

--- Comment #6 from Frédéric Wang (:fredw) <fred.wang at free.fr> ---
(In reply to Alexey Proskuryakov from comment #5)
> It's not great to remove tests, because that loses all the history. In this
> case, how would one know that there was discussion in
> https://bugs.webkit.org/show_bug.cgi?id=68551 ?
> 
> I guess at least there's a link from WPT source. But it's so much harder to
> find.

Yes, sorry about that. This came in the context of bug 177684, probably it would have make more sense for us to not apply the rule of splitting work into smaller patches here, so that we preserve the history link between the new file and old file (at least that's not worse than when people rename/move directories or files in WebKit). I'm adding a Bugzilla dependency to improve that a bit for future searches.

Anyway, I believe it makes sense to migrate to a WPT test and remove duplicate tests for this kind of bug reports, which could have been noticed earlier if the people who fixed the behavior in other browsers or clarified a spec had written new WPT tests and if one had sync CSSOM tests between WPT and WebKit.

For Web Platform development, it seems more helpful to me to link to interoperability tests, to the current specs or to other browsers' behaviors than to discussions that happened several years ago (not saying these discussions were not important of course).


Referenced Bugs:

https://bugs.webkit.org/show_bug.cgi?id=177684
[Bug 177684] Cannot unset transition with important
-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-unassigned/attachments/20180518/dc2daa57/attachment.html>


More information about the webkit-unassigned mailing list