150048 2015-10-12 14:08:59 -0700 [iOS] Avoid crash due to invalid screen bounds 2015-10-13 09:54:48 -0700 1 1 1 Unclassified WebKit Media WebKit Nightly Build iPhone / iPad All RESOLVED FIXED InRadar P2 Normal --- 1 bfulgham bfulgham bfulgham jer.noble thorton webkit-bug-importer oldest_to_newest 1132480 0 bfulgham 2015-10-12 14:08:59 -0700 Certain sites or media players modify layout settings such that a divide-by-zero error is encountered while computing the bounds that the video element should use for playback. When this happens, iOS throws an exception and kills the process. Instead, we should recognize when this has happened and use a reasonable set of bounds to display the video. 1132481 1 bfulgham 2015-10-12 14:09:25 -0700 <rdar://problem/22112664> 1132482 2 262921 bfulgham 2015-10-12 14:11:37 -0700 Created attachment 262921 Patch 1132783 3 bfulgham 2015-10-13 09:54:48 -0700 Committed r190992: <http://trac.webkit.org/changeset/190992> 262921 2015-10-12 14:11:37 -0700 2015-10-12 20:57:00 -0700 Patch bug-150048-20151012141102.patch text/plain 1703 bfulgham SW5kZXg6IFNvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi S2l0Mi9DaGFuZ2VMb2cJKHJldmlzaW9uIDE5MDg4NikKKysrIFNvdXJjZS9XZWJLaXQyL0NoYW5n ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE3IEBACisyMDE1LTEwLTEyICBCcmVudCBG dWxnaGFtICA8YmZ1bGdoYW1AYXBwbGUuY29tPgorCisgICAgICAgIFtpT1NdIEF2b2lkIGNyYXNo IGR1ZSB0byBpbnZhbGlkIHNjcmVlbiBib3VuZHMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtp dC5vcmcvc2hvd19idWcuY2dpP2lkPTE1MDA0OAorICAgICAgICA8cmRhcjovL3Byb2JsZW0vMjIx MTI2NjQ+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAg Q2hlY2sgZm9yIGFuIGludmFsaWQgYm91bmRzLCBhbmQgcmVzZXQgaXQgdG8gYSBrbm93biBzdGF0 ZSBiZWZvcmUKKyAgICAgICAgYXR0ZW1wdGluZyB0byB1c2UgaXQuCisKKyAgICAgICAgKiBXZWJQ cm9jZXNzL2lvcy9XZWJWaWRlb0Z1bGxzY3JlZW5NYW5hZ2VyLm1tOgorICAgICAgICAoV2ViS2l0 OjpXZWJWaWRlb0Z1bGxzY3JlZW5NYW5hZ2VyOjpzZXRWaWRlb0xheWVyRnJhbWVGZW5jZWQpOgor CiAyMDE1LTEwLTA5ICBBbmRlcnMgQ2FybHNzb24gIDxhbmRlcnNjYUBhcHBsZS5jb20+CiAKICAg ICAgICAgRG9uJ3QgYWxsb3cgcGx1Zy1pbnMgdG8gb3ZlcnJpZGUgaW1hZ2UgdHlwZXMgZm9yIDxl bWJlZD4gZWxlbWVudHMKSW5kZXg6IFNvdXJjZS9XZWJLaXQyL1dlYlByb2Nlc3MvaW9zL1dlYlZp ZGVvRnVsbHNjcmVlbk1hbmFnZXIubW0KPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291cmNlL1dlYktpdDIvV2Vi UHJvY2Vzcy9pb3MvV2ViVmlkZW9GdWxsc2NyZWVuTWFuYWdlci5tbQkocmV2aXNpb24gMTkwODU5 KQorKysgU291cmNlL1dlYktpdDIvV2ViUHJvY2Vzcy9pb3MvV2ViVmlkZW9GdWxsc2NyZWVuTWFu YWdlci5tbQkod29ya2luZyBjb3B5KQpAQCAtNTQwLDYgKzU0MCwxMSBAQCB2b2lkIFdlYlZpZGVv RnVsbHNjcmVlbk1hbmFnZXI6OnNldFZpZGVvCiAgICAgUmVmUHRyPFdlYlZpZGVvRnVsbHNjcmVl bkludGVyZmFjZUNvbnRleHQ+IGludGVyZmFjZTsKICAgICBzdGQ6OnRpZShtb2RlbCwgaW50ZXJm YWNlKSA9IGVuc3VyZU1vZGVsQW5kSW50ZXJmYWNlKGNvbnRleHRJZCk7CiAKKyAgICBpZiAoc3Rk Ojppc25hbihib3VuZHMueCgpKSB8fCBzdGQ6OmlzbmFuKGJvdW5kcy55KCkpIHx8IHN0ZDo6aXNu YW4oYm91bmRzLndpZHRoKCkpIHx8IHN0ZDo6aXNuYW4oYm91bmRzLmhlaWdodCgpKSkgeworICAg ICAgICBGbG9hdFJlY3QgY2xpZW50UmVjdCA9IGNsaWVudFJlY3RGb3JFbGVtZW50KG1vZGVsLT52 aWRlb0VsZW1lbnQoKSk7CisgICAgICAgIGJvdW5kcyA9IEZsb2F0UmVjdCgwLCAwLCBjbGllbnRS ZWN0LndpZHRoKCksIGNsaWVudFJlY3QuaGVpZ2h0KCkpOworICAgIH0KKyAgICAKICAgICBbQ0FU cmFuc2FjdGlvbiBiZWdpbl07CiAgICAgW0NBVHJhbnNhY3Rpb24gc2V0QW5pbWF0aW9uRHVyYXRp b246MF07CiAgICAgaWYgKGludGVyZmFjZS0+bGF5ZXJIb3N0aW5nQ29udGV4dCgpKQo=