[EME] Include pid in the log messages of GMPParent

RESOLVED FIXED in Firefox 39

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: jwwang, Assigned: jwwang)

Tracking

unspecified
mozilla39
Points:
---

Firefox Tracking Flags

(firefox39 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

4 years ago
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.
(Assignee)

Updated

4 years ago
Depends on: 1142917
(Assignee)

Comment 2

4 years ago
Created attachment 8577883 [details] [diff] [review]
1143532_include_pid_in_log_messages-v1.patch

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)
(Assignee)

Comment 3

4 years ago
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
Last Resolved: 4 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.