Closed
Bug 661755
Opened 13 years ago
Closed 13 years ago
[session restore cleanup] Fix line endings in tests
Categories
(Firefox :: Session Restore, defect)
Firefox
Session Restore
Tracking
()
RESOLVED
FIXED
Firefox 7
People
(Reporter: zpao, Assigned: zpao)
Details
Attachments
(1 file)
|
5.68 KB,
patch
|
dietrich
:
review+
|
Details | Diff | Splinter Review |
There are a number of tests that have non-unix line endings. Invisible, but I encountered them patching a test today and it bothered me. Dietrich, I have some other cleanup things laying around here (stripping out trailing whitespace, fixing some tests for consistency, etc.). Do these fall under the category of things that should be reviewed or can I just land them? (it might make blame a bit harder, but meh...)
Attachment #537076 -
Flags: review?(dietrich)
|
||
Comment 1•13 years ago
|
||
Comment on attachment 537076 [details] [diff] [review] Patch v0.1 r=me. yeah, no review required for minor test-only changes like these IMO. just watch the tree ;)
Attachment #537076 -
Flags: review?(dietrich) → review+
|
Assignee | |
Updated•13 years ago
|
Whiteboard: [incoming]
|
||
Comment 2•13 years ago
|
||
Pushed: http://hg.mozilla.org/mozilla-central/rev/f4b55635abb1
Assignee: nobody → paul
Status: NEW → RESOLVED
Closed: 13 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [incoming]
Target Milestone: --- → Firefox 7
Version: unspecified → Trunk
|
|
Assignee | |
Comment 3•13 years ago
|
||
in-testsuite- (just line endings changed, no tests needed)
Flags: in-testsuite+ → in-testsuite-
You need to log in
before you can comment on or make changes to this bug.
Description
•