Merge lp:~justinmcp/unity-chromium-extension/13.10 into lp:unity-chromium-extension/13.10

[Justin McPherson]

Infobars support has moved to the stable channel of chromium. This means
support is not patched in the way it was previously, and it's now necessary for the extension to request use.

Infobars support has moved to the stable channel of chromium. This means
support is not patched in the way it was previously, and it's now necessary for
the extension to request use.

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:237
http://jenkins.qa.ubuntu.com/job/unity-chromium-extension-13.10-ci/2/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-chromium-extension-13.10-saucy-amd64-ci/3
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-chromium-extension-13.10-saucy-armhf-ci/2

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-chromium-extension-13.10-ci/2/rebuild

review: Approve (continuous-integration)

[Robert Bruce Park (robru)]

Bah, this is way easier than I was expecting. Ok!

review: Approve

[David Barth (dbarth)]

Robert: I think this fix is very important. I will top-approve this branch: can you upload it to -proposed for a quick resolution of the problem. Thanks.

review: Approve

[Alexandre Abreu (abreu-alexandre)]

quick question: ... don't you also need to update the background.js page? doesn't it refer to chrome.experimental.infobars?

review: Needs Information

[Justin McPherson (justinmcp)]

@abreu-alexandre; Support was previously patched into the correct "namespace", so there is no need, good forward thinking :)

=== modified file 'chromium-extension/manifest.json.in'
--- chromium-extension/manifest.json.in	2013-09-19 16:19:40 +0000
+++ chromium-extension/manifest.json.in	2014-02-07 15:42:21 +0000
@@ -33,7 +33,7 @@
     "page": "background-page.html"
   },  
 
-  "permissions": ["tabs", "http://*/*", "notifications"],
+  "permissions": ["tabs", "http://*/*", "notifications", "infobars"],
   
   "options_page": "options.html",