Fix failing test from bug 708175

RESOLVED FIXED in mozilla14

Status

()

Core
Layout
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: jwir3, Assigned: jwir3)

Tracking

Trunk
mozilla14
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
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.
(Assignee)

Comment 1

5 years ago
Created attachment 615518 [details] [diff] [review]
b745993

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+
(Assignee)

Comment 3

5 years ago
(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.
(Assignee)

Comment 4

5 years ago
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
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.