570500 - (a11yperf_fx4) [meta] a11y performance improvements for Firefox 4

Status: RESOLVED FIXED

(Core :: Disability Access APIs, defect)

Description

[alexander :surkov (:asurkov)]

Please do not mark bugs as depended until they have (or going to have) real fix so that it can be tracker easily what was done.

Comment 1

[alexander :surkov (:asurkov)]

Attachment: LFC insertBefore

original test: http://hacks.mozilla.org/2010/05/better-performance-with-lazy-frame-construction/

Comment 2

[alexander :surkov (:asurkov)]

Attachment: LFC appendChild

original test: http://hacks.mozilla.org/2010/05/better-performance-with-lazy-frame-construction/

Comment 3

[alexander :surkov (:asurkov)]

Attachment: table mutation

original test: https://bug522847.bugzilla.mozilla.org/attachment.cgi?id=406846

Comment 4

[alexander :surkov (:asurkov)]

Attachment: remove children

Comment 5

[David Bolter [:davidb] (NeedInfo me for attention)]

Comment on attachment 449647 [details]
LFC insertBefore

>      var lastchild = document.getElementById("lastchild");
>      for (var i = 0; i < 8000; i++) {
>        var div = document.createElement("div");
>        container.insertBefore(div, lastchild);

I don't actually see an element with id="lastchild"...

Comment 6

[alexander :surkov (:asurkov)]

changing summary to more generic one

Comment 7

[alexander :surkov (:asurkov)]

Attachment: HTML name calculation

Comment 8

[alexander :surkov (:asurkov)]

Attachment: bug573469 Create Accessible Tree

DIV insertion: 470 ms
SPAN insertion: 44491 ms

without a11y the results are DIV: 470 ms, SPAN: 505 ms.

Comment 9

[David Bolter [:davidb] (NeedInfo me for attention)]

Comment on attachment 487590 [details]
bug573469 Create Accessible Tree

This should also show bug 563331.

Comment 10

[alexander :surkov (:asurkov)]

(In reply to comment #9)
> Comment on attachment 487590 [details]
> bug573469 Create Accessible Tree
> 
> This should also show bug 563331.

Sure, it's dupe in my mind.

Comment 11

[alexander :surkov (:asurkov)]

Attachment: bug573469 ARIA relation test

Comment 12

[Marco Zehe (:MarcoZ)]

(In reply to comment #8)
> Created attachment 487590 [details]
> bug573469 Create Accessible Tree
> 
> DIV insertion: 470 ms
> SPAN insertion: 44491 ms
> 
> without a11y the results are DIV: 470 ms, SPAN: 505 ms.

I'm at 230/313 MS with the try-server build that also includes bug 381599.

Comment 13

[Marco Zehe (:MarcoZ)]

Comment on attachment 489254 [details]
bug573469 ARIA relation test

This is the actual patch file, not a testcase. It's also having Application/octet stream as MIME type which is rather strange.

Comment 14

[alexander :surkov (:asurkov)]

Attachment: 613919: characterCount

before bug 613919: 74 sec, after 44. If I don't get characterCount twice then results are the same what's expected.

Comment 15

[alexander :surkov (:asurkov)]

Attachment: 573719: caretOffset

Comment 16

[alexander :surkov (:asurkov)]

Attachment: 613919: characterCount

Comment 17

[alexander :surkov (:asurkov)]

Attachment: 573719: caretOffset

Comment 18

[alexander :surkov (:asurkov)]

Attachment: bug573469 ARIA relation test

Comment 19

[alexander :surkov (:asurkov)]

schedule bug 573719 and bug 611537 to Firefox 5 since they aren't likely fixed in Fx4 timeframe. No other perf improvements is scheduled for Fx4 so we can mark this bug as fixed I think.