31259 2009-11-09 08:53:47 -0800 Web Inspector: Crash when inspecting WebCore\manual-tests\inspector\dom-mutation.html 2009-11-09 14:12:03 -0800 1 1 1 Unclassified WebKit Web Inspector (Deprecated) 528+ (Nightly build) All All RESOLVED FIXED P2 Normal --- 1 apavlov pfeldman bweinstein commit-queue joepeck keishi pfeldman pmuellr rik timothy oldest_to_newest 161594 0 apavlov 2009-11-09 08:53:47 -0800 Steps to reproduce: 1. Open WebCore\manual-tests\inspector\dom-mutation.html and open the Web Inspector's Elements panel 2. Expand the iframe body node (html/body/div[1]/iframe/html/body) in the elements tree. Watch the contents changing after the iframe content updates. 3. Click the "Test frame 2" button in the inspected page. The following crash occurs: > WebKit.dll!WTF::HashTable<WTF::RefPtr<WebCore::Node>,std::pair<WTF::RefPtr<WebCore::Node>,long>,WTF::PairFirstExtractor<std::pair<WTF::RefPtr<WebCore::Node>,long> >,WTF::PtrHash<WTF::RefPtr<WebCore::Node> >,WTF::PairHashTraits<WTF::HashTraits<WTF::RefPtr<WebCore::Node> >,WTF::HashTraits<long> >,WTF::HashTraits<WTF::RefPtr<WebCore::Node> > >::checkKey<WebCore::Node *,WTF::RefPtrHashMapRawKeyTranslator<WebCore::Node *,std::pair<WTF::RefPtr<WebCore::Node>,long>,WTF::PairHashTraits<WTF::HashTraits<WTF::RefPtr<WebCore::Node> >,WTF::HashTraits<long> >,WTF::PtrHash<WTF::RefPtr<WebCore::Node> > > >(WebCore::Node * const & key=0x00000000) Line 455 + 0x53 bytes C++ WebKit.dll!WTF::HashTable<WTF::RefPtr<WebCore::Node>,std::pair<WTF::RefPtr<WebCore::Node>,long>,WTF::PairFirstExtractor<std::pair<WTF::RefPtr<WebCore::Node>,long> >,WTF::PtrHash<WTF::RefPtr<WebCore::Node> >,WTF::PairHashTraits<WTF::HashTraits<WTF::RefPtr<WebCore::Node> >,WTF::HashTraits<long> >,WTF::HashTraits<WTF::RefPtr<WebCore::Node> > >::lookup<WebCore::Node *,WTF::RefPtrHashMapRawKeyTranslator<WebCore::Node *,std::pair<WTF::RefPtr<WebCore::Node>,long>,WTF::PairHashTraits<WTF::HashTraits<WTF::RefPtr<WebCore::Node> >,WTF::HashTraits<long> >,WTF::PtrHash<WTF::RefPtr<WebCore::Node> > > >(WebCore::Node * const & key=0x00000000) Line 471 C++ WebKit.dll!WTF::HashMap<WTF::RefPtr<WebCore::Node>,long,WTF::PtrHash<WTF::RefPtr<WebCore::Node> >,WTF::HashTraits<WTF::RefPtr<WebCore::Node> >,WTF::HashTraits<long> >::inlineGet(WebCore::Node * key=0x00000000) Line 270 + 0xc bytes C++ WebKit.dll!WTF::HashMap<WTF::RefPtr<WebCore::Node>,long,WTF::PtrHash<WTF::RefPtr<WebCore::Node> >,WTF::HashTraits<WTF::RefPtr<WebCore::Node> >,WTF::HashTraits<long> >::get(WebCore::Node * key=0x00000000) Line 281 C++ WebKit.dll!WebCore::InspectorDOMAgent::handleEvent(WebCore::ScriptExecutionContext * __formal=0x09c4303c, WebCore::Event * event=0x09abfcd8) Line 196 + 0x18 bytes C++ WebKit.dll!WebCore::EventTarget::fireEventListeners(WebCore::Event * event=0x09abfcd8) Line 297 + 0x35 bytes C++ WebKit.dll!WebCore::Node::handleLocalEvents(WebCore::Event * event=0x09abfcd8) Line 2384 C++ WebKit.dll!WebCore::Node::dispatchGenericEvent(WTF::PassRefPtr<WebCore::Event> prpEvent=NULL) Line 2515 + 0x1b bytes C++ WebKit.dll!WebCore::DOMWindow::dispatchLoadEvent() Line 1294 C++ WebKit.dll!WebCore::Document::dispatchWindowLoadEvent() Line 2904 C++ WebKit.dll!WebCore::Document::implicitClose() Line 1735 C++ WebKit.dll!WebCore::FrameLoader::checkCallImplicitClose() Line 1160 C++ WebKit.dll!WebCore::FrameLoader::checkCompleted() Line 1109 C++ WebKit.dll!WebCore::FrameLoader::finishedParsing() Line 1048 C++ WebKit.dll!WebCore::Document::finishedParsing() Line 4041 C++ WebKit.dll!WebCore::HTMLParser::finished() Line 1635 + 0x1a bytes C++ WebKit.dll!WebCore::HTMLTokenizer::end() Line 1864 C++ WebKit.dll!WebCore::HTMLTokenizer::finish() Line 1904 C++ WebKit.dll!WebCore::Document::finishParsing() Line 1879 + 0x1b bytes C++ WebKit.dll!WebCore::FrameLoader::endIfNotLoadingMainResource() Line 950 C++ WebKit.dll!WebCore::FrameLoader::end() Line 935 C++ WebKit.dll!WebCore::DocumentLoader::finishedLoading() Line 332 C++ WebKit.dll!WebCore::FrameLoader::finishedLoading() Line 2698 C++ WebKit.dll!WebCore::MainResourceLoader::didFinishLoading() Line 394 C++ WebKit.dll!WebCore::ResourceLoader::didFinishLoading(WebCore::ResourceHandle * __formal=0x08494d78) Line 403 + 0xf bytes C++ WebKit.dll!WebCore::didFinishLoading(_CFURLConnection * conn=0x0be86270, const void * clientInfo=0x08494d78) Line 232 + 0x1e bytes C++ CFNetwork.dll!71aaa0ad() 161762 1 42796 pfeldman 2009-11-09 13:38:40 -0800 Created attachment 42796 [PATCH] fix 161784 2 42796 commit-queue 2009-11-09 14:11:59 -0800 Comment on attachment 42796 [PATCH] fix Clearing flags on attachment: 42796 Committed r50691: <http://trac.webkit.org/changeset/50691> 161785 3 commit-queue 2009-11-09 14:12:03 -0800 All reviewed patches have been landed. Closing bug. 42796 2009-11-09 13:38:40 -0800 2009-11-09 14:11:59 -0800 [PATCH] fix dom_crash text/plain 1671 pfeldman ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg YjhlOTQ2YS4uYzM3MmU0OSAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi Q29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNSBAQAorMjAwOS0xMS0wOSAgUGF2ZWwgRmVsZG1h biAgPHBmZWxkbWFuQGNocm9taXVtLm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBUaW1vdGh5 IEhhdGNoZXIuCisKKyAgICAgICAgV2ViIEluc3BlY3RvcjogQ3Jhc2ggd2hlbiBpbnNwZWN0aW5n CisgICAgICAgIFdlYkNvcmVcbWFudWFsLXRlc3RzXGluc3BlY3Rvclxkb20tbXV0YXRpb24uaHRt bAorCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0zMTI1 OQorCisgICAgICAgICogaW5zcGVjdG9yL0luc3BlY3RvckRPTUFnZW50LmNwcDoKKyAgICAgICAg KFdlYkNvcmU6Okluc3BlY3RvckRPTUFnZW50OjpoYW5kbGVFdmVudCk6CisKIDIwMDktMTEtMDgg IFBhdmVsIEZlbGRtYW4gIDxwZmVsZG1hbkBjaHJvbWl1bS5vcmc+CiAKICAgICAgICAgUmV2aWV3 ZWQgYnkgVGltb3RoeSBIYXRjaGVyLgpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9pbnNwZWN0b3IvSW5z cGVjdG9yRE9NQWdlbnQuY3BwIGIvV2ViQ29yZS9pbnNwZWN0b3IvSW5zcGVjdG9yRE9NQWdlbnQu Y3BwCmluZGV4IGM3ZjdmNTMuLjVmODgwZmUgMTAwNjQ0Ci0tLSBhL1dlYkNvcmUvaW5zcGVjdG9y L0luc3BlY3RvckRPTUFnZW50LmNwcAorKysgYi9XZWJDb3JlL2luc3BlY3Rvci9JbnNwZWN0b3JE T01BZ2VudC5jcHAKQEAgLTE5Myw4ICsxOTMsOSBAQCB2b2lkIEluc3BlY3RvckRPTUFnZW50Ojpo YW5kbGVFdmVudChTY3JpcHRFeGVjdXRpb25Db250ZXh0KiwgRXZlbnQqIGV2ZW50KQogICAgICAg ICAgICAgLy8gUmUtYWRkIGZyYW1lIG93bmVyIGVsZW1lbnQgdG9nZXRoZXIgd2l0aCBpdHMgbmV3 IGNoaWxkcmVuLgogICAgICAgICAgICAgbG9uZyBwYXJlbnRJZCA9IG1fZG9jdW1lbnROb2RlVG9J ZE1hcC5nZXQoaW5uZXJQYXJlbnROb2RlKG5vZGUpKTsKICAgICAgICAgICAgIG1fZnJvbnRlbmQt PmNoaWxkTm9kZVJlbW92ZWQocGFyZW50SWQsIGZyYW1lT3duZXJJZCk7Ci0gICAgICAgICAgICBs b25nIHByZXZJZCA9IG1fZG9jdW1lbnROb2RlVG9JZE1hcC5nZXQoaW5uZXJQcmV2aW91c1NpYmxp bmcobm9kZSkpOwogICAgICAgICAgICAgU2NyaXB0T2JqZWN0IHZhbHVlID0gYnVpbGRPYmplY3RG b3JOb2RlKG5vZGUsIDAsICZtX2RvY3VtZW50Tm9kZVRvSWRNYXApOworICAgICAgICAgICAgTm9k ZSogcHJldmlvdXNTaWJsaW5nID0gaW5uZXJQcmV2aW91c1NpYmxpbmcobm9kZSk7CisgICAgICAg ICAgICBsb25nIHByZXZJZCA9IHByZXZpb3VzU2libGluZyA/IG1fZG9jdW1lbnROb2RlVG9JZE1h cC5nZXQocHJldmlvdXNTaWJsaW5nKSA6IDA7CiAgICAgICAgICAgICBtX2Zyb250ZW5kLT5jaGls ZE5vZGVJbnNlcnRlZChwYXJlbnRJZCwgcHJldklkLCB2YWx1ZSk7CiAgICAgICAgICAgICAvLyBJ bnZhbGlkYXRlIGNoaWxkcmVuIHJlcXVlc3RlZCBmbGFnIGZvciB0aGUgZWxlbWVudC4KICAgICAg ICAgICAgIG1fY2hpbGRyZW5SZXF1ZXN0ZWQucmVtb3ZlKG1fY2hpbGRyZW5SZXF1ZXN0ZWQuZmlu ZChmcmFtZU93bmVySWQpKTsK