UMR in nsAsyncEventRunner::Run, LoadNextSource::Run, and SelectResourceEvent::Run

RESOLVED FIXED

Status

()

RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: kinetik, Assigned: kinetik)

Tracking

({fixed1.9.1, valgrind})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

10 years ago
nsHTMLMediaElement does not initialize mCurrentLoadID.  This results in UMR warnings in the Run member function of nsAsyncEventRunner, LoadNextSource, and SelectResourceEvent.
(Assignee)

Comment 1

10 years ago
Created attachment 367555 [details] [diff] [review]
patch v0

Trivial fix.  Also clean up the order of initialization and initialize
Assignee: nobody → kinetik
Attachment #367555 - Flags: superreview?(roc)
Attachment #367555 - Flags: review?(roc)
(Assignee)

Comment 2

10 years ago
...initialize mPausedBeforeFreeze as well.
Attachment #367555 - Flags: superreview?(roc)
Attachment #367555 - Flags: superreview+
Attachment #367555 - Flags: review?(roc)
Attachment #367555 - Flags: review+
(Assignee)

Updated

10 years ago
Whiteboard: [needs landing]
http://hg.mozilla.org/mozilla-central/rev/c93612f649ab
Status: NEW → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Whiteboard: [needs landing] → [needs 191 landing]
You need to log in before you can comment on or make changes to this bug.