Merge pull request #640 from nunojusto/patch-1

Update launch.sh

Update launch.sh

Just a correction of port in use test algoritm.
This way we will not have problems when using port X and having some other service using zyX or any *X port

Release 0.6.1

Fixes version number mismatch (also adds more detailed instructions
for releasing).

Fixes #628

Release 0.6.0

This bumps the release version to 0.6.0

Fix missing mistyped setTimeout handler in UI

This commit fixes a mistyped setTimeout handler that would result
in the desired function not getting called (due to a missing set
of parentheses).  It also removes some uncessary anonymous functions,
and just passes the function objects directly to setTimeout.

Clarify that utils/launch.sh is MPL-2.0

Also, note in the top-level license file that the default noVNC
license for files that are not explicitly marked or mentioned in
the LICENSE.txt file are by default MPL-2.0 licensed.

Proper error handling for tight filters

Don't throw an exception when we encounter an unsupported tight
subencoding.

Update vnc_auto after the screen namechange

Commit 553864e85813f886b9c60b4825c8d4b714cae695 changed the name of
noVNC_screen to noVNC_container, vnc_auto.html was not updated
accordingly. Fixes #621

Merge branch 'ui-cleanup' of https://github.com/kanaka/noVNC

Split the setDesktopSize function (#618)

In order to follow the surrounding coding-standards, the
setDesktopSize client message is split from the public function which
now is called requestDesktopSize().

Consistent closing tags for void elements

Use a consistent naming convention for elements

* Element names we use "_" as word-delimiter, not "-"
* Element names use less camel-case
* Element names end with the type
* Element names always start with noVNC_

Change the names of settings-elements

This change was made in order to make it easier to distinguish
settings-elements from other elements. One example of the problem
that was solved is the two elements "noVNC_clip" and
"noVNC_clipboard" where the first is the setting for clipping mode.
That element was now renamed to "noVNC_setting_clip".

Timeouts should always use anonymous functions

While you CAN pass functions in strings, it is not the correct way
of doing it. The browsers are just being nice.

Change name of clipboard functions

There were a high risk of confusion between clipboard functions and
clipping functions.

Group together related functions and add dividers

Dividers between the different parts of the UI has been added in
order to easier get an overview.

Send the xvp keys from anonymous functions

Renamed and moved keyboard and mouse functions

Renamed and moved updateXvpButton

Move the connect and disconnect functions

Clarify comments and variable names for viewDrag

Renamed showExtraKeys to toggleExtraKeys

Moved and split the popupStatus function

Now if popupStatus is called twice it will refresh the duration of
the popup. If you want to wait 1.5 seconds and close the popup you
can click the popup itself.

Moved and changed order of the setting-functions

Remove whitespace from function definitions

Moved UpdateState and UpdateVisualState

Cleanup for the resize related functions

Renamed functions, added clarifying comments and moved the resize
related functions closer to the other viewport functions.

Switch names between the container and the screen

The noVNC_container now contains the logo and the screen. While the
noVNC_screen in turn contains the canvas.

Add fallback value for devicePixelRatio

In IE 10 for example, devicePixelRatio doesn't exist which caused the
code to fail by setting the thresholds to zero.

Add missing parenthesis (#600)

Respect the threshold in the viewdrag test (#600)

Merge branch 'SirCmpwn-fix-send-on-disconnect'

Updated patch to use the readyState constant even in the tests.

Fix failing test

It only makes sense to send data on a websocket if the readyState is
equal to 1.

Fix occasional error with sending while readyState !== 1

Under certain conditions, noVNC would attempt to flush the web socket
while it was disconnected, before the disconnected state was picked up.
This casues noVNC to crash ungracefully and the parent window is not
notified - leading to no chance at recovery without a page refresh.

Add a threshold for viewport dragging (#600)

Adds tap-to-click in viewport drag mode (#600)

Force clipping mode in Safari on all platforms

Re-arrange global UI variables

Merge pull request #602 from oneru/typo-fix

Adds missed line from PR 597

Adds missed line from PR 597

Merge pull request #597 from oneru/master

Handle missing leading slash in elem.pathname

Handle missing leading slash in elem.pathname

IE11 with compat mode turned off has been observed displaying this behavior.  This commit checks for and corrects this broken behavior.

Fixes #591

Merge pull request #582 from kanaka/bug/render-queue-corruption

Make sure to copy arrays when using render queue

Make sure to copy arrays when using render queue

This commit ensures that input arrays are copied to new storage when
they are pushed onto the render queue.  This ensures that they are
not overwritten before they are eventually used.

Fixes #571

Merge pull request #568 from kanaka/refactor/remove-refs-to-jsunzip

Cleanup/fixup unused code

Remove jsunzip, remove unused vars, fix vnc_perf.html

Merge branch 'feature/token-in-url'

Injects token into the path element if it's passed to
the noVNC page.

Closes #537
Fixes #536

Pass token into the path variable

If a token is already present in the path, the new variable
is ignored.  In order to properly manipulate the path,
a new method, `WebUtil.injectParamIfMissing` was introduced.

Fixes #536

[@directxman12: fix up path manipulation logic]

Merge pull request #547 from pigshell/hashargs

Add hash fragment as an optional method to supply config variables.

Closes #544

Fixed vnc_perf.html

This commit updates vnc_perf.html so that it works with the current
version of noVNC.  It also introduces a utility to convert noVNC session
recordings recorded in base64-mode to binary-mode recordings, since noVNC
no longer supports base64-mode.

Fixes #479.

Remove unused variables in the TIGHT server init

This commit removes some unused variables in the TIGHT server init
section of the server init handler, replacing them with documenting
comments and calls to rQskipBytes.

Fixes #440.

Remove remaining references to jsunzip.js

This commit removes remaining references to jsunzip.js,
which is no longer actually used in noVNC.

Closes #436.

Merge pull request #565 from kanaka/bug/dynamic-rq-resize

Resize Receive Queue to Fit Incoming Messages

On-Demand Dynamic Receive Queue Resizing

This commit causes the receive queue to dynamically
resize to fit incoming messages.

Fixes #557

Disable copyWithin Use in Websock.js

the `copyWithin` function of typed arrays has performance issues
in some versions of Chromium, and doesn't doesn't have enough
of a performance impact to justify leaving it enabled.

Merge pull request #555 from maditnerd/patch-1

Small typo in launch.sh

Small typo in launch.sh

Add hash fragment as an optional method to supply config variables.

Any config variable like host, port, password, token may be
specified either in the query string (like now), or in the URL hash
fragment. In case a given variable is present in both, the value in the
fragment takes precedence. Supplying variables in the fragment avoids
leaking them to the web server hosting the noVNC viewer HTML.

Merge pull request #543 from DvdGiessen/fix/vnc_sendpassword_auth

Fixes VNC standard auth using onPasswordRequired callback

Added return to correctly wait for password input before proceeding. Fixes #542

Merge pull request #532 from kanaka/bug/pako-issue

Make sure Pako always has enough room

Make sure Pako always has enough room

Previously, we used a fixed chunkSize of 100KiB for Pako's output
buffer.  Using a hardcoded size caused issues, since Pako would assume
we wanted to use multiple chunks, and we didn't deal with this.  Now,
`Inflator#inflate()` takes a new `expected` argument, which indicates
the expected output size.  If this is bigger than the current chunkSize,
Inflator allocates a new output buffer that's big enough to hold the
output.

Fixes #531

Moved resizeTimout to the UI object, no reason to have it outside anymore..

Merge pull request #524 from kanaka/bug/tight-buffer-overread

Fix buffer over-reads in handle_tight

Fix buffer over-reads in handle_tight

For performance reasons, the `handle_tight` function skips the
use of the receive queue API and uses the raw receive queue directly.
Because of the way that typed array receive queue gets reused, this
introduced the potential for buffer over-reads.

To address this, a new function, `rQwhole`, was introduced.
`rQwhole` simply returns a new view into the receive queue that
starts at 0 and ends at the current recorded end of the queue.

`handle_tight` now makes use of this function.

Fixes #522

Fix bug in non-true-color code

There was a bug caused by 38781d931ec18304f51ed3469faff8387e3cbc55
which prevented color map look-ups sent by rfb.js from working properly,
since display.js expected a single-item array, and rfb.js sent just them
item value itself (a number) instead.  This fixes that, and tweaks the
corresponding test to match that behavior.

Tests: Fixed bug in displayed assertion

This `displayed` assertion had a bug that was causing it to not
actually check anything (it was using obj.length instead of
data_cl.length).  This fixes that.

UI: Fix typo preventing reconnect after password

There was a typo in ui.js preventing the use of the connect button
after a password was entered.

Merge pull request #488 from kanaka/feature/more-perf-improvements

Performance Improvements

Only work with integers when panning to avoid getting a blurry image.
Also disable image-smoothing to avoid bugs seen on Android which were
also causing a blurry image while panning.

Skip unnecessary render queue object creation

This commit skips object creation for the render queue when not
needed.  Instead of pushing an object onto the queue, and then
immediately running the result, you call the function directly.
Then, if the render queue is not empty, an object is created and
pushed onto the queue.  Otherwise, the functionality is just run
directly.

Fix multi-line assertion messages in test runner

This commit prevents multi-line error messages from being
truncated in the local test runner ('tests/run_from_console.js').

Allow the use of the PhantomJS remote debugger

This commit adds the '--debugger <port>' option, which triggers
the PhantomJS remote debugger.  The initial output of the terminal
when running the debugger gives more information on how to use
it.

Use Typed Arrays for the send queue

This commit converts the send queue to use typed arrays, and converts
message creation functions in 'rfb.js' to create messages directly into
the socket's send queue.  This commit also removes the separate mouse array,
which is no longer needed.

Avoid Creating Small Objects Frequently

Creating lots of small objects frequently can drastically decrease
performance.  This commit introduces three fixes which avoid this:

- Use a preallocated palette and indexed-to-rgb destination Typed Array
  (the destination typed array is currently allocated at `4 * width *
  height`).

- Inline `getTightCLength`, which returned a two-item array.

- Pass RGBX data directly in a Typed Array to the Display, which
  avoids an extra loop, and only creates a new Typed Array View,
  instead of a whole new ArrayBuffer.

Use Typed Arrays for the Websock receive queue

**This commit removes Base64 (and Flash) support**

This commit converts websock.js to used Typed Arrays for the
receive queue (and tweaks rfb.js to ensure that it continues
to function, since only Firefox implements
`%TypedArray%.prototype.slice`).  Base64 support was removed
to simplify code paths, and pave the way for using Typed Arrays
for the send queue as well.

This provides two advantages: first, we allocate a buffer ahead
of time, meaning the browser doesn't have to do any work dynamically
increasing the receive queue size.  Secondly, we are now able to pass
around Typed Array Views (e.g. `Uint8Array`), which are lightweight, and
don't involve copying.

The downside is that we initially allocate more memory -- we currently
start out with 4 MiB, and then automatically double when it looks like
the amount unused is getting to small.

The commit also explicitly adds a check to the compacting logic that
avoids calling the copy functions if `_rQlen === _rQi`.

Remove unecessarily nested loop

This commit removes a nested loop in indexedToRGB, converting it into a
plain loop without multiplication.

WIP: Switch to Pako for zlib

This commit introduces an alternate implementation of the zlib
decompressor based on Pako (https://github.com/nodeca/pako).

Fix RFB.js JSHint Errors

This fixes a couple of JSHint errors in RFB.js caused
by using `==` instead of `===`.

Fix up vnc_playback.html and playback.js

This commit fixes vnc_playback.html and playback.js
so that they work with the current version of noVNC.

CI: Switch to Container-Based Travis

This commit switches the Travis tests over to Travis's new
container-based infrastructure.  It also tells Travis to
cache the node_modules directory for faster setup.

Merge pull request #499 from samhed/fullscreen

Add option to toggle fullscreen mode

* Hide the fullscreen toggle button on Safari since alphanumerical
  keyboard input doesn't work in fullscreen.
* Force clipping mode in Internet Explorer while in fullscreen since
  scrollbars doesn't work in fullscreen.

Adapt display to be able to handle a changing clipping-setting while connected

Fixes #498 - Add the ability to toggle fullscreen mode

Fix error from a previous commit, there is no data type called 'text'..

Renamed popupStatusPanel to popupStatus and removed the global variable
which kept track on if it was open or not (it automatically closes after
1.5 seconds anyway).

Clarified enableDisableClip (now called enableDisableViewClip) and
moved it to where the other clipping functions are.

Allow the popupStatusPanel to show any text but close it on a 1.5 second timer.

Clarify code with regards to the viewport drag functionality
* Fixes #502 so that the viewport drag functionality can't get stuck

Always show the viewport drag button on touch devices to keep the GUI from
"jumping around". Enable/disable the button instead of show/hide on these
devices.

Merge pull request #497 from MOZGIII/MOZGIII-patch-vnc-auto-fix

Fixed incorrect UI usage and minor updateState params errors in vnc_auto.html

Fixed incorrect UI usage and minor updateState params errors in vnc_auto.html

Reverting most of commit cb3e4de. The issue was not related to using keyboardinputReset.. the issue was infact a typo which was introduced in 3b8ec46. I must be too tired..

Stop using keyboardinputReset at the start of keyInput
* This fixes backspace when using the on-screen keyboard on Android

Update LICENSE.txt

Removed old no longer existing vnc.js and added playback.js

Merge pull request #483 from fabian-z/master

Make noVNC compatible with Content Security Policies

Move #keyboardinput declarations to base.css/ui.js

Re-fixes #428 which was broken by commit 58ded70
* Disable local cursor when the browser doesn't support data uri

Merge pull request #415 from jimdigriz/fixes

slip in PATH for 'npm test' as not everyone has karma

Merge pull request #474 from kanaka/bug/throw-error-from-constructor

Throw exceptions from RFB constructor