Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(317)

Issue 115759: Show Unicode IDNs for "On startup" setting and "Add page" dialog for it.... (Closed)

Created:
11 years, 7 months ago by tkent
Modified:
9 years, 7 months ago
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Show Unicode IDNs for "On startup" setting and "Add page" dialog for it. TEST=visit an IDN URL, and open the "Basics" tab of the "Options" dialog, select "Open the following pages", press "Add..." button, select the IDN URL, and confirm that the URL field is shown in Unicode, or paste a Punycode URL to the URL field of the "Add page" dialog, press "Add" button, and confirm that the "Open the followig pages" list shows the Unicode URL. BUG=3991 Landed: http://src.chromium.org/viewvc/chrome?view=rev&revision=16974

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -3 lines) Patch
M chrome/browser/views/options/general_page_view.cc View 2 chunks +4 lines, -1 line 0 comments Download
M chrome/browser/views/shelf_item_dialog.cc View 1 chunk +8 lines, -2 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
tkent_google.com
11 years, 7 months ago (2009-05-25 11:57:02 UTC) #1
brettw
LGTM
11 years, 7 months ago (2009-05-26 16:20:35 UTC) #2
jungshik at Google
LGTM
11 years, 7 months ago (2009-05-26 16:46:21 UTC) #3
tkent_google.com
Thank you. Would you commit this please?
11 years, 7 months ago (2009-05-27 00:44:22 UTC) #4
jungshik at Google
11 years, 7 months ago (2009-05-27 16:11:53 UTC) #5
Checked in ( http://src.chromium.org/viewvc/chrome?view=rev&revision=16974 )
Thank you.

On 2009/05/27 00:44:22, Kent wrote:
> Thank you.
> Would you commit this please?

Powered by Google App Engine
This is Rietveld 408576698