I meant that the name of the function should be consistent with the name of the property. So:

```js
onActivate: function onActivate(win) {
```

I guess your `onOpen` is a left-over.