117834 2005-12-07 03:14:44 +0000 konqueror html w/ javascript has high (80%) cpu usage 2006-01-22 03:08:01 +0000 1 1 2 Applications konqueror khtml unspecified Gentoo Packages Linux RESOLVED FIXED NOR normal --- 1 msimmons konqueror-bugs-null maksim 0 oldest_to_newest 396302 0 msimmons 2005-12-07 03:14:44 +0000 Version: (using KDE KDE 3.4.3) Installed from: Gentoo Packages Compiler: gcc version 3.4.4 OS: Linux The web page http://www.scied.science.doe.gov/scied/fast/about.html uses a javascript animation of some sort that uses 80% of the cpu on an AMD64 2GHz. Mozilla 1.7.12 with javascript enabled was not affected at all. The offending javascript follows: ================================= /* chaser.js * by Aaron Boodman v1.0 000919 * Copyright (c) 2000 Aaron Boodman. All Rights Reserved. [snip...get the full license agreement from the linked page] */ /*[snip comments]*/ var oChaser = { topMargin : 250, callRate : 10, slideTime : 1200, maxDiff : document.all ? document.body.clientHeight : window.innerHeight, isIE : document.all ? true : false, chaserDiv : document[document.all ? "all" : "layers"]["myChaser"] } /*snip comments*/ window.setInterval("oChaser.main( )", oChaser.callRate) /*snip comments*/ oChaser.main = function( ) { this.currentY = this.isIE ? this.chaserDiv.style.pixelTop : this.chaserDiv.top this.scrollTop = this.isIE ? document.body.scrollTop : window.pageYOffset var newTargetY = this.scrollTop + this.topMargin if ( this.currentY != newTargetY ) { if ( newTargetY != this.targetY ) { this.targetY = newTargetY this.slideInit( ) } this.slide( ) } } /*snip comments*/ oChaser.slideInit = function( ) { var now = new Date( ) this.A = this.targetY - this.currentY this.B = Math.PI / ( 2 * this.slideTime ) this.C = now.getTime( ) if (Math.abs(this.A) > this.maxDiff) { this.D = this.A > 0 ? this.targetY - this.maxDiff : this.targetY + this.maxDiff this.A = this.A > 0 ? this.maxDiff : -this.maxDiff } else { this.D = this.currentY } } /*snip comments*/ oChaser.slide = function( ) { var now = new Date( ) var newY = this.A * Math.sin( this.B * ( now.getTime( ) - this.C ) ) + this.D newY = Math.round( newY ) if (( this.A > 0 && newY > this.currentY ) || ( this.A < 0 && newY < this.currentY )) { if ( this.isIE )this.chaserDiv.style.pixelTop = newY else this.chaserDiv.top = newY } } 396303 1 13795 msimmons 2005-12-07 03:19:15 +0000 Created attachment 13795 the javascript in its entirity 396304 2 maksim 2005-12-07 03:24:29 +0000 LOL. That JS has no chance whatsoever of working on mozilla. And the CPU usage in konqueror? it fills the error log. There was some other bug report about it, I think I'll put in a cap 396306 3 msimmons 2005-12-07 03:41:22 +0000 I don't know enough about javascript one way or the other, I'll go ahead and close this if it isn't of any help. 396307 4 maksim 2005-12-07 03:45:23 +0000 No, this is very helpful, I am constantly bugging users in #kde that I don't have enough performance problems to fix :-). And the JavaScript on the page, and the reason for our slowness gave me a good laugh. Thanks again for reporting this 396311 5 msimmons 2005-12-07 04:04:19 +0000 Ok, after re-reading the comments I understand what you meant by filling the error log. I was wondering why there was all the cpu activity but the page was visually static. I'll leave it to someone who know what they are doing to close the bug. This is the first time I've reported a bug. 407997 6 maksim 2006-01-22 03:07:59 +0000 SVN commit 501092 by orlovich: Make this thing actually update incrementally and limit to 2K messages Fixes #117834 (I think there was some other report, too, but I can't find it) BUG:117834 M +5 -5 kjserrordlg.ui.h --- branches/KDE/3.5/kdelibs/khtml/kjserrordlg.ui.h #501091:501092 @@ -10,14 +10,14 @@ void KJSErrorDlg::init() { - _errorText->setText("<qt>"); + _errorText->setTextFormat(QTextEdit::LogText); + _errorText->setMaxLogLines(2048);//Limit to about 2K errors so we don't use much CPU } void KJSErrorDlg::addError( const QString & error ) { - // Argh why doesn't QText* interpret html tags unless I use setText()? - // This is really inefficient! - _errorText->setText(_errorText->text() + "<br>\n" + error); + + _errorText->append(error); } void KJSErrorDlg::setURL( const QString & url ) @@ -28,5 +28,5 @@ void KJSErrorDlg::clear() { _errorText->clear(); - _errorText->insert("<qt>"); + init(); } 13795 2005-12-07 03:19:15 +0000 2005-12-07 03:19:15 +0000 the javascript in its entirity chaser.js text/plain 3106 msimmons LyogY2hhc2VyLmpzDQogKiBieSBBYXJvbiBCb29kbWFuIHYxLjAgMDAwOTE5DQogKiBDb3B5cmln aHQgKGMpIDIwMDAgQWFyb24gQm9vZG1hbi4gQWxsIFJpZ2h0cyBSZXNlcnZlZC4NCiAqIENyZWF0 ZWQgZm9yIEdyZWF0RXF1YWxpemVyLmNvbSAoaHR0cDovL3d3dy5ncmVhdGVxdWFsaXplci5jb20v KSBhbmQNCiAqIGRvY3VtZW50ZWQgYXQgREhUTUwgTGFiIChodHRwOi8vd3d3LndlYnJlZmVyZW5j ZS5jb20vZGh0bWwvKQ0KICogTGljZW5zZSB0byB1c2UgaXMgZ3JhbnRlZCBpZiBhbmQgb25seSBp ZiB0aGlzIGVudGlyZQ0KICogY29weXJpZ2h0IG5vdGljZSBpcyBpbmNsdWRlZC4NCiAqIFRoaXMg cHJvZ3JhbSBpcyBmcmVlIHNvZnR3YXJlOyB5b3UgY2FuIHJlZGlzdHJpYnV0ZSBpdCBhbmQvb3Ig bW9kaWZ5DQogKiBpdCB1bmRlciB0aGUgdGVybXMgb2YgdGhlIEdOVSBHZW5lcmFsIFB1YmxpYyBM aWNlbnNlIGFzIHB1Ymxpc2hlZCBieQ0KICogdGhlIEZyZWUgU29mdHdhcmUgRm91bmRhdGlvbjsg ZWl0aGVyIHZlcnNpb24gMiBvZiB0aGUgTGljZW5zZSwgb3INCiAqIChhdCB5b3VyIG9wdGlvbikg YW55IGxhdGVyIHZlcnNpb24uDQoqLw0KDQovLyBUaGUgY2hhc2VyIG9iamVjdC4gU2luY2Ugd2Ug ZG9uJ3QgYW50aWNpcGF0ZSBoYXZpbmcgbW9yZQ0KLy8gdGhhbiBvbmUgb24gYSBwYWdlLCB3ZSBk b24ndCBib3RoZXIgbWFraW5nIHRoaXMgaW50bw0KLy8gYSBjbGFzcyBkZWZpbml0aW9uLiBBbGwg bmVjZXNzYXJ5IHByb3BlcnRpZXMgYXJlIHNldCBhcw0KLy8gcHJvcGVydGllcyBvZiB0aGlzIG9i amVjdCB0byBhdm9pZCBnbG9iYWwgdmFyaWFibGVzLg0KDQp2YXIgb0NoYXNlciA9IHsNCgl0b3BN YXJnaW4JOiAyNTAsDQoJY2FsbFJhdGUJOiAxMCwNCglzbGlkZVRpbWUJOiAxMjAwLA0KCW1heERp ZmYJCTogZG9jdW1lbnQuYWxsID8gZG9jdW1lbnQuYm9keS5jbGllbnRIZWlnaHQgOiB3aW5kb3cu aW5uZXJIZWlnaHQsDQoJaXNJRQkJOiBkb2N1bWVudC5hbGwgPyB0cnVlIDogZmFsc2UsDQoJY2hh c2VyRGl2CTogZG9jdW1lbnRbZG9jdW1lbnQuYWxsID8gImFsbCIgOiAibGF5ZXJzIl1bIm15Q2hh c2VyIl0NCn0NCg0KLy8gQXJyYW5nZSB0byBoYXZlIHRoZSBtYWluIGxvb3AgY2FsbGVkIGFzIG9m dGVuIGFzIHBvc3NpYmxlLCANCi8vIGJ1dCBub3QgbW9yZSBvZnRlbiB0aGFuIGV2ZXJ5IDM1IG1p bGxpc2Vjb25kcy4NCi8vIEV2ZW4gdGhvdWdoIHNvbWUgT1MncyBjYW4gYWNoZWl2ZSBiZXR0ZXIg dGhhbiB0aGF0LCB0aGVyZSdzIG5vIHJlYWwgcmVhc29uIHRvDQovLyBvdmVybG9hZCB0aGVtIGlm IGl0cyBub3QgZ29pbmcgdG8gaW1wcm92ZSB0aGUgYW5pbWF0aW9uIHF1YWxpdHkgdmVyeSBtdWNo Lg0KLy8gSSB0ZXN0ZWQsIGFuZCBiZXR0ZXIgdGhhbiAzNW1zIGRvZXNuJ3QgZG8gdmVyeSBtdWNo IHZpc3VhbGx5Lg0Kd2luZG93LnNldEludGVydmFsKCJvQ2hhc2VyLm1haW4oICkiLCBvQ2hhc2Vy LmNhbGxSYXRlKQ0KDQoNCi8vIE1haW4gbG9vcC4gVXBkYXRlcyB0YXJnZXRZLCBhbmQgZGVjaWRl cyB3aGV0aGVyIHRvIHN0YXJ0DQovLyB0aGUgYW5pbWF0aW9uIG92ZXIgYWdhaW4sIGNvbnRpbnVl IGFuIGV4aXN0aW5nIGFuaW1hdGlvbiwNCi8vIG9yIGRvIG5vdGhpbmcgYXQgYWxsLg0Kb0NoYXNl ci5tYWluID0gZnVuY3Rpb24oICkNCnsNCgl0aGlzLmN1cnJlbnRZCT0gdGhpcy5pc0lFID8gdGhp cy5jaGFzZXJEaXYuc3R5bGUucGl4ZWxUb3AgOiB0aGlzLmNoYXNlckRpdi50b3ANCgl0aGlzLnNj cm9sbFRvcAk9IHRoaXMuaXNJRSA/IGRvY3VtZW50LmJvZHkuc2Nyb2xsVG9wIDogd2luZG93LnBh Z2VZT2Zmc2V0DQoJdmFyIG5ld1RhcmdldFkJPSB0aGlzLnNjcm9sbFRvcCArIHRoaXMudG9wTWFy Z2luDQoJDQoJaWYgKCB0aGlzLmN1cnJlbnRZICE9IG5ld1RhcmdldFkgKSB7DQoNCgkJaWYgKCBu ZXdUYXJnZXRZICE9IHRoaXMudGFyZ2V0WSApIHsNCg0KCQkJdGhpcy50YXJnZXRZID0gbmV3VGFy Z2V0WQ0KCQkJdGhpcy5zbGlkZUluaXQoICkNCgkNCgkJfQ0KDQoJCXRoaXMuc2xpZGUoICkNCgkJ DQoJfQ0KfQ0KDQoNCg0KLy8gLnNsaWRlSW5pdCggKS4gSW5pdGlhbGl6ZXMgdGhlIHNsaWRlIGFu aW1hdGlvbi4gU2V0cyBwcm9wZXJ0aWVzDQovLyBvZiB0aGUgb0NoYXNlciBvYmplY3QgdGhhdCB3 aWxsIHJlcHJlc2VudCB0aGUgdmFyaW91cyBwYXJhbWF0ZXJzDQovLyBmb3IgdGhlIHNpbmUgd2F2 ZSBmdW5jdGlvbi4NCm9DaGFzZXIuc2xpZGVJbml0ID0gZnVuY3Rpb24oICkNCnsNCgl2YXIgbm93 CT0gbmV3IERhdGUoICkNCgkNCgl0aGlzLkEJCT0gdGhpcy50YXJnZXRZIC0gdGhpcy5jdXJyZW50 WQ0KCXRoaXMuQgkJPSBNYXRoLlBJIC8gKCAyICogdGhpcy5zbGlkZVRpbWUgKQ0KCXRoaXMuQwkJ PSBub3cuZ2V0VGltZSggKQ0KDQoJaWYgKE1hdGguYWJzKHRoaXMuQSkgPiB0aGlzLm1heERpZmYp IHsNCgkJdGhpcy5EID0gdGhpcy5BID4gMCA/IHRoaXMudGFyZ2V0WSAtIHRoaXMubWF4RGlmZiA6 IHRoaXMudGFyZ2V0WSArIHRoaXMubWF4RGlmZg0KCQl0aGlzLkEgPSB0aGlzLkEgPiAwID8gdGhp cy5tYXhEaWZmIDogLXRoaXMubWF4RGlmZg0KCX0gZWxzZSB7DQoJCXRoaXMuRCA9IHRoaXMuY3Vy cmVudFkNCgl9DQp9DQoNCg0KDQovLyAuc2xpZGUoICkuIE1vdmVzIHRoZSBvQ2hhc2VyIG9uZSBm cmFtZS4gSXRzIHJhdGUgZGVjcmVhc2VzIGFuZA0KLy8gaXMgZGVmaW5lZCBieSBhIHNpbmUgd2F2 ZS4NCm9DaGFzZXIuc2xpZGUgPSBmdW5jdGlvbiggKQ0Kew0KCXZhciBub3cJPSBuZXcgRGF0ZSgg KQ0KCXZhciBuZXdZCT0gdGhpcy5BICogTWF0aC5zaW4oIHRoaXMuQiAqICggbm93LmdldFRpbWUo ICkgLSB0aGlzLkMgKSApICsgdGhpcy5EDQoJbmV3WQkJPSBNYXRoLnJvdW5kKCBuZXdZICkNCg0K CWlmICgoIHRoaXMuQSA+IDAgJiYgbmV3WSA+IHRoaXMuY3VycmVudFkgKSB8fA0KCQkoIHRoaXMu QSA8IDAgJiYgbmV3WSA8IHRoaXMuY3VycmVudFkgKSkgew0KCQkJDQoJCQlpZiAoIHRoaXMuaXNJ RSApdGhpcy5jaGFzZXJEaXYuc3R5bGUucGl4ZWxUb3AgPSBuZXdZDQoJCQllbHNlCQkJdGhpcy5j aGFzZXJEaXYudG9wID0gbmV3WQ0KCX0NCn0NCg==