Crashes while trying to add a new location on N4

[Nekhelesh Ramananthan]

After rev 21 (https://code.launchpad.net/~martin-borho/ubuntu-weather-app/reboot-worker/+merge/252972), when opening weather app reboot on a freshly flashed mako (Nexus 4), adding a location results in weather app crashing.

Steps to reproduce:
1. Remove the weather app data in .local/share/com.ubuntu.weather and in .config/com.ubuntu.weather
2. Open weather app reboot
3. Try adding a location

What happens:
Weather app freezes and crashes.

Log:
qml: No settings, using defaults
qml: Database created
qml: Settings table added, Database upgraded to v0.2
qml: Removed old locations, Database upgraded to v0.3
qml: Migrating old data: {}
qml: No old data to migrate.
UbuntuWindow::handleSurfaceFocusChange(focused=true)
UbuntuWindow::handleSurfaceResize(width=768, height=1222)
UbuntuClipboard - Got invalid serialized mime data. Ignoring it.
qml: WARNING! Do not put Page/Tabs/PageStack inside another Page because that causes confusion which is the active page that sets the title and actions.
qml: WARNING! Do not put Page/Tabs/PageStack inside another Page because that causes confusion which is the active page that sets the title and actions.
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0
QIODevice::read: Called with maxSize < 0

[Nekhelesh Ramananthan (nik90)]

FYI, the crash happens only on the nexus 4. It works fine on a vivid desktop.

[Victor Thompson (vthompson)]

Is this on the RTM channel only? While sometimes locations fail to get added on my mako device running the devel-proposed channel, I haven't had it crash before.

[Nekhelesh Ramananthan (nik90)]

hmm I tested this on N4 running RTM. I didnt check on vivid. Let me do that and update this bug report.

[Victor Thompson (vthompson)]

Ok, it is crashing for me now on vivid. When I click the plus symbol to add a Location the app pauses for a second or two and then crashes. It wasn't previously though, perhaps we introduced this in a recent revision? Or perhaps a recent addition to the platform has introduced this?

[Martin Borho (martin-borho)]

Reason to remove the WorkerScript in the old app, was that the data given in at one side of the WorkerScript, arrived incomplete at the other side. And vice versa. Perhaps it's the same again. (Can't start the app on the device atm, some misconfiguration...)

[Victor Thompson (vthompson)]

If that is the case, perhaps we should revert the WorkerScript but leave in the initial retrieval from storage.

[Victor Thompson (vthompson)]

I can confirm that reverting the worker fixes the problem [1]

1 - https://code.launchpad.net/~vthompson/ubuntu-weather-app/reboot-revert-worker/+merge/253048

[Victor Thompson (vthompson)]

Correct MP is here: https://code.launchpad.net/~vthompson/ubuntu-weather-app/reboot-revert-worker/+merge/253050

[Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot)]

Fix committed into lp:ubuntu-weather-app/reboot at revision 22, scheduled for release in ubuntu-weather-app, milestone 2.0