Looks good to me. I don't have the time to test this myself, so I'll leave this open for others to kibitz for now.
I don't think it will do harm, so if no one else expresses an opinion after a while, you can probably nudge me into pushing it. Reviewed-by: Jeremy White <jwh...@codeweavers.com> Cheers, Jeremy On 09/27/2016 10:29 AM, Oliver Gutierrez wrote: > --- > enums.js | 9 ++++++ > main.js | 3 ++ > port.js | 85 > +++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > spice.html | 10 +++++++ > spice_auto.html | 10 +++++++ > spicemsg.js | 18 ++++++++++++ > utils.js | 7 +++++ > 7 files changed, 142 insertions(+) > create mode 100644 port.js > > diff --git a/enums.js b/enums.js > index 301fea0..b6e013c 100644 > --- a/enums.js > +++ b/enums.js > @@ -166,6 +166,15 @@ var SPICE_MSG_PLAYBACK_VOLUME = 105; > var SPICE_MSG_PLAYBACK_MUTE = 106; > var SPICE_MSG_PLAYBACK_LATENCY = 107; > > +var SPICE_MSG_SPICEVMC_DATA = 101; > +var SPICE_MSG_PORT_INIT = 201; > +var SPICE_MSG_PORT_EVENT = 202; > +var SPICE_MSG_END_PORT = 203; > + > +var SPICE_MSGC_SPICEVMC_DATA = 101; > +var SPICE_MSGC_PORT_EVENT = 201; > +var SPICE_MSGC_END_PORT = 202; > + > var SPICE_PLAYBACK_CAP_CELT_0_5_1 = 0; > var SPICE_PLAYBACK_CAP_VOLUME = 1; > var SPICE_PLAYBACK_CAP_LATENCY = 2; > diff --git a/main.js b/main.js > index 874a038..2d8a1ff 100644 > --- a/main.js > +++ b/main.js > @@ -59,6 +59,7 @@ function SpiceMainConn() > this.file_xfer_tasks = {}; > this.file_xfer_task_id = 0; > this.file_xfer_read_queue = []; > + this.ports = []; > } > > SpiceMainConn.prototype = Object.create(SpiceConn.prototype); > @@ -154,6 +155,8 @@ SpiceMainConn.prototype.process_channel_message = > function(msg) > this.cursor = new SpiceCursorConn(conn); > else if (chans.channels[i].type == SPICE_CHANNEL_PLAYBACK) > this.cursor = new SpicePlaybackConn(conn); > + else if (chans.channels[i].type == SPICE_CHANNEL_PORT) > + this.ports.push(new SpicePortConn(conn)); > else > { > if (! ("extra_channels" in this)) > diff --git a/port.js b/port.js > new file mode 100644 > index 0000000..ee22073 > --- /dev/null > +++ b/port.js > @@ -0,0 +1,85 @@ > +"use strict"; > +/* > + Copyright (C) 2016 by Oliver Gutierrez <ogut...@gmail.com> > + Miroslav Chodil <mcho...@redhat.com> > + > + This file is part of spice-html5. > + > + spice-html5 is free software: you can redistribute it and/or modify > + it under the terms of the GNU Lesser General Public License as published > by > + the Free Software Foundation, either version 3 of the License, or > + (at your option) any later version. > + > + spice-html5 is distributed in the hope that it will be useful, > + but WITHOUT ANY WARRANTY; without even the implied warranty of > + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the > + GNU Lesser General Public License for more details. > + > + You should have received a copy of the GNU Lesser General Public License > + along with spice-html5. If not, see <http://www.gnu.org/licenses/>. > +*/ > + > +/*---------------------------------------------------------------------------- > +** SpicePortConn > +** Drive the Spice Port Channel > +**--------------------------------------------------------------------------*/ > +function SpicePortConn() > +{ > + DEBUG > 0 && console.log('SPICE port: created SPICE port channel. > Args:', arguments); > + SpiceConn.apply(this, arguments); > + this.port_name = null; > +} > + > +SpicePortConn.prototype = Object.create(SpiceConn.prototype); > + > +SpicePortConn.prototype.process_channel_message = function(msg) > +{ > + if (msg.type == SPICE_MSG_PORT_INIT) > + { > + if (this.port_name === null) > + { > + var m = new SpiceMsgPortInit(msg.data); > + this.portName = arraybuffer_to_str(new Uint8Array(m.name)); > + this.portOpened = m.opened > + DEBUG > 0 && console.log('SPICE port: Port', this.portName, > 'initialized'); > + return true; > + } > + > + DEBUG > 0 && console.log('SPICE port: Port', this.port_name, 'is > already initialized.'); > + } > + else if (msg.type == SPICE_MSG_PORT_EVENT) > + { > + DEBUG > 0 && console.log('SPICE port: Port event received for', > this.portName, msg); > + var event = new CustomEvent('spice-port-event', { > + detail: { > + channel: this, > + spiceEvent: new Uint8Array(msg.data) > + }, > + bubbles: true, > + cancelable: true > + }); > + > + window.dispatchEvent(event); > + return true; > + } > + else if (msg.type == SPICE_MSG_SPICEVMC_DATA) > + { > + DEBUG > 0 && console.log('SPICE port: Data received in port', > this.portName, msg); > + var event = new CustomEvent('spice-port-data', { > + detail: { > + channel: this, > + data: msg.data > + }, > + bubbles: true, > + cancelable: true > + }); > + window.dispatchEvent(event); > + return true; > + } > + else > + { > + DEBUG > 0 && console.log('SPICE port: SPICE message type not > recognized:', msg) > + } > + > + return false; > +}; > diff --git a/spice.html b/spice.html > index c473678..f736c59 100644 > --- a/spice.html > +++ b/spice.html > @@ -42,6 +42,7 @@ > <script src="wire.js"></script> > <script src="spiceconn.js"></script> > <script src="display.js"></script> > + <script src="port.js"></script> > <script src="main.js"></script> > <script src="inputs.js"></script> > <script src="webm.js"></script> > @@ -142,6 +143,15 @@ > } > } > > + window.addEventListener('spice-port-data', function(event) { > + var msg_text = arraybuffer_to_str(new > Uint8Array(event.detail.data)); > + console.log('SPICE port', event.detail.channel.portName, > 'message text:', msg_text); > + }); > + > + window.addEventListener('spice-port-event', function(event) { > + console.log('SPICE port', event.detail.channel.portName, > 'event data:', event.detail.spiceEvent); > + }); > + > </script> > > </head> > diff --git a/spice_auto.html b/spice_auto.html > index 1179ebe..304655c 100644 > --- a/spice_auto.html > +++ b/spice_auto.html > @@ -42,6 +42,7 @@ > <script src="wire.js"></script> > <script src="spiceconn.js"></script> > <script src="display.js"></script> > + <script src="port.js"></script> > <script src="main.js"></script> > <script src="inputs.js"></script> > <script src="webm.js"></script> > @@ -182,6 +183,15 @@ > } > } > > + window.addEventListener('spice-port-data', function(event) { > + var msg_text = arraybuffer_to_str(new > Uint8Array(event.detail.data)); > + console.log('SPICE port', event.detail.channel.portName, > 'message text:', msg_text); > + }); > + > + window.addEventListener('spice-port-event', function(event) { > + console.log('SPICE port', event.detail.channel.portName, > 'event data:', event.detail.spiceEvent); > + }); > + > connect(); > </script> > > diff --git a/spicemsg.js b/spicemsg.js > index 0321cc7..3619996 100644 > --- a/spicemsg.js > +++ b/spicemsg.js > @@ -1278,3 +1278,21 @@ SpiceMsgDisplayInvalList.prototype = > } > }, > } > + > +function SpiceMsgPortInit(a, at) > +{ > + this.from_buffer(a,at); > +}; > + > +SpiceMsgPortInit.prototype = > +{ > + from_buffer: function (a, at) > + { > + at = at || 0; > + var dv = new SpiceDataView(a); > + var namesize = dv.getUint32(at, true); at += 4; > + var offset = dv.getUint32(at, true); at += 4; > + this.opened = dv.getUint8(at, true); at += 1; > + this.name = a.slice(offset, offset + namesize - 1); > + } > +} > diff --git a/utils.js b/utils.js > index 9093a24..a22d0ae 100644 > --- a/utils.js > +++ b/utils.js > @@ -100,6 +100,13 @@ function hexdump_buffer(a) > } > > > /*---------------------------------------------------------------------------- > +** Convert arraybuffer to string > +**--------------------------------------------------------------------------*/ > +function arraybuffer_to_str(buf) { > + return String.fromCharCode.apply(null, new Uint16Array(buf)); > +} > + > +/*---------------------------------------------------------------------------- > ** Converting keycodes to AT scancodes is very hard. > ** luckly there are some resources on the web and in the Xorg driver that > help > ** us figure out what browser dependent keycodes match to what scancodes. > _______________________________________________ Spice-devel mailing list Spice-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/spice-devel