Merge lp:~abreu-alexandre/webbrowser-app/fix-navigation-filtering into lp:webbrowser-app

Proposed by Alexandre Abreu
Status: Merged
Approved by: Olivier Tilloy
Approved revision: 343
Merged at revision: 350
Proposed branch: lp:~abreu-alexandre/webbrowser-app/fix-navigation-filtering
Merge into: lp:webbrowser-app
Diff against target: 33 lines (+7/-3)
1 file modified
src/app/Browser.qml (+7/-3)
To merge this branch: bzr merge lp:~abreu-alexandre/webbrowser-app/fix-navigation-filtering
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Olivier Tilloy Approve
Review via email: mp+187638@code.launchpad.net

Commit message

Fix issue w/ with invalid webapp component id/access when filtering urls

Description of the change

Fix issue w/ with invalid webapp component id/access when filtering urls

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
Olivier Tilloy (osomon) wrote :

LGTM.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/app/Browser.qml'
--- src/app/Browser.qml 2013-09-24 11:59:49 +0000
+++ src/app/Browser.qml 2013-09-26 02:31:47 +0000
@@ -311,13 +311,13 @@
311 }311 }
312312
313 Loader {313 Loader {
314 id: webappsComponentLoader
314 sourceComponent: (browser.webapp && tabsModel.currentIndex > -1) ? webappsComponent : undefined315 sourceComponent: (browser.webapp && tabsModel.currentIndex > -1) ? webappsComponent : undefined
315316
316 Component {317 Component {
317 id: webappsComponent318 id: webappsComponent
318319
319 UnityWebApps.UnityWebApps {320 UnityWebApps.UnityWebApps {
320 id: webapps
321 name: browser.webappName321 name: browser.webappName
322 bindee: tabsModel.currentWebview322 bindee: tabsModel.currentWebview
323 actionsContext: browser.actionManager.globalContext323 actionsContext: browser.actionManager.globalContext
@@ -423,8 +423,12 @@
423 var url = request.url.toString();423 var url = request.url.toString();
424424
425 // The list of url patterns defined by the webapp takes precedence over command line425 // The list of url patterns defined by the webapp takes precedence over command line
426 if (webapp && isRunningAsANamedWebapp() && webapps.model && webapps.model.exists(webapps.name)) {426 if (webapp && isRunningAsANamedWebapp()) {
427 if ( ! webapps.model.doesUrlMatchesWebapp(webapps.name, url)) {427 var webappComponent = webappsComponentLoader.item;
428
429 if (webappComponent != null &&
430 webappComponent.model.exists(webappComponent.name) &&
431 ! webappComponent.model.doesUrlMatchesWebapp(webappComponent.name, url)) {
428 action = WebView.IgnoreRequest;432 action = WebView.IgnoreRequest;
429 }433 }
430 }434 }

Subscribers

People subscribed via source and target branches

to status/vote changes: