103185 2012-11-24 11:44:28 -0800 [BlackBerry] RTSP should use the same MediaDocument as HTTP videos 2012-11-24 12:35:28 -0800 1 1 1 Unclassified WebKit WebKit BlackBerry 528+ (Nightly build) Other Other RESOLVED FIXED P3 Normal --- 1 lquinn lquinn eric.carlson feature-media-reviews mifenton rwlbuis tonikitoo webkit.review.bot oldest_to_newest 774869 0 lquinn 2012-11-24 11:44:28 -0800 Our RTSP handling currently generates its own HTML document instead of using the normal MediaDocument that is used for videos served by HTTP. We should use the same MediaDocument for both RTSP and HTTP. 774871 1 175863 lquinn 2012-11-24 11:51:41 -0800 Created attachment 175863 Patch 774876 2 175863 webkit.review.bot 2012-11-24 12:35:25 -0800 Comment on attachment 175863 Patch Clearing flags on attachment: 175863 Committed r135657: <http://trac.webkit.org/changeset/135657> 774877 3 webkit.review.bot 2012-11-24 12:35:28 -0800 All reviewed patches have been landed. Closing bug. 175863 2012-11-24 11:51:41 -0800 2012-11-24 12:35:25 -0800 Patch bug-103185-20121124144923.patch text/plain 2858 lquinn U3VidmVyc2lvbiBSZXZpc2lvbjogMTM1NjU0CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggMzA3MzZlMzk2NTQ0YjFm ZGM0MzRkMzY0YWM0Mzg3M2M4NGRiMzU4Zi4uNmMxNDNhYzI4ZjZiOWE0Y2JkNWYxMzlmMTJjNDZj OTk3ZTI3YjQ4ZCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE2IEBACisyMDEyLTExLTI0ICBMaWFt IFF1aW5uICA8bHF1aW5uQHJpbS5jb20+CisKKyAgICAgICAgW0JsYWNrQmVycnldIFJUU1Agc2hv dWxkIHVzZSB0aGUgc2FtZSBNZWRpYURvY3VtZW50IGFzIEhUVFAgdmlkZW9zLgorICAgICAgICBo dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTAzMTg1CisgICAgICAgIFJJ TSBQUiAyNTAxMTQKKworICAgICAgICBSZXZpZXdlZCBieSBHZW9yZ2UgU3RhaWtvcy4KKworICAg ICAgICBNYWtlIG91ciBNZWRpYVBsYXllciBpbXBsZW1lbnRhdGlvbiBpbmRpY2F0ZSB0aGF0IGl0 IHN1cHBvcnRzIFJUU1AuIFRoYXQgd2F5LCBET01JbXBsZW1lbnRhdGlvbjo6Y3JlYXRlRG9jdW1l bnQgd2lsbCBjcmVhdGUgdGhlIHNhbWUgTWVkaWFEb2N1bWVudCBhcyBpcyB1c2VkIGZvciBIVFRQ IHZpZGVvcy4KKworICAgICAgICAqIHBsYXRmb3JtL2dyYXBoaWNzL2JsYWNrYmVycnkvTWVkaWFQ bGF5ZXJQcml2YXRlQmxhY2tCZXJyeS5jcHA6CisgICAgICAgIChXZWJDb3JlOjpNZWRpYVBsYXll clByaXZhdGU6OnN1cHBvcnRzVHlwZSk6CisKIDIwMTItMTEtMjQgIEhheWF0byBJdG8gIDxoYXlh dG9AY2hyb21pdW0ub3JnPgogCiAgICAgICAgIEl0J3MgY29uZnVzaW5nIHRoYXQgcmV0dXJuIHZh bHVlcyBvZiAnYm9vbCBOb2RlOjpkaXNwYXRjaEV2ZW50KC4uLiknIGFuZCAnYm9vbCBOb2RlOjpk aXNwYXRjaE1vdXNlRXZlbnQoLi4pJyBoYXZlIHRoZSBvcHBvc2l0ZSBtZWFuaW5ncy4KZGlmZiAt LWdpdCBhL1NvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2JsYWNrYmVycnkvTWVkaWFQ bGF5ZXJQcml2YXRlQmxhY2tCZXJyeS5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFw aGljcy9ibGFja2JlcnJ5L01lZGlhUGxheWVyUHJpdmF0ZUJsYWNrQmVycnkuY3BwCmluZGV4IDc5 MTYyZjk1ZTg0ZTZmY2NiYWYxNjNiNzAyMmY4ZThlZDZhNjM1MzkuLjdlOWE3OWNiMDVkYWFkNWNi MjgzNGRjOWFhNWQ4N2E3YmZjYzI1Y2EgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL3BsYXRm b3JtL2dyYXBoaWNzL2JsYWNrYmVycnkvTWVkaWFQbGF5ZXJQcml2YXRlQmxhY2tCZXJyeS5jcHAK KysrIGIvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3MvYmxhY2tiZXJyeS9NZWRpYVBs YXllclByaXZhdGVCbGFja0JlcnJ5LmNwcApAQCAtNzYsMTkgKzc2LDIxIEBAIHZvaWQgTWVkaWFQ bGF5ZXJQcml2YXRlOjpnZXRTdXBwb3J0ZWRUeXBlcyhIYXNoU2V0PFdURjo6U3RyaW5nPiYgdHlw ZXMpCiAgICAgICAgIHR5cGVzLmFkZCgqaSk7CiB9CiAKLU1lZGlhUGxheWVyOjpTdXBwb3J0c1R5 cGUgTWVkaWFQbGF5ZXJQcml2YXRlOjpzdXBwb3J0c1R5cGUoY29uc3QgV1RGOjpTdHJpbmcmIHR5 cGUsIGNvbnN0IFdURjo6U3RyaW5nJiBjb2RlY3MsIGNvbnN0IEtVUkwmKQorTWVkaWFQbGF5ZXI6 OlN1cHBvcnRzVHlwZSBNZWRpYVBsYXllclByaXZhdGU6OnN1cHBvcnRzVHlwZShjb25zdCBXVEY6 OlN0cmluZyYgdHlwZSwgY29uc3QgV1RGOjpTdHJpbmcmIGNvZGVjcywgY29uc3QgS1VSTCYgdXJs KQogewotICAgIGlmICh0eXBlLmlzTnVsbCgpIHx8IHR5cGUuaXNFbXB0eSgpKSB7CisgICAgYm9v bCBpc1JUU1AgPSB1cmwucHJvdG9jb2xJcygicnRzcCIpOworCisgICAgaWYgKCFpc1JUU1AgJiYg KHR5cGUuaXNOdWxsKCkgfHwgdHlwZS5pc0VtcHR5KCkpKSB7CiAgICAgICAgIExPRyhNZWRpYSwg Ik1lZGlhUGxheWVyIGRvZXMgbm90IHN1cHBvcnQgdHlwZTsgdHlwZSBpcyBudWxsIG9yIGVtcHR5 LiIpOwogICAgICAgICByZXR1cm4gTWVkaWFQbGF5ZXI6OklzTm90U3VwcG9ydGVkOwogICAgIH0K IAogICAgIC8vIHNwZWMgc2F5cyB3ZSBzaG91bGQgbm90IHJldHVybiAicHJvYmFibHkiIGlmIHRo ZSBjb2RlY3Mgc3RyaW5nIGlzIGVtcHR5Ci0gICAgaWYgKFBsYXRmb3JtUGxheWVyOjptaW1lVHlw ZVN1cHBvcnRlZCh0eXBlLmFzY2lpKCkuZGF0YSgpKSkgewotICAgICAgICBMT0coTWVkaWEsICJN ZWRpYVBsYXllciBzdXBwb3J0cyB0eXBlOyBjYWNoZSBjb250YWlucyB0eXBlICclcycuIiwgdHlw ZS5hc2NpaSgpLmRhdGEoKSk7CisgICAgaWYgKGlzUlRTUCB8fCBQbGF0Zm9ybVBsYXllcjo6bWlt ZVR5cGVTdXBwb3J0ZWQodHlwZS5hc2NpaSgpLmRhdGEoKSkpIHsKKyAgICAgICAgTE9HKE1lZGlh LCAiTWVkaWFQbGF5ZXIgc3VwcG9ydHMgdHlwZSAlcy4iLCBpc1JUU1AgPyAicnRzcCIgOiB0eXBl LmFzY2lpKCkuZGF0YSgpKTsKICAgICAgICAgcmV0dXJuIGNvZGVjcy5pc0VtcHR5KCkgPyBNZWRp YVBsYXllcjo6TWF5QmVTdXBwb3J0ZWQgOiBNZWRpYVBsYXllcjo6SXNTdXBwb3J0ZWQ7CiAgICAg fQotICAgIExPRyhNZWRpYSwgIk1lZGlhUGxheWVyIGRvZXMgbm90IHN1cHBvcnQgdHlwZTsgY2Fj aGUgZG9lc24ndCBjb250YWluIHR5cGUgJyVzJy4iLCB0eXBlLmFzY2lpKCkuZGF0YSgpKTsKKyAg ICBMT0coTWVkaWEsICJNZWRpYVBsYXllciBkb2VzIG5vdCBzdXBwb3J0IHR5cGUgJXMuIiwgdHlw ZS5hc2NpaSgpLmRhdGEoKSk7CiAgICAgcmV0dXJuIE1lZGlhUGxheWVyOjpJc05vdFN1cHBvcnRl ZDsKIH0KIAo=