185420 2018-05-08 06:15:17 -0700 [EME][GStreamer] Dispatch the protection event from AppendPipeline to MediaPlayer 2018-05-10 18:26:40 -0700 1 1 1 Unclassified WebKit WebKitGTK WebKit Nightly Build Unspecified Unspecified RESOLVED INVALID P2 Normal --- 1 bandou.yacine webkit-unassigned bugs-noreply calvaris olivier.blin oldest_to_newest 1421700 0 bandou.yacine 2018-05-08 06:15:17 -0700 After the protection event support will be added in the bug 185245 the fucntion MediaPlayerPrivateGStreamerBase::handleProtectionEvent will be reached, so we should remove the ASSERT_NOT_REACHED. 1421706 1 339814 bandou.yacine 2018-05-08 07:06:49 -0700 Created attachment 339814 Patch 1422169 2 calvaris 2018-05-09 00:30:29 -0700 I think I explained myself poorly when I asked you to have this part of the code in a different bug. I meant that you should add the code to dispatch the event in a different bug other than the one for bug 185245. Would you have time to do it here? 1422204 3 bandou.yacine 2018-05-09 02:54:12 -0700 The bug is renamed from "Avoid a crashes after the protection event support is added in 185245" to "Dispatch the protection event from AppendPipeline to MediaPlayer" because I have removed the code which dispatches the Protection event from bug 185245 to this one. 1422978 4 bandou.yacine 2018-05-10 18:26:40 -0700 For more detail see 185245 339814 2018-05-08 07:06:49 -0700 2018-05-09 02:39:39 -0700 Patch bug-185420-20180508160651.patch text/plain 1784 bandou.yacine U3VidmVyc2lvbiBSZXZpc2lvbjogMjMxMDQ0CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggMDExN2JkMWI5YjQwYWFh YmU5ZTBiNGZlZjEyYjNhMTljNWY1MjAyYy4uY2QwMjUyNjliMmNkNzU0OTFhOTU3M2UzZWE1ODRi OWNhYThjMDdkZiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDE4LTA1LTA4ICBZYWNp bmUgQmFuZG91ICA8eWFjaW5lLmJhbmRvdV9leHRAc29mdGF0aG9tZS5jb20+CisKKyAgICAgICAg W0VNRV1bR1N0cmVhbWVyXSAgQXZvaWQgYSBjcmFzaGVzIGFmdGVyIHRoZSBwcm90ZWN0aW9uIGV2 ZW50IHN1cHBvcnQgaXMgYWRkZWQgaW4gMTg1MjQ1CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJr aXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xODU0MjAKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JP RFkgKE9PUFMhKS4KKworICAgICAgICBSZW1vdmUgdGhlIEFTU0VSVF9OT1RfUkVBQ0hFRCwgYmVj YXVzZSB3aXRoIHRoZSBwYXRjaCAxODUyNDUgdGhpcyBjb2RlIGlzIHJlYWNoZWQuCisgICAgICAg IFdpdGhvdXQgdGhpcyBwYXRjaCwgbW9zdCBlbmNyeXB0ZWQtbWVkaWEgV1BUIExheW91dFRlc3Rz IHdpbGwgY3Jhc2ggaW4gZGVidWcgY29uZi4KKworICAgICAgICAqIHBsYXRmb3JtL2dyYXBoaWNz L2dzdHJlYW1lci9NZWRpYVBsYXllclByaXZhdGVHU3RyZWFtZXJCYXNlLmNwcDoKKyAgICAgICAg KFdlYkNvcmU6Ok1lZGlhUGxheWVyUHJpdmF0ZUdTdHJlYW1lckJhc2U6OmhhbmRsZVByb3RlY3Rp b25FdmVudCk6CisKIDIwMTgtMDUtMDQgIFlhY2luZSBCYW5kb3UgIDx5YWNpbmUuYmFuZG91X2V4 dEBzb2Z0YXRob21lLmNvbT4KIAogICAgICAgICBbRU1FXVtHU3RyZWFtZXJdIEFkZCBhIGhhbmRs ZXIgZm9yIEdTdHJlYW1lciBwcm90ZWN0aW9uIGV2ZW50CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2Vi Q29yZS9wbGF0Zm9ybS9ncmFwaGljcy9nc3RyZWFtZXIvTWVkaWFQbGF5ZXJQcml2YXRlR1N0cmVh bWVyQmFzZS5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGljcy9nc3RyZWFtZXIv TWVkaWFQbGF5ZXJQcml2YXRlR1N0cmVhbWVyQmFzZS5jcHAKaW5kZXggMmU4NzY3NmNiY2QzMzc5 Y2M3YjU5ZmVmZmQwMjIyYWI5YWM0ZTgxZS4uNmY2MjVkNGFmYjg2YjE3OTQyOTM3YWRiNWMyOGZi MmFkMmM1ODg4OSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3Mv Z3N0cmVhbWVyL01lZGlhUGxheWVyUHJpdmF0ZUdTdHJlYW1lckJhc2UuY3BwCisrKyBiL1NvdXJj ZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2dzdHJlYW1lci9NZWRpYVBsYXllclByaXZhdGVH U3RyZWFtZXJCYXNlLmNwcApAQCAtMTI2OCw3ICsxMjY4LDYgQEAgdm9pZCBNZWRpYVBsYXllclBy aXZhdGVHU3RyZWFtZXJCYXNlOjpoYW5kbGVQcm90ZWN0aW9uRXZlbnQoR3N0RXZlbnQqIGV2ZW50 KQogICAgIGNvbnN0IGdjaGFyKiBldmVudEtleVN5c3RlbUlkID0gbnVsbHB0cjsKICAgICBnc3Rf ZXZlbnRfcGFyc2VfcHJvdGVjdGlvbihldmVudCwgJmV2ZW50S2V5U3lzdGVtSWQsIG51bGxwdHIs IG51bGxwdHIpOwogICAgIEdTVF9XQVJOSU5HKCJGSVhNRTogdW5oYW5kbGVkIHByb3RlY3Rpb24g ZXZlbnQgZm9yICVzIiwgZXZlbnRLZXlTeXN0ZW1JZCk7Ci0gICAgQVNTRVJUX05PVF9SRUFDSEVE KCk7CiB9CiAjZW5kaWYKIAo=