diff options
author | mgiuca <mgiuca@chromium.org> | 2015-06-11 16:44:30 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-06-11 23:44:53 +0000 |
commit | b102326b0da3ecb676d67fc9b0a1b22ac1e89d4c (patch) | |
tree | 92efb0e15fcc0fd9db0d53d4082e059b98ab3ab1 /base/BUILD.gn | |
parent | b77374302406941385acb0da512933b1afe88c40 (diff) | |
download | chromium_src-b102326b0da3ecb676d67fc9b0a1b22ac1e89d4c.zip chromium_src-b102326b0da3ecb676d67fc9b0a1b22ac1e89d4c.tar.gz chromium_src-b102326b0da3ecb676d67fc9b0a1b22ac1e89d4c.tar.bz2 |
Added ScopedPtrMap class, for maps that contain scoped_ptr values.
It is not currently possible to use scoped_ptr values in a std::map, due
to lack of C++11 library support; this class is a placeholder mapping
type that allows scoped_ptr values.
Internally, ScopedPtrMap does not use scoped_ptrs; it uses raw pointers
and automatically deletes its values when it is destroyed, or elements
are removed from the map. It is therefore safer to use than a map with
owned raw pointers as values, even when using an STLValueDeleter. It
also makes ownership much clearer, as you can insert elements into the
map via scoped_ptrs.
BUG=478594
Review URL: https://codereview.chromium.org/1076273004
Cr-Commit-Position: refs/heads/master@{#334083}
Diffstat (limited to 'base/BUILD.gn')
-rw-r--r-- | base/BUILD.gn | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/base/BUILD.gn b/base/BUILD.gn index fea9027..8a32ed6 100644 --- a/base/BUILD.gn +++ b/base/BUILD.gn @@ -177,6 +177,7 @@ component("base") { "containers/linked_list.h", "containers/mru_cache.h", "containers/scoped_ptr_hash_map.h", + "containers/scoped_ptr_map.h", "containers/small_map.h", "containers/stack_container.h", "cpu.cc", @@ -1100,6 +1101,7 @@ test("base_unittests") { "containers/linked_list_unittest.cc", "containers/mru_cache_unittest.cc", "containers/scoped_ptr_hash_map_unittest.cc", + "containers/scoped_ptr_map_unittest.cc", "containers/small_map_unittest.cc", "containers/stack_container_unittest.cc", "cpu_unittest.cc", |