Performance issues with the Dojo Editor in IE8/7

[Herman Lo]

Machine Spec: Intel Core Duo 2.2ghz, 3gb ram, Windows XP.

Under IE8, a dojo editor loaded with 3000 lines takes approximately 10˜ seconds to complete an operation. Attached a sample editor with the standard plugins plus three sample rudimentary plugins to demonstrate the issue (Move attached files to \dojo\dijit\tests\editor\).

this.editor. get('value');
this.editor.set('value', html);
this.editor.execCommand("insert html", 
      "<span class=\"cursorSpan\">Insert</span&g t;");

Given the performance issues with IE in general, some performance degradation is to be expected in comparison to other browsers but the performance gap is markedly noticeable. We can partially compensate for the issue by minimizing our manipulation of the text content and relying on progress dialogs but in certain cases where we need to manipulate the text in near real-time this may not be an option.

In a similar vein, we're experiencing issues with the editor after each edit. From my understanding, the editor will copy the text content into an internal array, so that it can maintain an undo history. However, in the case of the editor described above, the editor and browser will freeze for several seconds after each edit while the undo array is been populated.

Setting the customUndo flag to false, will disable the editor's undo functionality and instead fall back to the browser's native implementation of undo. However, the editor continues tracking the undo history after each edit. Is this the expected behavior?

[bill]

performance test

[bill]

plugin to run get test

[bill]

plugin to test insert

[bill]

plugin to test set

[bill]

patch to use cloneNode() and avoid apparently unnecessary node.outerHTML reference

[bill]

In [26975]:

Performance tweaks for editor's getNodeHtml(), gives about 10x performance boost on IE8 (see test_Editor_performance.html test case). Also adding small automated test for dijit/_editor/html.html. Fixes #14231 !strict, although I may check in further tweaks.

[bill]

In [26983]:

More performance tweaks for editor's getNodeHtml(), gives about 4x performance boost on IE6 (see test_performance.html test case). Refs #14231 !strict.

[bill]

In [26984]:

Test for escaping of HTML text content. Refs #14231 !strict.

[bill]

In [28084]:

IE9 prints style with a closing semicolon: style="width: 10%; height: 10%;". Refs #14231.

[bill]

In [28091]:

svn:eol-style, refs #14231

[Slavon]

​http://howtotreatpot.tumblr.com/