Issue 6913 Some test cases fail on Microsoft Edge

-removed "Extracting hostname from URL" test
-moved "Extracting hostname from URL with auth credentials" to "URL/host tools"
-skipped the failing test cases due to bugs in Edge on this platform

[geo, 2018-09-06 10:24:10]

https://codereview.adblockplus.org/29876558/diff/29876559/qunit/tests/url.js
File qunit/tests/url.js (right):

https://codereview.adblockplus.org/29876558/diff/29876559/qunit/tests/url.js#...
qunit/tests/url.js:21: const {extractHostFromFrame,
I know that this change is not related to this particular issue, but it's pretty
small, and I would have had to either use `let` for the platform import to be
consistent with this module, or `const` to be consistent with the rest of the
code. So I choose to be consistent with the rest of the code and updated this
import as well.

[Sebastian Noack, 2018-09-06 13:43:21]

https://codereview.adblockplus.org/29876558/diff/29876559/qunit/tests/url.js
File qunit/tests/url.js (right):

https://codereview.adblockplus.org/29876558/diff/29876559/qunit/tests/url.js#...
qunit/tests/url.js:21: const {extractHostFromFrame,
On 2018/09/06 10:24:10, geo wrote:
> I know that this change is not related to this particular issue, but it's
pretty
> small, and I would have had to either use `let` for the platform import to be
> consistent with this module, or `const` to be consistent with the rest of the
> code. So I choose to be consistent with the rest of the code and updated this
> import as well. 

I agree, but while on it, it also seems unnecessary to wrap this line.

https://codereview.adblockplus.org/29876558/diff/29876559/qunit/tests/url.js#...
qunit/tests/url.js:48: // Currently there are two bugs in Microsoft Edge
(EdgeHTML 17.17134)
Nit: It might read better with a blank line above.

https://codereview.adblockplus.org/29876558/diff/29876559/qunit/tests/url.js#...
qunit/tests/url.js:51: // See: https://issues.adblockplus.org/ticket/6913
Can we link to the Microsoft Edge bugs directly, rather than the issue in our
issue tracker?

[geo, 2018-09-06 20:10:47]

https://codereview.adblockplus.org/29876558/diff/29876559/qunit/tests/url.js
File qunit/tests/url.js (right):

https://codereview.adblockplus.org/29876558/diff/29876559/qunit/tests/url.js#...
qunit/tests/url.js:21: const {extractHostFromFrame,
On 2018/09/06 13:43:21, Sebastian Noack wrote:
> On 2018/09/06 10:24:10, geo wrote:
> > I know that this change is not related to this particular issue, but it's
> pretty
> > small, and I would have had to either use `let` for the platform import to
be
> > consistent with this module, or `const` to be consistent with the rest of
the
> > code. So I choose to be consistent with the rest of the code and updated
this
> > import as well. 
> 
> I agree, but while on it, it also seems unnecessary to wrap this line.

You're right. I was so focused on that let and const, that I didn't even
noticed. Thank you!

https://codereview.adblockplus.org/29876558/diff/29876559/qunit/tests/url.js#...
qunit/tests/url.js:48: // Currently there are two bugs in Microsoft Edge
(EdgeHTML 17.17134)
On 2018/09/06 13:43:21, Sebastian Noack wrote:
> Nit: It might read better with a blank line above.

Done.

https://codereview.adblockplus.org/29876558/diff/29876559/qunit/tests/url.js#...
qunit/tests/url.js:51: // See: https://issues.adblockplus.org/ticket/6913
On 2018/09/06 13:43:21, Sebastian Noack wrote:
> Can we link to the Microsoft Edge bugs directly, rather than the issue in our
> issue tracker?

Done.

[Sebastian Noack, 2018-09-06 21:49:35]

LGTM