1107259 - Remove the sync IPC getDPI() and getDefaultScale() during app startup (~15ms).

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Performance, defect)

Description

[[:fabrice] Fabrice Desré]

Attachment: getdpi.patch

This is technically a subset of 835679. This patch sends dpi and defaultScale in the SendShow() call along with other initialization data.

Comment 1

[[:fabrice] Fabrice Desré]

https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=28fbaad4337e

Comment 2

[Thinker Li [:sinker]]

Comment on attachment 8531730 [details] [diff] [review]
getdpi.patch

Review of attachment 8531730 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/ipc/TabParent.cpp
@@ +2028,5 @@
>      widget = nsContentUtils::WidgetForDocument(mFrameElement->OwnerDoc());
>    }
>  
>    if (widget) {
> +    printf_stderr("XXX TryCacheDPIAndScale parent=%p widget=%p\n", this, widget.get());

Do you remove this debug message?

::: widget/gonk/nsWindow.cpp
@@ +557,5 @@
>  
>  float
>  nsWindow::GetDPI()
>  {
> +    printf_stderr("XXX gonk nsWindow::GetDPI() %p\n", this);

Do you remove this debug message?

Comment 4

[[:fabrice] Fabrice Desré]

https://hg.mozilla.org/integration/b2g-inbound/rev/beac475d3863

Comment 5

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/beac475d3863