1. 08 Sep, 2010 2 commits
    • Joel Martin's avatar
      default_controls.js: Fix cursor setting init. · b5087acc
      Joel Martin authored
      The rfb variable wasn't available at the point settingsDisabled() was
      being called since it was called inline with RFB() initialization. To
      solve this we pass the updateState rfb variable so that the canvas can
      be queried for setting the cursor_uri value.
      b5087acc
    • Joel Martin's avatar
      JSLint. · d595e656
      Joel Martin authored
      d595e656
  2. 01 Sep, 2010 1 commit
  3. 31 Aug, 2010 2 commits
  4. 30 Aug, 2010 3 commits
  5. 27 Aug, 2010 3 commits
  6. 26 Aug, 2010 1 commit
    • Joel Martin's avatar
      Indexed receive queue. Up to 2X speedup in Chrome. · 67b4e987
      Joel Martin authored
      Generally, most servers send hextile updates as single updates
      containing many rects. Some servers send hextile updates as many small
      framebuffer updates with a few rects each (such as QEMU). This latter
      cases revealed that shifting off the beginning of the receive queue
      (which happens after each hextile FBU) performs poorly.
      
      This change switches to using an indexed receive queue (instead of
      actually shifting off the array). When the receive queue has grown to
      a certain size, then it is compacted all at once.
      
      The code is not as clean, but this change results in more than 2X
      speedup under Chrome for the pessimal case and 10-20% in firefox.
      67b4e987
  7. 11 Aug, 2010 1 commit
    • Joel Martin's avatar
      Treat RFB 3.6 as 3.3. · 1a5dd77d
      Joel Martin authored
      Apparently there are versions of UltraVNC that report version 3.6.
      This is not a legal version according to the spec, but we'll just
      force version 3.3 if we receive it. Thanks to Larry Rowe for the info.
      1a5dd77d
  8. 06 Aug, 2010 1 commit
    • Joel Martin's avatar
      Scroll render test and perf speedup. · 4ed717ad
      Joel Martin authored
      Turns out when Windows is running in QEMU and a window scroll happens,
      there are lots of little hextile rects sent. This is slow in noVNC.
      
      - Some recording/playback improvement.
      - Add test harness to drive playback of recordings.
      - By pulling off the rect header in one chunk we get a 3X speedup in
        Chrome and a 20% speedup in firefox (specifically for the scroll
        test).
      - Also, get rid of some noise from creating timers for handle_message.
        Check to make sure there isn't already a pending timer first.
      4ed717ad
  9. 04 Aug, 2010 3 commits
  10. 03 Aug, 2010 1 commit
  11. 02 Aug, 2010 2 commits
    • Joel Martin's avatar
      New API. Refactor Canvas and RFB objects. · 8db09746
      Joel Martin authored
      New API:
      
      To use the RFB object, you now must instantiate it (this allows more
      than one instance of it on the same page).
      
          rfb = new RFB(settings);
      
      The 'settings' variable is a namespace that contains initial default
      settings. These can also be set and read using 'rfb.set_FOO()' and
      'rfb.get_FOO()' where FOO is the setting name. The current settings
      are (and defaults) are:
          - target: the DOM Canvas element to use ('VNC_canvas').
          - encrypt: whether to encrypt the connection (false)
          - true_color: true_color or palette (true)
          - b64encode: base64 encode the WebSockets data (true)
          - local_cursor: use local cursor rendering (true if supported)
          - connectTimeout: milliseconds to wait for connect (2000)
          - updateState: callback when RFB state changes (none)
          - clipboardReceive: callback when clipboard data received (none)
      
      The parameters to the updateState callback have also changed. The
      function spec is now updateState(rfb, state, oldstate, msg):
          - rfb: the RFB object that this state change is for.
          - state: the new state
          - oldstate: the previous state
          - msg: a message associate with the state (not always set).
      
      The clipboardReceive spec is clipboardReceive(rfb, text):
          - rfb: the RFB object that this text is from.
          - text: the clipboard text received.
      
      Changes:
      
      - The RFB and Canvas namespaces are now more proper objects. Private
        implementation is no longer exposed and the public API has been made
        explicit. Also, instantiation allows more than one VNC connection
        on the same page (to complete this, DefaultControls will also need
        this same refactoring).
      
      - Added 'none' logging level.
      
      - Removed automatic stylesheet selection workaround in util.js and
        move it to defaultcontrols so that it doesn't interfere with
        intergration.
      
      - Also, some major JSLinting.
      
      - Fix input, canvas, and cursor tests to work with new model.
      8db09746
    • Joel Martin's avatar
      Catch exceptions in cursor detection. · 8171f4d8
      Joel Martin authored
      If cursor Data URI scheme detection threw an exception, it would cause
      canvas initialization to fail. cursor detection exceptions should just
      disable local cursor change support, not cause canvas init to fail.
      8171f4d8
  12. 30 Jul, 2010 1 commit
    • Joel Martin's avatar
      Preliminary scaling code (deactivated). · 125d8bbb
      Joel Martin authored
      Uses the CSS "scale()" operation. The main problem is that the DOM
      container is not rescaled, only the size of the displayed content
      within it so there will need to be some sort of mechanism to handle
      this better so other elements reflow to the new size. Or it might just
      not work and be removed later. The zoom property seems to do the right
      behavior, but it's not widely supported. Worth exploring though.
      125d8bbb
  13. 29 Jul, 2010 1 commit
    • Joel Martin's avatar
      Give other events chance to fire. · 29ad96c5
      Joel Martin authored
      After each complete framebufferUpdate, set a short timer to continue
      processing the receive queue. This gives other events a chance to
      fire. Especially important when noVNC is integrated into another
      website.
      29ad96c5
  14. 23 Jul, 2010 1 commit
    • Joel Martin's avatar
      Issue #15: noVNC falls behind. · 42b2246c
      Joel Martin authored
      noVNC was never processing more than one framebufferUpdate message per
      onmessage event. If noVNC receives an incomplete framebufferUpdate and
      then receives the rest of the framebufferUpdate plus another complete
      framebufferUpdate, then it will fall permanently behind.
      
      If there is more to process after a completed framebufferUpdate, then
      execute normal_msg again.
      
      All the render routines must return false if there is not enough data
      in the receive queue to process their current update, and true
      otherwise.
      42b2246c
  15. 22 Jul, 2010 5 commits
    • Joel Martin's avatar
      Query string cleanup and move it to util.js · a8edf9d8
      Joel Martin authored
      a8edf9d8
    • Joel Martin's avatar
      JSLint. RFB obj to rfb.js; vnc.js the 'loader'. · a7a89626
      Joel Martin authored
      Move the whole RFB object to rfb.js. vnc.js is now just the loader
      file. This allows an integrating project to easily replace vnc.js with
      an alternate loader mechanism (or just do it directly in the html
      file). Thanks for the idea primalmotion (http://github.com/primalmotion).
      
      Also, JSLint the various files.
      a7a89626
    • Joel Martin's avatar
      19463eac
    • Joel Martin's avatar
      f7f69d8e
    • Joel Martin's avatar
      API changes. Client cursor and settings menu. · da6dd893
      Joel Martin authored
      The following API changes may affect integrators:
      
          - Settings have been moved out of the RFB.connect() call. Each
            setting now has it's own setter function: setEncrypt, setBase64,
            setTrueColor, setCursor.
      
          - Encrypt and cursor settings now default to on.
      
          - CSS changes:
              - VNC_status_bar for input buttons switched to a element class.
      
              - VNC_buttons split into VNC_buttons_right and
                VNC_buttons_left
      
              - New id styles for VNC_settings_menu and VNC_setting
      
      Note: the encrypt, true_color and cursor, logging setting can all be
        set on load using query string variables (in addition to host, port
        and password).
      
      Client cursor (cursor pseudo-encoding) support has been polished and
      activated.
      
      The RFB settings are now presented as radio button list items in
      a drop-down "Settings" menu when using the default controls.
      
      Also, in the settings menu is the ability to select between alternate
      style-sheets.
      
      Cookie and stylesheet selection support added to util.js.
      da6dd893
  16. 20 Jul, 2010 2 commits
    • Joel Martin's avatar
      Fail if initial server connection fails. · f55b6b41
      Joel Martin authored
      f55b6b41
    • Joel Martin's avatar
      Add Cursor pseudo-encoding support (disabled for now). · 2c2b492c
      Joel Martin authored
      To change the appearance of the cursor, we use the CSS cursor style
      and set the url to a data URI scheme. The image data sent via the
      cursor pseudo-encoding has to be encoded to a CUR format file before
      being used in the data URI.
      
      During Canvas initialization we try and set a simple cursor to see if
      the browser has support. Opera is missing support for data URI scheme
      in cursor URLs.
      
      Disabled for now until we have a better way of specifying settings
      overall (too many settings for control bar now).
      2c2b492c
  17. 16 Jul, 2010 1 commit
    • Joel Martin's avatar
      State machine refactoring. · f00b1e37
      Joel Martin authored
      Add new states 'loaded', 'connect' and 'fatal':
      - Loaded state is first page state. Pass WebSockets mode message using
        this state.
      - Connect indicates that the user has issued a "connect" but we
        haven't gotten an WebSockets onopen yet.
      - Fatal is a condition that indicates inability to continue on: right
        now, lack of WebSockets/Flash or non-working canvas.
      
      Move much of the actual state transition code into updateState.
      
      Handle 'password' state better in default_controls.js; instead of
      disconnecting, prompt for password to send.
      
      Add comments to updateState indicating possible states.
      f00b1e37
  18. 14 Jul, 2010 1 commit
  19. 13 Jul, 2010 1 commit
  20. 06 Jul, 2010 2 commits
    • Joel Martin's avatar
      Remove unneeded mootools and FABridge test. · 447cd4ad
      Joel Martin authored
      mootools is no longer needed. The bug that the FABridge test was
      testing has been resolved in web-socket-js so it's no longer needed.
      447cd4ad
    • Joel Martin's avatar
      Refactor console logging code. · 81e5adaf
      Joel Martin authored
      Util.Debug, Util.Info, Util.Warn, Util.Error routines instead of
      direct calls to console.*. Add "logging=XXX" query variable that sets
      the logging level (default is "warn").
      
      Logging values:
          debug: code debug logging (many calls in performance path are also
                 commented for performance reasons).
          info: informative messages including timing information.
          warn: significant events
          error: something has gone wrong
      81e5adaf
  21. 05 Jul, 2010 1 commit
  22. 02 Jul, 2010 4 commits