[webkit-reviews] review granted: [Bug 203784] Add support for finding test references via <link rel="match"> : [Attachment 383132] Patch

[bugzilla-daemon at webkit.org]

youenn fablet <youennf at gmail.com> has granted Simon Fraser (smfr)
<simon.fraser at apple.com>'s request for review:
Bug 203784: Add support for finding test references via <link rel="match">
https://bugs.webkit.org/show_bug.cgi?id=203784

Attachment 383132: Patch

https://bugs.webkit.org/attachment.cgi?id=383132&action=review




--- Comment #11 from youenn fablet <youennf at gmail.com> ---
Comment on attachment 383132
  --> https://bugs.webkit.org/attachment.cgi?id=383132
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=383132&action=review

> Tools/Scripts/webkitpy/port/base.py:550
> +		   _log.warning('Multiple references are not supported.
Importing the first ref defined in {}'.format(test_name))

This code is coming from TestParser whose purpose is to import tests.
The purpose of this code is to run tests, so we should update the message
accordingly.
Or maybe just remove the warning altogether since we know this is a limitation
of webkitpy.