Nit: a multi-line statement like this is usually formatted thusly:

```
  let iframe = shell.contentBrowser.contentDocument.
               querySelector("iframe[data-frame-origin='" + origin + "']");
```

Also, doesn't origin need to be escaped in some fashion when used in a querySelector call like this one?