Merge lp:~justinmcp/unity-webapps-linkedin/expand-site-matches into lp:unity-webapps-linkedin

[Justin McPherson]

Expand site matching for linkedin to include https: variant.

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:66
http://jenkins.qa.ubuntu.com/job/unity-webapps-linkedin-ci/6/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-webapps-linkedin-trusty-amd64-ci/1
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-webapps-linkedin-trusty-armhf-ci/1

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-webapps-linkedin-ci/6/rebuild

review: Approve (continuous-integration)

[Robert Bruce Park (robru)]

Aren't you missing "https://www.linkedin.com" (eg, without the /* at the end)?

[Justin McPherson (justinmcp)]

I've test, and it works correctly in the container without. If you find
different let me know and I'll add it.

[Robert Bruce Park (robru)]

Nah, I didn't test anything, it just looked like it was missing since you had http with and without /* but https only had /*. No worries.

=== modified file 'manifest.json'
--- manifest.json	2013-03-26 15:43:55 +0000
+++ manifest.json	2014-03-24 07:14:27 +0000
@@ -1,1 +1,1 @@
-{"includes":["http://www.linkedin.com/*","http://www.linkedin.com"],"requires":["utils.js"],"name":"LinkedIn","scripts":["LinkedIn.user.js"],"maintainer":"Webapps Team <[e-mail]>","license":"GPL-3","manifest-version":"1.0","integration-version":"2.4.8","package-name":"LinkedIn","description":"Unity Webapp for LinkedIn","icons":{"128":"128/unity-webapps-linkedin.png","48":"48/unity-webapps-linkedin.png","52":"52/unity-webapps-linkedin.png","64":"64/unity-webapps-linkedin.png"},"domain":"linkedin.com","homepage":"http://www.linkedin.com"}
+{"includes":["http://www.linkedin.com/*","http://www.linkedin.com","https://www.linkedin.com/*"],"requires":["utils.js"],"name":"LinkedIn","scripts":["LinkedIn.user.js"],"maintainer":"Webapps Team <[e-mail]>","license":"GPL-3","manifest-version":"1.0","integration-version":"2.4.8","package-name":"LinkedIn","description":"Unity Webapp for LinkedIn","icons":{"128":"128/unity-webapps-linkedin.png","48":"48/unity-webapps-linkedin.png","52":"52/unity-webapps-linkedin.png","64":"64/unity-webapps-linkedin.png"},"domain":"linkedin.com","homepage":"http://www.linkedin.com"}