Closed
Bug 469513
Opened 16 years ago
Closed 16 years ago
453406-1.html : missing |dump('\n');| at end
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
VERIFIED
FIXED
mozilla1.9.1b3
People
(Reporter: sgautherie, Assigned: sgautherie)
References
()
Details
(Keywords: fixed1.9.1, polish)
Attachments
(1 file)
|
(deleted),
patch
|
sgautherie
:
review+
sgautherie
:
superreview+
|
Details | Diff | Splinter Review |
Current log is
{
REFTEST TEST-PASS | file:///E:/slave/trunk_win2k3-10/build/content/html/content/crashtests/451123-1.html | (LOAD ONLY)
aaabbbREFTEST TEST-PASS | file:///E:/slave/trunk_win2k3-10/build/content/html/content/crashtests/453406-1.html | (LOAD ONLY)
REFTEST TEST-PASS | file:///E:/slave/trunk_win2k3-10/build/content/svg/content/src/crashtests/307322-1.svg | (LOAD ONLY)
}
Adding an eol would allow the next test log to start at the beginning of the line, where it is +/- expected.
|
||
Comment 1•16 years ago
|
||
Oh, that dump('bbb') should just be removed. r+sr=bzbarsky to do that.
|
Assignee | |
Comment 2•16 years ago
|
||
(In reply to comment #1)
> Oh, that dump('bbb') should just be removed. r+sr=bzbarsky to do that.
Good, then what about |dump('aaa');| ?
|
|
||
Comment 3•16 years ago
|
||
Same thing.
|
|
Assignee | |
Comment 4•16 years ago
|
||
(In reply to comment #1)
> r+sr=bzbarsky
Assignee: nobody → sgautherie.bz
Status: NEW → ASSIGNED
Attachment #352911 -
Flags: superreview+
Attachment #352911 -
Flags: review+
|
|
Assignee | |
Comment 5•16 years ago
|
||
Comment on attachment 352911 [details] [diff] [review]
(Av1) Remove the |dump()|s
[Checkin: Comment 5 & 6]
http://hg.mozilla.org/mozilla-central/rev/6539acbc98f8
Attachment #352911 -
Attachment description: (Av1) Remove the |dump()|s → (Av1) Remove the |dump()|s
[Checkin: Comment 5]
|
|
Assignee | |
Updated•16 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [c-n: baking for 1.9.1]
Target Milestone: --- → mozilla1.9.2a1
|
|
Assignee | |
Comment 6•16 years ago
|
||
Comment on attachment 352911 [details] [diff] [review]
(Av1) Remove the |dump()|s
[Checkin: Comment 5 & 6]
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/58005c72de96
Attachment #352911 -
Attachment description: (Av1) Remove the |dump()|s
[Checkin: Comment 5] → (Av1) Remove the |dump()|s
[Checkin: Comment 5 & 6]
|
|
Assignee | |
Comment 7•16 years ago
|
||
{
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1229268401.1229271244.30216.gz&fulltext=1
Linux mozilla-central moz2-linux-slave07 dep unit test on 2008/12/14 07:26:41
REFTEST TEST-PASS | file:///builds/slave/trunk_linux-7/build/content/html/content/crashtests/451123-1.html | (LOAD ONLY)
REFTEST TEST-PASS | file:///builds/slave/trunk_linux-7/build/content/html/content/crashtests/453406-1.html | (LOAD ONLY)
REFTEST TEST-PASS | file:///builds/slave/trunk_linux-7/build/content/svg/content/src/crashtests/307322-1.svg | (LOAD ONLY)
}
V.Fixed
Status: RESOLVED → VERIFIED
Keywords: checkin-needed → fixed1.9.1
Whiteboard: [c-n: baking for 1.9.1]
Target Milestone: mozilla1.9.2a1 → mozilla1.9.1b3
You need to log in
before you can comment on or make changes to this bug.
Description
•