Closed Bug 1098552 Opened 10 years ago Closed 10 years ago

Bookmarks.jsm should use case-insensitive keywords

Categories

(Toolkit :: Places, defect)

defect
Not set
normal
Points:
2

Tracking

()

RESOLVED FIXED
mozilla37
Iteration:
37.2

People

(Reporter: mak, Assigned: mak)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

keywords should be case-insensitive, always lowercased in the backend.
Points: --- → 2
Flags: qe-verify-
Flags: firefox-backlog+
Assignee: nobody → mak77
Status: NEW → ASSIGNED
Iteration: --- → 37.1
Iteration: 37.1 → 37.2
Attached file MozReview Request: bz://1098552/mak (obsolete) (deleted) —
Attachment #8533806 - Flags: review?(mano)
/r/1357 - Bug 1098552 - Bookmarks.jsm should use case-insensitive keywords. r=Mano

Pull down this commit:

hg pull review -r 50db985a28194821d3e5e48d829e5af240ffdb31
https://reviewboard.mozilla.org/r/1355/#review787

::: toolkit/components/places/Bookmarks.jsm
(Diff revision 1)
> +    // Keywords are handled as case-insensitive, by always lowercasing them.

I think "Keywords are handled as case-insensitive." is enough.
Attachment #8533806 - Flags: review?(mano) → review+
https://hg.mozilla.org/integration/fx-team/rev/b741664b67a3
Flags: in-testsuite+
Target Milestone: --- → mozilla37
https://hg.mozilla.org/mozilla-central/rev/b741664b67a3
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Attachment #8533806 - Attachment is obsolete: true
Attachment #8618633 - Flags: review+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: