Bug 58101 – new-run-webkit-tests: minor cleanup to be consistent in handling missing files

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Dirk Pranke]

new-run-webkit-tests: minor cleanup to be consistent in handling missing files

[Dirk Pranke]

Created attachment 88741 [details]
Patch

[Dirk Pranke]

Created attachment 88755 [details]
Patch

[Dirk Pranke]

this patch is some cleanup needed prior to adding support for webaudio; it makes the changes in the webaudio patch easier to understand.

[Ojan Vafai]

Comment on attachment 88755 [details]
Patch

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

> Tools/ChangeLog:10
> +        new-run-webkit-tests: clean up the way we handle missing files,
> +        to be consistent. With this change, the Port.expected_X() and
> +        Driver.run_test() routines should return None if there is no
> +        output, not ''.
> +
> +        https://bugs.webkit.org/show_bug.cgi?id=58101

Usually the ChangeLog format is:

Reviewed by ...

one-line description that usually matches the bug title
http://bugs.webkit.org/...

detailed description here

* files modified

> Tools/Scripts/webkitpy/layout_tests/layout_package/single_test_runner.py:228
> +            self._port.compare_text(self._get_normalized_output_text(actual_text),
> +                                    # Assuming expected_text is already normalized.
> +                                    expected_text)):

This is difficult for me to read. Could you put it all on one line and put the comment above the if-statement?

[Dirk Pranke]

(In reply to comment #4)
> (From update of attachment 88755 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=88755&action=review
> 
> > Tools/ChangeLog:10
> > +        new-run-webkit-tests: clean up the way we handle missing files,
> > +        to be consistent. With this change, the Port.expected_X() and
> > +        Driver.run_test() routines should return None if there is no
> > +        output, not ''.
> > +
> > +        https://bugs.webkit.org/show_bug.cgi?id=58101
> 
> Usually the ChangeLog format is:
> 
> Reviewed by ...
> 
> one-line description that usually matches the bug title
> http://bugs.webkit.org/...
> 
> detailed description here
> 

Okay. I'm not sure how it is that I've done hundreds of commits and am learning this just now :)

> * files modified
> 
> > Tools/Scripts/webkitpy/layout_tests/layout_package/single_test_runner.py:228
> > +            self._port.compare_text(self._get_normalized_output_text(actual_text),
> > +                                    # Assuming expected_text is already normalized.
> > +                                    expected_text)):
> 
> This is difficult for me to read. Could you put it all on one line and put the comment above the if-statement?

Will do as part of landing this.

[Dirk Pranke]

Committed r83327: <http://trac.webkit.org/changeset/83327>