diff options
author | limasdf <limasdf@gmail.com> | 2015-12-08 19:58:45 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-12-09 03:59:36 +0000 |
commit | 3d1025433fba150119ec1519f9228f2febddd55c (patch) | |
tree | 1b00bc3d0334a7c624d7e49043182a6a9d6dd605 /chrome/browser/extensions/scripting_permissions_modifier_unittest.cc | |
parent | 9679230bb637a8aeca7598d08da802b1c64404fe (diff) | |
download | chromium_src-3d1025433fba150119ec1519f9228f2febddd55c.zip chromium_src-3d1025433fba150119ec1519f9228f2febddd55c.tar.gz chromium_src-3d1025433fba150119ec1519f9228f2febddd55c.tar.bz2 |
Use rvalue reference instead of extensions::ListBuilder::Pass()
C++ 11 enables rvalue reference with std::move() so that removing legacy ListBuilder::Pass() stuff.
TEST=unit_tests
BUG=563649
TBR=thakis@chromium.org
Review URL: https://codereview.chromium.org/1497753002
Cr-Commit-Position: refs/heads/master@{#363970}
Diffstat (limited to 'chrome/browser/extensions/scripting_permissions_modifier_unittest.cc')
-rw-r--r-- | chrome/browser/extensions/scripting_permissions_modifier_unittest.cc | 22 |
1 files changed, 12 insertions, 10 deletions
diff --git a/chrome/browser/extensions/scripting_permissions_modifier_unittest.cc b/chrome/browser/extensions/scripting_permissions_modifier_unittest.cc index 1b1a71d..6b37a5d 100644 --- a/chrome/browser/extensions/scripting_permissions_modifier_unittest.cc +++ b/chrome/browser/extensions/scripting_permissions_modifier_unittest.cc @@ -2,6 +2,8 @@ // Use of this source code is governed by a BSD-style license that can be // found in the LICENSE file. +#include <utility> + #include "base/strings/stringprintf.h" #include "chrome/browser/extensions/extension_service_test_base.h" #include "chrome/browser/extensions/extension_util.h" @@ -40,19 +42,19 @@ scoped_refptr<const Extension> CreateExtensionWithPermissions( } DictionaryBuilder script; - script.Set("matches", scriptable_host_list.Pass()) - .Set("js", ListBuilder().Append("foo.js")); + script.Set("matches", std::move(scriptable_host_list)) + .Set("js", std::move(ListBuilder().Append("foo.js"))); return ExtensionBuilder() .SetLocation(location) - .SetManifest( - DictionaryBuilder() - .Set("name", name) - .Set("description", "foo") - .Set("manifest_version", 2) - .Set("version", "0.1.2.3") - .Set("content_scripts", ListBuilder().Append(script.Pass())) - .Set("permissions", explicit_host_list.Pass())) + .SetManifest(DictionaryBuilder() + .Set("name", name) + .Set("description", "foo") + .Set("manifest_version", 2) + .Set("version", "0.1.2.3") + .Set("content_scripts", + std::move(ListBuilder().Append(script.Pass()))) + .Set("permissions", std::move(explicit_host_list))) .SetID(crx_file::id_util::GenerateId(name)) .Build(); } |