Closed Bug 1143532 Opened 8 years ago Closed 8 years ago

[EME] Include pid in the log messages of GMPParent

Categories

(Core :: Audio/Video, defect)

Product:
Core
Component:
Audio/Video
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla39
Tracking Flags:
Tracking Status
firefox39 --- fixed

People

(Reporter: jwwang, Assigned: jwwang)

References

Details

Attachments

(1 file)

1143532_include_pid_in_log_messages-v1.patch
10.68 KB, patch
eflores
: review+
Details | Diff | Splinter Review 
Since we could have many GMPParent instances in the chrome process, include the pid of the child process will let us know which GMPChild this GMPParent is talking to.
Depends on: 1142917
1. include pid of the child process to know which GMPChild we are talking to.
2. fix wrong class name in GMPStorageParent.cpp
Assignee: nobody → jwwang
Status: NEW → ASSIGNED
Attachment #8577883 - Flags: review?(edwin)
Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=66db21223ae2
Those oranges are not related to EME.
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/a29d23e9d4b6
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-firefox39: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
You need to log in before you can comment on or make changes to this bug.