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

Issue 185903002: HashTraits<RawPtr> uses 'nullptr' (Closed)

Created:
6 years, 9 months ago by Mikhail
Modified:
6 years, 9 months ago
Reviewers:
tkent, haraken
CC:
blink-reviews, Mikhail, adamk+blink_chromium.org, Inactive, abarth-chromium, Mads Ager (chromium), Erik Corry, tkent
Base URL:
https://chromium.googlesource.com/chromium/blink.git@master
Visibility:
Public.

Description

HashTraits<RawPtr> uses 'nullptr' It is now agreed in blink that 'nullptr' is used to represent a null value for smart pointers. This patch makes HashTraits<RawPtr> use 'nullptr_t' as 'EmptyValueType' making it consistent with HashTraits<RefPtr> and HashTraits<OwnPtr>. Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=168520

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+9 lines, -0 lines) Patch
M Source/wtf/HashTraits.h View 2 chunks +9 lines, -0 lines 0 comments Download

Messages

Total messages: 9 (0 generated)
Mikhail
PTAL
6 years, 9 months ago (2014-03-03 18:53:01 UTC) #1
tkent
lgtm
6 years, 9 months ago (2014-03-04 00:06:14 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/mikhail.pozdnyakov@intel.com/185903002/1
6 years, 9 months ago (2014-03-04 00:06:49 UTC) #3
haraken
LGTM
6 years, 9 months ago (2014-03-04 00:42:53 UTC) #4
Mikhail
The CQ bit was unchecked by mikhail.pozdnyakov@intel.com
6 years, 9 months ago (2014-03-04 09:45:18 UTC) #5
Mikhail
The CQ bit was checked by mikhail.pozdnyakov@intel.com
6 years, 9 months ago (2014-03-04 09:45:24 UTC) #6
Mikhail
The CQ bit was unchecked by mikhail.pozdnyakov@intel.com
6 years, 9 months ago (2014-03-05 06:20:44 UTC) #7
Mikhail
The CQ bit was checked by mikhail.pozdnyakov@intel.com
6 years, 9 months ago (2014-03-05 06:20:49 UTC) #8
commit-bot: I haz the power
6 years, 9 months ago (2014-03-05 19:34:46 UTC) #9
Message was sent while issue was closed.
Change committed as 168520

Powered by Google App Engine
This is Rietveld 408576698