Code review comment for lp:~lukas-kde/unity8/fix-shell-chrome

Michael Zanetti (mzanetti) wrote :

hmm... a couple of questions:

* A timer of 1 second seems quite long. Is that really what we want?

* afaict, this change now only loads the windowstatestorage stuff if we're starting up in windowed mode. However, it should still load the windowedX/Y/Width/Height even though we're in staged mode, just not use them yet while we're in staged mode. Once we're switching from staged to windowed, it should then use the previously loaded stuff.

review: Needs Information

« Back to merge proposal