Bookmarks.jsm should use case-insensitive keywords

RESOLVED FIXED in mozilla37

Status

()

defect
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: mak, Assigned: mak)

Tracking

(Blocks 1 bug)

Trunk
mozilla37
Points:
2
Dependency tree / graph
Bug Flags:
firefox-backlog +
in-testsuite +
qe-verify -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

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
Posted file MozReview Request: bz://1098552/mak (obsolete) —
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.
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: 5 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.