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

Proposed by Alexandre Abreu
Status: Merged
Approved by: David Barth
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
Alberto Mardegan (community) Approve
PS Jenkins bot (community) continuous-integration Approve
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.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Alberto Mardegan (mardy) wrote :

LGTM!

review: Approve
Revision history for this message
David Barth (dbarth) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/Ubuntu/UnityWebApps/UnityWebAppsUtils.js'
--- src/Ubuntu/UnityWebApps/UnityWebAppsUtils.js 2014-07-18 13:43:55 +0000
+++ src/Ubuntu/UnityWebApps/UnityWebAppsUtils.js 2014-10-08 17:18:04 +0000
@@ -184,7 +184,9 @@
184 // inject common function184 // inject common function
185185
186 proxy.navigateTo = function(url) {186 proxy.navigateTo = function(url) {
187 webViewId.url = url;187 if (url.length !== 0) {
188 webViewId.url = url;
189 }
188 };190 };
189 // called from the UnityWebApps side191 // called from the UnityWebApps side
190 proxy.onAppRaised = function () {192 proxy.onAppRaised = function () {

Subscribers

People subscribed via source and target branches

to all changes: