Version: (using KDE 4.3.1) OS: Linux Installed from: Fedora RPMs Akregator sometimes loads nspluginviewer when I read articles. This happens with both the plain text view and the the html view, even though it doesn't make any sense for the basic view and I think plugins and javascript are disabled in the html view. Note that I'm talking about the html "full website" article view, not about loading a page in a tab. I've found that I can reliably reproduce this with Bruce Sterling's feed, which often has embedded youtube videos: http://www.wired.com/beyond_the_beyond/feed/
Forgot to mention that the reason this is noticeable/annoying is bug #182869 - nspluginviewer takes a noticable amount of cpu time to do nothing.
I've been able to fix this by following https://bugs.kde.org/show_bug.cgi?id=182869 comment #17 and #18
So I set a breakpoint to see if I could find what calls nspluginviewer and why. Again on Bruce Sterling's feed: Breakpoint 2, PluginFactory::createPartObject (this=0xb2b1520, parentWidget= 0xa007f18, parent=0x9fdcee8, classname=0x3ee492e "Browser/View", args= QList<QString> = {...}) at /usr/src/debug/kdebase-4.4.2/apps/nsplugins/plugin_part.cpp:138 138 kDebug(1432) << "PluginFactory::create"; (gdb) bt #0 PluginFactory::createPartObject (this=0xb2b1520, parentWidget= 0xa007f18, parent=0x9fdcee8, classname=0x3ee492e "Browser/View", args= QList<QString> = {...}) at /usr/src/debug/kdebase-4.4.2/apps/nsplugins/plugin_part.cpp:138 #1 0x03e9d450 in KPluginFactory::create (this=0xb2b1520, iface= 0x3b361e0 "KParts::ReadOnlyPart", parentWidget=0xa007f18, parent= 0x9fdcee8, args=QList<QVariant> = {...}, keyword="") at /usr/src/debug/kdelibs-4.4.2/kdecore/util/kpluginfactory.cpp:171 #2 0x064e16cc in create<KParts::ReadOnlyPart> (this=0x9fdcee8, parentWidget=0xa007f18, parent=0x9fdcee8, mimetype= "application/x-shockwave-flash", serviceName="", serviceTypes= empty QList<QString>, params=QList<QString> = {...}) at /usr/src/debug/kdelibs-4.4.2/kdecore/util/kpluginfactory.h:515 #3 KHTMLPart::createPart (this=0x9fdcee8, parentWidget=0xa007f18, parent= 0x9fdcee8, mimetype="application/x-shockwave-flash", serviceName="", serviceTypes=empty QList<QString>, params=QList<QString> = {...}) at /usr/src/debug/kdelibs-4.4.2/khtml/khtml_part.cpp:4558 #4 0x064ca24d in KHTMLPart::processObjectRequest (this=0x9fdcee8, child= 0xaa316d0, _url=..., mimetype="application/x-shockwave-flash") ---Type <return> to continue, or q <return> to quit--- at /usr/src/debug/kdelibs-4.4.2/khtml/khtml_part.cpp:4350 #5 0x064d13ee in KHTMLPart::requestObject (this=0x9fdcee8, child= 0xaa316d0, url=..., _args=..., browserArgs=...) at /usr/src/debug/kdelibs-4.4.2/khtml/khtml_part.cpp:4258 #6 0x064d226a in KHTMLPart::requestObject (this=0x9fdcee8, frame= 0xb413ce0, url= "http://vimeo.com/moogaloop.swf?clip_id=10481179&server=vimeo.com&show_title=1&show_byline=1&show_portrait=0&color=&fullscreen=1", serviceType= "application/x-shockwave-flash", params=QList<QString> = {...}) at /usr/src/debug/kdelibs-4.4.2/khtml/khtml_part.cpp:4186 #7 0x065ac0ae in DOM::HTMLObjectBaseElementImpl::computeContent (this= 0xb413ce0) at /usr/src/debug/kdelibs-4.4.2/khtml/html/html_objectimpl.cpp:519 #8 0x065a6eb4 in DOM::HTMLPartContainerElementImpl::computeContentIfNeeded (this=0xb413ce0) at /usr/src/debug/kdelibs-4.4.2/khtml/html/html_objectimpl.cpp:90 #9 0x06569193 in khtml::KHTMLParser::popOneBlock (this=0xb1f3af8, delBlock= true) at /usr/src/debug/kdelibs-4.4.2/khtml/html/htmlparser.cpp:1863 #10 0x0656a26e in khtml::KHTMLParser::popBlock (this=0xb1f3af8, _id=72) ---Type <return> to continue, or q <return> to quit--- at /usr/src/debug/kdelibs-4.4.2/khtml/html/htmlparser.cpp:1804 #11 0x0656d224 in khtml::KHTMLParser::parseToken (this=0xb1f3af8, t= 0xb07d93c) at /usr/src/debug/kdelibs-4.4.2/khtml/html/htmlparser.cpp:254 #12 0x0656daf4 in khtml::HTMLTokenizer::processToken (this=0xb07d928) at /usr/src/debug/kdelibs-4.4.2/khtml/html/htmltokenizer.cpp:2056 #13 0x06574f3c in khtml::HTMLTokenizer::parseTag (this=0xb07d928, src=...) at /usr/src/debug/kdelibs-4.4.2/khtml/html/htmltokenizer.cpp:1529 #14 0x06577883 in khtml::HTMLTokenizer::write (this=0xb07d928, str=..., appendData=true) at /usr/src/debug/kdelibs-4.4.2/khtml/html/htmltokenizer.cpp:1810 #15 0x064bc12d in KHTMLPart::write (this=0x9fdcee8, str= "<div class=\"headerbox\" dir=\"ltr\">\n<div class=\"headertitle\" dir=\"ltr\">\n<a href=\"http://www.wired.com/beyond_the_beyond/2010/04/massimo-banzi-and-his-students/\">Massimo Banzi and his students</a></div>\n"...) at /usr/src/debug/kdelibs-4.4.2/khtml/khtml_part.cpp:2124 #16 0x0814b102 in Akregator::ArticleViewer::renderContent (this=0x9fe2c20, text= "<div class=\"headerbox\" dir=\"ltr\">\n<div class=\"headertitle\" dir=\"ltr\">\n<a href=\"http://www.wired.com/beyond_the_beyond/2010/04/massimo-banz---Type <return> to continue, or q <return> to quit--- i-and-his-students/\">Massimo Banzi and his students</a></div>\n"...) at /usr/src/debug/kdepim-4.4.2/akregator/src/articleviewer.cpp:411 #17 0x0814c23f in Akregator::ArticleViewer::showArticle (this=0x9fe2c20, article=...) at /usr/src/debug/kdepim-4.4.2/akregator/src/articleviewer.cpp:489 #18 0x081725d5 in Akregator::MainWidget::slotArticleSelected (this= 0x9f5bf60, article=...) at /usr/src/debug/kdepim-4.4.2/akregator/src/mainwidget.cpp:804 #19 0x08176f28 in Akregator::MainWidget::qt_metacall (this=0x9f5bf60, _c= InvokeMetaMethod, _id=<value optimized out>, _a=0xbffc2ad8) at /usr/src/debug/kdepim-4.4.2/i686-redhat-linux-gnu/akregator/src/mainwidget.moc:160 #20 0x008185db in QMetaObject::metacall (object=0x9f5bf60, cl= InvokeMetaMethod, idx=32, argv=0xbffc2ad8) at kernel/qmetaobject.cpp:237 #21 0x008274af in QMetaObject::activate (sender=0x9fe1c98, m=0x81c11b4, local_signal_index=1, argv=0xbffc2ad8) at kernel/qobject.cpp:3293 #22 0x081015c4 in Akregator::AbstractSelectionController::currentArticleChanged (this=0x9fe1c98, _t1=...) at /usr/src/debug/kdepim-4.4.2/i686-redhat-linux-gnu/akregator/src/abstra---Type <return> to continue, or q <return> to quit--- ctselectioncontroller.moc:107 #23 0x081290cf in Akregator::SelectionController::articleSelectionChanged ( this=0x9fe1c98) at /usr/src/debug/kdepim-4.4.2/akregator/src/selectioncontroller.cpp:286 #24 0x0812a663 in Akregator::SelectionController::qt_metacall (this= 0x9fe1c98, _c=InvokeMetaMethod, _id=<value optimized out>, _a= 0xbffc2c44) at /usr/src/debug/kdepim-4.4.2/i686-redhat-linux-gnu/akregator/src/selectioncontroller.moc:89 #25 0x008185db in QMetaObject::metacall (object=0x9fe1c98, cl= InvokeMetaMethod, idx=12, argv=0xbffc2c44) at kernel/qmetaobject.cpp:237 #26 0x008274af in QMetaObject::activate (sender=0xb6a9248, m=0x3794f10, local_signal_index=0, argv=0xbffc2c44) at kernel/qobject.cpp:3293 #27 0x0344a83a in QItemSelectionModel::selectionChanged (this=0xb6a9248, _t1=..., _t2=...) at .moc/release-shared/moc_qitemselectionmodel.cpp:152 #28 0x0344e898 in QItemSelectionModel::emitSelectionChanged (this= 0xb6a9248, newSelection=..., oldSelection=...) at itemviews/qitemselectionmodel.cpp:1536 #29 0x03450024 in QItemSelectionModel::select (this=0xb6a9248, selection= ---Type <return> to continue, or q <return> to quit--- ..., command=...) at itemviews/qitemselectionmodel.cpp:1099 #30 0x034372a4 in QTreeViewPrivate::select (this=0x9fdbcd0, topIndex=..., bottomIndex=..., command=...) at itemviews/qtreeview.cpp:3682 #31 0x03438025 in QTreeView::setSelection (this=0x9fdc1a0, rect=..., command=...) at itemviews/qtreeview.cpp:2246 #32 0x033f5513 in QAbstractItemView::mousePressEvent (this=0x9fdc1a0, event= 0xbffc3b40) at itemviews/qabstractitemview.cpp:1656 #33 0x03440a27 in QTreeView::mousePressEvent (this=0x9fdc1a0, event= 0xbffc3b40) at itemviews/qtreeview.cpp:1810 #34 0x08133c20 in Akregator::ArticleListView::mousePressEvent (this= 0x9fdc1a0, ev=0xbffc3b40) at /usr/src/debug/kdepim-4.4.2/akregator/src/articlelistview.cpp:336 #35 0x02ef6b11 in QWidget::event (this=0x9fdc1a0, event=0xbffc3b40) at kernel/qwidget.cpp:7994 #36 0x032c5064 in QFrame::event (this=0x9fdc1a0, e=0xbffc3b40) at widgets/qframe.cpp:557 #37 0x033531c3 in QAbstractScrollArea::viewportEvent (this=0x9fdc1a0, e= 0xbffc3b40) at widgets/qabstractscrollarea.cpp:1036 #38 0x033f9fd7 in QAbstractItemView::viewportEvent (this=0x9fdc1a0, event= ---Type <return> to continue, or q <return> to quit--- 0xbffc3b40) at itemviews/qabstractitemview.cpp:1610 #39 0x0343a491 in QTreeView::viewportEvent (this=0x9fdc1a0, event= 0xbffc3b40) at itemviews/qtreeview.cpp:1248 #40 0x03355665 in viewportEvent (this=0x9fdd448, o=0x9dc8400, e=0xbffc3b40) at widgets/qabstractscrollarea_p.h:100 #41 QAbstractScrollAreaFilter::eventFilter (this=0x9fdd448, o=0x9dc8400, e= 0xbffc3b40) at widgets/qabstractscrollarea_p.h:116 #42 0x00812e3a in QCoreApplicationPrivate::sendThroughObjectEventFilters ( this=0x9d8f3c8, receiver=0x9dc8400, event=0xbffc3b40) at kernel/qcoreapplication.cpp:819 #43 0x02ea3d0a in QApplicationPrivate::notify_helper (this=0x9d8f3c8, receiver=0x9dc8400, e=0xbffc3b40) at kernel/qapplication.cpp:4296 #44 0x02eab0fe in QApplication::notify (this=0xbffc43d4, receiver= 0x9dc8400, e=0xbffc3b40) at kernel/qapplication.cpp:3865 #45 0x061757cb in KApplication::notify (this=0xbffc43d4, receiver= 0x9dc8400, event=0xbffc3b40) at /usr/src/debug/kdelibs-4.4.2/kdeui/kernel/kapplication.cpp:302 #46 0x00813c03 in QCoreApplication::notifyInternal (this=0xbffc43d4, receiver=0x9dc8400, event=0xbffc3b40) at kernel/qcoreapplication.cpp:704 ---Type <return> to continue, or q <return> to quit--- #47 0x02ea9e68 in sendEvent (receiver=0x9dc8400, event=0xbffc3b40, alienWidget=0x9dc8400, nativeWidget=0x9fdc1a0, buttonDown=0x37a3138, lastMouseReceiver=..., spontaneous=true) at ../../src/corelib/kernel/qcoreapplication.h:215 #48 QApplicationPrivate::sendMouseEvent (receiver=0x9dc8400, event= 0xbffc3b40, alienWidget=0x9dc8400, nativeWidget=0x9fdc1a0, buttonDown= 0x37a3138, lastMouseReceiver=..., spontaneous=true) at kernel/qapplication.cpp:2965 #49 0x02f270d0 in QETWidget::translateMouseEvent (this=0x9fdc1a0, event= 0xbffc405c) at kernel/qapplication_x11.cpp:4373 #50 0x02f265e3 in QApplication::x11ProcessEvent (this=0xbffc43d4, event= 0xbffc405c) at kernel/qapplication_x11.cpp:3384 #51 0x02f524fa in x11EventSourceDispatch (s=0x9d98100, callback=0, user_data=0x0) at kernel/qguieventdispatcher_glib.cpp:146 #52 0x00f96fb8 in g_main_dispatch (context=0x9d97490) at gmain.c:1960 #53 IA__g_main_context_dispatch (context=0x9d97490) at gmain.c:2513 #54 0x00f9a8e8 in g_main_context_iterate (context=0x118eb0, block=1, dispatch=1, self=0x9d94098) at gmain.c:2591 #55 0x00f9aa14 in IA__g_main_context_iteration (context=0x9d97490, ---Type <return> to continue, or q <return> to quit--- may_block=1) at gmain.c:2654 #56 0x0083be46 in QEventDispatcherGlib::processEvents (this=0x9d6d728, flags=...) at kernel/qeventdispatcher_glib.cpp:412 #57 0x02f520e6 in QGuiEventDispatcherGlib::processEvents (this=0x9d6d728, flags=...) at kernel/qguieventdispatcher_glib.cpp:204 #58 0x008122da in QEventLoop::processEvents (this=0xbffc4354, flags=...) at kernel/qeventloop.cpp:149 #59 0x0081261a in QEventLoop::exec (this=0xbffc4354, flags=...) at kernel/qeventloop.cpp:201 #60 0x00814ce7 in QCoreApplication::exec () at kernel/qcoreapplication.cpp:981 #61 0x02ea3dd8 in QApplication::exec () at kernel/qapplication.cpp:3579 #62 0x0804f269 in main (argc=3, argv=0xbffc4534) at /usr/src/debug/kdepim-4.4.2/akregator/src/main.cpp:103
Created attachment 43005 [details] backtrace after breakpoint made on plugin loading Sorry, that was long. Here's the same backtrace I made after the breakpoint, attached as a file. I then single-stepped until nspluginviewer was launched (attached as another file). I hope this has been useful.
Created attachment 43006 [details] single-stepping in gdb until nspluginviewer launched with backtrace fragment
This still happens, though https://bugs.kde.org/show_bug.cgi?id=182869#c24 makes it much less noticeable. nspluginviewer now takes 1% CPU or less in the plain text view and 0% when I navigate to an article without embedded youtube. So nspluginviewer is still wrongly triggered, but now it's almost un-noticeable (apart from 4MB of USS according to smem).
This still happens, so I had another poke around with gdb. (gdb) bt #0 KHTMLPart::requestObject (this=0xa21550, frame=0xfd6970, url="http://www.youtube.com/v/d_OceOpCmf8?version=3", serviceType="application/x-shockwave-flash", params=QList<QString> = {...}) at /usr/src/debug/kdelibs-4.4.5/khtml/khtml_part.cpp:4176 #1 0x00000030be2c5240 in DOM::HTMLObjectBaseElementImpl::computeContent (this=0xfd6970) at /usr/src/debug/kdelibs-4.4.5/khtml/html/html_objectimpl.cpp:533 #2 0x00000030be286e23 in khtml::KHTMLParser::popOneBlock (this=0xc3ce60, delBlock=true) at /usr/src/debug/kdelibs-4.4.5/khtml/html/htmlparser.cpp:1863 #3 0x00000030be287e31 in khtml::KHTMLParser::popBlock (this=0xc3ce60, _id=72) at /usr/src/debug/kdelibs-4.4.5/khtml/html/htmlparser.cpp:1804 #4 0x00000030be28a98c in khtml::KHTMLParser::parseToken (this=0xc3ce60, t=0xea4408) at /usr/src/debug/kdelibs-4.4.5/khtml/html/htmlparser.cpp:254 #5 0x00000030be28b211 in khtml::HTMLTokenizer::processToken (this=0xea43e0) at /usr/src/debug/kdelibs-4.4.5/khtml/html/htmltokenizer.cpp:2056 #6 0x00000030be29241c in khtml::HTMLTokenizer::parseTag (this=0xea43e0, src=...) at /usr/src/debug/kdelibs-4.4.5/khtml/html/htmltokenizer.cpp:1529 #7 0x00000030be294bdd in khtml::HTMLTokenizer::write (this=0xea43e0, str=<value optimized out>, appendData=<value optimized out>) at /usr/src/debug/kdelibs-4.4.5/khtml/html/htmltokenizer.cpp:1810 #8 0x00000030be1f3626 in KHTMLPart::write (this=<value optimized out>, str=<value optimized out>) at /usr/src/debug/kdelibs-4.4.5/khtml/khtml_part.cpp:2124 #9 0x00007ffff011258b in Akregator::ArticleViewer::renderContent (this=0xa1e2d0, text="<div class=\"headerbox\" dir=\"ltr\">\n<div class=\"headertitle\" dir=\"ltr\">\n<a href=\"http://www.wired.com/beyond_the_beyond/2010/10/software-studies-4535-time-magazine-covers-1923-2009/\">Software Studies, 4"...) at /usr/src/debug/kdepim-4.4.6/akregator/src/articleviewer.cpp:411 ... frame 1 #1 0x00000030be2c5240 in DOM::HTMLObjectBaseElementImpl::computeContent (this=0xfd6970) at /usr/src/debug/kdelibs-4.4.5/khtml/html/html_objectimpl.cpp:533 533 if (!part->requestObject( this, effectiveURL, effectiveServiceType, params)) { (gdb) l 528 KHTMLPart* part = document()->part(); 529 clearChildWidget(); 530 531 kDebug(6031) << effectiveURL << effectiveServiceType << params; 532 533 if (!part->requestObject( this, effectiveURL, effectiveServiceType, params)) { 534 // Looks like we are gonna need alternative content after all... 535 m_renderAlternative = true; 536 } 537 (gdb) frame 2 #2 0x00000030be286e23 in khtml::KHTMLParser::popOneBlock (this=0xc3ce60, delBlock=true) at /usr/src/debug/kdelibs-4.4.5/khtml/html/htmlparser.cpp:1863 1863 current->close(); (gdb) l 1858 if((Elem->node != current)) { 1859 if (current->maintainsState() && document){ 1860 document->registerMaintainsState(current); 1861 document->attemptRestoreState(current); 1862 } 1863 current->close(); 1864 } 1865 #endif 1866 1867 removeForbidden(Elem->id, forbiddenTag); (gdb) frame 3 #3 0x00000030be287e31 in khtml::KHTMLParser::popBlock (this=0xc3ce60, _id=72) at /usr/src/debug/kdelibs-4.4.5/khtml/html/htmlparser.cpp:1804 1804 popOneBlock(); (gdb) l 1799 while (Elem) 1800 { 1801 if (Elem->id == _id) 1802 { 1803 int strayTable = inStrayTableContent; 1804 popOneBlock(); 1805 Elem = 0; 1806 1807 // This element was the root of some malformed content just inside an implicit or 1808 // explicit <tbody> or <tr>. (gdb) frame 4 #4 0x00000030be28a98c in khtml::KHTMLParser::parseToken (this=0xc3ce60, t=0xea4408) at /usr/src/debug/kdelibs-4.4.5/khtml/html/htmlparser.cpp:254 254 processCloseTag(t); (gdb) l 249 if(t->tid == ID_BR+ID_CLOSE_TAG && document->inCompatMode()) 250 t->tid -= ID_CLOSE_TAG; 251 252 if(t->tid > ID_CLOSE_TAG) 253 { 254 processCloseTag(t); 255 return; 256 } 257 258 // ignore spaces, if we're not inside a paragraph or other inline code The control flow doesn't seem continous here - maybe some functions are inline or something? Anyway, popOneBlock (triggered by bad markup?) ends up calling computeContent which doesn't check whether plugins are enabled before requesting something capable of handling flash. I see pluginsEnabled is checked earlier in page load, perhaps for other elements? E.g. (from another gdb run): (gdb) bt #0 KHTMLPart::pluginsEnabled (this=0xa21550) at /usr/src/debug/kdelibs-4.4.5/khtml/khtml_part.cpp:1422 #1 0x00000030be2c0b9a in DOM::HTMLObjectElementImpl::attach (this=<value optimized out>) at /usr/src/debug/kdelibs-4.4.5/khtml/html/html_objectimpl.cpp:833 #2 0x00000030be288407 in khtml::KHTMLParser::insertNode (this=0xfde8f0, n=0xe60cb0, flat=<value optimized out>) at /usr/src/debug/kdelibs-4.4.5/khtml/html/htmlparser.cpp:429 #3 0x00000030be28a85b in khtml::KHTMLParser::parseToken (this=0xfde8f0, t=0xd41c68) at /usr/src/debug/kdelibs-4.4.5/khtml/html/htmlparser.cpp:302 #4 0x00000030be28b211 in khtml::HTMLTokenizer::processToken (this=0xd41c40)
I checked this in F14 (which uses kde 4.5.2) and still see nspluginviewer launched. I also tried to test whether the kwebkitpart behaves the same, but akregator uses khtml in the article viewer regardless of the filetype settings.
I tested the suggested patch (pasted inline below) on F13, kdelibs 4.4.5, and it appears to solve the problem. nspluginviewer is not loaded for any of the recent Bruce Sterling articles in akregator - and flash still loads as expected when opening the article in a tab. Index: html/html_objectimpl.cpp =================================================================== --- khtml/html/html_objectimpl.cpp (revision 1191612) +++ khtml/html/html_objectimpl.cpp (working copy) @@ -509,6 +509,12 @@ KHTMLPart* p = document()->part(); if (!p || !p->javaEnabled()) newRenderAlternative = true; + } else { + // Similarly for plugins. + KHTMLPart* p = document()->part(); + + if (!p || !p->pluginsEnabled()) + newRenderAlternative = true; } // If there is no <embed> (here or as a child), and we don't have a type + url to go on, @@ -740,11 +746,6 @@ void HTMLEmbedElementImpl::attach() { - KHTMLPart* p = document()->part(); - - if (!p || !p->pluginsEnabled()) - m_renderAlternative = true; - if (parentNode()->id() == ID_OBJECT) NodeBaseImpl::attach(); else @@ -827,11 +828,6 @@ void HTMLObjectElementImpl::attach() { - KHTMLPart* p = document()->part(); - - if (!p || !p->pluginsEnabled()) - m_renderAlternative = true; - HTMLObjectBaseElementImpl::attach(); }
SVN commit 1193090 by orlovich: Make !pluginsEnabled() work proeprly rather than do some weirdo attach/detach dance BUG: 208780 M +7 -10 html_objectimpl.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1193090
SVN commit 1193091 by orlovich: Merged revision:r1193090 | orlovich | 2010-11-04 10:54:57 -0400 (Thu, 04 Nov 2010) | 4 lines Make !pluginsEnabled() work proeprly rather than do some weirdo attach/detach dance BUG: 208780 M +7 -10 html_objectimpl.cpp WebSVN link: http://websvn.kde.org/?view=rev&revision=1193091