core(non native url): fail test when url is not a native implementation

[wardpeet]

Summary

Bugfix: fixes websites overriding URL variable with something custom
When running node lighthouse-cli/index.js https://www.dailyo.in -G --verbose --devtools.throttling=provided you get the following output:

Sat, 28 Jul 2018 11:14:27 GMT method <= browser OK:verbose Runtime.evaluate {"result":{"type":"object","value":[{"err":{"message":"URL is not a constructor","stack":"TypeError: URL is not a constructor\n    at matches.forE
Sat, 28 Jul 2018 11:14:27 GMT method <= browser OK:verbose Runtime.evaluate {"result":{"type":"object","value":[{"err":{"message":"URL is not a constructor","stack":"TypeError: URL is not a constructor\n    at matches.forE

Related Issues/PRs

#5701

[devtools-bot]

Thanks chief! Appreciate you filing this bug. 👏

This is a known issue, most well described in #2784. So, we'll automatically close this as a duplicate.

However, if you believe your bug is different than the cases described there, please comment here with "necessarily-wide-alpaca" and I'll reopen this bug. 🤖 Beep beep boop.

[wardpeet]

necessarily-wide-alpaca

[paulirish]

Thanks. This fixes a pretty huge sentry bug. https://sentry.io/google-lighthouse/lighthouse/issues/495524891/?environment=production

[brendankenny]

🎉 🎉