Merge lp:~abreu-alexandre/webbrowser-app/with-oxide into lp:~osomon/webbrowser-app/with-oxide

Proposed by Alexandre Abreu
Status: Merged
Approved by: Olivier Tilloy
Approved revision: 476
Merged at revision: 487
Proposed branch: lp:~abreu-alexandre/webbrowser-app/with-oxide
Merge into: lp:~osomon/webbrowser-app/with-oxide
Diff against target: 29 lines (+3/-3)
1 file modified
src/app/webcontainer/WebApp.qml (+3/-3)
To merge this branch: bzr merge lp:~abreu-alexandre/webbrowser-app/with-oxide
Reviewer Review Type Date Requested Status
Olivier Tilloy Approve
Review via email: mp+211614@code.launchpad.net

Commit message

Container updates: Enable localstorage + some fixes

Description of the change

Container updates: Enable localstorage + some fixes

To post a comment you must log in.
Revision history for this message
Olivier Tilloy (osomon) wrote :

Do we need https://code.launchpad.net/~abreu-alexandre/unity-webapps-qml/oxide-integration/+merge/211615 to land first as a prerequisite, or does the landing order not matter?

review: Needs Information
Revision history for this message
Alexandre Abreu (abreu-alexandre) wrote :

> Do we need https://code.launchpad.net/~abreu-alexandre/unity-webapps-qml
> /oxide-integration/+merge/211615 to land first as a prerequisite, or does the
> landing order not matter?

Does not matter really there are no deps between the 2, but this fixes an issue further down the webapps chain where quite a few webapps use localStorage

Revision history for this message
Alexandre Abreu (abreu-alexandre) wrote :

> > Do we need https://code.launchpad.net/~abreu-alexandre/unity-webapps-qml
> > /oxide-integration/+merge/211615 to land first as a prerequisite, or does
> the
> > landing order not matter?
>
> Does not matter really there are no deps between the 2, but this fixes an
> issue further down the webapps chain where quite a few webapps use
> localStorage

You are right indeed, I missed/forgot the makeProxiesForWebViewBindee part,
yes there is a landing deps on https://code.launchpad.net/~abreu-alexandre/unity-webapps-qml
> /oxide-integration/+merge/211615

Revision history for this message
Olivier Tilloy (osomon) wrote :

Good. As long as the two MRs land together we should be fine. The change looks good to me.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/app/webcontainer/WebApp.qml'
--- src/app/webcontainer/WebApp.qml 2014-03-17 22:16:59 +0000
+++ src/app/webcontainer/WebApp.qml 2014-03-18 19:48:53 +0000
@@ -69,8 +69,6 @@
69 }69 }
70 height: parent.height - osk.height70 height: parent.height - osk.height
7171
72 //experimental.preferences.developerExtrasEnabled: developerExtrasEnabled
73
74 contextualActions: ActionList {72 contextualActions: ActionList {
75 Actions.CopyLink {73 Actions.CopyLink {
76 enabled: webview.contextualData.href.toString()74 enabled: webview.contextualData.href.toString()
@@ -116,6 +114,8 @@
116114
117 onNewTabRequested: Qt.openUrlExternally(url)115 onNewTabRequested: Qt.openUrlExternally(url)
118116
117 preferences.localStorageEnabled: true
118
119 // Small shim needed when running as a webapp to wire-up connections119 // Small shim needed when running as a webapp to wire-up connections
120 // with the webview (message received, etc…).120 // with the webview (message received, etc…).
121 // This is being called (and expected) internally by the webapps121 // This is being called (and expected) internally by the webapps
@@ -133,7 +133,7 @@
133 }133 }
134 }134 }
135 };135 };
136 return UnityWebAppsUtils.makeProxiesForQtWebViewBindee(webview, eventHandlers)136 return UnityWebAppsUtils.makeProxiesForWebViewBindee(webview, eventHandlers)
137 }137 }
138 }138 }
139139

Subscribers

People subscribed via source and target branches