Merge lp:~vrruiz/unity-webapps-linkedin/const-to-var into lp:unity-webapps-linkedin

[Víctor R. Ruiz]

Fix 'const' error (style_checker).

Fix 'const' error (style_checker).

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:47
http://jenkins.qa.ubuntu.com/job/unity-webapps-linkedin-ci/2/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-webapps-linkedin-saucy-amd64-ci/2/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-webapps-linkedin-saucy-armhf-ci/2/console

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

review: Needs Fixing (continuous-integration)

[Robert Bruce Park (robru)]

What's the deal with selfTest()? Why is it not defined for linkedin? Other apps seem to define their own...

[Robert Bruce Park (robru)]

Superceded by https://code.launchpad.net/~robru/unity-webapps-linkedin/appease-jslint/+merge/169550

review: Disapprove

Unmerged revisions

47. By Víctor R. Ruiz on 2013-06-11

Fix 'const' complain for style_checker

=== modified file 'LinkedIn.user.js'
--- LinkedIn.user.js	2013-06-07 15:33:16 +0000
+++ LinkedIn.user.js	2013-06-11 17:50:54 +0000
@@ -5,7 +5,7 @@
 // ==/UserScript==
 
 // This placeholder gets munged with real data at build time.
-const WebappsGettextDict = JSON.parse(unescape(
+var WebappsGettextDict = JSON.parse(unescape(
     "%7B%22GETTEXT%22%3A%22PLACEHOLDER%22%7D"
 ));