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

Issue 567653002: Use ConstructFromLiteral instead of implicit conversion (Closed)

Created:
6 years, 3 months ago by rwlbuis
Modified:
6 years, 3 months ago
Reviewers:
pdr., kouhei (in TOK)
CC:
blink-reviews, krit, fs, ed+blinkwatch_opera.com, f(malita), gyuyoung.kim_webkit.org, Stephen Chennney, kouhei+svg_chromium.org, pdr.
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Project:
blink
Visibility:
Public.

Description

Use ConstructFromLiteral instead of implicit conversion Since we are supposed to end up with an AtomicString, don't rely on implicit conversion, but provide it as static local variable while using ConstructFromLiteral to avoid using strlen. Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=181840

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -2 lines) Patch
M Source/core/svg/SVGUseElement.cpp View 1 chunk +3 lines, -2 lines 0 comments Download

Messages

Total messages: 7 (2 generated)
rwlbuis
PTAL
6 years, 3 months ago (2014-09-11 16:45:47 UTC) #2
kouhei (in TOK)
On 2014/09/11 16:45:47, rwlbuis wrote: > PTAL lgtm
6 years, 3 months ago (2014-09-11 17:14:17 UTC) #3
rwlbuis
On 2014/09/11 17:14:17, kouhei wrote: > On 2014/09/11 16:45:47, rwlbuis wrote: > > PTAL > ...
6 years, 3 months ago (2014-09-11 17:31:09 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patchset/567653002/1
6 years, 3 months ago (2014-09-11 17:53:35 UTC) #6
commit-bot: I haz the power
6 years, 3 months ago (2014-09-11 19:01:00 UTC) #7
Message was sent while issue was closed.
Committed patchset #1 (id:1) as 181840

Powered by Google App Engine
This is Rietveld 408576698