first stab at putting protocol handlers in all processes

[the8472]

it's not quite done, but i'd like some feedback. i had to untangle the gui and configuration concerns a bit to make it work in the child process.

[lidel]

Thank you. Went over it and I am okay with the direction of your changes.
Use of supressRecursion feels bit 'dirty', but I understand why it is there.

Some cosmetic things to keep in mind:

• Refactoring broke some UI tests (run via jpm test), which need to be fixed:
  • TypeError: button is undefined
  • TypeError: gw.enableHttpGatewayRedirect is not a function
• JSHint reported some issues:
  • I don't have a strong opinion regarding singlequotes, feel free to disable it if you wish (or use singlequotes).
  • Add "esnext": true, in jshint section of Gruntfile.js to suppress errors about arrow function syntax (=>) – use of ES6 is ok.

Let me know when you feel it is ready ( if we merge this first, #29 will need to rebase :-) )

[the8472]

All tests pass now. The last jshint thing seems bogus to me since function definitions are hoisted while consts have a temporal dead zone.

[lidel]

I agree, in this particular case moving function does not improve readability.
To whitelist, just add this comment:

function toggle(val) { /*jshint ignore:line*/

[the8472]

all green now

[lidel]

👍

[lidel]

@the8472 could you please take a look at #33 ?