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

Issue 26439002: Canonical encoding name : windows-949 => EUC-KR (Closed)

Created:
7 years, 2 months ago by jungshik at Google
Modified:
7 years, 2 months ago
Reviewers:
Nico, robertshield
CC:
chromium-reviews, grt+watch_chromium.org, amit, robertshield, jshin+watch_chromium.org
Visibility:
Public.

Description

Canonical encoding name : windows-949 => EUC-KR This is the chromium side of a blink change at https://codereview.chromium.org/25870005 Until the corresponding blink CL above is re-landed (it's landed but reverted), disable EUC-KR-related test temporarily. This is a copy of https://codereview.chromium.org/25769005/ (lgtm'd and approved). BUG=277027, 95963 TEST=browser test: BrowserEncodingTest.* and EncodingAlias*. TBR=thakis,robertshield Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=227506

Patch Set 1 #

Messages

Total messages: 4 (0 generated)
jungshik at Google
I'm TBR'ing because the CL was reviewed earlier. I had to copy the CL because ...
7 years, 2 months ago (2013-10-08 05:16:49 UTC) #1
jungshik at Google
This will go in before the blink change (that was reverted) with EUC-KR related tests ...
7 years, 2 months ago (2013-10-08 05:18:11 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/jshin@chromium.org/26439002/1
7 years, 2 months ago (2013-10-08 09:10:20 UTC) #3
commit-bot: I haz the power
7 years, 2 months ago (2013-10-08 12:38:28 UTC) #4
Message was sent while issue was closed.
Change committed as 227506

Powered by Google App Engine
This is Rietveld 408576698