Fix up the string manipulation in nsUpdateProcessor::ProcessUpdate

RESOLVED FIXED in mozilla15

Status

()

Toolkit
Application Update
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Away for a while, Assigned: Away for a while)

Tracking

unspecified
mozilla15
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

2.12 KB, patch
neil@parkwaycc.co.uk
: review+
Details | Diff | Splinter Review
(Assignee)

Description

6 years ago
See bug 307181 comment 190.
(Assignee)

Comment 1

6 years ago
Created attachment 627328 [details] [diff] [review]
Patch (v1)
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #627328 - Flags: review?(neil)

Comment 2

6 years ago
Comment on attachment 627328 [details] [diff] [review]
Patch (v1)

Bonus points if you remove the other unnecessary PromiseFlatCString too.
Attachment #627328 - Flags: review?(neil) → review+
(Assignee)

Comment 3

6 years ago
Done!

https://hg.mozilla.org/integration/mozilla-inbound/rev/be99d6a44513
Target Milestone: --- → mozilla15
https://hg.mozilla.org/mozilla-central/rev/be99d6a44513
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.