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

Issue 806043002: Use existing unicode.sky instead of re-inventing (Closed)

Created:
6 years ago by eseidel
Modified:
6 years ago
Reviewers:
esprehn
CC:
esprehn, abarth-chromium, mojo-reviews_chromium.org, ojan
Base URL:
git@github.com:domokit/mojo.git@master
Target Ref:
refs/heads/master
Project:
mojo
Visibility:
Public.

Description

Use existing unicode.sky instead of re-inventing I somehow missed this function the first time. This is already tested by tests/framework/xmlhttprequest/unicode-post.sky R=esprehn@chromium.org BUG= Committed: https://chromium.googlesource.com/external/mojo/+/d5ff26b6564f2069af0a2a9e5b7553190d38cec5

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -12 lines) Patch
M sky/framework/xmlhttprequest.sky View 2 chunks +2 lines, -12 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
eseidel
6 years ago (2014-12-16 00:18:22 UTC) #1
esprehn
lgtm
6 years ago (2014-12-16 00:28:37 UTC) #2
eseidel
6 years ago (2014-12-17 20:11:46 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
d5ff26b6564f2069af0a2a9e5b7553190d38cec5 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698