lp:~abreu-alexandre/unity-chromium-extension/support-chromium-29
- Get this branch:
- bzr branch lp:~abreu-alexandre/unity-chromium-extension/support-chromium-29
Branch merges
Branch information
Recent revisions
- 226. By Alexandre Abreu
-
Fix issue w/ installation (when message is being sent to background page from infobar, the sender.tab object is not set); updated some apis to newer versions. Fixes: https:/
/bugs.launchpad .net/bugs/ 1196943. Approved by Víctor R. Ruiz, PS Jenkins bot.
- 225. By Alexandre Abreu
-
[Impact]
Workaround for an issue with Chromium 27 and 28 and Webapps. Webapps stopped working altogether with those chromium versions. What happened is that any javascript callback function that was called (whatever the stage) from chromium's internal could not access the NPAPI plugin variable (_uwa) that is a javascript object that is a bridge to the WebApps NPAPI plugin. Accessing the variable throws with an exception "NPObject has been deleted" (which is not the case).
The workaround rebinds the variable for every callback to the plugin node.
TECHNICAL DETAILS:
-----------------There has been some changes recently in chromium 27+ around the v8 bindings' templates separated by world type (either page world or content script private world). Although I am not sure, it seemed to have an impact somewhere in the lookup location for V8NPObject v8 binding variables.
The "NPOBJect has been deleted" exception first occurs when webapps callback in the context_prepare() callback (from a native NPAPI function) that tries to call the plugin using _uwa. The variable is just closed over in the callback but is being looked up in the v8 binding for NPAPI (npObjectInvoke
Impl() in WebCore/ bindings/ v8/V8NPObject. cpp) in the wrong World (page world instead of content script world). The V8NPObject binding for the NPAPI plugin is apparently created properly (tagged w/ the proper world id) but somewhere along the line (copying?) something goes wrong.
[Test Case]
1. install the unity-chromium-
extension,
2. open chromium 28 and make sure that the extension is properly detected (chrome://extensions) ,
3. go to e.g. facebook.com or bbc.co.uk./news,
4. make sure that a launcher icon appears and the webapps integrates (notifications for bbc and messaging menu for facebook),[Regression Potential]
None since the fix is strictly confined in the webapps extension and webapps are currently broken.
Approved by PS Jenkins bot, Víctor R. Ruiz.
- 222. By Alexandre Abreu
-
Fix #1138473: libunity-webapps pulls gtk 3 in to Firefox. Fixes: https:/
/bugs.launchpad .net/bugs/ 1138473. Approved by PS Jenkins bot, Alexandre Abreu.
- 221. By Alexandre Abreu
-
Fix typo for setCanGoPrevious on the website side (proxy). Fixes: https:/
/bugs.launchpad .net/bugs/ 1139711. Approved by PS Jenkins bot, Alexandre Abreu.
- 220. By Alexandre Abreu
-
Fix #1124238: Launcher actions not showing up. Fixes: https:/
/bugs.launchpad .net/bugs/ 1124238. Approved by Víctor R. Ruiz.
- 219. By Alexandre Abreu
-
Fix 1099828: handle the case of the popup window glitches with the 'remove tabs in chromeless when only 1 tab' code.
Approved by Alexandre Abreu, PS Jenkins bot.
Branch metadata
- Branch format:
- Branch format 7
- Repository format:
- Bazaar repository format 2a (needs bzr 1.16 or later)
- Stacked on:
- lp:unity-chromium-extension/trunk