Bug 56376 - ASSERTION FAILED: !HashTranslator::equal(KeyTraits::emptyValue(), key)
Summary: ASSERTION FAILED: !HashTranslator::equal(KeyTraits::emptyValue(), key)
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (Deprecated) (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC Windows Vista
: P1 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-03-15 08:19 PDT by Berend-Jan Wever
Modified: 2011-03-16 13:05 PDT (History)
4 users (show)

See Also:


Attachments
Repro (39 bytes, text/html)
2011-03-15 08:19 PDT, Berend-Jan Wever
no flags Details
Patch (3.01 KB, patch)
2011-03-15 10:01 PDT, Pavel Feldman
yurys: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Berend-Jan Wever 2011-03-15 08:19:02 PDT
Created attachment 85806 [details]
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]
Comment 1 Eric Seidel (no email) 2011-03-15 10:01:18 PDT
Pfeldman, I choose you!
Comment 2 Pavel Feldman 2011-03-15 10:01:50 PDT
Created attachment 85821 [details]
Patch
Comment 3 Eric Seidel (no email) 2011-03-15 10:04:17 PDT
Wow, that was about the fastest fix ever. :)  (I hope pfeldman got the pokemon reference.)

How do we test this?
Comment 4 Pavel Feldman 2011-03-15 10:05:35 PDT
(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.
Comment 5 Yury Semikhatsky 2011-03-16 08:47:55 PDT
Comment on attachment 85821 [details]
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)) { ... ?
Comment 6 Pavel Feldman 2011-03-16 11:02:28 PDT
Committed r81259: <http://trac.webkit.org/changeset/81259>
Comment 7 WebKit Review Bot 2011-03-16 13:05:04 PDT
http://trac.webkit.org/changeset/81259 might have broken GTK Linux 64-bit Debug