Merge pull request #1179 from ssebs/super-key-feature

Super key feature

Add Windows Key Feature

Merge branch 'user_getters_and_setters_in_websock' of https://github.com/juanjoDiaz/noVNC

Use CustomEvent for playback events

Stop abusing Event as that doesn't work everwhere.

Avoid using String.prototype.startsWith()

IE doesn't support it.

Pre-convert recordings for playback

Convert the recordings ahead of time instead of during the playback.
That way we aren't messing up the profiling with time spent converting
data, rather than processing it.

Extract rQshift to common function

Add getters/setter to websock

Merge pull request #1165 from juanjoDiaz/throw_errors_instead_of_strings

Throw Error instead of String

Merge pull request #1166 from juanjoDiaz/simplify_EventTargetMixin

Simplify EventTargetMixin

Merge pull request #1168 from juanjoDiaz/use_new_error

Use `new` when constructing errors

Simplify EventTargetMixin

Use `new` when constructing errors

Throw Error instead of String

Dont reset touch mouse button state for view-only

Fixes bug introduced in 61f93180c8cbdc6259d0b083b51f169053f66f8c.

Fix chinese translation of "Connect"

Thanks to Lyon Hu (@chnhyg) for finding the issue.

Czech translation

Change copyright header (#1138)

* Change copyright header

This updates the copyright header to say "The noVNC Authors".  People
who previously had copyright listings are now under the AUTHORS file.

Add tests for Cursor encoding

Fix cursor encoding handling from earlier merge

Old code snuck in when merging the split of decoders to separate
classes. Restore the proper handling of cursors.

Merge branch 'style' of https://github.com/CendioOssman/noVNC

Merge branch 'decoders' of https://github.com/CendioOssman/noVNC

Merge pull request #1119 from patrakov/master

Show dot when there otherwise would be no visible cursor

Show dot when there is no visible cursor

Disabled by default.

Moved the "pixels + mask -> RGBA" logic to rfb.js

As requested by Pierre Ossman - he needs this for supporting other
cursor extensions.

Enforce switch colon spacing

Enforce space after function name

Enforce space before code block

Enforce explicit semi-colons

Enforce curly braces for control statements

Enforce no trailing whitespace

Enforce keyword spacing

Enforce object key spacing

Enforce function declaration style

Enforce function names

Enforce function call spacing

Enforce comma style

Enforce comma spacing

Enforce indentation

Don't lint xtscancodes.js

It's generated by another project so we cannot control the style
of it.

Enforce brace style

Merge pull request #1129 from novnc/vnclitecleanup

Cleanup of vnc_lite.html

Limit line length to 80 in vnc_lite

Add a screen element for vnc_lite.html

Makes it clearer where the remote screen will be created.

Simplify element names in vnc_lite

Create our own button for CtrlAltDel in vnc_lite

In order to have better control of the layout and to make the code
easier to follow. Using input type button or buttons will imply a lot of
built in styling that differs from browser to browser.

Use a simple prompt for passwords in vnc_lite

It's not password-masked, but it allows for a lot simpler code.

Order vnc_lite functions

Rename functions in vnc_lite

Give them obvious names to make the code easier to understand.

Use let and const instead of var in vnc_lite

The rest of noVNC has been converted already. This allows us to remove
the extra scope that was created for the VNC connection.

Remove support for the fragment and WebUtil dep

The only remaining use we had of WebUtil was getConfigVar(). Let's get
rid of that dependency and use our own, query-string-only and richly
commented version of that function. It's easier for people to get an
overview of vnc_lite if it's all in one file.

This commit removes support for the fragment, parameters can only be
passed using the query string from now on.

Cleanup non-essential options from vnc_lite

This is supposed to be a simple example, it shouldn't have this many
options. This commit removes the following options:
 * logging - the default level 'warn' is good enough
 * title - a weird thing to set from the query string anyway
 * token - not used by most setups
 * encrypt - looking at the URL is good enough
 * repeaterID - not used by most setups
 * shared - uncommon setting
 * resize - not supported by most servers

Note that the removal of 'encrypt' allows us to remove logic for
establishing a default port. The default port for wss is 443 and for ws
it's 80 anyway.

Add code comments to vnc_lite

Move decoders to separate classes

Makes things a lot clearer by letting each encoding handle its own
details and state.

Handle pseudo encodings directly

These have very special behaviour compared to normal data encodings,
so separate out them and handle them separately.

Remove statistics tracking

The profiles in the browsers are much better these days and give us
much better data than we can provide ourselves.

Give proper int argument to encodingName()

Simplify connected() function

Remove unneccessary code and bling from vnc_lite

Unused code, variables and unnecessary styles. The host/port check
would only have an effect if someone explicitly set them as empty in
the query string. The different colors of the status bar are not
necessary, nor is the styling of the background.

Move css rules for vnc_lite to the html

Easier to get an overview if it's all in one file.

Get rid of icons for vnc_lite

Icons aren't required and we want to get rid of the 'app/' dependency.

Merge branch 'hidpi_scale' of https://github.com/CendioOssman/noVNC

Replace bad sinon stub in mouse tests

It screwed up important calls inside the code being tested. Avoid
the stub by creating a temporary element with the desired properties.

Remove iOS specific code from vnc_lite

vnc_lite.html doesn't have touch support anyway

Remove machine control buttons from vnc_lite

The vnc_lite example is intended to be minimal and these buttons are
only useful in special cases.

Revert "Handle if desktopName isn't set.."

This reverts commit 22000b93d5aa3d12823e7440533e8eb25739e9c0. The
'desktopname' and the 'connect' events are dispatched by us in RFB and
are thus serial.

Merge pull request #1117 from novnc/bug/fix-test-playback

Fix test playback

Support password auth recordings for playback

When password auth is enabled on the server, the RFB object sends a
'credentialsrequired' event to the UI. This commit adds support for
this event to our recoding playback.

Handle if desktopName isn't set when connected

We can't guarantee that the desktopName event has been fired before the
connect event.

Merge pull request #1118 from novnc/disabledragwhilescale

Turn off view drag when scaling

Merge enableDisableViewClip and updateViewClip

Makes the code easier to follow and makes sure that viewDrag is
properly disabled when scaling. Fixes #1110.

Use macOS 10.13 for Safari tests

Safari 11 had a bug (#1125) which should be fixed in Safari 11.1 which comes with macOS 10.13.

Move dragThreshold definition to util/

In order to avoid multiple declarations that has to be updated in the
case of future updates.

Remove firebug comments

Firebug is discontinued and it's features are included in browser
development tools now a days.

Detail path to icon Makefile

Add a record flag to launch.sh

Add the `--record` flag to launch.js, for easy recording when testing.

Remove setViewDrag function

Unnecessary function only used in the toggle function above.

Remove unnecessary code

The enableDisableViewClip call in the fullscreen code didn't have any
effect and should have been removed when the special case for clipping
in IE and Safari fullscreen was removed in b18ef81.

The setViewDrag call claimed to disable view drag on UI state change.
The UI states are:

 init, connecting, connected, reconnecting, disconnecting, disconnected

The only state where the called function didn't immediately return was
"connected" and that's the only state where enabling view drag is
possible. Thus it could never have been enabled when changing to the
"connected" state.

Fix perf/playback tool

Somewhere along the way, the refactors broke playback.js.  This fixes
the actual functionality, and makes its JS loading match that in
vnc.html.

Handle fractional screen sizes

With high DPI systems we can end up with a container with a size that
is not an integer number of CSS pixels. Make sure we can handle those
cases by allowing a fractional size for the output canvas. Framebuffer
size and viewport coordinates are still restricted to integer dimensions
though.

Based on initial patch by Alexander E. Patrakov.

Merge branch 'master' of https://github.com/patrakov/noVNC

Documented browser cache issue

Use newer macOS test machine for Travis

Stop transpiling karma tests

This runs our code in the same manner as it would be used if loaded
directly in the browser. Includes the same kind of fallback for older
browsers.

Fix proper triggering of module fallback

We might be in the "interactive" readyState, which means that
DOMContentLoaded has already fired and we'll hang.

Don't stub out ES module imports

It is not allowed and only happens to work because babel doesn't
strictly follow the specification. It doesn't seem necessary for the
tests to run, so just remove it.

Better currentScript fallback

The previous heuristic didn't work under all circumstances, so try
something more robust.

Move sinon to karma framework

This frees us from manual imports, and makes things less magical
as those aren't ES modules even if the code suggest that the are.

Use the classic `function foo() { ... }` for top level functions or functions that depend on the scope

Use fat arrow functions `const foo = () => { ... };` for callbacks
and any other function that is passed around and it's not a top level function

Use ES6 classes
Always use the shorthand notation if the function is a method of an object or class `{ foo() { ... } }` or `class bar { foo() { ... } }`
unless it's a callback in which case you a fat arrow function should be used `{ cb: () => { ... } }`

Merge branch 'cursor' of https://github.com/CendioOssman/noVNC

Provide fallback cursor method

Some browsers don't support custom cursors, and there are cases
where the browsers refuse to show the cursor. Handle both of these
cases by letting the browser render the cursor via a floating
canvas.

This allows us to support a local cursor at all times.

Sort vkeys table

Merge branch 'fix/ie11-numpad5-compatibility' of https://github.com/vlastoun/noVNC

Fixes #1075

Rather than trying to pick a utility, we should be able to just use bash to check if a port is available or not.

We can probably assume bash is available due to the shebang declaring it.

Avoid TypedArray.slice() because of IE11

Use string assignment operator instead of concat()

The assignment operator is a lot faster.

Use the correct slicing for rQshiftStr

This didn't result in any error however since slice() handles such
mistakes gracefully.

Add rQshiftStr unit test for large strings