:Paolo Amadini Assignee
	Description • 7 years ago

This prevents synchronous reflows when opening subviews of the panelmultiview element.

	:Gijs (he/him)
	Comment 1 • 7 years ago

Paolo, are you still working on this set of bugs?

	:Paolo Amadini Assignee
	Comment 2 • 7 years ago

As we discussed at the performance planning today, reducing reflows in the location bar popup may be a higher priority.

	Code Review Bot [:reviewbot]
	Comment 4 • 6 years ago
mozreview-review

Comment on attachment 8954596 [details]
Bug 1420939 - Add an asynchronous version of the description height workaround.

https://reviewboard.mozilla.org/r/223692/#review229726


Code analysis found 2 defects in this patch:
 - 2 defects found by mozlint

You can run this analysis locally with:
 - `./mach lint path/to/file` (JS/Python)


If you see a problem in this automated review, please report it here: http://bit.ly/2y9N9Vx


::: browser/components/customizableui/PanelMultiView.jsm:1319
(Diff revision 1)
> -        continue;
> +          continue;
> -      }
> +        }
>  
> -      items.push({ element });
> +        items.push({ element });
> -    }
> +      }
> +    }

Error: Missing semicolon. [eslint: semi]

::: browser/components/customizableui/PanelMultiView.jsm:1338
(Diff revision 1)
>      // may contain wrapping text.
> +    let measureItems = () => {
> -    for (let item of items) {
> +      for (let item of items) {
> -      item.bounds = item.element.getBoundingClientRect();
> +        item.bounds = item.element.getBoundingClientRect();
> -    }
> +      }
> +    }

Error: Missing semicolon. [eslint: semi]

	:Gijs (he/him)
	Comment 5 • 6 years ago
mozreview-review

Comment on attachment 8954596 [details]
Bug 1420939 - Add an asynchronous version of the description height workaround.

https://reviewboard.mozilla.org/r/223692/#review230012

Besides the reviewbot nits this looks OK, assuming green try etc.

	:Paolo Amadini Assignee
	Comment 6 • 6 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=8cfee8ec0d2ef8c164f30deec2b667cd9a32e38d

	Pulsebot
	Comment 7 • 6 years ago

Pushed by paolo.mozmail@amadzone.org:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2b0380882e88
Add an asynchronous version of the description height workaround. r=Gijs

	Cristina Coroiu [:ccoroiu]
	Comment 8 • 6 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/2b0380882e88