regl-project / regl

👑 Functional WebGL
https://regl-project.github.io/
MIT License
5.22k stars 322 forks source link

Cross browser testing #143

Open mikolalysenko opened 8 years ago

mikolalysenko commented 8 years ago

We need to run regl on some non-v8/ANGLE based systems to make sure everything is still working. At least the following systems should all pass:

Maybe browserling would work?

Erkaman commented 8 years ago

We should perhaps also support Edge. Some guy on Reddit said that he had been able to run every single shader he'd come across in Edge. But he couldn't run the shadow volumes demo in Edge, for some reason.

mikolalysenko commented 8 years ago

We could try using Sauce Labs. It does CI for cross browser testing.

gregtatum commented 7 years ago

Debugger.html is using some interesting Firefox-based integration testing. Figured I'd drop it in this issue in case it's helpful. I haven't done as much cross-browser testing, so there could be some other better solutions out there.

danmarshall commented 6 years ago

I was not able to run the examples from http://regl.party/examples in Edge. Just got a black screen.

gregtatum commented 6 years ago

@danmarshall could you open the console and provide the error message if there is one?

danmarshall commented 6 years ago

@gregtatum yes I did that - no errors are reported 😬

danmarshall commented 6 years ago

@gregtatum - Whoops sorry, there are errors:

in bundle.js line 34994 this.db.createObjectStore(this.storeName, { keyPath: this.keyPath, autoIncrement: this.autoIncrement});

throws InvalidAccessError

dy commented 6 years ago

479 makes tests pass in all mentioned browsers, although it takes splitting batches, since not all browsers are comfortable handling 40k assertions.

As for CI solutions I can try to negotiate browserstack to provide an open-source account.

rreusser commented 6 years ago

@danmarshall I think the above traces through browser-module-sandbox, to browser-module-cache, and finally to level.js. In other words, it's the frame that lets you edit code and require node modules rather than an issue with regl. It'd be nice to fix it so that people don't assume regl fails.

dy commented 6 years ago

https://github.com/plotly/plotly.js/pull/2612#issuecomment-391121414