On Wed, May 16, 2012 at 7:44 AM, Benoit Chesneau <[email protected]> wrote:
> On Wed, May 16, 2012 at 7:42 AM, Paul Davis <[email protected]>
> wrote:
>> That JS test looks broken for JS environments that don't support
>> EventSource. Notably the CLI test suite but also browsers that don't
>> implement it. I'd either move the entire test into the conditional or
>> reimplement it by parsing the actual data sent across.
>>
>
>
> well there is an if() . Isn't it enough ?
>
> - benoit
i've changed the test from `if (window.EventSource) {` to `if
(!!window.EventSource) {` so every browser should be happy.
(I blame js to have a weird syntax)
- benoit
