Remove browser engine detection

It is out of date and not used anyway.

Skip some keyboard tests on IE and Edge

IE and Edge has some broken behaviour for keyboard events that
prevent the standard tests from running properly.

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

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

Error.error can be null in some cases

Use KeyboardEvent.key too look up keysyms

And emulate it on browsers where it is missing or incorrect. This
makes the code more future oriented as it primarily uses the standardised
fields.

Handle CapsLock on macOS

Modifiers behave a bit oddly on macOS, causing weird CapsLock events
to be sent by the browsers.

Clean up AltGraph handling

It doesn't need to be this general as the issue is mostly about
Windows. Also use the same modifier shuffle that RealVNC and
TigerVNC uses to get macOS working well.

Simplify pressed key handling

Prefer avoid having the server simulate multiple key presses by
refusing to use multiple keysyms for the same physical key.

Simplify handling of keypress

Use a dedicated variable to track a two stage key rather than
piggy-backing on the key state array.

Move keyboard handling in to Keyboard class

Replace the multi stage pipeline system with something simpler.
That level of abstraction is not needed.

Remove modifier synchronisation

The fields provided cannot tell us if it is the left or right
version of the key that's pressed, so they are inherently unreliable.
It is also not a huge problem in practice as we'll get in sync on
the next press or release of the modifier.

Remove QEMU key event handler

The normal event handler provides all the necessary information
now, so it is no longer needed.

Simplify keyboard event API

No need for an object for three static fields.

Send keyboard events from single place

This makes it easier to handle any needed variations, like different
types of messages.

Improve lookup of special keys

Look up keys that are independent of layout and state first,
followed by keys that are only mild variations in layouts.
This is more robust as there might be multiple physical keys
generating the same symbols, and Keysyms don't map directly to
Unicode in all cases.

At the same time switch over to using the modern, standardised
'code' field for lookup.

Improve character keysym lookup

Use the more modern 'key' field, and remove some legacy fallbacks
that are no longer required. This also removes the "stall" mechanism
as it is not needed with current browsers.

Use standard DOM identifiers for physical keys

Merge pull request #817 from univention/master

Remove double unescaping of title

Remove double unescaping of title

Commit fa5b334dcbe7a5dab21580b54cc2f5f142600379 by Joel Martin changed
getQueryVal() to always decode the value using decodeURIComponent(), but
unescape() is still used for extracting the title, leading to
double-unescaping.

As unescape() is deprecated anyway, remove the last user.

Cc: Joel Martin <github@martintribe.org>
Cc: Anthony Young <sleepsonthefloor@gmail.com>

Only load ES6 module loader polyfill if needed

Safari now has native support for modules, so we need to make sure
we only load our polyfill when actually needed.

Fix panning test for high DPI devices

There is a drag-threshold in the panning code which the tests didn't
account for. This caused the tests to fail when window.devicePixelRatio
was higher than 1.

Add Pierre as contributor in package.json

Update url's

Properly restore the default style

Setting a style to null does restore it in FF, Chrome, Safari and Edge.
But it does not work in Internet Explorer. The proper way to restore to
default values is to set it to the empty string. This works in all
browsers. Fixes issue #808.

Fixed the issue where novnc can't be relocated

Always send mouseUp events properly

If down is false we can't just toggle the current internal state, even
though this is correct most of the time. There are cases where we don't
get the down event and thus won't have a correct internal state. For
example, when clicking in the session after using the clipboard
textarea.

If down is false, we always want to send a mouse event with the button
in 'up'-state.

Merge pull request #801 from CendioOssman/errors

Improve fallback error handler

Generate source map for ES module loader rollup

Makes it easier to debug things as you get the original source and
proper line numbers.

Handle uncaught WebWorker exceptions

Try to populate error location on Promise errors

Only use unhandledrejection fallback when needed

Use common code to handle Promise errors

Handle partial error location information

We might not get line or column number, but the file is still
useful information.

ErrorEvent.error isn't always set

It's not just a matter of the browser lacking support.

Limit fallback error width to window width

Change the layout a bit to make sure the entire message can be read.

Use monospace font for unhandled errors

The browsers uses crude layout formatting that expects this.

Merge pull request #738 from patrakov/master

Changed the default compression level to 2

Hide internal keyboard functions

These may change and are not part of a stable API.

This also reindents the object functions to make sure they can
access private functions.

Remove character substitution

We can handle any Unicode codepoint now, so stop replacing symbols.

Simplify keysymdef.js

Some Unicode to Keysym mappings can be computed and can therefore
be left out of the huge lookup table.

Clean up keysymdef

Remove keysym names from keysymdef.js

They were incomplete and turned off in most cases so they served
little use besides adding complexity.

Get parse.js in sync with generated code

The generated keysymdef.js was recently converted to ES modules,
but the generating script was overlooked.

Define an initial shadow for spinner

IE and Edge refuse to animate attributes that weren't present in
the static definition.

Merge pull request #774 from novnc/refactor/es6-module-loader

[Refactor] ES6 Modules

Make vnc_playback.html functional once more

This commit makes vnc_playback.html functional once more, and completely
refactors tests/playback.js to make it usable in other scenarios.

In order for vnc_playback.js to properly load playback files now, they
must `export` their variables.

Fix vnc_auto.html

This commit fixes `vnc_auto.html` to work with the new changes.
Note that it is not translated over when `--with-app` is used
on `util/use_require.js`.  We'll probably want to deprecate it,
or do some longer-term cleanup.

Vendor in an IE11 polyfill for Promises

This commit introduces a polyfill to add support for Promises in IE11.
This means IE11 can be tested without first running
`utils/as_require.js`.

Move error handler into separate file

This commit moves the global error handler into a separate file,
so that it can catch module loading errors.

This also adds support for properly displaying error messages with
newlines in them (since the module loader may throw those)

Only apply settings with an RFB object

We should only apply settings if we have an RFB object available.
Some of the settings handlers didn't respect this, but do now.

Fix view_only clipboard handling

We need to actually consume the clipboard event message,
even if we don't act on it.

Move scripts in vnc.html head

This commit moves the vnc.html scripts into head.

Optimize ES6 Module Loader Polyfill

This commit makes the ES6 module loader polyfill use Web Workers,
so that Babel doesn't block the browser from animating.  It also
uses localStorage to cache the compiled results, only recompiling
on source changes, so it makes loading faster while developing noVNC.

This includes a vendored copy of the ES6 module loader, modified as
described above.

Use transition animation while loading

This shows the transition animation while loading, so users can see
something is going on.

Update tests to work with new structure

This updates the tests to work with the new structure, and removes the
old `utils/run_from_console.js` files in favor of just using Karma
directly.  The Karma debug page now displays the normal mocha HTML, so
we can use that instead of the HTML generation functionality of the old
test runner.

Note that PhantomJS does not work at the moment (PhantomJS 1.5 should
make it possible to test on PhantomJS again).

Refactor ES6 module structure/split up Util

This commit restructures many of the ES6 modules, splitting them
up to actual export multiple functions instead of a single object.

It also splits up Util into multiple sub-modules, to make it easier
to maintain.

Finally, localisation is renamed to localization.

Remove WebUtil.load_scripts

The only remaining user of WebUtil.load_scripts was for loading
localisation.  Instead, we now load the localization information
over XHR as a JSON blob.

Load translations over XHR

This commit causes translations to be loaded over XHR instead of
the import system.

Remove WebUtil.dirObj

`WebUtil.dirObj` wasn't being used anywhere, so this removes it.

Allow transforming to any format

This changes around `utils/use_require.js` to be able to generate any
of AMD (RequireJS), CommonJS, SystemJS, or UMD modules.  The three
former also include support for translating `vnc.html`, producing a full
"app" version of noVNC.

Use PNG for mouse cursors

The browsers' support for Microsoft's cursor format is a bit spotty,
so use the more common PNG format instead. This also allows us to
use a Canvas to generate the image, rather than coding it by hand.

Use ES6 modules natively via Polyfill

This commit introduces the "Browser ES Module Loader" polyfill
to support developing with native ES6 modules, without any compilation
step (files are passed through Babel in the browser).  This should not
be used in production -- a pre-compiled version passed through babel
ahead of time (as produced by the `npm install` hook or
`utils/use_require.js`) should be used instead.

Vendor an ES6-modules-compatible Pako

This commit vendors a converted, stripped-down version of Pako which
uses ES6 modules instead of CommonJS modules.

Uncomment ES6 module syntax

This removes the special comment part of the ES6 module syntax,
opting to enable ES6 module syntax by default.

It also appends `.js` to all import paths to better support in-browser
loading.

Fix typos in imports and i18n

`app/ui.js` had an incorrect import path which caused issues
when using the ES6 and/or CommonJS builds of noVNC.

`core/util.js` had a non-strict-compatible declaration of a variable
without a `let` or `var` (it now uses `let`).

This fixes both issues.

Merge branch 'settingswhileconnected' of https://github.com/samhed/noVNC

Closes pull request #778

Merge branch 'disableLabels' of https://github.com/samhed/noVNC

Closes pull request #776

Remove 512x512 icon

It is not a launcher icon, but rather used for display on Google
Play. As such it is not needed and removing it mitigates the issue
of some buggy browsers downloading each and every listed icon.

Protect against race in setCapture() polyfill

It might take a long time for a timer to fire, long enough for a
new grab to be initiated. Clearing out the capture element would
then cause a crash.

Disable labels for settings as well

Makes the settings UI easier to read.

Set input type number for port & reconnect_delay

Fix wrong colours on local cursor

The RGB order was backwards compared to the pixel format we've
requested, resulting in the red and blue channels getting swapped.

Allow local cursor to be changed while connected

Allow auto reconnect to be changed while connected

Allow view_only to be changed while connected

Don't send or recieve clipboard in view_only

Allow clipping to be changed while connected

Fix translateDOM for placeholders

Replaced 'in' with a comma, like in the surrounding code.

Proper workaround for Chrome's scrollbar bug

Avoid applying this workaround more than necessary. Also comment a
bit more why this hack is needed.

Update drag button at right time

It needs to update whenever the viewport or clipping changes, so
let's make sure that actually happens and not trigger on mildly
related events.

Update clipping before changing scaling

The scaling is relative the current viewport, so we need to make
sure it is correct before changing the scale setting.

Don't change stored setting when "forcing"

Don't overwrite the user's configured choice. Instead ignore
settings when they cannot be respected.

Fix status class for IE

IE apparently doesn't support removing several class specifiers at
once, so switch to several calls.

Allow scroll bars on Safari and IE

The previous problems were because we were scrolling the wrong element,
so we can enable this feature now.

Fix canvas size on IE

Properly encapsulate the scale in Display

Other parts of the code shouldn't have to care about this. Let Display
convert between canvas coordinates and framebuffer coordinates.

Set correct cursor style for setCapture()

Remove return value from event handlers

It's not used in modern event handlers.

Stop using window.event

It's an old propriatary IE thing that isn't necessary.

Remove Util.getEventPosition()

It mostly dealt with scrolling which we don't use. It also made mistakes
in some cases. Remove it and compute the coordinates directly in the
calling code.

Clean up some mouse workarounds

Adds comments that clarifies why they are needed. Also narrows the
handler for contextmenu a bit.

Improve setCapture polyfill

Fix some corner cases.

Fixes issue #773

Fix tests after changing auth negotiation

PhantomJS has a very basic implementation of Uint8Array, so we
need to help it out a bit.

Better selection of auth type

If no authentication is required then we should pick the None
option to avoid bothering the user.

Fix resize on Chrome

Chrome sucks at calculating the size and gets confused due to the
scrollbars. Hide the scroll bars while resizing.

Improve check for when to ignore mouse events

The browser tells us which element was under the cursor, so we don't
have to calculate it ourselves.

Use setCapture() polyfill everywhere

This makes sure we get consistent behaviour across all browsers.

More aggressive retaining of focus

Try to keep the virtual keyboard up even more. Only release focus
when it is absolutely necessary.

Expect console.debug(), not console.log(), in test

We use the more specific function now, even though it is usually an
alias.