Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Status

()

Firefox
General
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Kwan, Assigned: Kwan)

Tracking

Trunk
Firefox 14
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Work in bug 737792 showed up some errors in the test file browser_bug329212.js with misplaced brackets causing the error message to be inside the expression and thus not show up.
(Assignee)

Comment 1

5 years ago
Created attachment 608316 [details] [diff] [review]
Fix errors in the test

This fixes the errors in the test, moving misplaced brakets so the error message is an argument for ok() and not the function in the expression.  Also changed an ok(foo != bar) to isnot(foo, bar).
(Assignee)

Updated

5 years ago
Blocks: 329212

Updated

5 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true
Comment on attachment 608316 [details] [diff] [review]
Fix errors in the test

Thanks!
Attachment #608316 - Flags: review+

Updated

5 years ago
Assignee: nobody → moz-ian
Keywords: checkin-needed
(Assignee)

Comment 3

5 years ago
Comment on attachment 608316 [details] [diff] [review]
Fix errors in the test

Hmm, bzexport didn't update the patch description, so whoever checks it in will have to add the bug number & description.
http://hg.mozilla.org/integration/mozilla-inbound/rev/8889d78513a7
Keywords: checkin-needed
Target Milestone: --- → Firefox 14

Comment 5

5 years ago
https://hg.mozilla.org/mozilla-central/rev/8889d78513a7
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Blocks: 601091
You need to log in before you can comment on or make changes to this bug.