Commit d235cb6
authored
Fix toneMapping reference (#455)
AFAICT, the spec requires toneMapping period. Even if the browser
doesn't support toneMapping.mode = 'extended' it requires it still
requires it to exist. It will just be set to `'standard'` if the
implementation does not support `'extended'`.
The only other case is when the canvas has not been configured
in which case `getConfiguration` returns undefined.
Both `getConfiguration` and `toneMapping` were added to the spec
close enough that any implemetation that supports `getConfiguration`
should be requried to include `toneMapping`.1 parent 16dd65c commit d235cb6
1 file changed
Lines changed: 1 addition & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
355 | 355 | | |
356 | 356 | | |
357 | 357 | | |
358 | | - | |
| 358 | + | |
359 | 359 | | |
360 | 360 | | |
361 | 361 | | |
| |||
0 commit comments