Bug 85499 - [EFL] CSS tests needs gardening
Summary: [EFL] CSS tests needs gardening
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit EFL (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Thiago Marcos P. Santos
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-05-03 07:57 PDT by Thiago Marcos P. Santos
Modified: 2012-05-03 10:12 PDT (History)
4 users (show)

See Also:


Attachments
patch (deleted)
2012-05-03 09:26 PDT, Thiago Marcos P. Santos
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Thiago Marcos P. Santos 2012-05-03 07:57:08 PDT
What I'm doing now:

- Rebaselining of all failing CSS tests.
- Moving all CSS tests from Skipped list to test_expectations.txt and creating the appropriated bug.
Comment 1 Thiago Marcos P. Santos 2012-05-03 09:26:20 PDT
Created attachment 140034 [details]
patch
Comment 2 WebKit Review Bot 2012-05-03 09:32:26 PDT
Attachment 140034 [details] did not pass style-queue:

Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/plat..." exit_code: 1
Traceback (most recent call last):
  File "Tools/Scripts/check-webkit-style", line 48, in <module>
    sys.exit(CheckWebKitStyle().main())
  File "/mnt/git/webkit-style-queue/Tools/Scripts/webkitpy/style/main.py", line 154, in main
    patch_checker.check(patch)
  File "/mnt/git/webkit-style-queue/Tools/Scripts/webkitpy/style/patchreader.py", line 71, in check
    self._text_file_reader.process_file(file_path=path, line_numbers=None)
  File "/mnt/git/webkit-style-queue/Tools/Scripts/webkitpy/style/filereader.py", line 130, in process_file
    self._processor.process(lines, file_path, **kwargs)
  File "/mnt/git/webkit-style-queue/Tools/Scripts/webkitpy/style/checker.py", line 844, in process
    min_confidence)
  File "/mnt/git/webkit-style-queue/Tools/Scripts/webkitpy/style/checker.py", line 619, in dispatch
    min_confidence)
  File "/mnt/git/webkit-style-queue/Tools/Scripts/webkitpy/style/checker.py", line 593, in _create_checker
    checker = PNGChecker(file_path, handle_style_error)
  File "/mnt/git/webkit-style-queue/Tools/Scripts/webkitpy/style/checkers/png.py", line 43, in __init__
    self._fs = host.filesystem
AttributeError: 'NoneType' object has no attribute 'filesystem'


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 3 Thiago Marcos P. Santos 2012-05-03 09:35:04 PDT
(In reply to comment #2)
> Attachment 140034 [details] did not pass style-queue:
> 
> Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'LayoutTests/ChangeLog', u'LayoutTests/plat..." exit_code: 1
> Traceback (most recent call last):
>   File "Tools/Scripts/check-webkit-style", line 48, in <module>
>     sys.exit(CheckWebKitStyle().main())
>   File "/mnt/git/webkit-style-queue/Tools/Scripts/webkitpy/style/main.py", line 154, in main
>     patch_checker.check(patch)
>   File "/mnt/git/webkit-style-queue/Tools/Scripts/webkitpy/style/patchreader.py", line 71, in check
>     self._text_file_reader.process_file(file_path=path, line_numbers=None)
>   File "/mnt/git/webkit-style-queue/Tools/Scripts/webkitpy/style/filereader.py", line 130, in process_file
>     self._processor.process(lines, file_path, **kwargs)
>   File "/mnt/git/webkit-style-queue/Tools/Scripts/webkitpy/style/checker.py", line 844, in process
>     min_confidence)
>   File "/mnt/git/webkit-style-queue/Tools/Scripts/webkitpy/style/checker.py", line 619, in dispatch
>     min_confidence)
>   File "/mnt/git/webkit-style-queue/Tools/Scripts/webkitpy/style/checker.py", line 593, in _create_checker
>     checker = PNGChecker(file_path, handle_style_error)
>   File "/mnt/git/webkit-style-queue/Tools/Scripts/webkitpy/style/checkers/png.py", line 43, in __init__
>     self._fs = host.filesystem
> AttributeError: 'NoneType' object has no attribute 'filesystem'
> 
> 
> If any of these errors are false positives, please file a bug against check-webkit-style.

Thanks bot, but this was an issue (bug 85504) on the style checker that I already fixed but was not landed yet. :-P
Comment 4 WebKit Review Bot 2012-05-03 10:12:33 PDT
Comment on attachment 140034 [details]
patch

Clearing flags on attachment: 140034

Committed r115987: <http://trac.webkit.org/changeset/115987>
Comment 5 WebKit Review Bot 2012-05-03 10:12:44 PDT
All reviewed patches have been landed.  Closing bug.