[webkit-reviews] review granted: [Bug 201025] results.webkit.org: Escape html in changelog : [Attachment 377028] Patch

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Jonathan Bedard <jbedard at apple.com>'s
request for review:
Bug 201025: results.webkit.org: Escape html in changelog
https://bugs.webkit.org/show_bug.cgi?id=201025

Attachment 377028: Patch

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




--- Comment #11 from Darin Adler <darin at apple.com> ---
Comment on attachment 377028
  --> https://bugs.webkit.org/attachment.cgi?id=377028
Patch

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

While I’m not an expert at this, I’ll review since I spotted a bug earlier.

> Tools/resultsdbpy/resultsdbpy/view/templates/commit.html:64
> +		   return '';

Maybe this should return match instead of ''.