RESOLVED FIXED 166743
Start hittesting a clean tree in RenderEmbeddedObject::isReplacementObscured 
https://bugs.webkit.org/show_bug.cgi?id=166743
Summary Start hittesting a clean tree in RenderEmbeddedObject::isReplacementObscured
zalan
Reported 2017-01-05 14:37:00 PST
rdar://problem/29024384 0 com.apple.WebCore 0x00007fffb792c3e6 WebCore::InlineFlowBox::nodeAtPoint(WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::HitTestLocation const&, WebCore::LayoutPoint const&, WebCore::LayoutUnit, WebCore::LayoutUnit, WebCore::HitTestAction) + 1046 1 com.apple.WebCore 0x00007fffb7f8bf3b WebCore::RootInlineBox::nodeAtPoint(WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::HitTestLocation const&, WebCore::LayoutPoint const&, WebCore::LayoutUnit, WebCore::LayoutUnit, WebCore::HitTestAction) + 331 2 com.apple.WebCore 0x00007fffb7305332 WebCore::RenderLineBoxList::hitTest(WebCore::RenderBoxModelObject*, WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::HitTestLocation const&, WebCore::LayoutPoint const&, WebCore::HitTestAction) const + 914 3 com.apple.WebCore 0x00007fffb7e5091a WebCore::RenderBlockFlow::hitTestInlineChildren(WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::HitTestLocation const&, WebCore::LayoutPoint const&, WebCore::HitTestAction) + 74 4 com.apple.WebCore 0x00007fffb7304f85 WebCore::RenderBlock::hitTestContents(WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::HitTestLocation const&, WebCore::LayoutPoint const&, WebCore::HitTestAction) + 197 5 com.apple.WebCore 0x00007fffb7304c19 WebCore::RenderBlock::nodeAtPoint(WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::HitTestLocation const&, WebCore::LayoutPoint const&, WebCore::HitTestAction) + 1817 6 com.apple.WebCore 0x00007fffb7304f5c WebCore::RenderBlock::hitTestContents(WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::HitTestLocation const&, WebCore::LayoutPoint const&, WebCore::HitTestAction) + 156 7 com.apple.WebCore 0x00007fffb7304c19 WebCore::RenderBlock::nodeAtPoint(WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::HitTestLocation const&, WebCore::LayoutPoint const&, WebCore::HitTestAction) + 1817 8 com.apple.WebCore 0x00007fffb7304f5c WebCore::RenderBlock::hitTestContents(WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::HitTestLocation const&, WebCore::LayoutPoint const&, WebCore::HitTestAction) + 156 9 com.apple.WebCore 0x00007fffb7304c19 WebCore::RenderBlock::nodeAtPoint(WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::HitTestLocation const&, WebCore::LayoutPoint const&, WebCore::HitTestAction) + 1817 10 com.apple.WebCore 0x00007fffb7304f5c WebCore::RenderBlock::hitTestContents(WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::HitTestLocation const&, WebCore::LayoutPoint const&, WebCore::HitTestAction) + 156 11 com.apple.WebCore 0x00007fffb7304c19 WebCore::RenderBlock::nodeAtPoint(WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::HitTestLocation const&, WebCore::LayoutPoint const&, WebCore::HitTestAction) + 1817 12 com.apple.WebCore 0x00007fffb730446d WebCore::RenderObject::hitTest(WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::HitTestLocation const&, WebCore::LayoutPoint const&, WebCore::HitTestFilter) + 61 13 com.apple.WebCore 0x00007fffb730437d WebCore::RenderLayer::hitTestContents(WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::LayoutRect const&, WebCore::HitTestLocation const&, WebCore::HitTestFilter) const + 125 14 com.apple.WebCore 0x00007fffb730323c WebCore::RenderLayer::hitTestLayer(WebCore::RenderLayer*, WebCore::RenderLayer*, WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::LayoutRect const&, WebCore::HitTestLocation const&, bool, WebCore::HitTestingTransformState const*, double*) + 2060 15 com.apple.WebCore 0x00007fffb7ebad04 WebCore::RenderLayer::hitTestList(WTF::Vector<WebCore::RenderLayer*, 0ul, WTF::CrashOnOverflow, 16ul>*, WebCore::RenderLayer*, WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::LayoutRect const&, WebCore::HitTestLocation const&, WebCore::HitTestingTransformState const*, double*, double*, WebCore::HitTestingTransformState const*, bool) + 308 16 com.apple.WebCore 0x00007fffb7302f69 WebCore::RenderLayer::hitTestLayer(WebCore::RenderLayer*, WebCore::RenderLayer*, WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::LayoutRect const&, WebCore::HitTestLocation const&, bool, WebCore::HitTestingTransformState const*, double*) + 1337 17 com.apple.WebCore 0x00007fffb73029a0 WebCore::RenderLayer::hitTest(WebCore::HitTestRequest const&, WebCore::HitTestLocation const&, WebCore::HitTestResult&) + 592 18 com.apple.WebCore 0x00007fffb739e40a WebCore::RenderView::hitTest(WebCore::HitTestRequest const&, WebCore::HitTestLocation const&, WebCore::HitTestResult&) + 122 19 com.apple.WebCore 0x00007fffb739e788 WebCore::RenderWidget::nodeAtPoint(WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::HitTestLocation const&, WebCore::LayoutPoint const&, WebCore::HitTestAction) + 648 20 com.apple.WebCore 0x00007fffb730446d WebCore::RenderObject::hitTest(WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::HitTestLocation const&, WebCore::LayoutPoint const&, WebCore::HitTestFilter) + 61 21 com.apple.WebCore 0x00007fffb730437d WebCore::RenderLayer::hitTestContents(WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::LayoutRect const&, WebCore::HitTestLocation const&, WebCore::HitTestFilter) const + 125 22 com.apple.WebCore 0x00007fffb730323c WebCore::RenderLayer::hitTestLayer(WebCore::RenderLayer*, WebCore::RenderLayer*, WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::LayoutRect const&, WebCore::HitTestLocation const&, bool, WebCore::HitTestingTransformState const*, double*) + 2060 23 com.apple.WebCore 0x00007fffb7ebad04 WebCore::RenderLayer::hitTestList(WTF::Vector<WebCore::RenderLayer*, 0ul, WTF::CrashOnOverflow, 16ul>*, WebCore::RenderLayer*, WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::LayoutRect const&, WebCore::HitTestLocation const&, WebCore::HitTestingTransformState const*, double*, double*, WebCore::HitTestingTransformState const*, bool) + 308 24 com.apple.WebCore 0x00007fffb7302f69 WebCore::RenderLayer::hitTestLayer(WebCore::RenderLayer*, WebCore::RenderLayer*, WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::LayoutRect const&, WebCore::HitTestLocation const&, bool, WebCore::HitTestingTransformState const*, double*) + 1337 25 com.apple.WebCore 0x00007fffb7ebad04 WebCore::RenderLayer::hitTestList(WTF::Vector<WebCore::RenderLayer*, 0ul, WTF::CrashOnOverflow, 16ul>*, WebCore::RenderLayer*, WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::LayoutRect const&, WebCore::HitTestLocation const&, WebCore::HitTestingTransformState const*, double*, double*, WebCore::HitTestingTransformState const*, bool) + 308 26 com.apple.WebCore 0x00007fffb7302f69 WebCore::RenderLayer::hitTestLayer(WebCore::RenderLayer*, WebCore::RenderLayer*, WebCore::HitTestRequest const&, WebCore::HitTestResult&, WebCore::LayoutRect const&, WebCore::HitTestLocation const&, bool, WebCore::HitTestingTransformState const*, double*) + 1337 27 com.apple.WebCore 0x00007fffb73029a0 WebCore::RenderLayer::hitTest(WebCore::HitTestRequest const&, WebCore::HitTestLocation const&, WebCore::HitTestResult&) + 592 28 com.apple.WebCore 0x00007fffb739e40a WebCore::RenderView::hitTest(WebCore::HitTestRequest const&, WebCore::HitTestLocation const&, WebCore::HitTestResult&) + 122 29 com.apple.WebCore 0x00007fffb7e9090f WebCore::RenderEmbeddedObject::isReplacementObscured() const + 735 30 com.apple.WebKit 0x00007fffb85a3b8c WebKit::WebPage::createPlugin(WebKit::WebFrame*, WebCore::HTMLPlugInElement*, WebKit::Plugin::Parameters const&, WTF::String&) + 758 31 com.apple.WebKit 0x00007fffb8724832 WebKit::WebFrameLoaderClient::createPlugin(WebCore::IntSize const&, WebCore::HTMLPlugInElement*, WebCore::URL const&, WTF::Vector<WTF::String, 0ul, WTF::CrashOnOverflow, 16ul> const&, WTF::Vector<WTF::String, 0ul, WTF::CrashOnOverflow, 16ul> const&, WTF::String const&, bool) + 294 32 com.apple.WebCore 0x00007fffb80b73a6 WebCore::SubframeLoader::loadPlugin(WebCore::HTMLPlugInImageElement&, WebCore::URL const&, WTF::String const&, WTF::Vector<WTF::String, 0ul, WTF::CrashOnOverflow, 16ul> const&, WTF::Vector<WTF::String, 0ul, WTF::CrashOnOverflow, 16ul> const&, bool) + 454 33 com.apple.WebCore 0x00007fffb80b7745 WebCore::SubframeLoader::requestObject(WebCore::HTMLPlugInImageElement&, WTF::String const&, WTF::AtomicString const&, WTF::String const&, WTF::Vector<WTF::String, 0ul, WTF::CrashOnOverflow, 16ul> const&, WTF::Vector<WTF::String, 0ul, WTF::CrashOnOverflow, 16ul> const&) + 693 34 com.apple.WebCore 0x00007fffb789b12f WebCore::HTMLPlugInImageElement::requestObject(WTF::String const&, WTF::String const&, WTF::Vector<WTF::String, 0ul, WTF::CrashOnOverflow, 16ul> const&, WTF::Vector<WTF::String, 0ul, WTF::CrashOnOverflow, 16ul> const&) + 239 35 com.apple.WebCore 0x00007fffb73b9b02 WebCore::HTMLEmbedElement::updateWidget(WebCore::PluginCreationOption) + 226 36 com.apple.WebCore 0x00007fffb77f7968 WebCore::FrameView::updateEmbeddedObject(WebCore::RenderEmbeddedObject&) + 200 37 com.apple.WebCore 0x00007fffb77f7a18 WebCore::FrameView::updateEmbeddedObjects() + 104 38 com.apple.WebCore 0x00007fffb77f7b68 WebCore::FrameView::flushAnyPendingPostLayoutTasks() + 88 39 com.apple.WebCore 0x00007fffb7683059 WebCore::Document::updateLayoutIgnorePendingStylesheets(WebCore::Document::RunPostLayoutTasks) + 313 40 com.apple.WebCore 0x00007fffb789474b WebCore::HTMLObjectElement::renderWidgetLoadingPlugin
Attachments
Patch (1.88 KB, patch)
2017-01-05 14:38 PST, zalan 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
zalan
Comment 1 2017-01-05 14:38:46 PST
Created attachment 298136 [details] Patch
WebKit Commit Bot
Comment 2 2017-01-05 15:38:14 PST
Comment on attachment 298136 [details] Patch Clearing flags on attachment: 298136 Committed r210378: <http://trac.webkit.org/changeset/210378>
WebKit Commit Bot
Comment 3 2017-01-05 15:38:19 PST
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.