)]}'
{"id":"LineageOS%2Fandroid_frameworks_av~399422","triplet_id":"LineageOS%2Fandroid_frameworks_av~lineage-20.0~I8e6ba40fe3da30bf8753e7a16ad5c8cd5dfda40b","project":"LineageOS/android_frameworks_av","branch":"lineage-20.0","topic":"T_asb_2024-08","hashtags":[],"change_id":"I8e6ba40fe3da30bf8753e7a16ad5c8cd5dfda40b","subject":"libmediatranscoding: handle death recipient cookie ownership differently","status":"MERGED","created":"2024-08-09 16:46:57.000000000","updated":"2024-08-12 11:56:19.000000000","submitted":"2024-08-12 11:56:19.000000000","submitter":{"_account_id":15173,"name":"Kevin Haggerty","email":"haggertk@lineageos.org","username":"haggertk","avatars":[{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"399422-T_asb_2024-08","meta_rev_id":"6dfc68a7386bdb7146333e46cbf1a15f21681d3f","_number":399422,"virtual_id_number":399422,"owner":{"_account_id":15173,"name":"Kevin Haggerty","email":"haggertk@lineageos.org","username":"haggertk","avatars":[{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"actions":{},"labels":{"Verified":{"all":[{"value":0,"_account_id":15173,"name":"Kevin Haggerty","email":"haggertk@lineageos.org","username":"haggertk","avatars":[{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]}],"values":{"-1":"Fails"," 0":"No score","+1":"Verified"},"description":"","default_value":0},"Code-Review":{"all":[{"value":0,"_account_id":15173,"name":"Kevin Haggerty","email":"haggertk@lineageos.org","username":"haggertk","avatars":[{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]}],"values":{"-2":"Do not submit","-1":"I would prefer that you didn\u0027t submit this"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me, approved"},"description":"","default_value":0},"CI":{"all":[{"value":0,"_account_id":15173,"name":"Kevin Haggerty","email":"haggertk@lineageos.org","username":"haggertk","avatars":[{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]}],"values":{"-1":"Fail"," 0":"No score","+1":"Pass"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{},"pending_reviewers":{},"reviewer_updates":[],"messages":[{"id":"f3a68cb09359a297110ad089ebecfe978caa0017","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":15173,"name":"Kevin Haggerty","email":"haggertk@lineageos.org","username":"haggertk","avatars":[{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"date":"2024-08-09 16:46:57.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"6dfc68a7386bdb7146333e46cbf1a15f21681d3f","tag":"autogenerated:gerrit:merged","author":{"_account_id":15173,"name":"Kevin Haggerty","email":"haggertk@lineageos.org","username":"haggertk","avatars":[{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"date":"2024-08-12 11:56:19.000000000","message":"Change has been successfully pushed.","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"f21c760322b58d7cb92579cb18ff75869ddd345c","revisions":{"ab962689f840ba0a2d1a094c5bd98ddcd698b2c3":{"kind":"REWORK","_number":1,"created":"2024-08-09 16:46:57.000000000","uploader":{"_account_id":15173,"name":"Kevin Haggerty","email":"haggertk@lineageos.org","username":"haggertk","avatars":[{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"ref":"refs/changes/22/399422/1","fetch":{"anonymous http":{"url":"https://github.com/LineageOS/android_frameworks_av","ref":"refs/changes/22/399422/1","commands":{"Branch":"git fetch https://github.com/LineageOS/android_frameworks_av refs/changes/22/399422/1 \u0026\u0026 git checkout -b change-399422 FETCH_HEAD","Checkout":"git fetch https://github.com/LineageOS/android_frameworks_av refs/changes/22/399422/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://github.com/LineageOS/android_frameworks_av refs/changes/22/399422/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://github.com/LineageOS/android_frameworks_av refs/changes/22/399422/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://github.com/LineageOS/android_frameworks_av refs/changes/22/399422/1","Reset To":"git fetch https://github.com/LineageOS/android_frameworks_av refs/changes/22/399422/1 \u0026\u0026 git reset --hard FETCH_HEAD"}}},"commit":{"parents":[{"commit":"214685e2de63df75def32de6356a4d2853f5d5b7","subject":"Reapply \"Audio policy: anonymize Bluetooth MAC addresses\"","web_links":[{"name":"GitHub","tooltip":"Open in GitWeb","url":"https://github.com/LineageOS/android_frameworks_av/commit/214685e2de63df75def32de6356a4d2853f5d5b7"}]}],"author":{"name":"Devin Moore","email":"devinmoore@google.com","date":"2024-02-23 19:19:38.000000000","tz":0},"committer":{"name":"Kevin F. Haggerty","email":"haggertk@lineageos.org","date":"2024-08-09 16:08:59.000000000","tz":-360},"subject":"libmediatranscoding: handle death recipient cookie ownership differently","message":"libmediatranscoding: handle death recipient cookie ownership differently\n\nThe ownership of the death recipient cookie is now limited to the\nTranscodingResourcePolicy object and the binderDied callback.\nThey both must be able to delete the cookie object and they both must be\naware of it already being deleted.\n\nIn all cases, the TranscodingResourcePolicy object that needs to be\nunregistered will outlive the cookie and the death recipient.\n\nCalling unlinkToDeath is unneccessary because the last strong ref to the\nbinder that was linked to death is removed in the unregisterSelf method\nwhich will unlink the binder and death recipient.\n\nTest: atest CtsMediaTranscodingTestCases MediaSampleReaderNDKTests\nTest: adb shell kill -9 `pid  media.resource_observer`\nTest: delete mResourcePolicy.get() to force destructor after linkToDeath\nBug: 319210610\n(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:0c674f5ff68daa64b90e1a234061ba9bebe6173c)\n(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:0213a7d62ea5545b954145cc53e2bd65ed5dc609)\n(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:686987b9613678fe6540079e85e0d94519551738)\nMerged-In: I8e6ba40fe3da30bf8753e7a16ad5c8cd5dfda40b\nChange-Id: I8e6ba40fe3da30bf8753e7a16ad5c8cd5dfda40b\n","web_links":[{"name":"GitHub","tooltip":"Open in GitWeb","url":"https://github.com/LineageOS/android_frameworks_av/commit/ab962689f840ba0a2d1a094c5bd98ddcd698b2c3"}],"resolve_conflicts_web_links":[{"name":"GitHub","tooltip":"Open in GitWeb","url":"https://github.com/LineageOS/android_frameworks_av/commit/ab962689f840ba0a2d1a094c5bd98ddcd698b2c3"}]},"branch":"refs/heads/lineage-20.0"},"f21c760322b58d7cb92579cb18ff75869ddd345c":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2024-08-12 11:56:19.000000000","uploader":{"_account_id":15173,"name":"Kevin Haggerty","email":"haggertk@lineageos.org","username":"haggertk","avatars":[{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d32","height":32},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d56","height":56},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d100","height":100},{"url":"https://www.gravatar.com/avatar/258edfac858c1ce5f056ed4ca050a578.jpg?d\u003didenticon\u0026r\u003dpg\u0026s\u003d120","height":120}]},"ref":"refs/changes/22/399422/2","fetch":{"anonymous http":{"url":"https://github.com/LineageOS/android_frameworks_av","ref":"refs/changes/22/399422/2","commands":{"Branch":"git fetch https://github.com/LineageOS/android_frameworks_av refs/changes/22/399422/2 \u0026\u0026 git checkout -b change-399422 FETCH_HEAD","Checkout":"git fetch https://github.com/LineageOS/android_frameworks_av refs/changes/22/399422/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://github.com/LineageOS/android_frameworks_av refs/changes/22/399422/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://github.com/LineageOS/android_frameworks_av refs/changes/22/399422/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://github.com/LineageOS/android_frameworks_av refs/changes/22/399422/2","Reset To":"git fetch https://github.com/LineageOS/android_frameworks_av refs/changes/22/399422/2 \u0026\u0026 git reset --hard FETCH_HEAD"}}},"commit":{"parents":[{"commit":"679fa675dbb171cba01668a0f5f18b95cd99d140","subject":"Merge cherrypicks of [\u0027googleplex-android-review.googlesource.com/26599509\u0027] into security-aosp-tm-release.","web_links":[{"name":"GitHub","tooltip":"Open in GitWeb","url":"https://github.com/LineageOS/android_frameworks_av/commit/679fa675dbb171cba01668a0f5f18b95cd99d140"}]}],"author":{"name":"Devin Moore","email":"devinmoore@google.com","date":"2024-02-23 19:19:38.000000000","tz":0},"committer":{"name":"Android Build Coastguard Worker","email":"android-build-coastguard-worker@google.com","date":"2024-06-06 00:18:08.000000000","tz":0},"subject":"libmediatranscoding: handle death recipient cookie ownership differently","message":"libmediatranscoding: handle death recipient cookie ownership differently\n\nThe ownership of the death recipient cookie is now limited to the\nTranscodingResourcePolicy object and the binderDied callback.\nThey both must be able to delete the cookie object and they both must be\naware of it already being deleted.\n\nIn all cases, the TranscodingResourcePolicy object that needs to be\nunregistered will outlive the cookie and the death recipient.\n\nCalling unlinkToDeath is unneccessary because the last strong ref to the\nbinder that was linked to death is removed in the unregisterSelf method\nwhich will unlink the binder and death recipient.\n\nTest: atest CtsMediaTranscodingTestCases MediaSampleReaderNDKTests\nTest: adb shell kill -9 `pid  media.resource_observer`\nTest: delete mResourcePolicy.get() to force destructor after linkToDeath\nBug: 319210610\n(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:0c674f5ff68daa64b90e1a234061ba9bebe6173c)\n(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:0213a7d62ea5545b954145cc53e2bd65ed5dc609)\n(cherry picked from https://googleplex-android-review.googlesource.com/q/commit:686987b9613678fe6540079e85e0d94519551738)\nMerged-In: I8e6ba40fe3da30bf8753e7a16ad5c8cd5dfda40b\nChange-Id: I8e6ba40fe3da30bf8753e7a16ad5c8cd5dfda40b\n","web_links":[{"name":"GitHub","tooltip":"Open in GitWeb","url":"https://github.com/LineageOS/android_frameworks_av/commit/f21c760322b58d7cb92579cb18ff75869ddd345c"}],"resolve_conflicts_web_links":[{"name":"GitHub","tooltip":"Open in GitWeb","url":"https://github.com/LineageOS/android_frameworks_av/commit/f21c760322b58d7cb92579cb18ff75869ddd345c"}]},"branch":"refs/heads/lineage-20.0"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
