553834 - layout/reftests/first-letter/329069-2.html fails (not in manifest)

Status: RESOLVED FIXED

(Core :: Layout: Text and Fonts, defect)

Description

[Jesse Ruderman]

http://hg.mozilla.org/mozilla-central/rev/7b8c4eeac828 added files 329069-2.html and 329069-2-ref.html without adding them to the manifest.  Unfortunately, they currently fail, so I can't just add them.

Comment 1

[Simon Montagu :smontagu]

Attachment: Add to manifest, marked as failing

After confirming on tryserver that the test fails on all platforms, I suggest that we add it to the manifest as a failing test.

Comment 2

[Simon Montagu :smontagu]

Also we should change the summary of this bug to something like "Cluster iterator doesn't work correctly with Hangul Jamo".

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 433790 [details] [diff] [review]
Add to manifest, marked as failing

r=dbaron, but please do improve the summary of the bug, if you're going to use this bug for the failure.  (And of course don't mark it fixed when you check this in.)

Comment 4

[Jesse Ruderman]

I'd prefer if you marked this bug as fixed and used a new bug for the reftest.list comment.  It will make my dependency tracking saner.

Comment 5

[Simon Montagu :smontagu]

Filed bug 553981, and changed the patch to reference it.

Comment 6

[Simon Montagu :smontagu]

http://hg.mozilla.org/mozilla-central/rev/3425004f7c51