Skip to content

Merge Develop into Master for Eclipse Photon 1.1.0 release #172

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 26 commits into from
Jul 23, 2018
Merged

Conversation

jpwsutton
Copy link
Contributor

No description provided.

tmulkern and others added 26 commits September 6, 2017 10:29
* Removed reference to window and change to self

Removed reference to window and change to self. This is to allow the
client to be used in Web Workers see:
https://developer.mozilla.org/en/docs/Web/API/Window/self

Signed-off-by: Tadhg Mulkern<[email protected]>

* Updated Client-hareness.js

Updated Client-hareness.js, set global.window = global; to global.self =
global;

* Update .travis.yml to fix build error

Update .travis.yml to fix build error, set nodejs version to use the
latest stable version of 5.x
* Refactored Properties using Object.defineProperties

Refactored Properties using Object.defineProperties as seen here

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/defineProperties

This reduces the javascript code by removing repeated code. This will
have a impact on overall file size.

Signed-off-by: Tadhg Mulkern<[email protected]>

* Added additional cleanup of code

Added a piece of code to removed the need for all the Typed Array
checks, base on this method

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/ArrayBuffer/isView
and from the Stackoverflow answer
https://stackoverflow.com/questions/15251879/how-to-check-if-a-variable-is-a-typed-array-in-javascript/29651223#29651223
Changed usages of push on this._buffered_msg_queue and this._msg_queue
to use unshift

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/unshift,
removing the need to reverse the array to pop elements in fifo order

Signed-off-by: Tadhg Mulkern<[email protected]>
Signed-off-by: Nico Rikken <[email protected]>
Signed-off-by: Jannik Hunziker <[email protected]>
…ts. (#153)

* Fix global object

Signed-off-by: Manuel Iglesias <[email protected]>

* Remove references to self

Signed-off-by: Manuel Iglesias <[email protected]>

* Fix arguments conversion to array

Signed-off-by: Manuel Iglesias <[email protected]>

* Use simple in-memory fake of localStorage when global one is not available

Signed-off-by: Manuel Iglesias <[email protected]>
@jpwsutton jpwsutton self-assigned this Jul 23, 2018
@jpwsutton jpwsutton merged commit 413f729 into master Jul 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants