For "Copy link address" context menu, copy the URL with Unicode host name ins...

For "Copy link address" context menu, copy the URL with Unicode host name instead of Punycode host name.

TEST="Copy link address" copies Unicode host name for both of <a href="punycode URL"> and <a href="Unicode URL"> if Accept-Language preference is matched.
BUG=4008

[brettw, 2009-05-26 16:22:06]

LGTM

http://codereview.chromium.org/115752/diff/1/2
File chrome/browser/tab_contents/render_view_context_menu.cc (right):

http://codereview.chromium.org/115752/diff/1/2#newcode651
Line 651: // Unescaping path and query is not a good idea because other
Having a comment in the middle of a statement seems weird to me. Do you think it
would be just as clear to move it above the WriteTextToClipboard line?

[jungshik at Google, 2009-05-26 16:29:20]

LGTM with Brett's concern addressed.

[tkent_google.com, 2009-05-27 00:43:12]

Thanks.  Would you commit this patch please?

http://codereview.chromium.org/115752/diff/1/2
File chrome/browser/tab_contents/render_view_context_menu.cc (right):

http://codereview.chromium.org/115752/diff/1/2#newcode651
Line 651: // Unescaping path and query is not a good idea because other
On 2009/05/26 16:22:06, brettw wrote:
> Having a comment in the middle of a statement seems weird to me. Do you think
it
> would be just as clear to move it above the WriteTextToClipboard line?

Done.

[jungshik at Google, 2009-05-27 01:40:00]

On 2009/05/27 00:43:12, Kent wrote:
> Thanks.  Would you commit this patch please?

Sure. I'll after getting back the try bot results (I just sent the patch to the
trybot). 

Thank you for the patch.

[jungshik at Google, 2009-05-27 16:13:21]

On 2009/05/27 01:40:00, Jungshik Shin wrote:

> Sure. I'll after getting back the try bot results (I just sent the patch to
the

Checked in : http://src.chromium.org/viewvc/chrome?view=rev&revision=16973