The previous assignment was simpler because it was harmless. It is still harmless (a warning is a minor nuisance, but not harmful).

I would prefer to keep the logic simpler and go with

```
if (returnValue) {
    e.returnValue = returnValue;
}
```

And let the browser complain when a developer writes dubious code.