Can we have a null entry in "this.socksSessions[channelLabel]"? If so, it's strange... no? (I know you didn't write this... so you can fix now or add a TODO :-)  ... I just added a section to our style doc: https://docs.google.com/a/google.com/document/d/1ZV70oTNWO00bs0RcMwqr3FvKJXA6smYQ2kZ8Zr8YZnw/edit#heading=h.d0uggwuleeao