Bug 156912

Summary: testharnessreport.js should try to not produce reports with spaces at the end of the line.
Product: WebKit Reporter: youenn fablet <youennf>
Component: WebCore Misc.Assignee: youenn fablet <youennf>
Status: NEW    
Severity: Normal CC: ahmad.saleem792, annevk, ap, fred.wang, gsnedders, ntim
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

youenn fablet
Reported 2016-04-22 08:44:27 PDT
testharnessreport.js should try to not produce reports with spaces at the end of the line. git commit does not like it.
Attachments
Patch (deleted)
2016-04-22 08:52 PDT, youenn fablet 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
youenn fablet
Comment 1 2016-04-22 08:52:34 PDT
Created attachment 277064 [details] Patch
Ahmad Saleem
Comment 2 2023-05-22 09:01:03 PDT
Is this needed or we can close this, I think we keep 'testharnessreport.js' in sync with WPT, so we might not need to deviate from it and as such close this as 'RESOLVED INVALID'.
Sam Sneddon [:gsnedders]
Comment 3 2023-05-22 15:12:30 PDT
(In reply to Ahmad Saleem from comment #2) > Is this needed or we can close this, I think we keep 'testharnessreport.js' > in sync with WPT, so we might not need to deviate from it and as such close > this as 'RESOLVED INVALID'. testharnessreport.js is intended to be overridden by each test runner; thus we override it to make testharness.js tests work with run-webkit-tests. It's probably still reasonable to fix this, though not particularly high priority.
Note You need to log in before you can comment on or make changes to this bug.