Make extensions::DictionaryBuilder and extensions::ListValue unmovable.

chrome/browser/extensions/api/developer_private/developer_private_api_unittest.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <utility>
 
 #include "base/files/file_util.h"
 #include "base/macros.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 134 matching lines @@
   const char kName[] = "extension name";
   const char kVersion[] = "1.0.0.1";
   std::string id = crx_file::id_util::GenerateId(kName);
   DictionaryBuilder manifest;
   manifest.Set("name", kName)
           .Set("version", kVersion)
           .Set("manifest_version", 2)
           .Set("description", "an extension");
   scoped_refptr<const Extension> extension =
       ExtensionBuilder()
-          .SetManifest(std::move(manifest))
+          .SetManifest(manifest.Build())
           .SetLocation(Manifest::INTERNAL)
           .SetID(id)
           .Build();
   service()->AddExtension(extension.get());
   return extension.get();
 }
 
 void DeveloperPrivateApiUnitTest::TestExtensionPrefSetting(
     bool (*has_pref)(const std::string&, content::BrowserContext*),
     const std::string& key,
@@ skipping 343 matching lines @@
       error_test_util::CreateNewRuntimeError(extension->id(), "bar"));
   error_console->ReportError(
       error_test_util::CreateNewManifestError(extension->id(), "baz"));
   EXPECT_EQ(3u, error_console->GetErrorsForExtension(extension->id()).size());
 
   // Start by removing all errors for the extension of a given type (manifest).
   std::string type_string = api::developer_private::ToString(
       api::developer_private::ERROR_TYPE_MANIFEST);
   scoped_ptr<base::ListValue> args =
       ListBuilder()
-          .Append(std::move(DictionaryBuilder()
-                                .Set("extensionId", extension->id())
-                                .Set("type", type_string)))
+          .Append(DictionaryBuilder()
+                      .Set("extensionId", extension->id())
+                      .Set("type", type_string)
+                      .Build())
           .Build();
   scoped_refptr<UIThreadExtensionFunction> function =
       new api::DeveloperPrivateDeleteExtensionErrorsFunction();
   EXPECT_TRUE(RunFunction(function, *args)) << function->GetError();
   // Two errors should remain.
   const ErrorList& error_list =
       error_console->GetErrorsForExtension(extension->id());
   ASSERT_EQ(2u, error_list.size());
 
   // Next remove errors by id.
   int error_id = error_list[0]->id();
   args =
       ListBuilder()
-          .Append(std::move(
-              DictionaryBuilder()
-                  .Set("extensionId", extension->id())
-                  .Set("errorIds", std::move(ListBuilder().Append(error_id)))))
+          .Append(DictionaryBuilder()
+                      .Set("extensionId", extension->id())
+                      .Set("errorIds", ListBuilder().Append(error_id).Build())
+                      .Build())
           .Build();
   function = new api::DeveloperPrivateDeleteExtensionErrorsFunction();
   EXPECT_TRUE(RunFunction(function, *args)) << function->GetError();
   // And then there was one.
   EXPECT_EQ(1u, error_console->GetErrorsForExtension(extension->id()).size());
 
   // Finally remove all errors for the extension.
-  args = ListBuilder()
-             .Append(std::move(
-                 DictionaryBuilder().Set("extensionId", extension->id())))
-             .Build();
+  args =
+      ListBuilder()
+          .Append(
+              DictionaryBuilder().Set("extensionId", extension->id()).Build())
+          .Build();
   function = new api::DeveloperPrivateDeleteExtensionErrorsFunction();
   EXPECT_TRUE(RunFunction(function, *args)) << function->GetError();
   // No more errors!
   EXPECT_TRUE(error_console->GetErrorsForExtension(extension->id()).empty());
 }
 
 }  // namespace extensions