1380932 - Devtools should show unicode URLs where appropriate

Status: NEW

(DevTools :: General, enhancement, P3)

Description

[Valentin Gosu [:valentin] (he/him)]

After landing bug 1380617 nsIURI will by default return punycode .host/.hostPort/.spec

Parts of the devtools that need to display the URL in a unicode format should be updated to use the nsIURI.displayHost/.displayHostPort/.displaySpec methods.

In order to test this, set network.standard-url.punycode-host to true, go to a IDNA domain such as http://valentin.xn--gou-2lb.ro/ open the devtools networking panel, and see if <Request URL> and <Domain> display the correct thing.

Other parts of devtools that may be affected:
* Storage
* Debugger
* others?

We should also decide _if_ we want to show unicode/punycode in the devtools. The Chrome Devtools have different behaviours for some of these fields.
We should also consider adding tests for this scenario so we don't regress in the future.

Comment 1

[Patrick Brosset <:pbro>]

Thanks for filing.
Let's make this a meta bug with dependencies for each tool.

(In reply to Valentin Gosu [:valentin] from comment #0)
> In order to test this, set network.standard-url.punycode-host to true, go to
> a IDNA domain such as http://valentin.xn--gou-2lb.ro/ open the devtools
> networking panel, and see if <Request URL> and <Domain> display the correct
> thing.
Tested on a new profile in 57, this pref already seems to be set to true for me. 
Request URL and Domain both show http://valentin.xn--gou-2lb.ro/ which is good.

> Other parts of devtools that may be affected:
> * Storage
> * Debugger
> * others?
The inspector and console also show URLs in their UIs. So they will be affected.
 
> We should also decide _if_ we want to show unicode/punycode in the devtools.
> The Chrome Devtools have different behaviours for some of these fields.
Do you mind expanding on how does Chrome differ on this?
I would tend to think that we should show punycode consistently in DevTools.

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

The meta keyword is there, the bug doesn't depend on other bugs and there is no activity for 12 months.
:pbro, maybe it's time to close this bug?

Comment 3

[Patrick Brosset <:pbro>]

I think we still want this bug on file.

Comment 4

[BugBot [:suhaib / :marco/ :calixte]]

The meta keyword is there, the bug doesn't depend on other bugs and there is no activity for 12 months.
:Honza, maybe it's time to close this bug?

Comment 5

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Feels like we still want this.

Honza

Comment 6

[BugBot [:suhaib / :marco/ :calixte]]

The meta keyword is there, the bug doesn't depend on other bugs and there is no activity for 12 months.
:Honza, maybe it's time to close this bug?

Comment 7

[Jan Honza Odvarko [:Honza] (always need-info? me)]

The bug seems to be still valid, but removing the meta flag.