24759 2009-03-23 09:02:42 -0700 Exceptions in removeStyleClass and enclosingNodeOrSelfWithNodeName 2009-03-24 10:09:29 -0700 1 1 1 Unclassified WebKit Web Inspector (Deprecated) 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 1 yurys yurys dglazkov oldest_to_newest 114778 0 yurys 2009-03-23 09:02:42 -0700 WebCore/inspector/front-end/SourceFrame.js adds method removeStyleClass to Element.prototype in content iframe(http://trac.webkit.org/browser/trunk/WebCore/inspector/front-end/SourceFrame.js#L227). Though the method calls removeMatchingStyleClasses which is not defined for Element. The same problem with Node.prototype.enclosingNodeOrSelfWithNodeName and Node.prototype.enclosingNodeOrSelfWithNodeNameInArray. The problem is repeatable with Chrome and for some reason is not repeatable with Safari. 114794 1 28857 yurys 2009-03-23 10:40:24 -0700 Created attachment 28857 a patch definging missing methods in the iframe 114803 2 28857 darin 2009-03-23 11:31:16 -0700 Comment on attachment 28857 a patch definging missing methods in the iframe r=me 114836 3 dglazkov 2009-03-23 13:22:00 -0700 I'll land. 114957 4 dglazkov 2009-03-24 10:09:29 -0700 Landed as http://trac.webkit.org/changeset/41940. 28857 2009-03-23 10:40:24 -0700 2009-03-23 11:31:16 -0700 a patch definging missing methods in the iframe SourceFramePatch.txt text/plain 1938 yurys SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n CShyZXZpc2lvbiA0MTkxMCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA IC0xLDMgKzEsMTggQEAKKzIwMDktMDMtMjMgIHl1cnlzICA8WXVyeSBTZW1pa2hhdHNreSA8eXVy eXNAY2hyb21pdW0ub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor CisgICAgICAgIEFkZCBtaXNzaW5nIG1ldGhvZHMKKyAgICAgICAgRWxlbWVudC5wcm90b3R5cGUu cmVtb3ZlTWF0Y2hpbmdTdHlsZUNsYXNzZXMgYW5kCisgICAgICAgIE5vZGUucHJvdG90eXBlLmVu Y2xvc2luZ05vZGVPclNlbGZXaXRoTm9kZU5hbWVJbkFycmF5CisgICAgICAgIHRvIFNvdXJjZUZy YW1lIGNvbnRlbnQKKyAgICAgICAgaWZyYW1lLiBUaGVzZSBtZXRob2RzIGFyZSBjYWxsZWQgYnkK KyAgICAgICAgRWxlbWVudC5wcm90b3R5cGUucmVtb3ZlU3R5bGVDbGFzcyBhbmQKKyAgICAgICAg Tm9kZS5wcm90b3R5cGUuZW5jbG9zaW5nTm9kZU9yU2VsZldpdGhOb2RlTmFtZS4KKworICAgICAg ICAqIGluc3BlY3Rvci9mcm9udC1lbmQvU291cmNlRnJhbWUuanM6CisgICAgICAgIChXZWJJbnNw ZWN0b3IuU291cmNlRnJhbWUucHJvdG90eXBlLl9sb2FkZWQpOgorCiAyMDA5LTAzLTIzICBFcmlj IENhcmxzb24gIDxlcmljLmNhcmxzb25AYXBwbGUuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5 IEFkYW0gUm9iZW4uCkluZGV4OiBXZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvU291cmNlRnJh bWUuanMKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09 PT09PT09PT09PT09PT09PQotLS0gV2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQtZW5kL1NvdXJjZUZy YW1lLmpzCShyZXZpc2lvbiAzOTIwOCkKKysrIFdlYkNvcmUvaW5zcGVjdG9yL2Zyb250LWVuZC9T b3VyY2VGcmFtZS5qcwkod29ya2luZyBjb3B5KQpAQCAtMjI1LDggKzIyNSwxMCBAQCBXZWJJbnNw ZWN0b3IuU291cmNlRnJhbWUucHJvdG90eXBlID0gewogCiAgICAgICAgIHRoaXMuZWxlbWVudC5j b250ZW50V2luZG93LkVsZW1lbnQucHJvdG90eXBlLmFkZFN0eWxlQ2xhc3MgPSBFbGVtZW50LnBy b3RvdHlwZS5hZGRTdHlsZUNsYXNzOwogICAgICAgICB0aGlzLmVsZW1lbnQuY29udGVudFdpbmRv dy5FbGVtZW50LnByb3RvdHlwZS5yZW1vdmVTdHlsZUNsYXNzID0gRWxlbWVudC5wcm90b3R5cGUu cmVtb3ZlU3R5bGVDbGFzczsKKyAgICAgICAgdGhpcy5lbGVtZW50LmNvbnRlbnRXaW5kb3cuRWxl bWVudC5wcm90b3R5cGUucmVtb3ZlTWF0Y2hpbmdTdHlsZUNsYXNzZXMgPSBFbGVtZW50LnByb3Rv dHlwZS5yZW1vdmVNYXRjaGluZ1N0eWxlQ2xhc3NlczsKICAgICAgICAgdGhpcy5lbGVtZW50LmNv bnRlbnRXaW5kb3cuRWxlbWVudC5wcm90b3R5cGUuaGFzU3R5bGVDbGFzcyA9IEVsZW1lbnQucHJv dG90eXBlLmhhc1N0eWxlQ2xhc3M7CiAgICAgICAgIHRoaXMuZWxlbWVudC5jb250ZW50V2luZG93 Lk5vZGUucHJvdG90eXBlLmVuY2xvc2luZ05vZGVPclNlbGZXaXRoTm9kZU5hbWUgPSBOb2RlLnBy b3RvdHlwZS5lbmNsb3NpbmdOb2RlT3JTZWxmV2l0aE5vZGVOYW1lOworICAgICAgICB0aGlzLmVs ZW1lbnQuY29udGVudFdpbmRvdy5Ob2RlLnByb3RvdHlwZS5lbmNsb3NpbmdOb2RlT3JTZWxmV2l0 aE5vZGVOYW1lSW5BcnJheSA9IE5vZGUucHJvdG90eXBlLmVuY2xvc2luZ05vZGVPclNlbGZXaXRo Tm9kZU5hbWVJbkFycmF5OwogCiAgICAgICAgIHRoaXMuX2FkZEV4aXN0aW5nTWVzc2FnZXNUb1Nv dXJjZSgpOwogICAgICAgICB0aGlzLl9hZGRFeGlzdGluZ0JyZWFrcG9pbnRzVG9Tb3VyY2UoKTsK