56376 2011-03-15 08:19:02 -0700 ASSERTION FAILED: !HashTranslator::equal(KeyTraits::emptyValue(), key) 2024-05-22 14:42:11 -0700 1 1 1 Unclassified WebKit Web Inspector (Deprecated) 528+ (Nightly build) PC Windows Vista RESOLVED FIXED P1 Normal --- 1 skylined webkit-unassigned abarth dev+webkit eric pfeldman webkit.review.bot oldest_to_newest 367664 0 85806 skylined 2011-03-15 08:19:02 -0700 Created attachment 85806 Repro Chromium: http://code.google.com/p/chromium/issues/detail?id=76190 Repro: Open the below file in Chromium, open inspector (CTRL+SHIFT+J), refresh page. <body onload="document.open();"></body> Stack: 0037d8bc 56be7f8f chrome_56720000!WTF::HashTable<WebCore::Node *,std::pair<WebCore::Node *,unsigned int>,WTF::PairFirstExtractor<std::pair<WebCore::Node *,unsigned int> >,WTF::PtrHash<WebCore::Node *>,WTF::PairHashTraits<WTF::HashTraits<WebCore::Node *>,WTF::HashTraits<unsigned int> >,WTF::HashTraits<WebCore::Node *> >::checkKey<WebCore::Node *,WTF::IdentityHashTranslator<WebCore::Node *,std::pair<WebCore::Node *,unsigned int>,WTF::PtrHash<WebCore::Node *> > >+0x6a [c:\src\chromium\internal\src\third_party\webkit\source\javascriptcore\wtf\hashtable.h @ 465] 0037d8e4 56be7baa chrome_56720000!WTF::HashTable<WebCore::Node *,std::pair<WebCore::Node *,unsigned int>,WTF::PairFirstExtractor<std::pair<WebCore::Node *,unsigned int> >,WTF::PtrHash<WebCore::Node *>,WTF::PairHashTraits<WTF::HashTraits<WebCore::Node *>,WTF::HashTraits<unsigned int> >,WTF::HashTraits<WebCore::Node *> >::lookup<WebCore::Node *,WTF::IdentityHashTranslator<WebCore::Node *,std::pair<WebCore::Node *,unsigned int>,WTF::PtrHash<WebCore::Node *> > >+0x2f [c:\src\chromium\internal\src\third_party\webkit\source\javascriptcore\wtf\hashtable.h @ 481] 0037d8f4 56be7703 chrome_56720000!WTF::HashTable<WebCore::Node *,std::pair<WebCore::Node *,unsigned int>,WTF::PairFirstExtractor<std::pair<WebCore::Node *,unsigned int> >,WTF::PtrHash<WebCore::Node *>,WTF::PairHashTraits<WTF::HashTraits<WebCore::Node *>,WTF::HashTraits<unsigned int> >,WTF::HashTraits<WebCore::Node *> >::lookup+0x1a [c:\src\chromium\internal\src\third_party\webkit\source\javascriptcore\wtf\hashtable.h @ 343] 0037d908 56be6c33 chrome_56720000!WTF::HashMap<WebCore::Node *,unsigned int,WTF::PtrHash<WebCore::Node *>,WTF::HashTraits<WebCore::Node *>,WTF::HashTraits<unsigned int> >::get+0x23 [c:\src\chromium\internal\src\third_party\webkit\source\javascriptcore\wtf\hashmap.h @ 277] 0037d920 56be670b chrome_56720000!WebCore::InspectorBrowserDebuggerAgent::hasBreakpoint+0x43 [c:\src\chromium\internal\src\third_party\webkit\source\webcore\inspector\inspectorbrowserdebuggeragent.cpp @ 331] 0037d97c 56971fbf chrome_56720000!WebCore::InspectorBrowserDebuggerAgent::willRemoveDOMNode+0x10b [c:\src\chromium\internal\src\third_party\webkit\source\webcore\inspector\inspectorbrowserdebuggeragent.cpp @ 275] 0037d98c 56ac46fd chrome_56720000!WebCore::InspectorInstrumentation::willRemoveDOMNodeImpl+0x2f [c:\src\chromium\internal\src\third_party\webkit\source\webcore\inspector\inspectorinstrumentation.cpp @ 127] 0037d9a0 56ac44b5 chrome_56720000!WebCore::InspectorInstrumentation::willRemoveDOMNode+0x2d [c:\src\chromium\internal\src\third_party\webkit\source\webcore\inspector\inspectorinstrumentation.h @ 346] 0037d9fc 56ac266e chrome_56720000!WebCore::dispatchChildRemovalEvents+0x55 [c:\src\chromium\internal\src\third_party\webkit\source\webcore\dom\containernode.cpp @ 1055] 0037da40 56ac236e chrome_56720000!WebCore::willRemoveChildren+0xae [c:\src\chromium\internal\src\third_party\webkit\source\webcore\dom\containernode.cpp @ 393] 0037dac8 568f9288 chrome_56720000!WebCore::ContainerNode::removeChildren+0x4e [c:\src\chromium\internal\src\third_party\webkit\source\webcore\dom\containernode.cpp @ 513] 0037dadc 568f907c chrome_56720000!WebCore::Document::implicitOpen+0x28 [c:\src\chromium\internal\src\third_party\webkit\source\webcore\dom\document.cpp @ 1977] 0037dafc 56caf3ae chrome_56720000!WebCore::Document::open+0x13c [c:\src\chromium\internal\src\third_party\webkit\source\webcore\dom\document.cpp @ 1940] 0037dba0 576316ba chrome_56720000!WebCore::V8HTMLDocument::openCallback+0x29e [c:\src\chromium\internal\src\third_party\webkit\source\webcore\bindings\v8\custom\v8htmldocumentcustom.cpp @ 166] 0037dca0 5762e550 chrome_56720000!v8::internal::HandleApiCallHelper<0>+0x32a [c:\src\chromium\internal\src\v8\src\builtins.cc @ 1069] 0037dcb0 5762e4c8 chrome_56720000!v8::internal::Builtin_Impl_HandleApiCall+0x10 [c:\src\chromium\internal\src\v8\src\builtins.cc @ 1086] 0037dcc0 245e02ae chrome_56720000!v8::internal::Builtin_HandleApiCall+0x18 [c:\src\chromium\internal\src\v8\src\builtins.cc @ 1085] 367727 1 eric 2011-03-15 10:01:18 -0700 Pfeldman, I choose you! 367728 2 85821 pfeldman 2011-03-15 10:01:50 -0700 Created attachment 85821 Patch 367733 3 eric 2011-03-15 10:04:17 -0700 Wow, that was about the fastest fix ever. :) (I hope pfeldman got the pokemon reference.) How do we test this? 367735 4 pfeldman 2011-03-15 10:05:35 -0700 (In reply to comment #3) > Wow, that was about the fastest fix ever. :) (I hope pfeldman got the pokemon reference.) > > How do we test this? I'm trying to repro this with an iframe, if I succeed we will have a test. 368340 5 85821 yurys 2011-03-16 08:47:55 -0700 Comment on attachment 85821 Patch View in context: https://bugs.webkit.org/attachment.cgi?id=85821&action=review > Source/WebCore/inspector/InspectorBrowserDebuggerAgent.cpp:270 > + Node* parentNode = InspectorDOMAgent::innerParentNode(node); Should it be changed to else if(Node* parentNode = InspectorDOMAgent::innerParentNode(node)) { ... ? 368389 6 pfeldman 2011-03-16 11:02:28 -0700 Committed r81259: <http://trac.webkit.org/changeset/81259> 368476 7 webkit.review.bot 2011-03-16 13:05:04 -0700 http://trac.webkit.org/changeset/81259 might have broken GTK Linux 64-bit Debug 2037149 8 frances_c 2024-05-22 14:42:11 -0700 *** Bug 16918 has been marked as a duplicate of this bug. *** 85806 2011-03-15 08:19:02 -0700 2011-03-15 08:19:02 -0700 Repro repro2.html text/html 39 skylined PGJvZHkgb25sb2FkPSJkb2N1bWVudC5vcGVuKCk7Ij48L2JvZHk+ 85821 2011-03-15 10:01:50 -0700 2011-03-16 08:47:54 -0700 Patch bug-56376-20110315200148.patch text/plain 3079 pfeldman U3VidmVyc2lvbiBSZXZpc2lvbjogODExMzIKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0No YW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCA0Yjk1ZDBmZWE4YmIwNjc3 YTJjMWZkNzYzZmNjMTUxYmMzNzJlYTc3Li5lM2IxM2YzOTE2YjRjOWIwZDA4ODdkOGU2NDVlNzVj ZDk2NWNhMTk1IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvU291 cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDUgKzEsMTcgQEAKIDIwMTEtMDMtMTUgIFBhdmVs IEZlbGRtYW4gIDxwZmVsZG1hbkBjaHJvbWl1bS5vcmc+CiAKKyAgICAgICAgUmV2aWV3ZWQgYnkg Tk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgV2ViIEluc3BlY3RvcjogQVNTRVJUSU9OIEZBSUxF RDogIUhhc2hUcmFuc2xhdG9yOjplcXVhbChLZXlUcmFpdHM6OmVtcHR5VmFsdWUoKSwga2V5KQor ICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTYzNzYKKwor ICAgICAgICAqIGluc3BlY3Rvci9JbnNwZWN0b3JCcm93c2VyRGVidWdnZXJBZ2VudC5jcHA6Cisg ICAgICAgIChXZWJDb3JlOjpJbnNwZWN0b3JCcm93c2VyRGVidWdnZXJBZ2VudDo6d2lsbFJlbW92 ZURPTU5vZGUpOgorICAgICAgICAqIGluc3BlY3Rvci9mcm9udC1lbmQvRE9NQWdlbnQuanM6Cisg ICAgICAgIChXZWJJbnNwZWN0b3IuRE9NQWdlbnQucHJvdG90eXBlLl9zZXREb2N1bWVudCk6CisK KzIwMTEtMDMtMTUgIFBhdmVsIEZlbGRtYW4gIDxwZmVsZG1hbkBjaHJvbWl1bS5vcmc+CisKICAg ICAgICAgUmV2aWV3ZWQgYnkgWXVyeSBTZW1pa2hhdHNreS4KIAogICAgICAgICBXZWIgSW5zcGVj dG9yOiBjcmFzaCB1cG9uICIvL2h0bWwvL0BpZCIgc2VhcmNoIGluIGVsZW1lbnRzIHBhbmVsLgpk aWZmIC0tZ2l0IGEvU291cmNlL1dlYkNvcmUvaW5zcGVjdG9yL0luc3BlY3RvckJyb3dzZXJEZWJ1 Z2dlckFnZW50LmNwcCBiL1NvdXJjZS9XZWJDb3JlL2luc3BlY3Rvci9JbnNwZWN0b3JCcm93c2Vy RGVidWdnZXJBZ2VudC5jcHAKaW5kZXggZTdiNWVlNzI0ZmJiMDEwZmZlZTA0MjcwZDVmZDk5YTYw MGNlZTk1Yi4uZWE3NjNmNGI1OThhOWM3ZmU1OGEzNzBmMzY3ZDc3MzliZDMxNDcwNSAxMDA2NDQK LS0tIGEvU291cmNlL1dlYkNvcmUvaW5zcGVjdG9yL0luc3BlY3RvckJyb3dzZXJEZWJ1Z2dlckFn ZW50LmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvSW5zcGVjdG9yQnJvd3NlckRl YnVnZ2VyQWdlbnQuY3BwCkBAIC0yNjcsMTIgKzI2NywxMyBAQCB2b2lkIEluc3BlY3RvckJyb3dz ZXJEZWJ1Z2dlckFnZW50Ojp3aWxsUmVtb3ZlRE9NTm9kZShOb2RlKiBub2RlKQogICAgIGlmICgh ZGVidWdnZXJBZ2VudCkKICAgICAgICAgcmV0dXJuOwogCisgICAgTm9kZSogcGFyZW50Tm9kZSA9 IEluc3BlY3RvckRPTUFnZW50Ojppbm5lclBhcmVudE5vZGUobm9kZSk7CiAgICAgaWYgKGhhc0Jy ZWFrcG9pbnQobm9kZSwgTm9kZVJlbW92ZWQpKSB7CiAgICAgICAgIFJlZlB0cjxJbnNwZWN0b3JP YmplY3Q+IGV2ZW50RGF0YSA9IEluc3BlY3Rvck9iamVjdDo6Y3JlYXRlKCk7CiAgICAgICAgIGRl c2NyaXB0aW9uRm9yRE9NRXZlbnQobm9kZSwgTm9kZVJlbW92ZWQsIGZhbHNlLCBldmVudERhdGEu Z2V0KCkpOwogICAgICAgICBldmVudERhdGEtPnNldFN0cmluZygiYnJlYWtwb2ludFR5cGUiLCBk b21OYXRpdmVCcmVha3BvaW50VHlwZSk7CiAgICAgICAgIGRlYnVnZ2VyQWdlbnQtPmJyZWFrUHJv Z3JhbShOYXRpdmVCcmVha3BvaW50RGVidWdnZXJFdmVudFR5cGUsIGV2ZW50RGF0YS5yZWxlYXNl KCkpOwotICAgIH0gZWxzZSBpZiAoaGFzQnJlYWtwb2ludChJbnNwZWN0b3JET01BZ2VudDo6aW5u ZXJQYXJlbnROb2RlKG5vZGUpLCBTdWJ0cmVlTW9kaWZpZWQpKSB7CisgICAgfSBlbHNlIGlmIChw YXJlbnROb2RlICYmIGhhc0JyZWFrcG9pbnQocGFyZW50Tm9kZSwgU3VidHJlZU1vZGlmaWVkKSkg ewogICAgICAgICBSZWZQdHI8SW5zcGVjdG9yT2JqZWN0PiBldmVudERhdGEgPSBJbnNwZWN0b3JP YmplY3Q6OmNyZWF0ZSgpOwogICAgICAgICBkZXNjcmlwdGlvbkZvckRPTUV2ZW50KG5vZGUsIFN1 YnRyZWVNb2RpZmllZCwgZmFsc2UsIGV2ZW50RGF0YS5nZXQoKSk7CiAgICAgICAgIGV2ZW50RGF0 YS0+c2V0U3RyaW5nKCJicmVha3BvaW50VHlwZSIsIGRvbU5hdGl2ZUJyZWFrcG9pbnRUeXBlKTsK ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL2luc3BlY3Rvci9mcm9udC1lbmQvRE9NQWdlbnQu anMgYi9Tb3VyY2UvV2ViQ29yZS9pbnNwZWN0b3IvZnJvbnQtZW5kL0RPTUFnZW50LmpzCmluZGV4 IGM2MGE5ZDk1ODI0NTYwYzJlY2U0MzUxNGUwZjlmYzhiZTRkNTNkNWYuLjIzMWI4ZjMwNTM4ZTU4 YmQxMDk0ZGE1ZDAxYjA4NzVlOGIxN2ZhODQgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJDb3JlL2lu c3BlY3Rvci9mcm9udC1lbmQvRE9NQWdlbnQuanMKKysrIGIvU291cmNlL1dlYkNvcmUvaW5zcGVj dG9yL2Zyb250LWVuZC9ET01BZ2VudC5qcwpAQCAtNDAyLDcgKzQwMiw4IEBAIFdlYkluc3BlY3Rv ci5ET01BZ2VudC5wcm90b3R5cGUgPSB7CiAgICAgICAgIGlmIChwYXlsb2FkICYmICJpZCIgaW4g cGF5bG9hZCkgewogICAgICAgICAgICAgdGhpcy5fZG9jdW1lbnQgPSBuZXcgV2ViSW5zcGVjdG9y LkRPTURvY3VtZW50KHRoaXMsIHBheWxvYWQpOwogICAgICAgICAgICAgdGhpcy5faWRUb0RPTU5v ZGVbcGF5bG9hZC5pZF0gPSB0aGlzLl9kb2N1bWVudDsKLSAgICAgICAgICAgIHRoaXMuX2JpbmRO b2Rlcyh0aGlzLl9kb2N1bWVudC5jaGlsZHJlbik7CisgICAgICAgICAgICBpZiAodGhpcy5fZG9j dW1lbnQuY2hpbGRyZW4pCisgICAgICAgICAgICAgICAgdGhpcy5fYmluZE5vZGVzKHRoaXMuX2Rv Y3VtZW50LmNoaWxkcmVuKTsKICAgICAgICAgfSBlbHNlCiAgICAgICAgICAgICB0aGlzLl9kb2N1 bWVudCA9IG51bGw7CiAgICAgICAgIHRoaXMuZGlzcGF0Y2hFdmVudFRvTGlzdGVuZXJzKFdlYklu c3BlY3Rvci5ET01BZ2VudC5FdmVudHMuRG9jdW1lbnRVcGRhdGVkLCB0aGlzLl9kb2N1bWVudCk7 Cg==