Closed Bug 1192193 Opened 9 years ago Closed 6 years ago

It takes too much time to open "alwaysLowered" window

Categories

(Firefox OS Graveyard :: Gaia::System::Window Mgmt, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: azasypkin, Unassigned)

References

Details

Here is the very simple test case [1] that shows the problem - we just open "alwaysLowered" window using the the URL with almost empty HTML document. There are 2 important performance marks: first right before we call window.open, second is the right after.

I've profiled this with Raptor (10 runs) and here are results:

| Metric                        | Mean    | Median  | Min     | Max      | StdDev  | p95      |
| ----------------------------- | ------- | ------- | ------- | -------- | ------- | -------- |
| coldlaunch.before open window | 204.400 | 204.500 | 185.000 | 225.000  | 11.681  | 225.000  |
| coldlaunch.after open window  | 795.500 | 764.500 | 673.000 | 1362.000 | 193.603 | 1362.000 |
| coldlaunch.fullyLoaded        | 795.600 | 764.500 | 673.000 | 1363.000 | 193.896 | 1363.000 |
| coldlaunch.pss                | 12.760  | 12.800  | 12.700  | 12.800   | 0.049   | 12.800   |
| coldlaunch.rss                | 26.190  | 26.200  | 26.100  | 26.200   | 0.030   | 26.200   |
| coldlaunch.uss                | 9.110   | 9.100   | 9.100   | 9.200    | 0.030   | 9.200    |

P95 values look quite scary :)

[1] https://github.com/azasypkin/sw-tests#07.08.2015
Hey Etienne,

I've just remeasured alwaysLowered window with the latest build and I see numbers are improved significantly (looks like we shaved ~200-300ms \0/) [1].

I'm just wondering if you know about any ongoing work related to window management in Gecko or System app? So that we can refer to it in case we see regressions in the future :)

| Metric                        | Mean    | Median  | Min     | Max     | StdDev | p95     |
| ----------------------------- | ------- | ------- | ------- | ------- | ------ | ------- |
| coldlaunch.before open window | 227.100 | 227.500 | 183.000 | 277.000 | 24.191 | 269.000 |
| coldlaunch.after open window  | 399.550 | 396.000 | 349.000 | 499.000 | 43.183 | 499.000 |
| coldlaunch.fullyLoaded        | 400.000 | 396.000 | 350.000 | 500.000 | 43.305 | 500.000 |
| coldlaunch.uss                | 8.409   | 8.410   | 8.395   | 8.430   | 0.009  | 8.428   |
| coldlaunch.pss                | 12.047  | 12.063  | 11.983  | 12.099  | 0.034  | 12.089  |
| coldlaunch.rss                | 25.653  | 25.656  | 25.621  | 25.684  | 0.019  | 25.682  |


[1] https://github.com/nga-crash-tests/always-lowered-window#09.09.2015
Flags: needinfo?(etienne)
Not sure... lots of people are looking at app launch time but nothing in particular comes to mind that landed recently.

Maybe the screenshot cleanups in bug 1194547 helped a bit (screenshoting is really **** the main thread). Or something that I didn't review/wrote :)
Flags: needinfo?(etienne)
Blocks: 1203469
Okay, thanks! We'll keep eye on it then :)
Well, silly me, I was too happy to notice that window.open silently fails due to the lack of "open-hidden-window" permission. So the numbers are still quite bad :/

| Metric                        | Mean    | Median  | Min     | Max      | StdDev  | p95      |
| ----------------------------- | ------- | ------- | ------- | -------- | ------- | -------- |
| coldlaunch.before open window | 313.500 | 309.000 | 264.000 | 421.000  | 37.407  | 395.000  |
| coldlaunch.after open window  | 815.100 | 772.000 | 664.000 | 1459.000 | 160.255 | 1188.000 |
| coldlaunch.fullyLoaded        | 815.550 | 772.000 | 665.000 | 1460.000 | 160.278 | 1188.500 |
| coldlaunch.uss                | 9.195   | 9.184   | 9.148   | 9.277    | 0.034   | 9.267    |
| coldlaunch.pss                | 12.800  | 12.805  | 12.512  | 12.907   | 0.079   | 12.899   |
| coldlaunch.rss                | 26.389  | 26.379  | 26.352  | 26.461   | 0.033   | 26.453   |

Sorry for the noise.
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.