RESOLVED FIXED 87072
Assertion failure when mouse points to a direct child text node of shadow root 
https://bugs.webkit.org/show_bug.cgi?id=87072
Summary Assertion failure when mouse points to a direct child text node of shadow root
Hayato Ito
Reported Tuesday, May 22, 2012 3:30:28 AM UTC
Let me quote the test case from https://bugs.webkit.org/show_bug.cgi?id=86795#c3. <div id="container"></div> <script> var shadowRoot = new WebKitShadowRoot(document.getElementById('container')); shadowRoot.innerHTML = 'a'; </script> Moving the cursor over the 'a' results in the following assert: ASSERTION FAILED: !m_node->isShadowRoot() ../../third_party/WebKit/Source/WebCore/dom/ComposedShadowTreeWalker.h(81) : void WebCore::ComposedShadowTreeWalker::assertPrecondition() const
Attachments
remove an assertion (6.69 KB, patch)
2012-05-21 21:00 PDT, Hayato Ito 		no flags
DetailsFormatted DiffDiff
Patch for landing (6.69 KB, patch)
2012-05-22 21:50 PDT, Hayato Ito 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Hayato Ito
Comment 1 Tuesday, May 22, 2012 5:00:32 AM UTC
Created attachment 143186 [details] remove an assertion
Dimitri Glazkov (Google)
Comment 2 Tuesday, May 22, 2012 5:24:32 PM UTC
Comment on attachment 143186 [details] remove an assertion View in context: https://bugs.webkit.org/attachment.cgi?id=143186&action=review > Source/WebCore/dom/ComposedShadowTreeWalker.cpp:57 > + // assertPrecondition(); We usually don't land commented code like this. Perhaps your FIXME description could mention it.
Hayato Ito
Comment 3 Wednesday, May 23, 2012 2:57:43 AM UTC
Thank you for the review. (In reply to comment #2) > (From update of attachment 143186 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=143186&action=review > > > Source/WebCore/dom/ComposedShadowTreeWalker.cpp:57 > > + // assertPrecondition(); > > We usually don't land commented code like this. Perhaps your FIXME description could mention it. Okay. I'll mention that in FIXME. Let me land this after that, merging with ToT.
Hayato Ito
Comment 4 Wednesday, May 23, 2012 5:50:09 AM UTC
Created attachment 143458 [details] Patch for landing
WebKit Review Bot
Comment 5 Wednesday, May 23, 2012 6:39:37 AM UTC
Comment on attachment 143458 [details] Patch for landing Clearing flags on attachment: 143458 Committed r118125: <http://trac.webkit.org/changeset/118125>
WebKit Review Bot
Comment 6 Wednesday, May 23, 2012 6:39:44 AM UTC
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.