Merge lp:~abreu-alexandre/unity-webapps-qml/guard-url-navigate-to into lp:unity-webapps-qml

Proposed by Alexandre Abreu on 2014-10-08
Status: Merged
Approved by: David Barth on 2014-10-09
Approved revision: 133
Merged at revision: 134
Proposed branch: lp:~abreu-alexandre/unity-webapps-qml/guard-url-navigate-to
Merge into: lp:unity-webapps-qml
Diff against target: 14 lines (+3/-1)
1 file modified
src/Ubuntu/UnityWebApps/UnityWebAppsUtils.js (+3/-1)
To merge this branch: bzr merge lp:~abreu-alexandre/unity-webapps-qml/guard-url-navigate-to
Reviewer Review Type Date Requested Status
David Barth (community) Approve on 2014-10-09
Alberto Mardegan (community) 2014-10-08 Approve on 2014-10-09
PS Jenkins bot (community) continuous-integration Approve on 2014-10-08
Review via email: mp+237645@code.launchpad.net

Commit message

Add small guard before url navigation

Description of the change

Add small guard before url navigation

To post a comment you must log in.
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Alberto Mardegan (mardy) wrote :

LGTM!

review: Approve
David Barth (dbarth) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/Ubuntu/UnityWebApps/UnityWebAppsUtils.js'
2--- src/Ubuntu/UnityWebApps/UnityWebAppsUtils.js 2014-07-18 13:43:55 +0000
3+++ src/Ubuntu/UnityWebApps/UnityWebAppsUtils.js 2014-10-08 17:18:04 +0000
4@@ -184,7 +184,9 @@
5 // inject common function
6
7 proxy.navigateTo = function(url) {
8- webViewId.url = url;
9+ if (url.length !== 0) {
10+ webViewId.url = url;
11+ }
12 };
13 // called from the UnityWebApps side
14 proxy.onAppRaised = function () {

Subscribers

People subscribed via source and target branches

to all changes: