jshint shouldn't throw errors - wp-admin/revisions.js

[adamsilverstein]

The file wp-admin/revisions.js throws many errors in the jshint section of the build process.

The errors generated are:

W116: Expected '{' and instead saw ... - thrown in the many places where curly brackets are omitted from single line if/else blocks

W081: Too many var statements. - thrown in several places where var is used separately in a single function block instead of being combined into a single var statement at the top of the function

W117: ? is not defined. (Backbone, '_' Underscore, _wpRevisionsSettings) thrown for variables that are defined/passed from external sources; also several spots where var declaration missing ahead of variable use

W098: ? is defined but never used. - thrown where function signatures include variables that are never used and are therefore not needed

[adamsilverstein]

fix jshint for revisions.js

[adamsilverstein]

As a check (per azaozz's suggestion), I compared the minified versions of the original and patched files (kaleidoscope helpful since its all one line). differences were minimal: better variable name optimization where variables now declared properly. the one other change was that I switched console reference to window.console to clear errors - the other approach would be to add console to the file's global declaration.

[azaozz]

In 26035:

jshint: fixes for revisions.js, props adamsilverstein, fixes #25864.

[nacin]

In 26207:

Fix JSHint errors in two files. see #25957, #25864. (Similar names but separate files.)