Opened 9 years ago

Last modified 9 years ago

#6354 confirmed Bug

The i() function takes a lot more time in 3.4.1 than in 3.4.0

Reported by: Niko Viitala Owned by:
Priority: Normal Milestone:
Component: QA Version: 3.4.1
Keywords: Firefox Cc:

Description

I have 24 CKEDitor instances in a page. With the latest Firefox using Firebug profiler I get

i()    24    71.74%    5361.136ms    5361.49ms    223.395ms    39.052ms    383.467ms   
ckeditor.js (line 18)
function () {
    var i = !!this.$.offsetHeight &&
        this.getComputedStyle("visibility") != "hidden", j, k;
    if (i && (b.webkit || b.opera)) {
        j = this.getWindow();
        if (!j.equals(a.document.getWindow()) && (k = j.$.frameElement)) {
            i = (new h(k)).isVisible();
        }
    }
    return i;
}

and the page freezes for a long time.

The most time consuming function in 3.4.0 is getNext():

getNext()   24   18.71%   387.901ms   393.449ms   16.394ms   15.711ms   20.288ms   
ckeditor.js (line 14)

So you see the load differences.

Change History (1)

comment:1 Changed 9 years ago by Krzysztof Studnik

Component: GeneralQA
Keywords: Firefox added; load freezing removed
Status: newconfirmed

Thank You for valuable information. I hope we write more optimal code for next versions.

Note: See TracTickets for help on using tickets.
© 2003 – 2019 CKSource – Frederico Knabben. All rights reserved. | Terms of use | Privacy policy