Fix tunnel issue

[mcampa]

Apparently BrowserSync tries to refresh the browser before bs.io is defined causing the following issues:

• Local Tunnel error: Cannot read property 'sockets' of undefined #852
• Tunnel Issue on reload() #978

The issue happens when option tunnel: true

bs.init({
    open: false,
    port: 3000,
    files:
    [ '/Users/mario.campa/stencil/templates',
        '/Users/mario.campa/stencil/lang' ],
    watchOptions:
    { ignoreInitial: true,
        ignored:
        [ '/Users/mario.campa/stencil/assets/scss',
            '/Users/mario.campa/stencil/assets/less',
            '/Users/mario.campa/stencil/assets/css',
            '/Users/mario.campa/stencil/assets/dist' ] },
    proxy: 'localhost:3001',
    tunnel: true,
});

/usr/local/lib/node_modules/browser-sync/node_modules/rx/dist/rx.js:77
    throw e;
    ^

TypeError: Cannot read property 'sockets' of undefined
    at EventEmitter.file:reload (/usr/local/lib/node_modules/browser-sync/lib/internal-events.js:17:18)
    at emitOne (events.js:101:20)
    at EventEmitter.emit (events.js:188:7)
    at /usr/local/lib/node_modules/browser-sync/lib/internal-events.js:100:35
    at Array.forEach (native)
    at AnonymousObserver._onNext (/usr/local/lib/node_modules/browser-sync/lib/internal-events.js:98:25)
    at AnonymousObserver.Rx.AnonymousObserver.AnonymousObserver.next (/usr/local/lib/node_modules/browser-sync/node_modules/rx/dist/rx.js:1828:12)
    at AnonymousObserver.Rx.internals.AbstractObserver.AbstractObserver.onNext (/usr/local/lib/node_modules/browser-sync/node_modules/rx/dist/rx.js:1762:31)
    at AnonymousObserver.tryCatcher (/usr/local/lib/node_modules/browser-sync/node_modules/rx/dist/rx.js:63:31)
    at AutoDetachObserverPrototype.next (/usr/local/lib/node_modules/browser-sync/node_modules/rx/dist/rx.js:5883:51)

This fixes the issue and the tunnel is started with no issues

[coveralls]

Coverage decreased (-0.003%) to 95.4% when pulling c49ca6c on mcampa:tunnelBug into cbe2ccb on BrowserSync:master.

[mcampa]

anyone wants to comment on this issue?

[shakyShane]

@mcampa looks great thanks! I just need to create a test-case for it :)

[junedkazi]

Do we have any update on this PR ?

[junedkazi]

@shakyShane any updates on this PR ?

[bookernath]

@shakyShane just wanted to follow up on this - we'd love to move back to using canonical browser-sync and this is our only reason for maintaining a fork today