lp:~cmiller/unity-chromium-extension/extension-dir

Created by Chad Miller on 2013-09-08 and last modified on 2013-09-08
Get this branch:
bzr branch lp:~cmiller/unity-chromium-extension/extension-dir
Only Chad Miller can upload to this branch. If you are Chad Miller please log in for upload directions.

Branch merges

Related bugs

Related blueprints

Branch information

Owner:
Chad Miller
Project:
WebApps: unity-chromium-extensions
Status:
Development

Recent revisions

228. By Chad Miller on 2013-09-08

Move extension manifest to FHS-supported directory. Chromium 29 in ubuntu
supports it. (LP: #1222565)

227. By Robert Bruce Park on 2013-07-12

Release 2.4.7-0ubuntu1 to Saucy.

Approved by Ken VanDine, PS Jenkins bot.

226. By Alexandre Abreu on 2013-07-11

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 on 2013-07-08

[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 (npObjectInvokeImpl() 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.

224. By Alexandre Abreu on 2013-06-28

Support chromium 28

223. By Alexandre Abreu on 2013-03-14

Release 2.4.6 to R.

Approved by PS Jenkins bot, Víctor R. Ruiz.

222. By Alexandre Abreu on 2013-03-12

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 on 2013-03-05

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 on 2013-02-18

Fix #1124238: Launcher actions not showing up. Fixes: https://bugs.launchpad.net/bugs/1124238.

Approved by Víctor R. Ruiz.

219. By Alexandre Abreu on 2013-02-13

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
This branch contains Public information 
Everyone can see this information.

Subscribers