They were relying on the Query.parse interface, which was removed
separately.

https://bugs.torproject.org/31126#comment:5

So that you can click on the text as well as the slider.

There were two problems having to do with looking up in the UNITS
object. (1) It was checking for undefined keys by comparing to null,
rather than undefined. (2) It was finding Object.prototype keys like
"toString".

This standard interface does the same thing as Query.parse did, with a
different API on the return value. It doesn't have the problems with
keys inherited from Object.prototype that Query.parse did.

It was formerly used for the "broker" and "relay" query string
parameters, which were removed in
debe9c37.

This is now necessary since the webextension changed to default to on.

Follow up to 095f4a05

From trac 31067

In Firefox, leaving out the STUN port caused connections to the STUN
server to fail

Some files were omitted in the .eslintignore, left as an exercise to the
reader.

We probably want to reduce amount of globals overall and use proper es
modules.

Requires at least node v8.x

This was probably meant to be `@relay = null` is the coffeescript.

No idea why but, with it, when a client disconnects the datachannel
doesn't close.  This wasn't an issue prior to 359600dd since the
configuration wasn't being passed to the peerconnection constructor.

This probably just makes the url invalid and falls back to a default in
libwebrtc.

I guess alternatively, just use an if/else block.

With,

  ./node_modules/.bin/coffee -b -c Cakefile `find . -path ./node_modules -prune -o -name '*.coffee'`

After auditing for breaking changes [0] so that the output produced uses
ES2015 classes.

[0] https://coffeescript.org/#breaking-changes)

But note that maybe we should be constructing a new PCConfig each time
Config is instantiated, rather than sharing it on the prototype.

From https://coffeescript.org/#breaking-changes-super-this

The webextension icons were missing in the Mozilla addons upload. We
need a newer version to update it.

Messed up the git tag so just going for v0.0.3

So that it compiles to,

```
if (!(name in result)) {
  result[name] = value;
}
```

instead of,

```
if (!(indexOf.call(result, name) >= 0)) {
  result[name] = value;
}
```

since `result` is an object, not an array.

Trac 31067

Prevents repeat calls to update() on startup

Now that we're setting the active attribute of pair to true before
accepting a client offer, we should be careful to reset it to false if
receiving the offer fails