24874 2009-03-26 21:33:27 -0700 HTMLMediaElement: 'duration' defaults to NaN, deal with it 2009-03-27 05:59:44 -0700 1 1 1 Unclassified WebKit WebCore Misc. 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 0 eric.carlson webkit-unassigned oldest_to_newest 115489 0 eric.carlson 2009-03-26 21:33:27 -0700 As per the spec, the duration of a media file is NaN when no media data is available The HTMLMediaElement needs to special case that state when it does comparisons with "duration()". 115490 1 29004 eric.carlson 2009-03-26 21:45:42 -0700 Created attachment 29004 proposed patch 115526 2 eric.carlson 2009-03-27 05:59:44 -0700 Sending WebCore/ChangeLog Sending WebCore/html/HTMLMediaElement.cpp Transmitting file data .. Committed revision 42029. 29004 2009-03-26 21:45:42 -0700 2009-03-26 21:53:11 -0700 proposed patch patch.txt text/plain 1808 eric.carlson SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA0MjAyOCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTggQEAKKzIwMDktMDMtMjYgIEVyaWMgQ2FybHNvbiAgPGVyaWMuY2FybHNvbkBh cHBsZS5jb20+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAg ICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTI0ODc0CisgICAgICAg IDI0ODc0OiBIVE1MTWVkaWFFbGVtZW50OiAnZHVyYXRpb24nIGRlZmF1bHRzIHRvIE5hTiwgZGVh bCB3aXRoIGl0CisgICAgICAgIAorICAgICAgICBUaGUgJ2R1cmF0aW9uJyBhdHRyaWJ1dGUgaXMg TmFOIHdoZW4gbm8gbWVkaWEgaXMgYXZhaWxhYmxlLCBzbyB0aGUKKyAgICAgICAgbWVkaWEgZWxl bWVudCBzaG91bGQgdGFrZSBjYXJlIHdoZW4gd29ya2luZyB3aXRoIGR1cmF0aW9uKCkuCisKKyAg ICAgICAgKiBodG1sL0hUTUxNZWRpYUVsZW1lbnQuY3BwOgorICAgICAgICAoV2ViQ29yZTo6SFRN TE1lZGlhRWxlbWVudDo6bWVkaWFQbGF5ZXJUaW1lQ2hhbmdlZCk6IERvbid0IGNvbXBhcmUgd2l0 aCBkdXJhdGlvbiB3aGVuCisgICAgICAgIGl0IGlzIE5hTi4KKyAgICAgICAgKFdlYkNvcmU6OkhU TUxNZWRpYUVsZW1lbnQ6OmVuZGVkUGxheWJhY2spOiBEaXR0by4KKwogMjAwOS0wMy0yNiAgTWFy ayBSb3dlICA8bXJvd2VAYXBwbGUuY29tPgogCiAgICAgICAgIEZpeCB0aGUgYnVpbGQuCkluZGV4 OiBXZWJDb3JlL2h0bWwvSFRNTE1lZGlhRWxlbWVudC5jcHAKPT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29y ZS9odG1sL0hUTUxNZWRpYUVsZW1lbnQuY3BwCShyZXZpc2lvbiA0MjAyNSkKKysrIFdlYkNvcmUv aHRtbC9IVE1MTWVkaWFFbGVtZW50LmNwcAkod29ya2luZyBjb3B5KQpAQCAtMTEyNyw3ICsxMTI3 LDcgQEAgdm9pZCBIVE1MTWVkaWFFbGVtZW50OjptZWRpYVBsYXllclRpbWVDaAogICAgIAogICAg IGZsb2F0IG5vdyA9IGN1cnJlbnRUaW1lKCk7CiAgICAgZmxvYXQgZHVyID0gZHVyYXRpb24oKTsK LSAgICBpZiAobm93ID49IGR1cikgeworICAgIGlmICghaXNuYW4oZHVyKSAmJiBkdXIgJiYgbm93 ID49IGR1cikgewogICAgICAgICBpZiAobG9vcCgpKSB7CiAgICAgICAgICAgICBFeGNlcHRpb25D b2RlIGlnbm9yZWRFeGNlcHRpb247CiAgICAgICAgICAgICBtX3NlbnRFbmRFdmVudCA9IGZhbHNl OwpAQCAtMTIyNCw3ICsxMjI0LDExIEBAIGJvb2wgSFRNTE1lZGlhRWxlbWVudDo6cG90ZW50aWFs bHlQbGF5aW4KIAogYm9vbCBIVE1MTWVkaWFFbGVtZW50OjplbmRlZFBsYXliYWNrKCkgY29uc3QK IHsKLSAgICByZXR1cm4gbV9wbGF5ZXIgJiYgbV9yZWFkeVN0YXRlID49IEhBVkVfTUVUQURBVEEg JiYgY3VycmVudFRpbWUoKSA+PSBkdXJhdGlvbigpICYmICFsb29wKCk7CisgICAgaWYgKCFtX3Bs YXllciB8fCBtX3JlYWR5U3RhdGUgPCBIQVZFX01FVEFEQVRBKQorICAgICAgICByZXR1cm4gZmFs c2U7CisgICAgCisgICAgZmxvYXQgZHVyID0gZHVyYXRpb24oKTsKKyAgICByZXR1cm4gIWlzbmFu KGR1cikgJiYgY3VycmVudFRpbWUoKSA+PSBkdXIgJiYgIWxvb3AoKTsKIH0KIAogYm9vbCBIVE1M TWVkaWFFbGVtZW50OjpzdG9wcGVkRHVlVG9FcnJvcnMoKSBjb25zdAo=