Bug 90030
Summary: | When reviewing patches, render tests and references | ||
---|---|---|---|
Product: | WebKit | Reporter: | Simon Fraser (smfr) <simon.fraser> |
Component: | Tools / Tests | Assignee: | Nobody <webkit-unassigned> |
Status: | NEW | ||
Severity: | Normal | CC: | abarth, ap, eric, rniwa, simon.fraser |
Priority: | P2 | ||
Version: | 528+ (Nightly build) | ||
Hardware: | Unspecified | ||
OS: | Unspecified |
Simon Fraser (smfr)
When reviewing a patch like https://bugs.webkit.org/attachment.cgi?id=149641&action=review I have a hard time evaluating the test and its reference, because I can't easily see what the test looks like (there's no pixel result).
I'd love for the patch reviewing UI to have some way to show the test and reference in iframes, so I can visually inspect them.
Attachments | ||
---|---|---|
Add attachment proposed patch, testcase, etc. |
Ojan Vafai
Agree this would be nice. Could even do a pixel diff in JS. It's a little tricker in cases where you're modifying an existing reftest, but even in that case, we already have the ability to apply the modification to tip-of-tree to get the full HTML file.