804968 - Adjusting the height of the awesomebar results popup requires too many layout flushes

Status: RESOLVED FIXED

(Toolkit :: Autocomplete, defect)

Description

[Jared Wein [:jaws] (please needinfo? me)]

Attachment: Patch

STR:
1) Open a tab with an average size data URI as the address, for example the cat image from http://dopiaza.org/tools/datauri/examples/
2) Open a new tab, and start typing 'data' in to the address bar.

The SPS profiler shows that a ton of time is spent within adjustHeight of the rich-result-popup in autocomplete.xml. The code within adjustHeight calls getBoundingClientRect for two rows, and then computes the difference between the top of the first row and the bottom of the last row.

The heights of these rows shouldn't change while the program is running, so we should be able to cache the height of the row the first time this method is called, and then use the cached value from then-on.

See these two profiles for examples of the jank I am seeing (filter on adjustHeight):
http://people.mozilla.com/~bgirard/cleopatra/?report=f8d697a5720acd9d00d0fabc01c8a26ff88256c9
http://people.mozilla.com/~bgirard/cleopatra/?report=7331ec1c982940482150016f005b77c16e8b75ec

Comment 1

[Neil Deakin]

Comment on attachment 674604 [details] [diff] [review]
Patch

This patch doesn't allow the row height to change nor allows having rows of different heights. I assume that is ok.

Comment 2

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 674604 [details] [diff] [review]
Patch

I'll change this so it only affects the awesomebar.

Comment 3

[Jared Wein [:jaws] (please needinfo? me)]

Attachment: Patch v2

Comment 4

[Dão Gottwald [:dao]]

Comment on attachment 674893 [details] [diff] [review]
Patch v2

Firefox is the primary consumer of the rich-results binding, so I think we should take the first patch.

Comment 5

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 674604 [details] [diff] [review]
Patch

Re-setting the r+ from enn.

https://hg.mozilla.org/integration/mozilla-inbound/rev/0fc382c36b4c

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/0fc382c36b4c