Bug 64554

Summary: Remove the notion of TestExpectationLine.valid, start storing parsing errors in expectations themselves.
Product: WebKit Reporter: Dimitri Glazkov (Google) <dglazkov>
Component: Tools / TestsAssignee: Dimitri Glazkov (Google) <dglazkov>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 64531    
Bug Blocks: 64385, 64559    
Attachments:
Description Flags
Patch none

Dimitri Glazkov (Google)
Reported 2011-07-14 13:01:59 PDT
Remove the notion of TestExpectationLine.valid, start storing parsing errors in expectations themselves.
Attachments
Patch (20.61 KB, patch)
2011-07-14 13:09 PDT, Dimitri Glazkov (Google) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Dimitri Glazkov (Google)
Comment 1 2011-07-14 13:09:53 PDT
Created attachment 100852 [details] Patch
Adam Barth
Comment 2 2011-07-14 22:47:43 PDT
Comment on attachment 100852 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=100852&action=review > Tools/Scripts/webkitpy/layout_tests/models/test_expectations.py:726 > + modifiers = [o for o in options if o in self.MODIFIERS] normally we'd use the variable option instead of o
Adam Barth
Comment 3 2011-07-14 22:48:40 PDT
Comment on attachment 100852 [details] Patch I mean, I feel like I don't fully understand this change, which makes me hesitant to review it. However, the direction looks good.
Dimitri Glazkov (Google)
Comment 4 2011-07-15 08:51:43 PDT
Comment on attachment 100852 [details] Patch Clearing flags on attachment: 100852 Committed r91071: <http://trac.webkit.org/changeset/91071>
Dimitri Glazkov (Google)
Comment 5 2011-07-15 08:51:48 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.