Closed
Bug 1086714
Opened 10 years ago
Closed 10 years ago
Change test_selection_underline.html's log output so that it can be used with reftest-analyzer.xhtml
Categories
(Core :: Layout, defect)
Core
Layout
Tracking
()
RESOLVED
FIXED
mozilla36
People
(Reporter: jwatt, Assigned: jwatt)
Details
Attachments
(1 file)
3.27 KB,
patch
|
dholbert
:
review+
jwatt
:
checkin+
|
Details | Diff | Splinter Review |
No description provided.
![]() |
Assignee | |
Comment 1•10 years ago
|
||
Getting reftests failing under this test, similar to bug 1084650. Let's fix the test so that it's output can be compared with the analyzer.
![]() |
Assignee | |
Comment 2•10 years ago
|
||
Attachment #8508863 -
Flags: review?(dholbert)
Comment 3•10 years ago
|
||
Comment on attachment 8508863 [details] [diff] [review]
patch
>diff --git a/layout/generic/test/test_selection_underline.html b/layout/generic/test/test_selection_underline.html
>+ var description = "(selection: " + aTest.selection.typeName +
>+ ", style: " + aTest.decoration.styleName +
>+ ", relativeSize: " + aTest.decoration.relativeSize +
>+ ", font: " + aTest.font.family +
>+ ", default font size: " + aTest.font.defaultSize + ")";
>+
^
Nit: whitespace on blank line here (a single space character) -- remove.
r=me
Attachment #8508863 -
Flags: review?(dholbert) → review+
![]() |
Assignee | |
Comment 4•10 years ago
|
||
Comment on attachment 8508863 [details] [diff] [review]
patch
https://hg.mozilla.org/integration/mozilla-inbound/rev/e1f058f9565b
Attachment #8508863 -
Flags: checkin+
Comment 5•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in
before you can comment on or make changes to this bug.
Description
•