Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(66)

Issue 1170153002: SVG: Escape non-ASCII characters in console messages. (Closed)

Created:
4 years, 10 months ago by tkent
Modified:
4 years, 10 months ago
Reviewers:
keishi
CC:
blink-reviews, krit, kouhei+svg_chromium.org, fs, f(malita), gyuyoung2, Stephen Chennney, pdr+svgwatchlist_chromium.org, rwlbuis
Target Ref:
refs/heads/master
Project:
blink
Visibility:
Public.

Description

SVG: Escape non-ASCII characters in console messages. BUG=498103 Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=196815

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -5 lines) Patch
M LayoutTests/TestExpectations View 1 chunk +12 lines, -0 lines 0 comments Download
M LayoutTests/svg/dom/path-parser-expected.txt View 1 chunk +2 lines, -2 lines 0 comments Download
M LayoutTests/svg/dom/preserve-aspect-ratio-parser-expected.txt View 1 chunk +1 line, -1 line 0 comments Download
M LayoutTests/svg/dom/transform-parser-expected.txt View Binary file 0 comments Download
M Source/core/svg/SVGElement.cpp View 2 chunks +2 lines, -2 lines 0 comments Download

Messages

Total messages: 7 (3 generated)
tkent
Keishi, please review this.
4 years, 10 months ago (2015-06-10 00:39:03 UTC) #3
keishi
lgtm
4 years, 10 months ago (2015-06-10 01:56:00 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1170153002/20001
4 years, 10 months ago (2015-06-10 01:59:42 UTC) #6
commit-bot: I haz the power
4 years, 10 months ago (2015-06-10 02:03:37 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=196815

Powered by Google App Engine
This is Rietveld 408576698