Strip null bytes in document titles when setting the chrome window's title.

RESOLVED FIXED in seamonkey2.19

Status

RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: philip.chee, Assigned: philip.chee)

Tracking

Trunk
seamonkey2.19

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment on attachment 725422 [details] [diff] [review]
Patch v1.0 Proposed fix.

>+              docTitle = this.contentTitle.replace("\0", "", "g");
Nit: Please use the RegExp form /\0+/g instead.

>-              if (docElement.getAttribute("chromehidden").indexOf("location") != -1) {
>+              if (docElement.getAttribute("chromehidden").contains("location")) {
[Unless this is a stray leftover then these sorts of changes should be done in a dedicated bug.]
Attachment #725422 - Flags: review?(neil) → review+
(Assignee)

Comment 3

6 years ago
Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/bf49737b12dc

> Nit: Please use the RegExp form /\0+/g instead.
Fixed.

> [Unless this is a stray leftover then these sorts of changes should be done in a dedicated bug.]
Removed.
(Assignee)

Updated

6 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.19
You need to log in before you can comment on or make changes to this bug.