Closed Bug 745993 Opened 10 years ago Closed 10 years ago

Fix failing test from bug 708175

Categories

(Core :: Layout, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla14

People

(Reporter: jwir3, Assigned: jwir3)

References

Details

Attachments

(1 file)

There was a mistake in the checkin for bug 708175 that caused a test to not be run correctly, leading to a checked-in failing test, which is currently commented-out.

See bug 708175, comment 43.
Attached patch b745993Splinter Review
The reference file should have had font-size equal to 34, not 12.
Attachment #615518 - Flags: review?(dbaron)
Comment on attachment 615518 [details] [diff] [review]
b745993

r=dbaron, though against a current mozilla-inbound tree the reftest.list change should look like uncommenting a line.  (Don't try landing on central, because then you'll leave the inbound-central merger to deal with the mess.)
Attachment #615518 - Flags: review?(dbaron) → review+
(In reply to David Baron [:dbaron] from comment #2)
> Comment on attachment 615518 [details] [diff] [review]
> b745993
> 
> r=dbaron, though against a current mozilla-inbound tree the reftest.list
> change should look like uncommenting a line.  (Don't try landing on central,
> because then you'll leave the inbound-central merger to deal with the mess.)

Yup, I had planned on merging locally with m-i before pushing back up to inbound.
Pushed to inbound:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9e11f5d4b7d9
Status: NEW → UNCONFIRMED
Ever confirmed: false
Target Milestone: --- → mozilla14
Status: UNCONFIRMED → NEW
Ever confirmed: true
https://hg.mozilla.org/mozilla-central/rev/9e11f5d4b7d9
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.