don't needlessly construct nsAutoCString temporaries when serializing GMPVideoCodec

RESOLVED FIXED in Firefox 48

Status

()

Core
Audio/Video: GMP
P2
normal
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: froydnj, Assigned: froydnj)

Tracking

unspecified
mozilla48
Points:
---

Firefox Tracking Flags

(firefox48 fixed)

Details

Attachments

(1 attachment)

We have nsDependentCString to avoid needless memory traffic here.  We
need an explicit cast, however, because we don't have a ParamTraits
specialization for nsDependentCString, only for nsCString, and template
specialization doesn't take derived classes into account.
Created attachment 8738575 [details] [diff] [review]
don't needlessly construct nsAutoCString temporaries when serializing GMPVideoCodec
Attachment #8738575 - Flags: review?(cpearce)
Attachment #8738575 - Flags: review?(cpearce) → review+
Component: Audio/Video → Audio/Video: GMP
Priority: -- → P2

Comment 2

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/c7d9bb9a69b8

Comment 3

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/c7d9bb9a69b8
Status: NEW → RESOLVED
Last Resolved: 2 years ago
status-firefox48: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
You need to log in before you can comment on or make changes to this bug.