Bug 833104 - Dont block the camera preview from loading during initialis... #7900
Conversation
| return r; | ||
| } | ||
|
|
||
| api.init = function() { |
There was a problem hiding this comment.
Why not have this module initialize itself as soon as it loads? That is, get rid of the init() function and just call asyncStorage.getItem right away? If you did that, you might want to move this script and asyncStorage up to the head of the document so they start running as soon as possible.
There was a problem hiding this comment.
I kept having varying results with attempting to do asyncstorage reads as well as anything else before the previewstream loaded, they all made it slower to some varying degree. right now the only thing that happens before the previewstream is some dom code required to show the preview stream, and some filmstrip self initialisation which didnt have a large impact (although it may be worth looking at in a follow up)
Plus it doesnt make any difference to our code whether its loaded early or not, we still have to handle the cases where asyncstorage hasnt loaded before the ui is activated
Bug 833104 - Dont block the camera preview from loading during initialis...
Bug 833104 - Dont block the camera preview from loading during initialis...
Bug 833104 - Dont block the camera preview from loading during initialis...
...ation