put unexpected passes into their own table

[Ojan Vafai]

put unexpected passes into their own table

[Ojan Vafai]

Created attachment 90382 [details]
Patch

[Eric Seidel (no email)]

Comment on attachment 90382 [details]
Patch

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

> Tools/Scripts/webkitpy/layout_tests/layout_package/json_results.html:11
> +tbody tr:first-of-type:hover {

I'm confused why you want to match the first tr in the table body and only on hover.

[Ojan Vafai]

(In reply to comment #2)
> (From update of attachment 90382 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=90382&action=review
> 
> > Tools/Scripts/webkitpy/layout_tests/layout_package/json_results.html:11
> > +tbody tr:first-of-type:hover {
> 
> I'm confused why you want to match the first tr in the table body and only on hover.

This change is actually just so that we don't change opacity when you hover over the thead.

We put each row in it's own tbody. Then we use a second tr for the expected results. We don't want to change the opacity of the expected results on hover. So the HTML looks something like the following:

<table>
  <thead>...</thead>
  <tbody>
    <tr>test name + links to results</tr>
    <tr>iframes/imgs to the inline results</tr>
  </tbody>
  <tbody>
    <tr>test name + links to results</tr>
    <tr>iframes/imgs to the inline results</tr>
  </tbody>
</table>

This way, when we sort the rows, we can sort the tbody elements and the inline results sort along with the test + result links.

[Ojan Vafai]

Comment on attachment 90382 [details]
Patch

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

>>> Tools/Scripts/webkitpy/layout_tests/layout_package/json_results.html:11
>>> +tbody tr:first-of-type:hover {
>> 
>> I'm confused why you want to match the first tr in the table body and only on hover.
> 
> This change is actually just so that we don't change opacity when you hover over the thead.
> 
> We put each row in it's own tbody. Then we use a second tr for the expected results. We don't want to change the opacity of the expected results on hover. So the HTML looks something like the following:
> 
> <table>
>   <thead>...</thead>
>   <tbody>
>     <tr>test name + links to results</tr>
>     <tr>iframes/imgs to the inline results</tr>
>   </tbody>
>   <tbody>
>     <tr>test name + links to results</tr>
>     <tr>iframes/imgs to the inline results</tr>
>   </tbody>
> </table>
> 
> This way, when we sort the rows, we can sort the tbody elements and the inline results sort along with the test + result links.

I can cut this or do this in a separate patch if you need.

[Tony Chang]

Comment on attachment 90382 [details]
Patch

Ojan says he has a version that is rebased with the globalState() change.

[Ojan Vafai]

Created attachment 90554 [details]
Patch

[Ojan Vafai]

Committed r84518: <http://trac.webkit.org/changeset/84518>

[WebKit Review Bot]

http://trac.webkit.org/changeset/84518 might have broken Qt Linux ARMv7 Release and Qt Windows 32-bit Debug